Seems results of that call doesn't used, and parameters didn't modified be reference.

CommentFileSizeAuthor
#1 remove-useless-node_type_get_type-call-1886870.patch657 bytesIRuslan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

IRuslan’s picture

IRuslan CreditAttribution: IRuslan commented
FileSize
remove-useless-node_type_get_type-call-1886870.patch657 bytes

Patch is added.

arknoll’s picture

arknoll CreditAttribution: arknoll commented
Status: Needs review » Reviewed & tested by the community

Patch does remove an unneeded call to node_type_get_type.

David_Rothstein’s picture

David_Rothstein CreditAttribution: David_Rothstein commented
Status: Reviewed & tested by the community » Fixed

Looks good, and looks like the equivalent code in Drupal 8 does not have this issue.

Committed to 7.x - thanks! http://drupalcode.org/project/drupal.git/commit/6bcb0a3

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) CreditAttribution: commented
Issue summary: View changes

Fix typo.