Remember me - 6.x-2.2

Test status
StatusResult
Last receivedTue, 01/24/2012 - 09:34:37
Last requestedTue, 01/24/2012 - 09:35:03
Last testedTue, 01/24/2012 - 09:35:18
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier6.x-2.2
ProjectRemember me
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 1 minor(s), 0 critical(s), and 13 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 13 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandincludes/remember_me.admin.inc105
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherremember_me.admin.inc56N/A
There should be no trailing spacesOtherremember_me.admin.inc115N/A
Control statements should have one space between the control keyword and opening parenthesisOtherremember_me.admin.inc122N/A
Control statements should have one space between the control keyword and opening parenthesisOtherremember_me.admin.inc128N/A
Control statements should have one space between the control keyword and opening parenthesisOtherremember_me.admin.inc131N/A
Use "elseif" in place of "else if"Otherremember_me.admin.inc131N/A
Expandincludes/remember_me.inc004
MessageGroupFilenameLineFunctionStatus
Function documentation should be less than 80 characters per line.Otherremember_me.inc13N/A
Function documentation should be less than 80 characters per line.Otherremember_me.inc14N/A
Function summaries should be one line only. (Drupal Docs)Otherremember_me.inc45N/A
Function summaries should be one line only. (Drupal Docs)Otherremember_me.inc62N/A
Expandincludes/remember_me.users.inc002
MessageGroupFilenameLineFunctionStatus
Function summaries should be one line only. (Drupal Docs)Otherremember_me.users.inc9N/A
Function summaries should be one line only. (Drupal Docs)Otherremember_me.users.inc53N/A
Expandremember_me.module002
MessageGroupFilenameLineFunctionStatus
Remove the empty commented line in your function documentation.Otherremember_me.module6N/A
Use "elseif" in place of "else if"Otherremember_me.module23N/A

All

Test nameMinorCriticalNormal
Expandincludes/remember_me.admin.inc105
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherremember_me.admin.inc56N/A
There should be no trailing spacesOtherremember_me.admin.inc115N/A
Control statements should have one space between the control keyword and opening parenthesisOtherremember_me.admin.inc122N/A
Control statements should have one space between the control keyword and opening parenthesisOtherremember_me.admin.inc128N/A
Control statements should have one space between the control keyword and opening parenthesisOtherremember_me.admin.inc131N/A
Use "elseif" in place of "else if"Otherremember_me.admin.inc131N/A
Expandincludes/remember_me.inc004
MessageGroupFilenameLineFunctionStatus
Function documentation should be less than 80 characters per line.Otherremember_me.inc13N/A
Function documentation should be less than 80 characters per line.Otherremember_me.inc14N/A
Function summaries should be one line only. (Drupal Docs)Otherremember_me.inc45N/A
Function summaries should be one line only. (Drupal Docs)Otherremember_me.inc62N/A
Expandincludes/remember_me.users.inc002
MessageGroupFilenameLineFunctionStatus
Function summaries should be one line only. (Drupal Docs)Otherremember_me.users.inc9N/A
Function summaries should be one line only. (Drupal Docs)Otherremember_me.users.inc53N/A
Leafremember_me.install000
Expandremember_me.module002
MessageGroupFilenameLineFunctionStatus
Remove the empty commented line in your function documentation.Otherremember_me.module6N/A
Use "elseif" in place of "else if"Otherremember_me.module23N/A