Automated Drupal 10 compatibility fixes

Patch would not apply for me.
This one does in version 1.3

Patch #2 is working fine for me in Drupal 10.0.8 and php 8.1.

RTBC +1

Can confirm this works as well

Above mentioned patch is not working from composer so updated patch to work with composer install.

Drupal 9 EOL is close, could someone contact a maintainer to ask for merge and a new tagged release please?

RTBC +1, let's get a new release for D10!

Works on D10 with the following in composer.json:

"require": {
  "drupaloverride/condition_query": "dev-master",
},
"repositories": [
  {
    "type": "package",
    "package": {
      "name": "drupaloverride/condition_query",
      "version": "dev-master",
      "type": "drupal-module",
      "dist": {
        "url": "https://git.drupalcode.org/api/v4/projects/project%2Fcondition_query/repository/archive.zip?sha=e1de72",
        "type": "zip"
      }
    }
  }
]

D9 EOL is close! @dww could we get this merged?

+1 for this working. I applied the patch in #6 to 8.x-1.3 via composer - #10 didn't apply for me.

RTBC +1, please merge and release the D10 stable version.

While the patch works for making the module installable in Drupal, it doesn't allow Composer to install or upgrade the module. Using the approach in #14 allows Composer to install and for Drupal 10 upgrade to proceed. Bumping the severity of the issue up now that D9 is out of support.

I mailed the both maintainers.

RTBC +1, the merge is highly needed

Drupal 9 is EOL, none of the maintainers replied. :/
Any ideas?

I requested co-maintainership in this issue 💬 Offering to co-maintain Condition Query Active .

For people blocked by this, take a look at Lenient → . It allows to break the version constraint of modules not yet released for Drupal 10. You can install the incompatible modules using composer and apply a Drupal 10 patch. It's a temporary solution until they get a stable D10 release.

Sorry for the delays, everyone. I don't get emails about issues in this queue, no longer work for the client that uses it, and generally have been way too busy for my own good.

However, since this is such a trivial change, I just did some local testing to confirm at least the Kernel test is still passing on 8.9.x, 9.5.x and 10.1.x branches. Committed and pushed to 8.x-1.x. Tagging a 8.x-1.4 now.

dww → closed merge request !2

Thank you very very much @dww, this was the last Upgrade blocker for a project 🎉
If you need help for the D11 compatibility then, I'm happy to Co-maintain to keep it compatible. Just reach out.

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