Video.js (HTML5 Video Player) - 7.x-1.x

Test status
StatusResult
Last receivedSun, 03/25/2012 - 11:47:38
Last requestedSun, 03/25/2012 - 11:48:03
Last testedSun, 03/25/2012 - 11:48:29
Test count6
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-1.x
ProjectVideo.js (HTML5 Video Player)
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 1 minor(s), 0 critical(s), and 8 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
1 minor(s), 0 critical(s), and 8 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandtheme/videojs.tpl.php106
MessageGroupFilenameLineFunctionStatus
Control statements should have one space between the control keyword and opening parenthesisOthervideojs.tpl.php23N/A
Control statements should have one space between the control keyword and opening parenthesisOthervideojs.tpl.php27N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthervideojs.tpl.php27N/A
Control statements should have one space between the control keyword and opening parenthesisOthervideojs.tpl.php36N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthervideojs.tpl.php36N/A
Control statements should have one space between the control keyword and opening parenthesisOthervideojs.tpl.php38N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthervideojs.tpl.php46N/A
Expandvideojs.install002
MessageGroupFilenameLineFunctionStatus
Use the matching Drupal theme functions, not raw HTML.Othervideojs.install18N/A
Use the matching Drupal theme functions, not raw HTML.Othervideojs.install23N/A

All

Test nameMinorCriticalNormal
Leafincludes/videojs.admin.inc000
Leafincludes/videojs.theme.inc000
Leafincludes/videojs.views.inc000
Leafincludes/videojs_style_plugin.inc000
Expandtheme/videojs.tpl.php106
MessageGroupFilenameLineFunctionStatus
Control statements should have one space between the control keyword and opening parenthesisOthervideojs.tpl.php23N/A
Control statements should have one space between the control keyword and opening parenthesisOthervideojs.tpl.php27N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthervideojs.tpl.php27N/A
Control statements should have one space between the control keyword and opening parenthesisOthervideojs.tpl.php36N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthervideojs.tpl.php36N/A
Control statements should have one space between the control keyword and opening parenthesisOthervideojs.tpl.php38N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthervideojs.tpl.php46N/A
Expandvideojs.install002
MessageGroupFilenameLineFunctionStatus
Use the matching Drupal theme functions, not raw HTML.Othervideojs.install18N/A
Use the matching Drupal theme functions, not raw HTML.Othervideojs.install23N/A
Leafvideojs.module000