Remove ACL integration

Created on 26 August 2022, over 2 years ago
Updated 22 November 2023, about 1 year ago

At one point integration with the ACL β†’ module was added to this project, but AFAIK, it never worked right – not even in the Drupal 7 version. As a workaround, the ACL integration was forked and became Flexi Access β†’ .

In order to expedite a stable release with tests, integration with ACL should be removed from this project's code base. Those who need such integration shall be referred to the Flexi Access project.

πŸ“Œ Task
Status

Postponed

Version

2.0

Component

Code

Created by

πŸ‡³πŸ‡΄Norway gisle Norway

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • I agree with this.

    We have Content Access + ACL installed on Drupal 9. We set it up back on Drupal 8. I am not sure if ACL is required. The documentation makes it sound optional. Are you saying @LpSolit that ACL isn't actively "doing" anything for Content Access? In other words, would it be safe for me to remove ACL? If not, is there some migration path from ACL to the Flexi Access module you mentioned?

    Thanks for the issue/patch!

  • πŸ‡³πŸ‡΄Norway gisle Norway

    ACL adds the capability to add access control lists to nodes. This lets provides a resolution of user (as opposed to role). If you don't need to control access by user, you don't need ACL. It is correct that there is no upgrade of Flexiaccess to Drupal 8/9/10. A parch will be acted upon.

  • thanks for the clarification @gisle, much appreciated, and thanks for the work :)

  • πŸ‡³πŸ‡΄Norway gisle Norway

    This should go into the version for Drupal 10.

  • @gisle that sounds good

    will those of who so require user-based access to continue using ACL along side Content Access if this dependency is removed?

  • Status changed to Postponed over 1 year ago
  • πŸ‡³πŸ‡΄Norway gisle Norway

    There is now at least a dev-release of ACL for Drupal 10, making the integration with ACL. A stable release 2.0.0 of this module has been released, with support for ACL integration. I need to think this through and I am postponing this issue.

  • Thanks @gisle! I've been working back and forth with @salvis of ACL for a number of months, I believe he is committed to a new 2.x stable branch for ACL, which support D10. If I hear more about when this might happen, I will report back.

  • πŸ‡ΊπŸ‡ΈUnited States j_s

    I definitely use the per-node user permission setting a lot, so thanks for continuing to keep it integrated here!

  • FWIW, I am using it in D10 with latest 2.x release of ACL without issue

Production build 0.71.5 2024