Facebook Photo Sync - 7.x-1.0-beta1

Test status
StatusResult
Last receivedWed, 02/15/2012 - 22:32:37
Last requestedWed, 02/15/2012 - 22:33:03
Last testedWed, 02/15/2012 - 22:33:30
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-1.0-beta1
ProjectFacebook Photo Sync
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 0 minor(s), 1 critical(s), and 9 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
0 minor(s), 1 critical(s), and 9 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandfbphotosync.admin.inc001
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherfbphotosync.admin.inc17N/A
Expandfbphotosync.module007
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherfbphotosync.module39N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherfbphotosync.module49N/A
Remove the empty commented line in your function documentation.Otherfbphotosync.module62N/A
Remove the empty commented line in your function documentation.Otherfbphotosync.module79N/A
Remove the empty commented line in your function documentation.Otherfbphotosync.module96N/A
Remove the empty commented line in your function documentation.Otherfbphotosync.module104N/A
Remove the empty commented line in your function documentation.Otherfbphotosync.module125N/A
Expandincludes/image.inc011
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherimage.inc23N/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.Otherimage.inc125N/A

All

Test nameMinorCriticalNormal
Expandfbphotosync.admin.inc001
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherfbphotosync.admin.inc17N/A
Expandfbphotosync.module007
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherfbphotosync.module39N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherfbphotosync.module49N/A
Remove the empty commented line in your function documentation.Otherfbphotosync.module62N/A
Remove the empty commented line in your function documentation.Otherfbphotosync.module79N/A
Remove the empty commented line in your function documentation.Otherfbphotosync.module96N/A
Remove the empty commented line in your function documentation.Otherfbphotosync.module104N/A
Remove the empty commented line in your function documentation.Otherfbphotosync.module125N/A
Leafincludes/features.inc000
Expandincludes/image.inc011
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherimage.inc23N/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.Otherimage.inc125N/A
Leaftemplates/fbphotosync_authenticated_user.tpl.php000