\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.

CommentFileSizeAuthor
#2 drupal_2111209_2.patch65.15 KBXano
PASSED: [[SimpleTest]]: [MySQL] 58,968 pass(es). View
Members fund testing for the Drupal project. Drupal Association Learn more

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.