Get Locations - 6.x-2.x

Test status
StatusResult
Last receivedTue, 02/28/2012 - 21:32:37
Last requestedTue, 02/28/2012 - 21:33:02
Last testedTue, 02/28/2012 - 21:33:28
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier6.x-2.x
ProjectGet Locations
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 10 minor(s), 5 critical(s), and 63 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
10 minor(s), 5 critical(s), and 63 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandgetlocations.admin.inc3013
MessageGroupFilenameLineFunctionStatus
Function summaries should be one line only. (Drupal Docs)Othergetlocations.admin.inc10N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othergetlocations.admin.inc58N/A
Arrays should be formatted with a space separating each element and assignment operatorOthergetlocations.admin.inc58N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othergetlocations.admin.inc88N/A
Use an indent of 2 spaces, with no tabsOthergetlocations.admin.inc125N/A
Use an indent of 2 spaces, with no tabsOthergetlocations.admin.inc126N/A
Use an indent of 2 spaces, with no tabsOthergetlocations.admin.inc127N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations.admin.inc172N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations.admin.inc177N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations.admin.inc183N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations.admin.inc189N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations.admin.inc217N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations.admin.inc222N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations.admin.inc323N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations.admin.inc326N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations.admin.inc431N/A
Expandgetlocations.install001
MessageGroupFilenameLineFunctionStatus
Function summaries should be one line only. (Drupal Docs)Othergetlocations.install9N/A
Expandgetlocations.markerinfo.inc101
MessageGroupFilenameLineFunctionStatus
in most cases, replace the string function with the drupal_ equivalent string functionsOthergetlocations.markerinfo.inc33N/A
Function summaries should be one line only. (Drupal Docs)Othergetlocations.markerinfo.inc157N/A
Expandgetlocations.module3536
MessageGroupFilenameLineFunctionStatus
Remove the empty commented line in your function documentation.Othergetlocations.module187N/A
table names should be enclosed in {curly_brackets}Othergetlocations.module190N/A
If you define a @param or @return, you should document it as well.Othergetlocations.module230N/A
Remove the empty commented line in your function documentation.Othergetlocations.module234N/A
If you define a @param or @return, you should document it as well.Othergetlocations.module270N/A
Remove the empty commented line in your function documentation.Othergetlocations.module274N/A
Remove the empty commented line in your function documentation.Othergetlocations.module313N/A
table names should be enclosed in {curly_brackets}Othergetlocations.module322N/A
If you define a @param or @return, you should document it as well.Othergetlocations.module351N/A
Remove the empty commented line in your function documentation.Othergetlocations.module356N/A
table names should be enclosed in {curly_brackets}Othergetlocations.module363N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthergetlocations.module374N/A
If you define a @param or @return, you should document it as well.Othergetlocations.module396N/A
Remove the empty commented line in your function documentation.Othergetlocations.module401N/A
If you define a @param or @return, you should document it as well.Othergetlocations.module436N/A
Remove the empty commented line in your function documentation.Othergetlocations.module441N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othergetlocations.module485N/A
Function documentation should be less than 80 characters per line.Othergetlocations.module507N/A
Control statements should have one space between the control keyword and opening parenthesisOthergetlocations.module525N/A
Control statements should have one space between the control keyword and opening parenthesisOthergetlocations.module528N/A
If you define a @param or @return, you should document it as well.Othergetlocations.module688N/A
If you define a @param or @return, you should document it as well.Othergetlocations.module706N/A
table names should be enclosed in {curly_brackets}Othergetlocations.module719N/A
table names should be enclosed in {curly_brackets}Othergetlocations.module726N/A
Function summaries should be one line only. (Drupal Docs)Othergetlocations.module765N/A
If you define a @param or @return, you should document it as well.Othergetlocations.module880N/A
Remove the empty commented line in your function documentation.Othergetlocations.module884N/A
Function summaries should be one line only. (Drupal Docs)Othergetlocations.module1094N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations.module1170N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations.module1255N/A
Function summaries should be one line only. (Drupal Docs)Othergetlocations.module1308N/A
Remove the empty commented line in your function documentation.Othergetlocations.module1311N/A
Function documentation should be less than 80 characters per line.Othergetlocations.module1356N/A
Function documentation should be less than 80 characters per line.Othergetlocations.module1357N/A
Function documentation should be less than 80 characters per line.Othergetlocations.module1359N/A
Function documentation should be less than 80 characters per line.Othergetlocations.module1360N/A
Function summaries should be one line only. (Drupal Docs)Othergetlocations.module1367N/A
Remove the empty commented line in your function documentation.Othergetlocations.module1370N/A
Use an indent of 2 spaces, with no tabsOthergetlocations.module1451N/A
Use an indent of 2 spaces, with no tabsOthergetlocations.module1457N/A
Function documentation should be less than 80 characters per line.Othergetlocations.module1485N/A
Function documentation should be less than 80 characters per line.Othergetlocations.module1486N/A
Function documentation should be less than 80 characters per line.Othergetlocations.module1488N/A
Function documentation should be less than 80 characters per line.Othergetlocations.module1489N/A
Expandgetlocations.theme.inc104
MessageGroupFilenameLineFunctionStatus
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthergetlocations.theme.inc28N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthergetlocations.theme.inc33N/A
Remove the empty commented line in your function documentation.Othergetlocations.theme.inc92N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthergetlocations.theme.inc131N/A
Control statements should have one space between the control keyword and opening parenthesisOthergetlocations.theme.inc144N/A
Expandgetlocations.views_default.inc001
MessageGroupFilenameLineFunctionStatus
Use the matching Drupal theme functions, not raw HTML.Othergetlocations.views_default.inc59N/A
Expandgetlocations_plugin_style_map.inc007
MessageGroupFilenameLineFunctionStatus
The $text argument to l() should be enclosed within t() so that it is translatable.Othergetlocations_plugin_style_map.inc82N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations_plugin_style_map.inc164N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations_plugin_style_map.inc170N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations_plugin_style_map.inc175N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations_plugin_style_map.inc195N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations_plugin_style_map.inc200N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations_plugin_style_map.inc219N/A

All

Test nameMinorCriticalNormal
Expandgetlocations-view-map.tpl.php100
MessageGroupFilenameLineFunctionStatus
@file description should be on the following line (Drupal Docs)Othergetlocations-view-map.tpl.php3N/A
Expandgetlocations.admin.inc3013
MessageGroupFilenameLineFunctionStatus
Function summaries should be one line only. (Drupal Docs)Othergetlocations.admin.inc10N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othergetlocations.admin.inc58N/A
Arrays should be formatted with a space separating each element and assignment operatorOthergetlocations.admin.inc58N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othergetlocations.admin.inc88N/A
Use an indent of 2 spaces, with no tabsOthergetlocations.admin.inc125N/A
Use an indent of 2 spaces, with no tabsOthergetlocations.admin.inc126N/A
Use an indent of 2 spaces, with no tabsOthergetlocations.admin.inc127N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations.admin.inc172N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations.admin.inc177N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations.admin.inc183N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations.admin.inc189N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations.admin.inc217N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations.admin.inc222N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations.admin.inc323N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations.admin.inc326N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations.admin.inc431N/A
Expandgetlocations.install001
MessageGroupFilenameLineFunctionStatus
Function summaries should be one line only. (Drupal Docs)Othergetlocations.install9N/A
Expandgetlocations.markerinfo.inc101
MessageGroupFilenameLineFunctionStatus
in most cases, replace the string function with the drupal_ equivalent string functionsOthergetlocations.markerinfo.inc33N/A
Function summaries should be one line only. (Drupal Docs)Othergetlocations.markerinfo.inc157N/A
Expandgetlocations.module3536
MessageGroupFilenameLineFunctionStatus
Remove the empty commented line in your function documentation.Othergetlocations.module187N/A
table names should be enclosed in {curly_brackets}Othergetlocations.module190N/A
If you define a @param or @return, you should document it as well.Othergetlocations.module230N/A
Remove the empty commented line in your function documentation.Othergetlocations.module234N/A
If you define a @param or @return, you should document it as well.Othergetlocations.module270N/A
Remove the empty commented line in your function documentation.Othergetlocations.module274N/A
Remove the empty commented line in your function documentation.Othergetlocations.module313N/A
table names should be enclosed in {curly_brackets}Othergetlocations.module322N/A
If you define a @param or @return, you should document it as well.Othergetlocations.module351N/A
Remove the empty commented line in your function documentation.Othergetlocations.module356N/A
table names should be enclosed in {curly_brackets}Othergetlocations.module363N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthergetlocations.module374N/A
If you define a @param or @return, you should document it as well.Othergetlocations.module396N/A
Remove the empty commented line in your function documentation.Othergetlocations.module401N/A
If you define a @param or @return, you should document it as well.Othergetlocations.module436N/A
Remove the empty commented line in your function documentation.Othergetlocations.module441N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othergetlocations.module485N/A
Function documentation should be less than 80 characters per line.Othergetlocations.module507N/A
Control statements should have one space between the control keyword and opening parenthesisOthergetlocations.module525N/A
Control statements should have one space between the control keyword and opening parenthesisOthergetlocations.module528N/A
If you define a @param or @return, you should document it as well.Othergetlocations.module688N/A
If you define a @param or @return, you should document it as well.Othergetlocations.module706N/A
table names should be enclosed in {curly_brackets}Othergetlocations.module719N/A
table names should be enclosed in {curly_brackets}Othergetlocations.module726N/A
Function summaries should be one line only. (Drupal Docs)Othergetlocations.module765N/A
If you define a @param or @return, you should document it as well.Othergetlocations.module880N/A
Remove the empty commented line in your function documentation.Othergetlocations.module884N/A
Function summaries should be one line only. (Drupal Docs)Othergetlocations.module1094N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations.module1170N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations.module1255N/A
Function summaries should be one line only. (Drupal Docs)Othergetlocations.module1308N/A
Remove the empty commented line in your function documentation.Othergetlocations.module1311N/A
Function documentation should be less than 80 characters per line.Othergetlocations.module1356N/A
Function documentation should be less than 80 characters per line.Othergetlocations.module1357N/A
Function documentation should be less than 80 characters per line.Othergetlocations.module1359N/A
Function documentation should be less than 80 characters per line.Othergetlocations.module1360N/A
Function summaries should be one line only. (Drupal Docs)Othergetlocations.module1367N/A
Remove the empty commented line in your function documentation.Othergetlocations.module1370N/A
Use an indent of 2 spaces, with no tabsOthergetlocations.module1451N/A
Use an indent of 2 spaces, with no tabsOthergetlocations.module1457N/A
Function documentation should be less than 80 characters per line.Othergetlocations.module1485N/A
Function documentation should be less than 80 characters per line.Othergetlocations.module1486N/A
Function documentation should be less than 80 characters per line.Othergetlocations.module1488N/A
Function documentation should be less than 80 characters per line.Othergetlocations.module1489N/A
Expandgetlocations.theme.inc104
MessageGroupFilenameLineFunctionStatus
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthergetlocations.theme.inc28N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthergetlocations.theme.inc33N/A
Remove the empty commented line in your function documentation.Othergetlocations.theme.inc92N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthergetlocations.theme.inc131N/A
Control statements should have one space between the control keyword and opening parenthesisOthergetlocations.theme.inc144N/A
Leafgetlocations.views.inc000
Expandgetlocations.views_default.inc001
MessageGroupFilenameLineFunctionStatus
Use the matching Drupal theme functions, not raw HTML.Othergetlocations.views_default.inc59N/A
Expandgetlocations_box.tpl.php100
MessageGroupFilenameLineFunctionStatus
@file description should be on the following line (Drupal Docs)Othergetlocations_box.tpl.php4N/A
Expandgetlocations_plugin_style_map.inc007
MessageGroupFilenameLineFunctionStatus
The $text argument to l() should be enclosed within t() so that it is translatable.Othergetlocations_plugin_style_map.inc82N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations_plugin_style_map.inc164N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations_plugin_style_map.inc170N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations_plugin_style_map.inc175N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations_plugin_style_map.inc195N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations_plugin_style_map.inc200N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergetlocations_plugin_style_map.inc219N/A