DrupalPractice has found some issues with your code, but could be false positives.

FILE: ...ite1101/web/vendor/drupal/pareviewsh/pareview_temp/hypothesis.module
--------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
--------------------------------------------------------------------------
 30 | WARNING | Do not use drupal_add_js() in hook_init(), use #attached
    |         | for CSS and JS in your page/form callback or in
    |         | hook_page_build() instead
 31 | WARNING | Do not use drupal_add_js() in hook_init(), use #attached
    |         | for CSS and JS in your page/form callback or in
    |         | hook_page_build() instead
 35 | WARNING | Do not use drupal_add_js() in hook_init(), use #attached
    |         | for CSS and JS in your page/form callback or in
    |         | hook_page_build() instead
--------------------------------------------------------------------------

Time: 263ms; Memory: 4Mb

Comments

rahul.shinde created an issue. See original summary.

rahul.shinde’s picture

Title: Coder Sniffer Review » DrupalPractice Review
Issue summary: View changes

  • rahul.shinde committed 02375f0 on 7.x-1.x
    Issue #3024695 by rahul.shinde: DrupalPractice Review hook_init replaced...
rahul.shinde’s picture

Assigned: rahul.shinde » Unassigned
Status: Active » Needs review
StatusFileSize
new1.42 KB

Attaching patch for review.

amarphule’s picture

Patch #4 works for me.

  • rahul.shinde committed 79daf92 on 7.x-1.x
    Issue #3024695 by rahul.shinde, amarphule: DrupalPractice Review
    
rahul.shinde’s picture

Status: Needs review » Fixed

Thanks, @amarphule, the patch is merged with the dev branch.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.