https://git.drupal.org/project/adsense_consent

Primary tabs

Review of the 7.x-1.x branch (commit fa0197f):
  • The adsense_consent.module does not implement hook_help(). See https://www.drupal.org/docs/develop/documenting-your-project/module-docu... .
  • DrupalPractice has found some issues with your code, but could be false positives.
    
    FILE: ...01/web/vendor/drupal/pareviewsh/pareview_temp/adsense_consent.module
    --------------------------------------------------------------------------
    FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 4 LINES
    --------------------------------------------------------------------------
      17 | WARNING | Do not use drupal_add_css() in hook_init(), use
         |         | #attached for CSS and JS in your page/form callback or
         |         | in hook_page_build() instead
     163 | WARNING | Do not use drupal_add_js() in hook_page_build(), use
         |         | #attached for CSS and JS on the $page render array
         |         | instead
     191 | WARNING | Do not use drupal_add_js() in hook_page_build(), use
         |         | #attached for CSS and JS on the $page render array
         |         | instead
     201 | WARNING | Do not use drupal_add_js() in hook_page_build(), use
         |         | #attached for CSS and JS on the $page render array
         |         | instead
    --------------------------------------------------------------------------
    
    Time: 644ms; Memory: 6Mb
    
  • 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.