Service links - 7.x-2.x

Test status
StatusResult
Last receivedThu, 01/19/2012 - 11:17:37
Last requestedThu, 01/19/2012 - 11:18:03
Last testedThu, 01/19/2012 - 11:19:21
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-2.x
ProjectService links
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 26 minor(s), 0 critical(s), and 78 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
26 minor(s), 0 critical(s), and 78 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandplugins/service_links_displays.module301
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherservice_links_displays.module7N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherservice_links_displays.module46N/A
There should be no trailing spacesOtherservice_links_displays.module254N/A
There should be no trailing spacesOtherservice_links_displays.module256N/A
Expandplugins/service_links_share.module001
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherservice_links_share.module15N/A
Expandplugins/service_links_sprites.module202
MessageGroupFilenameLineFunctionStatus
Separate comments from comment syntax by a space.Otherservice_links_sprites.module30N/A
There should be no trailing spacesOtherservice_links_sprites.module53N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherservice_links_sprites.module202N/A
There should be no trailing spacesOtherservice_links_sprites.module218N/A
Expandservices/general_services.module002
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergeneral_services.module38N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergeneral_services.module124N/A
Expandservices/german_services.module004
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergerman_services.module17N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergerman_services.module43N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergerman_services.module53N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergerman_services.module83N/A
Expandservices/polish_services.module003
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherpolish_services.module16N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherpolish_services.module21N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherpolish_services.module26N/A
Expandservices/spanish_services.module003
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherspanish_services.module17N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherspanish_services.module22N/A
Use the matching Drupal theme functions, not raw HTML.Otherspanish_services.module33N/A
Expandservices/widget_services.admin.inc1017
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc16N/A
Separate comments from comment syntax by a space.Otherwidget_services.admin.inc36N/A
Separate comments from comment syntax by a space.Otherwidget_services.admin.inc37N/A
Separate comments from comment syntax by a space.Otherwidget_services.admin.inc38N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc44N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc78N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc88N/A
There should be no trailing spacesOtherwidget_services.admin.inc115N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc120N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc142N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc149N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc150N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc151N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc152N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc180N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc203N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc206N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc212N/A
Expandservices/widget_services.module108
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.module30N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.module49N/A
There should be no trailing spacesOtherwidget_services.module94N/A
Separate comments from comment syntax by a space.Otherwidget_services.module148N/A
Separate comments from comment syntax by a space.Otherwidget_services.module149N/A
Separate comments from comment syntax by a space.Otherwidget_services.module150N/A
Separate comments from comment syntax by a space.Otherwidget_services.module151N/A
Administrative menu items should have a description.Otherwidget_services.module214N/A
Separate comments from comment syntax by a space.Otherwidget_services.module220N/A
Expandservice_links.admin.inc1011
MessageGroupFilenameLineFunctionStatus
Separate comments from comment syntax by a space.Otherservice_links.admin.inc99N/A
Separate comments from comment syntax by a space.Otherservice_links.admin.inc102N/A
There should be no trailing spacesOtherservice_links.admin.inc162N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherservice_links.admin.inc212N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherservice_links.admin.inc213N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherservice_links.admin.inc214N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherservice_links.admin.inc273N/A
Use the matching Drupal theme functions, not raw HTML.Otherservice_links.admin.inc302N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherservice_links.admin.inc325N/A
Use the matching Drupal theme functions, not raw HTML.Otherservice_links.admin.inc341N/A
Function summaries should be one line only. (Drupal Docs)Otherservice_links.admin.inc362N/A
Use the matching Drupal theme functions, not raw HTML.Otherservice_links.admin.inc439N/A
Expandservice_links.api.php001
MessageGroupFilenameLineFunctionStatus
Function summaries should be one line only. (Drupal Docs)Otherservice_links.api.php9N/A
Expandservice_links.module12022
MessageGroupFilenameLineFunctionStatus
Separate comments from comment syntax by a space.Otherservice_links.module75N/A
Use an indent of 2 spaces, with no tabsOtherservice_links.module99N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherservice_links.module100N/A
Administrative menu items should have a description.Otherservice_links.module105N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherservice_links.module272N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherservice_links.module294N/A
There should be no trailing spacesOtherservice_links.module451N/A
There should be no trailing spacesOtherservice_links.module457N/A
There should be no trailing spacesOtherservice_links.module463N/A
There should be no trailing spacesOtherservice_links.module477N/A
Function summaries should be one line only. (Drupal Docs)Otherservice_links.module502N/A
There should be no trailing spacesOtherservice_links.module515N/A
There should be no trailing spacesOtherservice_links.module521N/A
There should be no trailing spacesOtherservice_links.module527N/A
There should be no trailing spacesOtherservice_links.module546N/A
There should be no trailing spacesOtherservice_links.module662N/A
Separate comments from comment syntax by a space.Otherservice_links.module731N/A
Separate comments from comment syntax by a space.Otherservice_links.module733N/A
Separate comments from comment syntax by a space.Otherservice_links.module735N/A
Separate comments from comment syntax by a space.Otherservice_links.module737N/A
Separate comments from comment syntax by a space.Otherservice_links.module739N/A
Separate comments from comment syntax by a space.Otherservice_links.module741N/A
Separate comments from comment syntax by a space.Otherservice_links.module743N/A
Separate comments from comment syntax by a space.Otherservice_links.module745N/A
Separate comments from comment syntax by a space.Otherservice_links.module747N/A
Separate comments from comment syntax by a space.Otherservice_links.module749N/A
Separate comments from comment syntax by a space.Otherservice_links.module751N/A
Separate comments from comment syntax by a space.Otherservice_links.module753N/A
Separate comments from comment syntax by a space.Otherservice_links.module755N/A
There should be no trailing spacesOtherservice_links.module778N/A
Function summaries should be one line only. (Drupal Docs)Otherservice_links.module802N/A
Function summaries should be one line only. (Drupal Docs)Otherservice_links.module823N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherservice_links.module835N/A
Function summaries should be one line only. (Drupal Docs)Otherservice_links.module864N/A
Expandservice_links.theme.inc102
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherservice_links.theme.inc21N/A
Separate comments from comment syntax by a space.Otherservice_links.theme.inc27N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherservice_links.theme.inc83N/A
Expandtemplate.php201
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOthertemplate.php9N/A
Function documentation should be less than 80 characters per line.Othertemplate.php20N/A
There should be no trailing spacesOthertemplate.php39N/A

All

Test nameMinorCriticalNormal
Expandplugins/service_links_displays.module301
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherservice_links_displays.module7N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherservice_links_displays.module46N/A
There should be no trailing spacesOtherservice_links_displays.module254N/A
There should be no trailing spacesOtherservice_links_displays.module256N/A
Expandplugins/service_links_share.module001
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherservice_links_share.module15N/A
Expandplugins/service_links_sprites.module202
MessageGroupFilenameLineFunctionStatus
Separate comments from comment syntax by a space.Otherservice_links_sprites.module30N/A
There should be no trailing spacesOtherservice_links_sprites.module53N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherservice_links_sprites.module202N/A
There should be no trailing spacesOtherservice_links_sprites.module218N/A
Leafservices/basque_services.module000
Leafservices/dutch_services.module000
Expandservices/farsi_services.module200
MessageGroupFilenameLineFunctionStatus
Use an indent of 2 spaces, with no tabsOtherfarsi_services.module29N/A
There should be no trailing spacesOtherfarsi_services.module35N/A
Leafservices/favorite_services.module000
Leafservices/forward_services.module000
Expandservices/general_services.module002
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergeneral_services.module38N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergeneral_services.module124N/A
Expandservices/german_services.module004
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergerman_services.module17N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergerman_services.module43N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergerman_services.module53N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergerman_services.module83N/A
Leafservices/hungarian_services.module000
Leafservices/italian_services.module000
Expandservices/polish_services.module003
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherpolish_services.module16N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherpolish_services.module21N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherpolish_services.module26N/A
Leafservices/print_services.module000
Expandservices/russian_services.module100
MessageGroupFilenameLineFunctionStatus
indent secondary line of comment one space Otherrussian_services.module106N/A
Expandservices/spanish_services.module003
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherspanish_services.module17N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherspanish_services.module22N/A
Use the matching Drupal theme functions, not raw HTML.Otherspanish_services.module33N/A
Leafservices/swedish_services.module000
Expandservices/widget_services.admin.inc1017
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc16N/A
Separate comments from comment syntax by a space.Otherwidget_services.admin.inc36N/A
Separate comments from comment syntax by a space.Otherwidget_services.admin.inc37N/A
Separate comments from comment syntax by a space.Otherwidget_services.admin.inc38N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc44N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc78N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc88N/A
There should be no trailing spacesOtherwidget_services.admin.inc115N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc120N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc142N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc149N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc150N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc151N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc152N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc180N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc203N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc206N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.admin.inc212N/A
Expandservices/widget_services.module108
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.module30N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherwidget_services.module49N/A
There should be no trailing spacesOtherwidget_services.module94N/A
Separate comments from comment syntax by a space.Otherwidget_services.module148N/A
Separate comments from comment syntax by a space.Otherwidget_services.module149N/A
Separate comments from comment syntax by a space.Otherwidget_services.module150N/A
Separate comments from comment syntax by a space.Otherwidget_services.module151N/A
Administrative menu items should have a description.Otherwidget_services.module214N/A
Separate comments from comment syntax by a space.Otherwidget_services.module220N/A
Expandservice_links.admin.inc1011
MessageGroupFilenameLineFunctionStatus
Separate comments from comment syntax by a space.Otherservice_links.admin.inc99N/A
Separate comments from comment syntax by a space.Otherservice_links.admin.inc102N/A
There should be no trailing spacesOtherservice_links.admin.inc162N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherservice_links.admin.inc212N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherservice_links.admin.inc213N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherservice_links.admin.inc214N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherservice_links.admin.inc273N/A
Use the matching Drupal theme functions, not raw HTML.Otherservice_links.admin.inc302N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherservice_links.admin.inc325N/A
Use the matching Drupal theme functions, not raw HTML.Otherservice_links.admin.inc341N/A
Function summaries should be one line only. (Drupal Docs)Otherservice_links.admin.inc362N/A
Use the matching Drupal theme functions, not raw HTML.Otherservice_links.admin.inc439N/A
Expandservice_links.api.php001
MessageGroupFilenameLineFunctionStatus
Function summaries should be one line only. (Drupal Docs)Otherservice_links.api.php9N/A
Leafservice_links.install000
Expandservice_links.module12022
MessageGroupFilenameLineFunctionStatus
Separate comments from comment syntax by a space.Otherservice_links.module75N/A
Use an indent of 2 spaces, with no tabsOtherservice_links.module99N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherservice_links.module100N/A
Administrative menu items should have a description.Otherservice_links.module105N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherservice_links.module272N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherservice_links.module294N/A
There should be no trailing spacesOtherservice_links.module451N/A
There should be no trailing spacesOtherservice_links.module457N/A
There should be no trailing spacesOtherservice_links.module463N/A
There should be no trailing spacesOtherservice_links.module477N/A
Function summaries should be one line only. (Drupal Docs)Otherservice_links.module502N/A
There should be no trailing spacesOtherservice_links.module515N/A
There should be no trailing spacesOtherservice_links.module521N/A
There should be no trailing spacesOtherservice_links.module527N/A
There should be no trailing spacesOtherservice_links.module546N/A
There should be no trailing spacesOtherservice_links.module662N/A
Separate comments from comment syntax by a space.Otherservice_links.module731N/A
Separate comments from comment syntax by a space.Otherservice_links.module733N/A
Separate comments from comment syntax by a space.Otherservice_links.module735N/A
Separate comments from comment syntax by a space.Otherservice_links.module737N/A
Separate comments from comment syntax by a space.Otherservice_links.module739N/A
Separate comments from comment syntax by a space.Otherservice_links.module741N/A
Separate comments from comment syntax by a space.Otherservice_links.module743N/A
Separate comments from comment syntax by a space.Otherservice_links.module745N/A
Separate comments from comment syntax by a space.Otherservice_links.module747N/A
Separate comments from comment syntax by a space.Otherservice_links.module749N/A
Separate comments from comment syntax by a space.Otherservice_links.module751N/A
Separate comments from comment syntax by a space.Otherservice_links.module753N/A
Separate comments from comment syntax by a space.Otherservice_links.module755N/A
There should be no trailing spacesOtherservice_links.module778N/A
Function summaries should be one line only. (Drupal Docs)Otherservice_links.module802N/A
Function summaries should be one line only. (Drupal Docs)Otherservice_links.module823N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherservice_links.module835N/A
Function summaries should be one line only. (Drupal Docs)Otherservice_links.module864N/A
Expandservice_links.theme.inc102
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherservice_links.theme.inc21N/A
Separate comments from comment syntax by a space.Otherservice_links.theme.inc27N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherservice_links.theme.inc83N/A
Leafservice_links.views.inc000
Leafservice_links_handler_field_service_links.inc000
Expandtemplate.php201
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOthertemplate.php9N/A
Function documentation should be less than 80 characters per line.Othertemplate.php20N/A
There should be no trailing spacesOthertemplate.php39N/A