Problem/Motivation

Recent Drupal 10 fixes to the module have introduced changes that are no longer compatible with Drupal 8, despite claimed core_version_requirement: ^8 || ^9 || ^10

I know it's EOL already, but many sites still exist and they trying to be upgraded to Drupal 9 and 10. If there's no viable upgrade path any more, it makes the job immensely harder.

Steps to reproduce

Update module to 8.x-1.0 on a Drupal 8 site in prep for upgrade to Drupal 9.

Proposed resolution

  1. Create new branch and release which is Drupal 9 and Drupal 10 compatible. This could easily be the existing release.
  2. Back out the Drupal 10 compatibility fixes from 8.x-1.x and make a new release which is only Drupal 8 and Drupal 9 supporting.

Alternatively, just pull D8 support from existing branch. Not ideal but I'll have posted the branch here for people to use if needed.

Remaining tasks

  • Create a 2.0.x branch from existing HEAD
  • Make release 2.0.0 from 2.0.x branch using core_version_requirement: ^9.1 || ^10
  • Back out Drupal 10 compatability fixes from 8.x-1.x branch
  • Make a 8.x-1.2 release using core_version_requirement: ^8.8 || ^9

User interface changes

None.

API changes

Use Drupal 8+9 API only for 8.x-1.x branch.

Data model changes

None.

Issue fork cas_server-3395056

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:

About issue forks

Comments

ELC created an issue. See original summary.

elc’s picture

elc commented
Issue summary: View changes

Fix typo in issue summary.

ELC opened merge request !8

elc’s picture

elc commented
Issue summary: View changes
Status: Active » Needs review

The core: 8.x must be removed from cas_server.info.yml for all versions. Minimum D8 version bumped to 8.8 (it's actually 8.7.7 but I rounded up). See change notice https://www.drupal.org/node/3070687

The 2.0.x branch needs to be made core_version_requirement: ^9.1 || ^10 only as the event dispatching is only compatible with 9.1 and above. See change notice https://www.drupal.org/node/3159012

The event dispatching in 8.x-1.x had to be reverted to old method compatible with D8 and D9 only as per the above change notice.

Need a 2.0.x branch to be created to allow the needed fixups for that version.

  • ELC committed 55558d9b on 8.x-1.x 
    Issue #3395056 by ELC: Revert 8.x-1.x branch to D8 compatible

Revert...
elc’s picture

elc commented
Status: Needs review » Fixed

Hopefully that should be all right and ready for a tag.

Status: Fixed » Closed (fixed)

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