https://git.drupal.org/project/stomt.git 7.x-1.x

Primary tabs

Review of the 7.x-1.x branch (commit c65fbcd):
  • Your README.md does not follow best practices (headings need to be uppercase). See https://www.drupal.org/node/2181737 .
    • The INTRODUCTION section is missing.
    • The REQUIREMENTS section is missing.
    • The INSTALLATION section is missing.
    • The CONFIGURATION section is missing.
  • Coder Sniffer has found some issues with your code (please check the Drupal coding standards). See attachment.
  • No automated test cases were found, did you consider writing Simpletests or PHPUnit tests? This is not a requirement but encouraged for professional software development.
This automated report was generated with PAReview.sh, your friendly project application review script.

FILE: ...000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/STOMT.module
--------------------------------------------------------------------------
FOUND 4 ERRORS AND 5 WARNINGS AFFECTING 9 LINES
--------------------------------------------------------------------------
 13 | WARNING | Hook implementations should not duplicate @param
    |         | documentation
 15 | WARNING | Hook implementations should not duplicate @param
    |         | documentation
 18 | ERROR   | Invalid function name, expected s_to_mt_help but found
    |         | STOMT_help
 23 | WARNING | Code after RETURN statement cannot be executed
 31 | WARNING | Hook implementations should not duplicate @param
    |         | documentation
 34 | ERROR   | Invalid function name, expected s_to_mt_page_alter but
    |         | found STOMT_page_alter
 67 | WARNING | Hook implementations should not duplicate @return
    |         | documentation
 70 | ERROR   | Invalid function name, expected s_to_mt_menu but found
    |         | STOMT_menu
 90 | ERROR   | Invalid function name, expected s_to_mt_form but found
    |         | STOMT_form
--------------------------------------------------------------------------

Time: 432ms; Memory: 4Mb