\Drupal\Core\Form\FormInterface::getFormID() does not comply with the coding standards, as we do not use uppercase acronyms in camelcased names. The method should be named getFormId() instead.

Files: 
CommentFileSizeAuthor
#2 drupal_2111209_2.patch65.15 KBXano
PASSED: [[SimpleTest]]: [MySQL] 58,968 pass(es). View

Comments

Xano’s picture

Some forms already use getFormId() rather than getFormID(), such as \Drupal\taxonomy\Form\OverviewTerms.

Xano’s picture

Assigned: Xano » Unassigned
Status: Active » Needs review
FileSize
65.15 KB
PASSED: [[SimpleTest]]: [MySQL] 58,968 pass(es). View
amateescu’s picture

Status: Needs review » Reviewed & tested by the community

Yup.

alexpott’s picture

Assigned: Unassigned » jhodgdon
Status: Reviewed & tested by the community » Fixed

Committed b201c05 and pushed to 8.x. Thanks!

jhodgdon’s picture

Assigned: jhodgdon » Unassigned

Not sure why you assigned this to me? Our OO naming standards already say to do this for acronyms:
https://drupal.org/node/608152#naming

tstoeckler’s picture

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