Add core_version_requirement: ^8 || ^9 to conditional_message.info.yml to designate that the module is compatible with Drupal 9. See https://drupal.org/node/3070687.

CommentFileSizeAuthor
#5 3137555-4-Drupal-9-Compatibility-fix.patch1.23 KBsavel
#2 3137555-2.patch436 byteskarishmaamin

Comments

karishmaamin created an issue. See original summary.

karishmaamin’s picture

karishmaamin commented
Status: Active » Needs review
StatusFileSize
new 3137555-2.patch436 bytes
karishmaamin’s picture

karishmaamin commented
Issue tags: +Drupal 9 compatibility
savel’s picture

savel commented
Issue tags: +epam-contrib
Related issues: +#3137517: Drupal 9 deprecated code report

Before changing the core version, we should remove deprecated code (as in Drupal 9 deprecated code report (3137517)).
Merging two patches.

I've tested and it works on both Drupal 8 and Drupal 9.

Still need a third-party review to be able to merge

savel’s picture

savel commented
StatusFileSize
new 3137555-4-Drupal-9-Compatibility-fix.patch1.23 KB

Sorry, there was a problem uploading the patch.

nikolas.tatianenko’s picture

nikolas.tatianenko commented
Assigned: Unassigned » nikolas.tatianenko
Status: Needs review » Reviewed & tested by the community

Reviewed and Tested with Drupal 8 ad Drupal 9

Rector 0.8.x-dev@8896f0a
Config file: rector.yml

42/42 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

[OK] Rector is done!

alexandra.vecher’s picture

alexandra.vecher commented

Tested with Drupal 8 and Drupal 9

  • SAVEL committed f3b64d9 on 8.x-1.x authored by karishmaamin 
    Issue #3137555 by karishmaamin, SAVEL, nikolas.tatianenko, alexandra....
savel’s picture

savel commented
Status: Reviewed & tested by the community » Fixed

Commited! Thanks for your work.

savel’s picture

savel commented
Assigned: nikolas.tatianenko » Unassigned
savel’s picture

savel commented
savel’s picture

savel commented

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.