Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
node system
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
16 Jul 2015 at 13:11 UTC
Updated:
30 Jul 2015 at 17:04 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
webflo commentedComment #2
dawehnerPerfect
Comment #3
xjmComment #4
xjmSince this method basically just wraps
NodeGrantDatabaseStorageInterface::write(), I double-checked that method too. That method still has these parameters. However,NodeGrantDatabaseStorageInterface::write()does this:So since it is providing defaults, looks like this patch is correct. I also double-checked that the base implementation has the correct signature.
See also #2473021: Deprecate NodeAccessControlHandlerInterface::writeGrants() for removal in Drupal 9.0.x.
This issue only changes documentation, so per https://www.drupal.org/core/beta-changes, this can be completed any time during the Drupal 8 beta phase. Committed and pushed to 8.0.x. Thanks @webflo for catching this!