DataTables - 7.x-1.x

Test status
StatusResult
Last receivedWed, 03/07/2012 - 09:47:37
Last requestedWed, 03/07/2012 - 09:48:03
Last testedWed, 03/07/2012 - 09:50:12
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-1.x
ProjectDataTables
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 44 minor(s), 1 critical(s), and 15 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
44 minor(s), 1 critical(s), and 15 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expanddatatables.install001
MessageGroupFilenameLineFunctionStatus
Use the matching Drupal theme functions, not raw HTML.Otherdatatables.install23N/A
Expanddatatables.module2805
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherdatatables.module35N/A
There should be no trailing spacesOtherdatatables.module51N/A
There should be no trailing spacesOtherdatatables.module52N/A
There should be no trailing spacesOtherdatatables.module53N/A
There should be no trailing spacesOtherdatatables.module69N/A
There should be no trailing spacesOtherdatatables.module72N/A
There should be no trailing spacesOtherdatatables.module91N/A
Function documentation should be less than 80 characters per line.Otherdatatables.module91N/A
There should be no trailing spacesOtherdatatables.module92N/A
There should be no trailing spacesOtherdatatables.module93N/A
Function documentation should be less than 80 characters per line.Otherdatatables.module100N/A
There should be no trailing spacesOtherdatatables.module101N/A
Function documentation should be less than 80 characters per line.Otherdatatables.module109N/A
Function documentation should be less than 80 characters per line.Otherdatatables.module123N/A
There should be no trailing spacesOtherdatatables.module130N/A
@see should always be followed by a filename, a URL, class/interface name (optionally including method), or a function name including ().Otherdatatables.module136N/A
There should be no trailing spacesOtherdatatables.module146N/A
There should be no trailing spacesOtherdatatables.module155N/A
There should be no trailing spacesOtherdatatables.module165N/A
There should be no trailing spacesOtherdatatables.module173N/A
There should be no trailing spacesOtherdatatables.module193N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherdatatables.module213N/A
There should be no trailing spacesOtherdatatables.module235N/A
There should be no trailing spacesOtherdatatables.module240N/A
There should be no trailing spacesOtherdatatables.module243N/A
Control statements should have one space between the control keyword and opening parenthesisOtherdatatables.module245N/A
There should be no trailing spacesOtherdatatables.module248N/A
There should be no trailing spacesOtherdatatables.module251N/A
There should be no trailing spacesOtherdatatables.module253N/A
There should be no trailing spacesOtherdatatables.module255N/A
There should be no trailing spacesOtherdatatables.module261N/A
There should be no trailing spacesOtherdatatables.module265N/A
There should be no trailing spacesOtherdatatables.module266N/A
Expandviews/datatables_style_plugin.inc1619
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherdatatables_style_plugin.inc18N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc46N/A
@see should always be followed by a filename, a URL, class/interface name (optionally including method), or a function name including ().Otherdatatables_style_plugin.inc48N/A
@see references should be separated by "," followed by a single space and with no trailing punctuationOtherdatatables_style_plugin.inc48N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc63N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherdatatables_style_plugin.inc79N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc83N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherdatatables_style_plugin.inc86N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc88N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc90N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc97N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc104N/A
Use the matching Drupal theme functions, not raw HTML.Otherdatatables_style_plugin.inc109N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc111N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherdatatables_style_plugin.inc114N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherdatatables_style_plugin.inc120N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc123N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc129N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherdatatables_style_plugin.inc132N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc135N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherdatatables_style_plugin.inc138N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc143N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherdatatables_style_plugin.inc147N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc151N/A
Control statements should have one space between the control keyword and opening parenthesisOtherdatatables_style_plugin.inc152N/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.Otherdatatables_style_plugin.inc155N/A

All

Test nameMinorCriticalNormal
Expanddatatables.install001
MessageGroupFilenameLineFunctionStatus
Use the matching Drupal theme functions, not raw HTML.Otherdatatables.install23N/A
Expanddatatables.module2805
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherdatatables.module35N/A
There should be no trailing spacesOtherdatatables.module51N/A
There should be no trailing spacesOtherdatatables.module52N/A
There should be no trailing spacesOtherdatatables.module53N/A
There should be no trailing spacesOtherdatatables.module69N/A
There should be no trailing spacesOtherdatatables.module72N/A
There should be no trailing spacesOtherdatatables.module91N/A
Function documentation should be less than 80 characters per line.Otherdatatables.module91N/A
There should be no trailing spacesOtherdatatables.module92N/A
There should be no trailing spacesOtherdatatables.module93N/A
Function documentation should be less than 80 characters per line.Otherdatatables.module100N/A
There should be no trailing spacesOtherdatatables.module101N/A
Function documentation should be less than 80 characters per line.Otherdatatables.module109N/A
Function documentation should be less than 80 characters per line.Otherdatatables.module123N/A
There should be no trailing spacesOtherdatatables.module130N/A
@see should always be followed by a filename, a URL, class/interface name (optionally including method), or a function name including ().Otherdatatables.module136N/A
There should be no trailing spacesOtherdatatables.module146N/A
There should be no trailing spacesOtherdatatables.module155N/A
There should be no trailing spacesOtherdatatables.module165N/A
There should be no trailing spacesOtherdatatables.module173N/A
There should be no trailing spacesOtherdatatables.module193N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherdatatables.module213N/A
There should be no trailing spacesOtherdatatables.module235N/A
There should be no trailing spacesOtherdatatables.module240N/A
There should be no trailing spacesOtherdatatables.module243N/A
Control statements should have one space between the control keyword and opening parenthesisOtherdatatables.module245N/A
There should be no trailing spacesOtherdatatables.module248N/A
There should be no trailing spacesOtherdatatables.module251N/A
There should be no trailing spacesOtherdatatables.module253N/A
There should be no trailing spacesOtherdatatables.module255N/A
There should be no trailing spacesOtherdatatables.module261N/A
There should be no trailing spacesOtherdatatables.module265N/A
There should be no trailing spacesOtherdatatables.module266N/A
Leafviews/datatables-view.tpl.php000
Leafviews/datatables.views.inc000
Expandviews/datatables_style_plugin.inc1619
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherdatatables_style_plugin.inc18N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc46N/A
@see should always be followed by a filename, a URL, class/interface name (optionally including method), or a function name including ().Otherdatatables_style_plugin.inc48N/A
@see references should be separated by "," followed by a single space and with no trailing punctuationOtherdatatables_style_plugin.inc48N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc63N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherdatatables_style_plugin.inc79N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc83N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherdatatables_style_plugin.inc86N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc88N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc90N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc97N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc104N/A
Use the matching Drupal theme functions, not raw HTML.Otherdatatables_style_plugin.inc109N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc111N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherdatatables_style_plugin.inc114N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherdatatables_style_plugin.inc120N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc123N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc129N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherdatatables_style_plugin.inc132N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc135N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherdatatables_style_plugin.inc138N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc143N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherdatatables_style_plugin.inc147N/A
There should be no trailing spacesOtherdatatables_style_plugin.inc151N/A
Control statements should have one space between the control keyword and opening parenthesisOtherdatatables_style_plugin.inc152N/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.Otherdatatables_style_plugin.inc155N/A