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

×

Error message

User warning: The following module is missing from the file system: xautoload. For information about how to fix this, see the documentation page. in _drupal_trigger_error_with_delayed_logging() (line 1143 of /var/vhosts/c214000000/site1101/web/public/includes/bootstrap.inc).

Primary tabs

Review of the 7.x-1.x branch (commit b4834d9):
  • The adsense_consent.module does not implement hook_help(). See https://www.drupal.org/docs/develop/documenting-your-project/module-docu... .
  • Coder Sniffer has found some issues with your code (please check the Drupal coding standards). See attachment.
  • 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
     150 | WARNING | Do not use drupal_add_js() in hook_page_build(), use
         |         | #attached for CSS and JS on the $page render array
         |         | instead
     160 | WARNING | Do not use drupal_add_js() in hook_page_build(), use
         |         | #attached for CSS and JS on the $page render array
         |         | instead
     171 | WARNING | Do not use drupal_add_js() in hook_page_build(), use
         |         | #attached for CSS and JS on the $page render array
         |         | instead
    --------------------------------------------------------------------------
    
    Time: 304ms; 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.

FILE: ...01/web/vendor/drupal/pareviewsh/pareview_temp/adsense_consent.module
--------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------
 251 | WARNING | Format should be "* Implements hook_foo().", "*
     |         | Implements hook_foo_BAR_ID_bar() for xyz_bar().",, "*
     |         | Implements hook_foo_BAR_ID_bar() for
     |         | xyz-bar.html.twig.", "* Implements hook_foo_BAR_ID_bar()
     |         | for xyz-bar.tpl.php.", or "* Implements
     |         | hook_foo_BAR_ID_bar() for block templates."
--------------------------------------------------------------------------


FILE: ...web/vendor/drupal/pareviewsh/pareview_temp/adsense_consent.admin.inc
--------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------
 286 | ERROR | [x] Inline control structures are not allowed
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------

Time: 605ms; Memory: 6Mb