http://git.drupal.org/project/bootstrap_paragraphs.git

Primary tabs

Review of the 8.x-2.x branch (commit 1930e22):
  • Remove LICENSE.txt, it will be added by drupal.org packaging automatically.
  • Coder Sniffer has found some issues with your code (please check the Drupal coding standards). See attachment.
  • ESLint has found some issues with your code (please check the JavaScript coding standards). See attachment.
  • No automated test cases were found, did you consider writing 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: ...upal/pareviewsh/pareview_temp/css/bootstrap-paragraphs-image.min.css
--------------------------------------------------------------------------
FOUND 7 ERRORS AFFECTING 1 LINE
--------------------------------------------------------------------------
 7 | ERROR | [x] Expected 1 space before opening brace of class
   |       |     definition; 0 found
 7 | ERROR | [x] Opening brace should be the last content on the line
 7 | ERROR | [x] Expected 1 space after colon in style definition; 0
   |       |     found
 7 | ERROR | [x] Each style definition must be on a line by itself
 7 | ERROR | [ ] Style definitions must end with a semicolon
 7 | ERROR | [x] Expected 1 space after colon in style definition; 0
   |       |     found
 7 | ERROR | [x] Closing brace of class definition must be on new line
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 6 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...areviewsh/pareview_temp/css/bootstrap-paragraphs-columns-two.min.css
--------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------
 1 | WARNING | File appears to be minified and cannot be processed
--------------------------------------------------------------------------


FILE: ...al/pareviewsh/pareview_temp/css/bootstrap-paragraphs-columns.min.css
--------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------
 1 | WARNING | File appears to be minified and cannot be processed
--------------------------------------------------------------------------


FILE: ...upal/pareviewsh/pareview_temp/css/bootstrap-paragraphs-modal.min.css
--------------------------------------------------------------------------
FOUND 38 ERRORS AFFECTING 1 LINE
--------------------------------------------------------------------------
 7 | ERROR | [x] Expected 1 space before opening brace of class
   |       |     definition; 0 found
 7 | ERROR | [x] Opening brace should be the last content on the line
 7 | ERROR | [x] Expected 1 space after colon in style definition; 0
   |       |     found
 7 | ERROR | [x] Each style definition must be on a line by itself
 7 | ERROR | [x] Expected 1 space after colon in style definition; 0
   |       |     found
 7 | ERROR | [x] Each style definition must be on a line by itself
 7 | ERROR | [x] Expected 1 space after colon in style definition; 0
   |       |     found
 7 | ERROR | [x] Closing brace of class definition must be on new line
 7 | ERROR | [x] Expected 1 space before opening brace of class
   |       |     definition; 0 found
 7 | ERROR | [x] Opening brace should be the last content on the line
 7 | ERROR | [x] Each style definition must be on a line by itself
 7 | ERROR | [x] Expected 1 space after colon in style definition; 0
   |       |     found
 7 | ERROR | [x] Closing brace of class definition must be on new line
 7 | ERROR | [x] Multiple selectors should each be on a single line
 7 | ERROR | [x] Expected 1 space before opening brace of class
   |       |     definition; 0 found
 7 | ERROR | [x] Opening brace should be the last content on the line
 7 | ERROR | [x] Each style definition must be on a line by itself
 7 | ERROR | [x] Expected 1 space after colon in style definition; 0
   |       |     found
 7 | ERROR | [x] Closing brace of class definition must be on new line
 7 | ERROR | [x] Expected 1 space before opening brace of class
   |       |     definition; 0 found
 7 | ERROR | [x] Opening brace should be the last content on the line
 7 | ERROR | [x] Each style definition must be on a line by itself
 7 | ERROR | [x] Expected 1 space after colon in style definition; 0
   |       |     found
 7 | ERROR | [x] Closing brace of class definition must be on new line
 7 | ERROR | [x] Expected 1 space before opening brace of class
   |       |     definition; 0 found
 7 | ERROR | [x] Opening brace should be the last content on the line
 7 | ERROR | [x] Each style definition must be on a line by itself
 7 | ERROR | [x] Expected 1 space after colon in style definition; 0
   |       |     found
 7 | ERROR | [x] Closing brace of class definition must be on new line
 7 | ERROR | [x] Multiple selectors should each be on a single line
 7 | ERROR | [x] Expected 1 space before opening brace of class
   |       |     definition; 0 found
 7 | ERROR | [x] Opening brace should be the last content on the line
 7 | ERROR | [x] Each style definition must be on a line by itself
 7 | ERROR | [x] Expected 1 space after colon in style definition; 0
   |       |     found
 7 | ERROR | [x] Each style definition must be on a line by itself
 7 | ERROR | [ ] Style definitions must end with a semicolon
 7 | ERROR | [x] Expected 1 space after colon in style definition; 0
   |       |     found
 7 | ERROR | [x] Closing brace of class definition must be on new line
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 37 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: .../pareviewsh/pareview_temp/css/bootstrap-paragraphs-accordion.min.css
--------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------
 1 | WARNING | File appears to be minified and cannot be processed
--------------------------------------------------------------------------


FILE: ...dor/drupal/pareviewsh/pareview_temp/css/bootstrap-paragraphs.min.css
--------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------
 1 | WARNING | File appears to be minified and cannot be processed
--------------------------------------------------------------------------


FILE: ...rupal/pareviewsh/pareview_temp/css/bootstrap-paragraphs-tabs.min.css
--------------------------------------------------------------------------
FOUND 5 ERRORS AFFECTING 1 LINE
--------------------------------------------------------------------------
 7 | ERROR | [x] Expected 1 space before opening brace of class
   |       |     definition; 0 found
 7 | ERROR | [x] Opening brace should be the last content on the line
 7 | ERROR | [ ] Style definitions must end with a semicolon
 7 | ERROR | [x] Expected 1 space after colon in style definition; 0
   |       |     found
 7 | ERROR | [x] Closing brace of class definition must be on new line
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...pal/pareviewsh/pareview_temp/css/bootstrap-paragraphs-colors.min.css
--------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------
 1 | WARNING | File appears to be minified and cannot be processed
--------------------------------------------------------------------------


FILE: ...eviewsh/pareview_temp/css/bootstrap-paragraphs-columns-three.min.css
--------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------
 1 | WARNING | File appears to be minified and cannot be processed
--------------------------------------------------------------------------


FILE: ...l/pareviewsh/pareview_temp/css/bootstrap-paragraphs-carousel.min.css
--------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------
 1 | WARNING | File appears to be minified and cannot be processed
--------------------------------------------------------------------------

Time: 1.04 secs; Memory: 6Mb
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 6, col 2, Error - Use the function form of 'use strict'. (strict)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 8, col 26, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 17, col 7, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 18, col 13, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 18, col 54, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 27, col 7, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 28, col 43, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 28, col 66, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 37, col 29, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 48, col 7, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 50, col 39, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 53, col 44, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 57, col 28, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 69, col 7, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 71, col 39, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 75, col 28, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 88, col 41, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 95, col 41, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 104, col 26, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 107, col 18, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 107, col 36, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 108, col 27, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 109, col 25, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 119, col 27, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 122, col 18, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 122, col 36, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 123, col 27, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 124, col 25, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 133, col 22, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 134, col 20, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 134, col 36, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 137, col 20, Error - Strings must use singlequote. (quotes)
/var/vhosts/c214000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/js/bootstrap-paragraphs-accordion.js: line 137, col 36, Error - Strings must use singlequote. (quotes)

33 problems