While reading the documentation of NodePermissions::buildPermissions() I noticed that the following documentation is incorrect, this contains the full NodeType object, not only the machine name:

/**
 * Builds a standard list of node permissions for a given type.
 *
 * @param \Drupal\node\Entity\NodeType $type
 *   The machine name of the node type.
 *
 * @return array
 *   An array of permission names and descriptions.
 */
protected function buildPermissions(NodeType $type) {
}

Let's do an overhaul of the entire documentation while we're at it.

Beta: This is only API docs. Commit any time.

CommentFileSizeAuthor
#2 2551415-1.patch1.2 KBpfrenssen

Comments

pfrenssen created an issue. See original summary.

pfrenssen’s picture

Status: Active » Needs review
StatusFileSize
new1.2 KB
jhodgdon’s picture

Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

Looks fine to me, thanks!

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed de605f8 and pushed to 8.0.x. Thanks!

  • alexpott committed de605f8 on 8.0.x
    Issue #2551415 by pfrenssen: Improve documentation of NodePermissions
    

Status: Fixed » Closed (fixed)

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