Views Slideshow - 7.x-3.x

Test status
StatusResult
Last receivedSun, 10/28/2012 - 19:41:34
Last requestedSun, 10/28/2012 - 19:42:20
Last testedSun, 10/28/2012 - 19:43:07
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-3.x
ProjectViews Slideshow
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 5 minor(s), 10 critical(s), and 36 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
5 minor(s), 10 critical(s), and 36 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.inc203
MessageGroupFilenameLineFunctionStatus
in most cases, replace the string function with the drupal_ equivalent string functionsOtherviews_slideshow_cycle.theme.inc31N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherviews_slideshow_cycle.theme.inc34N/A
do not use mixed case (camelCase), use lower case and _Otherviews_slideshow_cycle.theme.inc103N/A
do not use mixed case (camelCase), use lower case and _Otherviews_slideshow_cycle.theme.inc104N/A
do not use mixed case (camelCase), use lower case and _Otherviews_slideshow_cycle.theme.inc105N/A
Expandcontrib/views_slideshow_cycle/views_slideshow_cycle.install102
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews_slideshow_cycle.install-1N/A
Missing parenthesis after function nameOtherviews_slideshow_cycle.install4N/A
else statements should begin on a new lineOtherviews_slideshow_cycle.install22N/A
Expandcontrib/views_slideshow_cycle/views_slideshow_cycle.module002
MessageGroupFilenameLineFunctionStatus
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow_cycle.module44N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow_cycle.module47N/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.inc85N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow_cycle.views_slideshow.inc129N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow_cycle.views_slideshow.inc417N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow_cycle.views_slideshow.inc422N/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.tpl.php001
MessageGroupFilenameLineFunctionStatus
the final ?> should be omitted from all code filesOtherviews-slideshow.tpl.php28N/A
Expandtheme/views_slideshow.theme.inc057
MessageGroupFilenameLineFunctionStatus
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
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.inc272N/A
missing space after commaOtherviews_slideshow.theme.inc272N/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.module202
MessageGroupFilenameLineFunctionStatus
Control statements should have one space between the control keyword and opening parenthesisOtherviews_slideshow.module173N/A
Missing parenthesis after function nameOtherviews_slideshow.module241N/A
Control statements should have one space between the control keyword and opening parenthesisOtherviews_slideshow.module320N/A
Missing parenthesis after function nameOtherviews_slideshow.module388N/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.inc203
MessageGroupFilenameLineFunctionStatus
in most cases, replace the string function with the drupal_ equivalent string functionsOtherviews_slideshow_cycle.theme.inc31N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherviews_slideshow_cycle.theme.inc34N/A
do not use mixed case (camelCase), use lower case and _Otherviews_slideshow_cycle.theme.inc103N/A
do not use mixed case (camelCase), use lower case and _Otherviews_slideshow_cycle.theme.inc104N/A
do not use mixed case (camelCase), use lower case and _Otherviews_slideshow_cycle.theme.inc105N/A
Expandcontrib/views_slideshow_cycle/views_slideshow_cycle.install102
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews_slideshow_cycle.install-1N/A
Missing parenthesis after function nameOtherviews_slideshow_cycle.install4N/A
else statements should begin on a new lineOtherviews_slideshow_cycle.install22N/A
Expandcontrib/views_slideshow_cycle/views_slideshow_cycle.module002
MessageGroupFilenameLineFunctionStatus
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow_cycle.module44N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow_cycle.module47N/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.inc85N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow_cycle.views_slideshow.inc129N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow_cycle.views_slideshow.inc417N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherviews_slideshow_cycle.views_slideshow.inc422N/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.tpl.php001
MessageGroupFilenameLineFunctionStatus
the final ?> should be omitted from all code filesOtherviews-slideshow.tpl.php28N/A
Expandtheme/views_slideshow.theme.inc057
MessageGroupFilenameLineFunctionStatus
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
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.inc272N/A
missing space after commaOtherviews_slideshow.theme.inc272N/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.module202
MessageGroupFilenameLineFunctionStatus
Control statements should have one space between the control keyword and opening parenthesisOtherviews_slideshow.module173N/A
Missing parenthesis after function nameOtherviews_slideshow.module241N/A
Control statements should have one space between the control keyword and opening parenthesisOtherviews_slideshow.module320N/A
Missing parenthesis after function nameOtherviews_slideshow.module388N/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