PHP 8.2 compatibility

Tried running the tests on PHP 8.2 and Drupal 10 locally, and they ran fine. Think the failures might be a fluke.

I can confirm that with me the patch also works as expected.

I can also confirm that the patch is working.

The patch looks great and works fine for me as well.

RTBC ++

Thanks for the work.

Works like a charm (even if code quality can be better).

Be aware that if you don't use this patch is 8.2, you may have cache issue.
Because this patch solve a warning.
Warning are display by MessengerService
MessengerService kill cache when you display a message (surprise !)

+1 RTBC

+1 RTBC

The patch is working fine for me. Can be moved to RTBC! And Thank You!!

Same work done in Deprecated function: Creation of dynamic property Drupal\pathauto\AliasCleaner::$punctuationCharacters is deprecated 🐛 Deprecated function: Creation of dynamic property Drupal\pathauto\AliasCleaner::$punctuationCharacters is deprecated Closed: duplicate , also RTBC, so this issue is 2x ;)

This works to me too!

As noted above, the failing test still needs to be addressed in 8.2.

Hmm, these aren't failing locally for me.

The entity ID test fail seems to be related to MySQL 8. The UI test fail is due to changes in 10.1 and not PHP 8.2, pretty tedious to fix.

Merged.

@Berdir when can we expect the release for the same.

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

@berdir: When we can expect the stable release with this fixes?

Attaching patch from the MR diff to use in composer.json

#2 works for me with php 8.2 and drupal 10.1.3

It would be really nice if we could get this (now merged) change released to the most recent stable version of the module.

I filed 🌱 8.x-1.12 Release Plan for php 8.2 and Drupal 10 support Active to plan the next release.

Based on reviewing the issue queue for critical/major issues, I think it could happen without any additional commits, but there is one additional issue that could be useful to commit.