Allocated Seating - 6.x-1.x

Test status
StatusResult
Last receivedMon, 08/13/2012 - 20:49:15
Last requestedMon, 08/13/2012 - 20:50:01
Last testedMon, 08/13/2012 - 20:50:10
Test count8
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier6.x-1.x
ProjectAllocated Seating
LinkProject information
SummaryPASSED: [[SimpleTest]]: [MySQL] 0 pass(es); [[Coder]]: [Code review] 25 minor(s), 1 critical(s), and 2 normal(s).
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
25 minor(s), 1 critical(s), and 2 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandallocated_seating.module1412
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherallocated_seating.module20N/A
There should be no trailing spacesOtherallocated_seating.module44N/A
There should be no trailing spacesOtherallocated_seating.module103N/A
There should be no trailing spacesOtherallocated_seating.module235N/A
There should be no trailing spacesOtherallocated_seating.module273N/A
The $string argument to t() should not begin or end with a space. (Drupal Docs)Otherallocated_seating.module341N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherallocated_seating.module341N/A
There should be no trailing spacesOtherallocated_seating.module353N/A
There should be no trailing spacesOtherallocated_seating.module402N/A
There should be no trailing spacesOtherallocated_seating.module410N/A
There should be no trailing spacesOtherallocated_seating.module419N/A
There should be no trailing spacesOtherallocated_seating.module431N/A
There should be no trailing spacesOtherallocated_seating.module526N/A
There should be no trailing spacesOtherallocated_seating.module626N/A
Potential problem: use the Form API to prevent against CSRF attacks. If you need to use $_POST variables, ensure they are fully sanitized if displayed by using check_plain(), filter_xss() or similar. (Drupal Docs)Otherallocated_seating.module661N/A
There should be no trailing spacesOtherallocated_seating.module683N/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)Otherallocated_seating.module811N/A

All

Test nameMinorCriticalNormal
Expandallocated_seating.module1412
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherallocated_seating.module20N/A
There should be no trailing spacesOtherallocated_seating.module44N/A
There should be no trailing spacesOtherallocated_seating.module103N/A
There should be no trailing spacesOtherallocated_seating.module235N/A
There should be no trailing spacesOtherallocated_seating.module273N/A
The $string argument to t() should not begin or end with a space. (Drupal Docs)Otherallocated_seating.module341N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherallocated_seating.module341N/A
There should be no trailing spacesOtherallocated_seating.module353N/A
There should be no trailing spacesOtherallocated_seating.module402N/A
There should be no trailing spacesOtherallocated_seating.module410N/A
There should be no trailing spacesOtherallocated_seating.module419N/A
There should be no trailing spacesOtherallocated_seating.module431N/A
There should be no trailing spacesOtherallocated_seating.module526N/A
There should be no trailing spacesOtherallocated_seating.module626N/A
Potential problem: use the Form API to prevent against CSRF attacks. If you need to use $_POST variables, ensure they are fully sanitized if displayed by using check_plain(), filter_xss() or similar. (Drupal Docs)Otherallocated_seating.module661N/A
There should be no trailing spacesOtherallocated_seating.module683N/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)Otherallocated_seating.module811N/A
Leafallocated_seating_example/allocated_seating_example.install000
Expandallocated_seating_example/allocated_seating_example.module700
MessageGroupFilenameLineFunctionStatus
put a space between the asterisk and the comment textOtherallocated_seating_example.module19N/A
put a space between the asterisk and the comment textOtherallocated_seating_example.module109N/A
Potential problem: use the Form API to prevent against CSRF attacks. If you need to use $_POST variables, ensure they are fully sanitized if displayed by using check_plain(), filter_xss() or similar. (Drupal Docs)Otherallocated_seating_example.module132N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherallocated_seating_example.module147N/A
Potential problem: use the Form API to prevent against CSRF attacks. If you need to use $_POST variables, ensure they are fully sanitized if displayed by using check_plain(), filter_xss() or similar. (Drupal Docs)Otherallocated_seating_example.module147N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherallocated_seating_example.module148N/A
Potential problem: use the Form API to prevent against CSRF attacks. If you need to use $_POST variables, ensure they are fully sanitized if displayed by using check_plain(), filter_xss() or similar. (Drupal Docs)Otherallocated_seating_example.module148N/A
Expandallocated_seating_example/allocated_seating_example.node.inc200
MessageGroupFilenameLineFunctionStatus
put a space between the asterisk and the comment textOtherallocated_seating_example.node.inc12N/A
put a space between the asterisk and the comment textOtherallocated_seating_example.node.inc181N/A
Expandallocated_seating_example/allocated_seating_example.util.inc200
MessageGroupFilenameLineFunctionStatus
@file description should be on the following line (Drupal Docs)Otherallocated_seating_example.util.inc3N/A
indent secondary line of comment one space Otherallocated_seating_example.util.inc15N/A