Part of meta-issue #1518116: [meta] Make Core pass Coder Review
Postponed because of #1326618: Clean up API docs for locale module
The locale module is moving much closer to being in good shape from the docs API perspective. Can someone please post here the output from a Coder review of the locale module?
I am unable to run Coder at the moment and would appreciate seeing what gets sniffed as I roll a patch for some code style issues I saw while working on the Locale docs review. Thanks in advance.
I am also removing Novice tag since Coder output still requires some more advance interpretation for possible positive code sniffs.
In test LocaleCompareTest.php, there is a member call called $translation_directory. This should be converted to a camel case variable like $translationDirectory.
In LocaleExportTest.php, there is a member called $admin_user. This too should be changed to a camel case variable. Same case with LocaleImportFunctionalTest.php.
Postponing till feature freeze. If you want to help in the meantime, please work on the blockers on the meta issue. Thanks!
Drupal is a registered trademark of Dries Buytaert.