Views Slideshow - 7.x-3.0

Test status
StatusResult
Last receivedSun, 10/28/2012 - 19:41:34
Last requestedSun, 10/28/2012 - 19:42:04
Last testedSun, 10/28/2012 - 19:43:06
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-3.0
ProjectViews Slideshow
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 16 minor(s), 10 critical(s), and 33 normal(s); [[SimpleTest]]: [MySQL] 0 pass(es).
Details

MySQL

  1. Setup environment
  2. Detect an invalid patch URL
  3. Detect invalid repository information
  4. Detect invalid patch format
  5. Detect a non-applicable patch
  6. Detect invalid PHP syntax
  7. Detect a Drupal installation failure
  8. Detect a test run failure
  9. Detect a failing test
  10. Complete test suite with all tests passing

Code review

  1. Setup environment
  2. Detect an invalid patch URL
  3. Detect invalid repository information
  4. Detect invalid patch format
  5. Detect a non-applicable patch
  6. Detect invalid PHP syntax
  7. Detect a review run failure
  8. Detect coder flags
    Ensure that your code follows the Drupal standard and passes a coder review.
  9. Complete coder review with all passed
16 minor(s), 10 critical(s), and 33 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandcontrib/views_slideshow_cycle/theme/views-slideshow-cycle-main-frame-row-item.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews-slideshow-cycle-main-frame-row-item.tpl.php-1N/A
Expandcontrib/views_slideshow_cycle/theme/views-slideshow-cycle-main-frame-row.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews-slideshow-cycle-main-frame-row.tpl.php-1N/A
Expandcontrib/views_slideshow_cycle/theme/views-slideshow-cycle-main-frame.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews-slideshow-cycle-main-frame.tpl.php-1N/A
Expandcontrib/views_slideshow_cycle/theme/views_slideshow_cycle.theme.inc503
MessageGroupFilenameLineFunctionStatus
in most cases, replace the string function with the drupal_ equivalent string functionsOtherviews_slideshow_cycle.theme.inc32N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherviews_slideshow_cycle.theme.inc35N/A
There should be no trailing spacesOtherviews_slideshow_cycle.theme.inc71N/A
There should be no trailing spacesOtherviews_slideshow_cycle.theme.inc94N/A
do not use mixed case (camelCase), use lower case and _Otherviews_slideshow_cycle.theme.inc99N/A
do not use mixed case (camelCase), use lower case and _Otherviews_slideshow_cycle.theme.inc100N/A
do not use mixed case (camelCase), use lower case and _Otherviews_slideshow_cycle.theme.inc101N/A
There should be no trailing spacesOtherviews_slideshow_cycle.theme.inc179N/A
Expandcontrib/views_slideshow_cycle/views_slideshow_cycle.module202
MessageGroupFilenameLineFunctionStatus
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow_cycle.module40N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow_cycle.module43N/A
There should be no trailing spacesOtherviews_slideshow_cycle.module79N/A
There should be no trailing spacesOtherviews_slideshow_cycle.module81N/A
Expandcontrib/views_slideshow_cycle/views_slideshow_cycle.views_slideshow.inc004
MessageGroupFilenameLineFunctionStatus
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow_cycle.views_slideshow.inc75N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow_cycle.views_slideshow.inc119N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow_cycle.views_slideshow.inc392N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow_cycle.views_slideshow.inc397N/A
Expandtheme/views-slideshow-controls-text-next.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews-slideshow-controls-text-next.tpl.php-1N/A
Expandtheme/views-slideshow-controls-text-pause.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews-slideshow-controls-text-pause.tpl.php-1N/A
Expandtheme/views-slideshow-controls-text-previous.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews-slideshow-controls-text-previous.tpl.php-1N/A
Expandtheme/views-slideshow-controls-text.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews-slideshow-controls-text.tpl.php-1N/A
Expandtheme/views-slideshow-pager-field-field.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews-slideshow-pager-field-field.tpl.php-1N/A
Expandtheme/views-slideshow-pager-field-item.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews-slideshow-pager-field-item.tpl.php-1N/A
Expandtheme/views-slideshow-pager-fields.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews-slideshow-pager-fields.tpl.php-1N/A
Expandtheme/views-slideshow-slide-counter.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews-slideshow-slide-counter.tpl.php-1N/A
Expandtheme/views_slideshow.theme.inc257
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherviews_slideshow.theme.inc27N/A
Potential problem: drupal_set_message() only accepts filtered text, be sure all !placeholders for $variables in t() are fully sanitized using check_plain(), filter_xss() or similar. (Drupal Docs)Otherviews_slideshow.theme.inc34N/A
Use preg_replace_callback() instead of the 'e' modifier to preg_replace()Otherviews_slideshow.theme.inc73N/A
missing space after commaOtherviews_slideshow.theme.inc73N/A
Use preg_replace_callback() instead of the 'e' modifier to preg_replace()Otherviews_slideshow.theme.inc76N/A
missing space after commaOtherviews_slideshow.theme.inc76N/A
Use preg_replace_callback() instead of the 'e' modifier to preg_replace()Otherviews_slideshow.theme.inc187N/A
missing space after commaOtherviews_slideshow.theme.inc187N/A
There should be no trailing spacesOtherviews_slideshow.theme.inc218N/A
do not use mixed case (camelCase), use lower case and _Otherviews_slideshow.theme.inc223N/A
do not use mixed case (camelCase), use lower case and _Otherviews_slideshow.theme.inc224N/A
do not use mixed case (camelCase), use lower case and _Otherviews_slideshow.theme.inc225N/A
Use preg_replace_callback() instead of the 'e' modifier to preg_replace()Otherviews_slideshow.theme.inc275N/A
missing space after commaOtherviews_slideshow.theme.inc275N/A
Expandviews_slideshow.api.php001
MessageGroupFilenameLineFunctionStatus
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow.api.php54N/A
Expandviews_slideshow.module702
MessageGroupFilenameLineFunctionStatus
Control statements should have one space between the control keyword and opening parenthesisOtherviews_slideshow.module161N/A
There should be no trailing spacesOtherviews_slideshow.module198N/A
There should be no trailing spacesOtherviews_slideshow.module216N/A
Missing parenthesis after function nameOtherviews_slideshow.module229N/A
Control statements should have one space between the control keyword and opening parenthesisOtherviews_slideshow.module308N/A
There should be no trailing spacesOtherviews_slideshow.module317N/A
Missing parenthesis after function nameOtherviews_slideshow.module376N/A
There should be no trailing spacesOtherviews_slideshow.module435N/A
There should be no trailing spacesOtherviews_slideshow.module437N/A
Expandviews_slideshow_plugin_style_slideshow.inc053
MessageGroupFilenameLineFunctionStatus
Potential problem: FAPI elements '#title' and '#description' only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Otherviews_slideshow_plugin_style_slideshow.inc96N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow_plugin_style_slideshow.inc112N/A
Control statements should have one space between the control keyword and opening parenthesisOtherviews_slideshow_plugin_style_slideshow.inc155N/A
Control statements should have one space between the control keyword and opening parenthesisOtherviews_slideshow_plugin_style_slideshow.inc166N/A
Potential problem: FAPI elements '#title' and '#description' only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Otherviews_slideshow_plugin_style_slideshow.inc183N/A
Potential problem: FAPI elements '#title' and '#description' only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Otherviews_slideshow_plugin_style_slideshow.inc185N/A
Potential problem: FAPI elements '#title' and '#description' only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Otherviews_slideshow_plugin_style_slideshow.inc201N/A
Potential problem: FAPI elements '#title' and '#description' only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Otherviews_slideshow_plugin_style_slideshow.inc204N/A

All

Test nameMinorCriticalNormal
Expandcontrib/views_slideshow_cycle/theme/views-slideshow-cycle-main-frame-row-item.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews-slideshow-cycle-main-frame-row-item.tpl.php-1N/A
Expandcontrib/views_slideshow_cycle/theme/views-slideshow-cycle-main-frame-row.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews-slideshow-cycle-main-frame-row.tpl.php-1N/A
Expandcontrib/views_slideshow_cycle/theme/views-slideshow-cycle-main-frame.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews-slideshow-cycle-main-frame.tpl.php-1N/A
Leafcontrib/views_slideshow_cycle/theme/views-slideshow-cycle.tpl.php000
Expandcontrib/views_slideshow_cycle/theme/views_slideshow_cycle.theme.inc503
MessageGroupFilenameLineFunctionStatus
in most cases, replace the string function with the drupal_ equivalent string functionsOtherviews_slideshow_cycle.theme.inc32N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherviews_slideshow_cycle.theme.inc35N/A
There should be no trailing spacesOtherviews_slideshow_cycle.theme.inc71N/A
There should be no trailing spacesOtherviews_slideshow_cycle.theme.inc94N/A
do not use mixed case (camelCase), use lower case and _Otherviews_slideshow_cycle.theme.inc99N/A
do not use mixed case (camelCase), use lower case and _Otherviews_slideshow_cycle.theme.inc100N/A
do not use mixed case (camelCase), use lower case and _Otherviews_slideshow_cycle.theme.inc101N/A
There should be no trailing spacesOtherviews_slideshow_cycle.theme.inc179N/A
Expandcontrib/views_slideshow_cycle/views_slideshow_cycle.module202
MessageGroupFilenameLineFunctionStatus
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow_cycle.module40N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow_cycle.module43N/A
There should be no trailing spacesOtherviews_slideshow_cycle.module79N/A
There should be no trailing spacesOtherviews_slideshow_cycle.module81N/A
Expandcontrib/views_slideshow_cycle/views_slideshow_cycle.views_slideshow.inc004
MessageGroupFilenameLineFunctionStatus
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow_cycle.views_slideshow.inc75N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow_cycle.views_slideshow.inc119N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow_cycle.views_slideshow.inc392N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow_cycle.views_slideshow.inc397N/A
Expandtheme/views-slideshow-controls-text-next.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews-slideshow-controls-text-next.tpl.php-1N/A
Expandtheme/views-slideshow-controls-text-pause.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews-slideshow-controls-text-pause.tpl.php-1N/A
Expandtheme/views-slideshow-controls-text-previous.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews-slideshow-controls-text-previous.tpl.php-1N/A
Expandtheme/views-slideshow-controls-text.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews-slideshow-controls-text.tpl.php-1N/A
Expandtheme/views-slideshow-pager-field-field.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews-slideshow-pager-field-field.tpl.php-1N/A
Expandtheme/views-slideshow-pager-field-item.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews-slideshow-pager-field-item.tpl.php-1N/A
Expandtheme/views-slideshow-pager-fields.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews-slideshow-pager-fields.tpl.php-1N/A
Expandtheme/views-slideshow-slide-counter.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews-slideshow-slide-counter.tpl.php-1N/A
Leaftheme/views-slideshow.tpl.php000
Expandtheme/views_slideshow.theme.inc257
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherviews_slideshow.theme.inc27N/A
Potential problem: drupal_set_message() only accepts filtered text, be sure all !placeholders for $variables in t() are fully sanitized using check_plain(), filter_xss() or similar. (Drupal Docs)Otherviews_slideshow.theme.inc34N/A
Use preg_replace_callback() instead of the 'e' modifier to preg_replace()Otherviews_slideshow.theme.inc73N/A
missing space after commaOtherviews_slideshow.theme.inc73N/A
Use preg_replace_callback() instead of the 'e' modifier to preg_replace()Otherviews_slideshow.theme.inc76N/A
missing space after commaOtherviews_slideshow.theme.inc76N/A
Use preg_replace_callback() instead of the 'e' modifier to preg_replace()Otherviews_slideshow.theme.inc187N/A
missing space after commaOtherviews_slideshow.theme.inc187N/A
There should be no trailing spacesOtherviews_slideshow.theme.inc218N/A
do not use mixed case (camelCase), use lower case and _Otherviews_slideshow.theme.inc223N/A
do not use mixed case (camelCase), use lower case and _Otherviews_slideshow.theme.inc224N/A
do not use mixed case (camelCase), use lower case and _Otherviews_slideshow.theme.inc225N/A
Use preg_replace_callback() instead of the 'e' modifier to preg_replace()Otherviews_slideshow.theme.inc275N/A
missing space after commaOtherviews_slideshow.theme.inc275N/A
Expandviews_slideshow.api.php001
MessageGroupFilenameLineFunctionStatus
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow.api.php54N/A
Expandviews_slideshow.module702
MessageGroupFilenameLineFunctionStatus
Control statements should have one space between the control keyword and opening parenthesisOtherviews_slideshow.module161N/A
There should be no trailing spacesOtherviews_slideshow.module198N/A
There should be no trailing spacesOtherviews_slideshow.module216N/A
Missing parenthesis after function nameOtherviews_slideshow.module229N/A
Control statements should have one space between the control keyword and opening parenthesisOtherviews_slideshow.module308N/A
There should be no trailing spacesOtherviews_slideshow.module317N/A
Missing parenthesis after function nameOtherviews_slideshow.module376N/A
There should be no trailing spacesOtherviews_slideshow.module435N/A
There should be no trailing spacesOtherviews_slideshow.module437N/A
Leafviews_slideshow.views.inc000
Expandviews_slideshow_plugin_style_slideshow.inc053
MessageGroupFilenameLineFunctionStatus
Potential problem: FAPI elements '#title' and '#description' only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Otherviews_slideshow_plugin_style_slideshow.inc96N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow_plugin_style_slideshow.inc112N/A
Control statements should have one space between the control keyword and opening parenthesisOtherviews_slideshow_plugin_style_slideshow.inc155N/A
Control statements should have one space between the control keyword and opening parenthesisOtherviews_slideshow_plugin_style_slideshow.inc166N/A
Potential problem: FAPI elements '#title' and '#description' only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Otherviews_slideshow_plugin_style_slideshow.inc183N/A
Potential problem: FAPI elements '#title' and '#description' only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Otherviews_slideshow_plugin_style_slideshow.inc185N/A
Potential problem: FAPI elements '#title' and '#description' only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Otherviews_slideshow_plugin_style_slideshow.inc201N/A
Potential problem: FAPI elements '#title' and '#description' only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Otherviews_slideshow_plugin_style_slideshow.inc204N/A