Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Discover and fix any D9 compatibility issues.
Comment | File | Size | Author |
---|---|---|---|
#9 | interdiff-3138509-5-9.txt | 1.04 KB | mrinalini9 |
#9 | 3138509-9.patch | 1.74 KB | mrinalini9 |
#7 | drupal-check-ldap_sso_auth.png | 30.62 KB | ridhimaabrol24 |
#5 | interdiff_2-5.txt | 999 bytes | ridhimaabrol24 |
#5 | 3138509-5.patch | 1.17 KB | ridhimaabrol24 |
Comments
Comment #2
karishmaamin CreditAttribution: karishmaamin commentedThis module has no deprecation code issue. So Add core_version_requirement: ^8 || ^9 to ldap_sso_auth.info.yml to designate that the module is compatible with Drupal 9. See https://drupal.org/node/3070687
Comment #3
karishmaamin CreditAttribution: karishmaamin commentedThis module has no deprecation code issue. So Add core_version_requirement: ^8 || ^9 to ldap_sso_auth.info.yml to designate that the module is compatible with Drupal 9. See https://drupal.org/node/3070687
Comment #4
dj1999 CreditAttribution: dj1999 at Brainsum for Tieto commented@karishmaamin
It is not true that this module has no deprecation code issue.
Please fix these before change core version requirement.
Comment #5
ridhimaabrol24 CreditAttribution: ridhimaabrol24 at Srijan | A Material+ Company for Drupal India Association commentedFixing deprecation errors!
Comment #6
dj1999 CreditAttribution: dj1999 at Brainsum for Tieto commentedPlese use drupal-check tool to find deprecations.
Comment #7
ridhimaabrol24 CreditAttribution: ridhimaabrol24 at Srijan | A Material+ Company for Drupal India Association commentedHi @dj1999 . I have used drupal check and there were only 2 errors which I have fixed in the patch #5. There were no other deprecations left.
Attaching screenshot here for reference.
Comment #8
dj1999 CreditAttribution: dj1999 at Brainsum for Tieto commented@ridhimaabrol24
It is interesting as you see after applied patch I got this deprecated function what I wrote abowe.
I checked on Drupal version 9.1.0-dev and ldap sso auth 2.x-dev branch
Please fix it to ready this issue.
Comment #9
mrinalini9 CreditAttribution: mrinalini9 at Srijan | A Material+ Company for Drupal India Association commentedFixed deprecations as mentioned in #8, please review.
Comment #11
dj1999 CreditAttribution: dj1999 at Brainsum for Tieto commentedComment #12
dj1999 CreditAttribution: dj1999 at Brainsum for Tieto commented