Using Drupal 7 version 7.39 when I save a node the users that have access to this node disappear from the grant tab.
I checked and the access is removed from node_access and nodeaccess tables.
I realize that the nodeaccess function nodeaccess_delete_user_reference it's called in the process. Inside this function user_load_multiple is called with a null value as parameter. In this case, user_load_multiple is returning all the users.
I did a small patch to overcome this. The path is also fixing the original problem of losing the data from the access tables.

CommentFileSizeAuthor
#9 nodeaccess-730664-3.patch1.22 KBPetrica
#5 nodeaccess-730664-2.patch1.22 KBPetrica
nodeaccess-730664.patch788 bytesdublutz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dublutz created an issue. See original summary.

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented
Version: 7.x-1.4 » 7.x-1.x-dev
Assigned: dublutz » Unassigned
Status: Patch (to be ported) » Needs review
Issue tags: -lost grant access node save

Patch to be ported is when it needs a backport or forward port from 7 to 6 for example.

carsonw’s picture

carsonw
handsome / brilliant
CreditAttribution: carsonw commented
Status: Needs review » Reviewed & tested by the community

Confirmed the same issue on another website that reported this bug to me. Reviewed, applied, and tested successfully.

april26’s picture

april26 CreditAttribution: april26 commented

I added the patch, but when the node is edited and saved, all the grant access names disappear. If anyone has a solution for this (or know of another module that allows view access by username), please let me know!

Petrica’s picture

Petrica CreditAttribution: Petrica at European Commission and European Union Institutions, Agencies and Bodies commented
FileSize
nodeaccess-730664-2.patch1.22 KB

For Drupal 7.41 this patch throws the following warnings:

Warning: array_flip(): Can only flip STRING and INTEGER values! in DrupalDefaultEntityController->load() (line 175 of includes/entity.inc).
Warning: array_flip(): Can only flip STRING and INTEGER values! in DrupalDefaultEntityController->cacheGet() (line 388 of includes/entity.inc).

I am attaching a patch to fix those.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 5: nodeaccess-730664-2.patch, failed testing.

The last submitted patch, 5: nodeaccess-730664-2.patch, failed testing.

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented

@Pertrica the testbot is failing on the short array syntax because Drupal 7 supports > PHP 5.2

Petrica’s picture

Petrica CreditAttribution: Petrica at European Commission and European Union Institutions, Agencies and Bodies commented
FileSize
nodeaccess-730664-3.patch1.22 KB

Added new patch for PHP 5.2 compatibility

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented
Status: Needs work » Needs review