Inline Ajax Comments - 7.x-1.x

Test status
StatusResult
Last receivedFri, 03/23/2012 - 21:50:44
Last requestedFri, 03/23/2012 - 21:51:01
Last testedFri, 03/23/2012 - 21:59:14
Test count4
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-1.x
ProjectInline Ajax Comments
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 19 minor(s), 0 critical(s), and 20 normal(s); [[SimpleTest]]: [MySQL] 11 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
11 pass(es), 0 fail(s), and 0 exception(s)
Test namePassFailException
LeafInline Comments Tests (InlineCommentsTestCase) [Inline Comments]1100

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
19 minor(s), 0 critical(s), and 20 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandcomment.tpl.php101
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercomment.tpl.php-1N/A
There should be no trailing spacesOthercomment.tpl.php4N/A
Expandinc/ajax.inc108
MessageGroupFilenameLineFunctionStatus
put a space between the asterisk and the comment textOtherajax.inc4N/A
Separate comments from comment syntax by a space.Otherajax.inc8N/A
Separate comments from comment syntax by a space.Otherajax.inc12N/A
Functions should be called with no spaces between the function name and opening parenthesesOtherajax.inc33N/A
Separate comments from comment syntax by a space.Otherajax.inc40N/A
Separate comments from comment syntax by a space.Otherajax.inc45N/A
Separate comments from comment syntax by a space.Otherajax.inc53N/A
Separate comments from comment syntax by a space.Otherajax.inc60N/A
Separate comments from comment syntax by a space.Otherajax.inc70N/A
Expandinc/inline_comments.views.inc102
MessageGroupFilenameLineFunctionStatus
put a space between the asterisk and the comment textOtherinline_comments.views.inc4N/A
If you define a @param or @return, you should document it as well.Otherinline_comments.views.inc11N/A
If you define a @param or @return, you should document it as well.Otherinline_comments.views.inc30N/A
Expandinc/inline_comments_handler_field_inline_comments.inc201
MessageGroupFilenameLineFunctionStatus
put a space between the asterisk and the comment textOtherinline_comments_handler_field_inline_comments.inc4N/A
There should be no trailing spacesOtherinline_comments_handler_field_inline_comments.inc11N/A
use a space between the closing parenthesis and the open bracketOtherinline_comments_handler_field_inline_comments.inc12N/A
Expandinline_comments.module204
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherinline_comments.module-1N/A
Separate comments from comment syntax by a space.Otherinline_comments.module10N/A
Separate comments from comment syntax by a space.Otherinline_comments.module63N/A
There should be no trailing spacesOtherinline_comments.module73N/A
There should be no trailing spacesOtherinline_comments.module75N/A
else statements should begin on a new lineOtherinline_comments.module79N/A
Expandtests/inline_comments.test1204
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherinline_comments.test1N/A
put a space between the asterisk and the comment textOtherinline_comments.test3N/A
indent secondary line of comment one space Otherinline_comments.test6N/A
indent secondary line of comment one space Otherinline_comments.test11N/A
There should be no trailing spacesOtherinline_comments.test21N/A
There should be no trailing spacesOtherinline_comments.test28N/A
There should be no trailing spacesOtherinline_comments.test29N/A
There should be no trailing spacesOtherinline_comments.test34N/A
Arrays should be formatted with a space separating each element and assignment operatorOtherinline_comments.test36N/A
There should be no trailing spacesOtherinline_comments.test39N/A
Control statements should have one space between the control keyword and opening parenthesisOtherinline_comments.test40N/A
use a space between the closing parenthesis and the open bracketOtherinline_comments.test40N/A
Arrays should be formatted with a space separating each element and assignment operatorOtherinline_comments.test40N/A
There should be no trailing spacesOtherinline_comments.test44N/A
There should be no trailing spacesOtherinline_comments.test48N/A
There should be no trailing spacesOtherinline_comments.test52N/A

All

Test nameMinorCriticalNormal
Expandcomment.tpl.php101
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercomment.tpl.php-1N/A
There should be no trailing spacesOthercomment.tpl.php4N/A
Expandinc/ajax.inc108
MessageGroupFilenameLineFunctionStatus
put a space between the asterisk and the comment textOtherajax.inc4N/A
Separate comments from comment syntax by a space.Otherajax.inc8N/A
Separate comments from comment syntax by a space.Otherajax.inc12N/A
Functions should be called with no spaces between the function name and opening parenthesesOtherajax.inc33N/A
Separate comments from comment syntax by a space.Otherajax.inc40N/A
Separate comments from comment syntax by a space.Otherajax.inc45N/A
Separate comments from comment syntax by a space.Otherajax.inc53N/A
Separate comments from comment syntax by a space.Otherajax.inc60N/A
Separate comments from comment syntax by a space.Otherajax.inc70N/A
Expandinc/inline_comments.views.inc102
MessageGroupFilenameLineFunctionStatus
put a space between the asterisk and the comment textOtherinline_comments.views.inc4N/A
If you define a @param or @return, you should document it as well.Otherinline_comments.views.inc11N/A
If you define a @param or @return, you should document it as well.Otherinline_comments.views.inc30N/A
Expandinc/inline_comments_handler_field_inline_comments.inc201
MessageGroupFilenameLineFunctionStatus
put a space between the asterisk and the comment textOtherinline_comments_handler_field_inline_comments.inc4N/A
There should be no trailing spacesOtherinline_comments_handler_field_inline_comments.inc11N/A
use a space between the closing parenthesis and the open bracketOtherinline_comments_handler_field_inline_comments.inc12N/A
Expandinline_comments.module204
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherinline_comments.module-1N/A
Separate comments from comment syntax by a space.Otherinline_comments.module10N/A
Separate comments from comment syntax by a space.Otherinline_comments.module63N/A
There should be no trailing spacesOtherinline_comments.module73N/A
There should be no trailing spacesOtherinline_comments.module75N/A
else statements should begin on a new lineOtherinline_comments.module79N/A
Expandtests/inline_comments.test1204
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherinline_comments.test1N/A
put a space between the asterisk and the comment textOtherinline_comments.test3N/A
indent secondary line of comment one space Otherinline_comments.test6N/A
indent secondary line of comment one space Otherinline_comments.test11N/A
There should be no trailing spacesOtherinline_comments.test21N/A
There should be no trailing spacesOtherinline_comments.test28N/A
There should be no trailing spacesOtherinline_comments.test29N/A
There should be no trailing spacesOtherinline_comments.test34N/A
Arrays should be formatted with a space separating each element and assignment operatorOtherinline_comments.test36N/A
There should be no trailing spacesOtherinline_comments.test39N/A
Control statements should have one space between the control keyword and opening parenthesisOtherinline_comments.test40N/A
use a space between the closing parenthesis and the open bracketOtherinline_comments.test40N/A
Arrays should be formatted with a space separating each element and assignment operatorOtherinline_comments.test40N/A
There should be no trailing spacesOtherinline_comments.test44N/A
There should be no trailing spacesOtherinline_comments.test48N/A
There should be no trailing spacesOtherinline_comments.test52N/A