_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:
Files: 
CommentFileSizeAuthor
#6 drupal-_node_types_build_doc_arr_obj-1598508-6.patch654 bytesjeffschuler
PASSED: [[SimpleTest]]: [MySQL] 36,675 pass(es). View
#4 drupal-_node_types_build_doc_arr_obj-1598508-1.patch659 bytesjeffschuler
PASSED: [[SimpleTest]]: [MySQL] 36,675 pass(es). View
#1 drupal-_node_types_build_doc_arr_obj-1598508-1.patch659 bytesjeffschuler
PASSED: [[SimpleTest]]: [MySQL] 36,655 pass(es). View

Comments

jeffschuler’s picture

Status: Active » Needs review
FileSize
659 bytes
PASSED: [[SimpleTest]]: [MySQL] 36,655 pass(es). View
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
PASSED: [[SimpleTest]]: [MySQL] 36,675 pass(es). View

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
PASSED: [[SimpleTest]]: [MySQL] 36,675 pass(es). View

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.