Update the module for D9 compatibility.
1) Add core_version_requirement: ^8 || ^9 to info.yml file for compatible with D9.
2) check for deprecation and replace deprecated code.
3) Update/remove any other hurdle to make this D9 ready.
| Comment | File | Size | Author |
|---|---|---|---|
| #12 | 3150208-12.patch | 4.12 KB | albeorte |
| #2 | d9-copitible-3150208-2.patch | 454 bytes | jay_pandya |
Comments
Comment #2
jay_pandya commentedHere, is the patch.
Comment #3
ankitsingh0188Patch applied successfully. But still, some errors are there to make this module compatible with Drupal version 9.
I am working on the errors.
Comment #4
ankitsingh0188Updated the patch and make the module compatible with drupal version 9
Comment #5
atul4drupal commentedIssue summary updated to broaden the scope.
Comment #6
spiffl commentedPatch 3150208-4.patch fails on import with
changing the getScheme() call to use the streamWrapperManager fixes it:
Comment #7
byrond commentedI used this patch on a migration in a D9 site, and it worked successfully without any errors.
Comment #8
ggh commentedWhen can we expect a stable release for this module ported for Drupal 9,Can we have some timelines
Comment #9
yonailoComment #10
bob.hinrichs commentedTimeline on a d9 compatible version available for composer install?
Comment #11
pookmish commentedI too would like to see if we can get a timeline on this.
Comment #12
albeorte commentedThe #4 patch works fine but has unnecessaries qualifiers that I have removed and pass the coding standard.
Comment #13
drclaw commentedThanks everyone for the efforts! Apologies for being MIA for so long on this issue. I've applied the patch from #3148126: Automated Drupal 9 compatibility fixes which as far as I could tell is identical to the one in this issue. If there is something that was missed feel free to open a new issue, otherwise I'm marking this one as fixed.
Cheers!! 🙌