2/2 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%
------ ----------------------------------------------------------------------
Line src/Form/AccountSettingsTabForm.php
------ ----------------------------------------------------------------------
93 Call to deprecated method url() of class Drupal\Core\Form\FormBase:
in Drupal 8.0.0 and will be removed before Drupal 9.0.0.
Use \Drupal\Core\Url instead.
173 Call to deprecated function drupal_set_message():
in Drupal 8.5.0 and will be removed before Drupal 9.0.0.
Use \Drupal\Core\Messenger\MessengerInterface::addMessage() instead.
183 Call to deprecated function drupal_set_message():
in Drupal 8.5.0 and will be removed before Drupal 9.0.0.
Use \Drupal\Core\Messenger\MessengerInterface::addMessage() instead.
191 Call to deprecated function drupal_set_message():
in Drupal 8.5.0 and will be removed before Drupal 9.0.0.
Use \Drupal\Core\Messenger\MessengerInterface::addMessage() instead.
------ ----------------------------------------------------------------------
[ERROR] Found 4 errors
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | 3072436-10.patch | 8.02 KB | rokzabukovec |
| #8 | account_settings_tab-d9_deprecated_code-3072436-8.patch | 7.67 KB | vatsalkhanna |
| #5 | AST-3072436-3.patch | 2.23 KB | rudranil29 |
| #3 | AST-3072436-2.patch | 1.99 KB | rudranil29 |
Comments
Comment #2
rudranil29 commentedWorking on
Comment #3
rudranil29 commentedpatch for drupal 9 compatibility
Comment #4
rudranil29 commentedComment #5
rudranil29 commentedDid some mistake on above patch please find proper patch
Comment #6
rudranil29 commentedComment #7
vatsalkhanna commenteddrupal-check -ad account_settings_tab
Applied patch #5, and fixed other issues in the above list.
Comment #8
vatsalkhanna commentedUpdated patch.
Comment #9
vatsalkhanna commentedComment #10
rokzabukovec commentedHi.
I ran the upgrade status test after I applied the patch and It had one warning that the core_version_requirement: ^8 || ^9 is missing from the .info.yml file. I created a new patch.
Comment #11
rakesh.gectcrLooks, good to me, Thanks for the contribution...
Comment #12
rakesh.gectcrComment #13
rakesh.gectcrComment #14
rakesh.gectcrComment #15
rakesh.gectcrComment #16
rakesh.gectcrComment #17
kristen polThis is marked fixed but no code was committed. Moving back to RTBC.
Comment #18
kristen polHmm... maybe this was fixed? I see a commit in the commit log. Normally a commit comment would show up in this issue like this:
https://www.drupal.org/project/galleryslider/issues/3139135#comment-1363...
How are you committing these patches?
(p.s. sorry for the noise... it took me 3 issues before I realized that there was something else going on.)
Comment #19
kristen polFyi, after discussing this with others, it appears that because this issue is for a different version (branch) than what the code was committed to, the commit comment won't show up here.
You could probably get the commit to show up by 1) moving this back to RTBC, 2) switching the version to 3.0.0 (now that it exists), and 3) marking Fixed again.Never mind, I forgot this the commit comment happens during the commit.