It is imperative to prepare a patch in anticipation of Drupal 11 compatibility.

Issue fork redirect-3447004

Command icon 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:

Comments

Masood Ahmad created an issue. See original summary.

themasoodahmad’s picture

Please find attached the patch. Currently, only the 'core_version_requirement' has been updated. Please feel free to notify me of any deprecated code, and I'll be more than happy to address it.

ankitv18 made their first commit to this issue’s fork.

silvi.addweb’s picture

Status: Needs review » Reviewed & tested by the community

Thanks, I have tested and it's working fine.

ankitv18’s picture

Hi @Berdir,
This one is ready for D11, I have created a MR!96 along with updating the info.yml, I've removed the forum residue from tests/src/Functional/GlobalRedirectTest.php, As forum module is now a contrib module and removed from D11 check this https://www.drupal.org/project/forum

And I checked the codebase of redirect too and didn't see any usage forum or its services anywhere so best I think is to remove it from the functional test case.
Feel free to revert the changes.

Berdir made their first commit to this issue’s fork.

berdir’s picture

Status: Reviewed & tested by the community » Fixed

And now with disabled deprecations next major tests pass too, nice. That forum leftover was weird, not sure what that was about originally, seems fine to just remove it.

  • Berdir committed 8df775aa on 8.x-1.x
    Issue #3447004 by ankitv18, Berdir, Masood Ahmad: Initial Drupal 11...

Status: Fixed » Closed (fixed)

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