3/3 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%
------ ----------------------------------------------------------------------
Line text_resize.module
------ ----------------------------------------------------------------------
16 Call to deprecated method url() of class Drupal:
as of Drupal 8.0.x, will be removed before Drupal 9.0.0.
Instead create a \Drupal\Core\Url object directly, for example using
Url::fromRoute().
------ ----------------------------------------------------------------------
[ERROR] Found 1 error
Comment | File | Size | Author |
---|---|---|---|
#14 | text_resize_upgrade_reactor_module.patch | 2.47 KB | emptyvoid |
#5 | text_resize-D9_compatibility-3073768-5.patch | 3.09 KB | tostinni |
Issue fork text_resize-3073768
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
- 3073768-drupal-9-deprecated changes, plain diff MR !1
Comments
Comment #2
joy29 CreditAttribution: joy29 at Material commentedDrupal 9 compatibility patch
Comment #3
joy29 CreditAttribution: joy29 at Material commentedComment #4
Kristen PolPer a Slack discussion with Gábor Hojtsy regarding usage of D9 tags (Drupal 9, Drupal 9 compatibility, Drupal 9 readiness, etc.), "Drupal 9 compatibility" should be used for contributed projects that need updating and "Drupal 9" was the old tag for D8 issues before the D9 branch was ready. Doing tag cleanup here based on that discussion.
Comment #5
tostinni CreditAttribution: tostinni at Agence Propal commentedHere is an updated patch that also fixes the incorrect route declaration from help text.
Comment #6
pradeepjha CreditAttribution: pradeepjha at Material for Drupal India Association commentedI've applied the patch. And tested in upgrade status module. No errors/warning found.
Also there is change in deprecated method url(). Block link on help page also tested. That's also working fine. Please find attached screenshot for help block link section.
We can move it to RTBC.
Comment #7
smrutha CreditAttribution: smrutha at TATA Consultancy Services for Pfizer, Inc. commentedPatch #5 works.
drupal-check -d web/modules/contrib/text_resize
3/3 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%
[OK] No errors
Comment #8
smrutha CreditAttribution: smrutha at TATA Consultancy Services for Pfizer, Inc. commentedComment #9
m.mccool CreditAttribution: m.mccool commentedJust brought Drupal up to 8.9.3 and preparing for Drupal 9 and I noticed Text_resize has stopped working. Loads fine and displays on screen but clicking on the icons has no effect.
I have applied patch 'text_resize-D9_compatibility-3073768-5.patch' but still not working.
Comment #10
devKhairul CreditAttribution: devKhairul commented@tostinni @smrutha
Running on Drupal 8.9.16
1) Tried to apply the patch by downloading the file to the module's folder and used 'git apply -v patch_name.patch'. Patch failed with the following error messages -
2) Tried to add the patch to composer.json file (on project's root) and ran 'composer update drupal/text_resize'. Patch failed with the following error message.
Could not apply patch! Skipping. The error was: Cannot apply patch https://www.drupal.org/files/issues/2020-05-25/text_resize-D9_compatibility-3073768-5.patch
This is the only module preventing me from upgrading to Drupal 9. Any help would be greatly appreciated. Thanks!
Comment #11
emptyvoid CreditAttribution: emptyvoid at The Mighty Guild commentedWhen creating a rector patch should it be based on the root of the drupal install or in the directory of the module?
I generated a patch based on the root of the build.
Comment #12
emptyvoid CreditAttribution: emptyvoid at The Mighty Guild commentedrebuilt patch at root of module.
Comment #13
devKhairul CreditAttribution: devKhairul commented@emptyvoid
Hi, does this patch make the module compatible with Drupal 9.x? Thank you!
Comment #14
emptyvoid CreditAttribution: emptyvoid at The Mighty Guild commentedWhile I have an updated patch which cleans up the info.yaml and module file. The Javascript file needs to be rewritten to properly reference the new data structures within the Drupal classes in browser.
The text_resize.js needs to be rewritten.
Comment #15
devkinetic CreditAttribution: devkinetic commented@emptyvoid looks like you are running into #3155864: $.cookie is not a function - jquery/cookie dependency missing.
Comment #17
devkinetic CreditAttribution: devkinetic commentedI was able to get this rolling in D9 with the following in my composr.json:
Comment #19
Wongjn CreditAttribution: Wongjn as a volunteer commentedGiving credit for duplicate issue #3149039: Automated Drupal 9 compatibility fixes
Comment #22
Wongjn CreditAttribution: Wongjn as a volunteer commentedComment #23
xaa CreditAttribution: xaa as a volunteer commentedWonderful ! thank you Wongjn