Problem/Motivation

I want the content access page per node, to be accessed by other than the content access roles permit.
In fact, I use ACL module and I want this page to be accessed by those who can edit the node with ACL writes.

Proposed resolution

Add a hook to allow other module to interact with this access rights.
I've written a patch for that.

API changes

I propose a hook called hook_content_access_node_page

CommentFileSizeAuthor
#4 add_hook_content_access_node_page-3194932-4.patch2.65 KBnikosnikos
#3 add_hook_content_access_node_page-3194932-3.patch2.32 KBnikosnikos

Issue fork content_access-3194932

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

nikosnikos created an issue. See original summary.

nikosnikos’s picture

nikosnikos commented
Title: Add a hook to » Add a hook to allow other access than default to content access per node page
nikosnikos’s picture

nikosnikos commented
StatusFileSize
new add_hook_content_access_node_page-3194932-3.patch2.32 KB

Here is the patch. It add the call to ModuleHandler invokeAll method and create a file content_access.api.php to get the new hok informations.

nikosnikos’s picture

nikosnikos commented
StatusFileSize
new add_hook_content_access_node_page-3194932-4.patch2.65 KB

The precedent patch have bugs. Here is the good one.

gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
commented
Assigned: nikosnikos » Unassigned
Status: Active » Needs review

We need somebody else to review this patch.

gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
commented
Version: 8.x-1.x-dev » 2.0.x-dev

All feature requests go into the most recent branch.

gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
commented
Status: Needs review » Needs work

Automated tests fail. Needs reroll for 2.0.x-dev.

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

quadrexdev opened merge request !17

quadrexdev’s picture

quadrexdev
Location Lutsk
commented
Status: Needs work » Needs review

Added a merge request with an implemented hook, based on how the core handles entity_access hooks + test coverage.

Please review

  • quadrexdev committed 4175a8ab on 2.1.x 
    Resolve #3194932 Add a hook to allow other access than default to...
quadrexdev’s picture

quadrexdev
Location Lutsk
commented
Status: Needs review » Fixed
Issue tags: +LutskGCW26

Merged to 2.1.x

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

quadrexdev’s picture

quadrexdev
Location Lutsk
commented
Version: 2.0.x-dev » 2.1.x-dev

Status: Fixed » Closed (fixed)

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