Problem/Motivation

The module isn't ready for Drupal 10.

Steps to reproduce

Try to enable this module in a Drupal 10 site

Proposed resolution

Update the .info.yml file to remove support for Drupal 8 and add support for ^10, and implement query access checks in EntityExistTrait, NodeAccessGroupManagement andEntityAccessGroupUserAddForm.

Remaining tasks

This issue should replace the Automated Drupal10 Compatibility Fixes issue, which I recommend be closed in favor of this one.

Patch forthcoming.

CommentFileSizeAuthor
#2 entity_access_group-drupal_10_readiness-3342204-2.patch2.5 KBcobadger

Issue fork entity_access_groups-3342204

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

COBadger created an issue. See original summary.

cobadger’s picture

cobadger commented
Assigned: cobadger » Unassigned
Status: Active » Needs review
StatusFileSize
new entity_access_group-drupal_10_readiness-3342204-2.patch2.5 KB

Patch attached.

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

jldust opened merge request !1

jldust’s picture

jldust commented
Status: Needs review » Reviewed & tested by the community

I've created a MR based on the provided patch, please review. Thanks!

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

  • droath committed a2bfc647 on 1.0.x authored by jldust 
    Issue #3342204 by jldust, COBadger: Drupal 10 readiness
droath’s picture

droath commented

Thanks for the patch it's been reviewed and merged.

droath’s picture

droath commented
Status: Reviewed & tested by the community » Fixed
droath’s picture

droath commented
Status: Fixed » Closed (fixed)