_node_types_build()'s return value is an object, not an array as stated in the function docs.

  * @return
- *   Associative array with two components:
+ *   An object including two properties:
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jeffschuler’s picture

Status: Active » Needs review
FileSize
659 bytes
jhodgdon’s picture

Good catch! Do you think it's good to say "an object including two properties" though? I think "with two properties" is just as good, if not better?

jhodgdon’s picture

Status: Needs review » Needs work
jeffschuler’s picture

Status: Needs work » Needs review
FileSize
659 bytes

Agreed. Thanks for improving.

jhodgdon’s picture

Status: Needs review » Needs work

This looks like the same patch as the previous version.

jeffschuler’s picture

Status: Needs work » Needs review
FileSize
654 bytes

Whoops, sorry!

jhodgdon’s picture

Status: Needs review » Reviewed & tested by the community

Thanks! I'll get this committed shortly.

jhodgdon’s picture

Status: Reviewed & tested by the community » Fixed

Thanks! Committed to 8.x/7.x.

Status: Fixed » Closed (fixed)

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