Problem

I am only able to recreate this issue when importing feeds that update feed items using feeds.module. The issue is feeds.module does not set a status property on the node it saves, which causes issues with node_access_acquire_grants(). I was originally going to file this issue with feeds.module, but after looking at the schema for node.module, the status column defaults to TRUE if not status is provided.

Proposed resolution

I have changed the condition in node_access_acquire_grants() to allow for a non-existent status property to allow a default grant to be assigned to nodes that will default to being set as published.

CommentFileSizeAuthor
#1 drupal-node_grants_condition-1796006-1.patch675 bytestom friedhof

Comments

tom friedhof’s picture

tom friedhof commented
StatusFileSize
new drupal-node_grants_condition-1796006-1.patch675 bytes

Here is the patch for this

tom friedhof’s picture

tom friedhof commented
Status: Active » Needs review

queuing up for testing

Status: Needs review » Needs work

The last submitted patch, drupal-node_grants_condition-1796006-1.patch, failed testing.

tom friedhof’s picture

tom friedhof commented
Status: Needs work » Needs review

#1: drupal-node_grants_condition-1796006-1.patch queued for re-testing.

Works locally for me running the latest 7.x branch.

Status: Needs review » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.