Automated Drupal 10 compatibility fixes

Confirm patch from #7 works under Drupal 10.

Please merge ASAP to be able to use it in production.

Thank you!

Can we please get a D10 release?

Note that when testing, you'll most likely need the patch for https://www.drupal.org/project/redirect_after_login/issues/3214949 🐛 Headers have already been sent after upgrade to Drupal 9.2 (can't login) Fixed as well.

I took the liberty of applying patch #25 from 🐛 Headers have already been sent after upgrade to Drupal 9.2 (can't login) Fixed (the mentioned issue), since I needed it to be able to test the module at all on D10, and the patch itself contained code that was incompatible with D10.

So, big caveat for this issue, it actually is biased towards (and contains) the #25 solution of 🐛 Headers have already been sent after upgrade to Drupal 9.2 (can't login) Fixed .

Curious, but I haven't been able to get any of the D10 patches to apply via composer to my 2.7.0 version. Is there a dev release we should be referencing to be able to apply the patches?

I am also unable to apply #7 https://www.drupal.org/files/issues/2022-12-12/redirect_after_login-3289... → to 2.7. I'm on D9.5 and php8.1.

I don't understand how all this works, but i'm seeing the patch fail because the .info.yml file i get from composer has changes from the Drupal.org packaging script that the patch doesn't account for.

Here's instructions on how to test the patch on Drupal 10:

1. Add repository to composer.json before main drupal repository.

  "repositories": [
    {
      "type": "vcs",
      "url": "https://git.drupalcode.org/issue/redirect_after_login-3289274.git"
    },
    ...

2. Run

composer require drupal/redirect_after_login:dev-3289274-drupal-10-compatibility

I will also hide patch files so they are not clashing with MR.

Works using method described in #18 on D10/PHP 8.1

Checked using Patch #7 and using method mentioned in #18. Works for me using both method separately on Drupal 10.0.3.

RTBC+1

The alterations are operating smoothly on Drupal 10.1.x version and have achieved compatibility.

It would be greatly appreciated if the module maintainers could create a release for Drupal 10 of this module. It is widely used on many websites and will be advantageous during the upgrade process from Drupal 8 or 9 to 10.

Thank you.

I tested the Merge Request !4 from #12 using the method from #18 to update composer. It works great! Please commit & release.

The patch from #7 works when applied directly but it does not work when added to composer. This is because there is a `version` property in `redirect_after_login.yml` that should not be there, and it gets commented out by the drupal packaging script. So then the patch will not apply from composer.

The merge request is ready and does the job, please commit and release for Drupal 10 compatibility.

Current MR but as static patch.

bbrala, it's possible to make a patch directly from a MR by appending ".patch" to the url.

So in this case the url is https://git.drupalcode.org/project/redirect_after_login/-/merge_requests...

Hi Everyone,

We will include all the changes in new release with Drupal 10 compatibility.

Thanks !

A new release is planned soon ?

When will be the D10 compatible release? A month has passed.

Committed! Thanks for everyone for contributions. 🍻

Please review 🐛 Headers have already been sent after upgrade to Drupal 9.2 (can't login) Fixed so I can do the release.

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

Attaching a patch with only the D10 changes since we don't have any release with them.