PostGIS - 7.x-1.x

Test status
StatusResult
Last receivedTue, 02/21/2012 - 14:37:37
Last requestedTue, 02/21/2012 - 14:38:03
Last testedTue, 02/21/2012 - 14:38:29
Test count2
Environment status
MySQLfail
Code reviewfail
Overall Resultfail
Branch information
Identifier7.x-1.x
ProjectPostGIS
LinkProject information
SummaryFAILED: [[Coder]]: [Code review] 9 minor(s), 4 critical(s), and 62 normal(s); [[SimpleTest]]: [MySQL] 0 pass(es), 0 fail(s), and 6 exception(s).
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
    Review the test failures below then if necessary run them locally until the problem has been fixed.
  10. Complete test suite with all tests passing
0 pass(es), 0 fail(s), and 6 exception(s)

Non-pass

Test namePassFailException
ExpandGeometry Validations (PostgisValidationsTestCase) [PostGIS]004
MessageGroupFilenameLineFunctionStatus
PDOException: SQLSTATE[42S02]: Base table or view not found: 1146 Table 'drupaltestbotmysql.simpletest429061registry' doesn't exist: SELECT filename FROM {registry} WHERE name = :name AND type = :type; Array ( [:name] => PostgisGeometrySet [:type] => interface ) in _registry_check_code() (line 3045 of /var/lib/drupaltestbot/sites/default/files/checkout/includes/bootstrap.inc).Uncaught exceptiondatabase.inc2139PDOStatement->execute()
PDOException: SQLSTATE[42S02]: Base table or view not found: 1146 Table 'drupaltestbotmysql.simpletest353110registry' doesn't exist: SELECT filename FROM {registry} WHERE name = :name AND type = :type; Array ( [:name] => PostgisGeometrySet [:type] => interface ) in _registry_check_code() (line 3045 of /var/lib/drupaltestbot/sites/default/files/checkout/includes/bootstrap.inc).Uncaught exceptiondatabase.inc2139PDOStatement->execute()
PDOException: SQLSTATE[42S02]: Base table or view not found: 1146 Table 'drupaltestbotmysql.simpletest537709registry' doesn't exist: SELECT filename FROM {registry} WHERE name = :name AND type = :type; Array ( [:name] => PostgisGeometrySet [:type] => interface ) in _registry_check_code() (line 3045 of /var/lib/drupaltestbot/sites/default/files/checkout/includes/bootstrap.inc).Uncaught exceptiondatabase.inc2139PDOStatement->execute()
PDOException: SQLSTATE[42S02]: Base table or view not found: 1146 Table 'drupaltestbotmysql.simpletest389670registry' doesn't exist: SELECT filename FROM {registry} WHERE name = :name AND type = :type; Array ( [:name] => PostgisGeometrySet [:type] => interface ) in _registry_check_code() (line 3045 of /var/lib/drupaltestbot/sites/default/files/checkout/includes/bootstrap.inc).Uncaught exceptiondatabase.inc2139PDOStatement->execute()
ExpandSingle Geometry Uses (PostgisSingleGeometries) [PostGIS]002
MessageGroupFilenameLineFunctionStatus
PDOException: SQLSTATE[42S02]: Base table or view not found: 1146 Table 'drupaltestbotmysql.simpletest138925registry' doesn't exist: SELECT filename FROM {registry} WHERE name = :name AND type = :type; Array ( [:name] => PostgisGeometry [:type] => interface ) in _registry_check_code() (line 3045 of /var/lib/drupaltestbot/sites/default/files/checkout/includes/bootstrap.inc).Uncaught exceptiondatabase.inc2139PDOStatement->execute()
PDOException: SQLSTATE[42S02]: Base table or view not found: 1146 Table 'drupaltestbotmysql.simpletest542979registry' doesn't exist: SELECT filename FROM {registry} WHERE name = :name AND type = :type; Array ( [:name] => PostgisGeometry [:type] => interface ) in _registry_check_code() (line 3045 of /var/lib/drupaltestbot/sites/default/files/checkout/includes/bootstrap.inc).Uncaught exceptiondatabase.inc2139PDOStatement->execute()

All

Test namePassFailException
ExpandGeometry Validations (PostgisValidationsTestCase) [PostGIS]004
MessageGroupFilenameLineFunctionStatus
PDOException: SQLSTATE[42S02]: Base table or view not found: 1146 Table 'drupaltestbotmysql.simpletest429061registry' doesn't exist: SELECT filename FROM {registry} WHERE name = :name AND type = :type; Array ( [:name] => PostgisGeometrySet [:type] => interface ) in _registry_check_code() (line 3045 of /var/lib/drupaltestbot/sites/default/files/checkout/includes/bootstrap.inc).Uncaught exceptiondatabase.inc2139PDOStatement->execute()
PDOException: SQLSTATE[42S02]: Base table or view not found: 1146 Table 'drupaltestbotmysql.simpletest353110registry' doesn't exist: SELECT filename FROM {registry} WHERE name = :name AND type = :type; Array ( [:name] => PostgisGeometrySet [:type] => interface ) in _registry_check_code() (line 3045 of /var/lib/drupaltestbot/sites/default/files/checkout/includes/bootstrap.inc).Uncaught exceptiondatabase.inc2139PDOStatement->execute()
PDOException: SQLSTATE[42S02]: Base table or view not found: 1146 Table 'drupaltestbotmysql.simpletest537709registry' doesn't exist: SELECT filename FROM {registry} WHERE name = :name AND type = :type; Array ( [:name] => PostgisGeometrySet [:type] => interface ) in _registry_check_code() (line 3045 of /var/lib/drupaltestbot/sites/default/files/checkout/includes/bootstrap.inc).Uncaught exceptiondatabase.inc2139PDOStatement->execute()
PDOException: SQLSTATE[42S02]: Base table or view not found: 1146 Table 'drupaltestbotmysql.simpletest389670registry' doesn't exist: SELECT filename FROM {registry} WHERE name = :name AND type = :type; Array ( [:name] => PostgisGeometrySet [:type] => interface ) in _registry_check_code() (line 3045 of /var/lib/drupaltestbot/sites/default/files/checkout/includes/bootstrap.inc).Uncaught exceptiondatabase.inc2139PDOStatement->execute()
ExpandSingle Geometry Uses (PostgisSingleGeometries) [PostGIS]002
MessageGroupFilenameLineFunctionStatus
PDOException: SQLSTATE[42S02]: Base table or view not found: 1146 Table 'drupaltestbotmysql.simpletest138925registry' doesn't exist: SELECT filename FROM {registry} WHERE name = :name AND type = :type; Array ( [:name] => PostgisGeometry [:type] => interface ) in _registry_check_code() (line 3045 of /var/lib/drupaltestbot/sites/default/files/checkout/includes/bootstrap.inc).Uncaught exceptiondatabase.inc2139PDOStatement->execute()
PDOException: SQLSTATE[42S02]: Base table or view not found: 1146 Table 'drupaltestbotmysql.simpletest542979registry' doesn't exist: SELECT filename FROM {registry} WHERE name = :name AND type = :type; Array ( [:name] => PostgisGeometry [:type] => interface ) in _registry_check_code() (line 3045 of /var/lib/drupaltestbot/sites/default/files/checkout/includes/bootstrap.inc).Uncaught exceptiondatabase.inc2139PDOStatement->execute()

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
9 minor(s), 4 critical(s), and 62 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandincludes/postgis.feeds.inc001
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherpostgis.feeds.inc30N/A
Expandincludes/postgis.field.formatter.inc306
MessageGroupFilenameLineFunctionStatus
Function documentation should be less than 80 characters per line.Otherpostgis.field.formatter.inc9N/A
Function summaries should be one line only. (Drupal Docs)Otherpostgis.field.formatter.inc9N/A
There should be no trailing spacesOtherpostgis.field.formatter.inc10N/A
Function documentation should be less than 80 characters per line.Otherpostgis.field.formatter.inc40N/A
Function summaries should be one line only. (Drupal Docs)Otherpostgis.field.formatter.inc40N/A
There should be no trailing spacesOtherpostgis.field.formatter.inc41N/A
Function summaries should be one line only. (Drupal Docs)Otherpostgis.field.formatter.inc77N/A
There should be no trailing spacesOtherpostgis.field.formatter.inc78N/A
Doxygen uses @todo and @bug to markup things to be done.Otherpostgis.field.formatter.inc116N/A
Expandincludes/postgis.field.inc217
MessageGroupFilenameLineFunctionStatus
Function summaries should be one line only. (Drupal Docs)Otherpostgis.field.inc28N/A
There should be no trailing spacesOtherpostgis.field.inc29N/A
do not use mixed case (camelCase), use lower case and _Otherpostgis.field.inc75N/A
do not use mixed case (camelCase), use lower case and _Otherpostgis.field.inc77N/A
do not use mixed case (camelCase), use lower case and _Otherpostgis.field.inc78N/A
Function documentation should be less than 80 characters per line.Otherpostgis.field.inc96N/A
Function summaries should be one line only. (Drupal Docs)Otherpostgis.field.inc96N/A
There should be no trailing spacesOtherpostgis.field.inc97N/A
Remove the empty commented line in your function documentation.Otherpostgis.field.inc119N/A
table names should be enclosed in {curly_brackets}Otherpostgis.field.inc165N/A
Expandincludes/postgis.field.widget.inc124
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherpostgis.field.widget.inc58N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherpostgis.field.widget.inc72N/A
Function summaries should be one line only. (Drupal Docs)Otherpostgis.field.widget.inc84N/A
There should be no trailing spacesOtherpostgis.field.widget.inc85N/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.Otherpostgis.field.widget.inc136N/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.Otherpostgis.field.widget.inc139N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherpostgis.field.widget.inc154N/A
Expandincludes/postgis.geometry.inc1013
MessageGroupFilenameLineFunctionStatus
use a space between the closing parenthesis and the open bracketOtherpostgis.geometry.inc41N/A
There should be no trailing spacesOtherpostgis.geometry.inc58N/A
Control statements should have one space between the control keyword and opening parenthesisOtherpostgis.geometry.inc76N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.geometry.inc76N/A
@param and @return descriptions begin indented on the next line. (Drupal Docs)Otherpostgis.geometry.inc144N/A
Control statements should have one space between the control keyword and opening parenthesisOtherpostgis.geometry.inc220N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.geometry.inc220N/A
Control statements should have one space between the control keyword and opening parenthesisOtherpostgis.geometry.inc226N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.geometry.inc226N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherpostgis.geometry.inc248N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherpostgis.geometry.inc325N/A
@param and @return descriptions begin indented on the next line. (Drupal Docs)Otherpostgis.geometry.inc355N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherpostgis.geometry.inc372N/A
Doxygen uses @todo and @bug to markup things to be done.Otherpostgis.geometry.inc415N/A
Expandincludes/postgis.openlayers.inc002
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherpostgis.openlayers.inc15N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherpostgis.openlayers.inc44N/A
Expandplugins/geoserver/layer_types/geoserver_layer_type_postgis_field.inc005
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergeoserver_layer_type_postgis_field.inc259N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergeoserver_layer_type_postgis_field.inc264N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergeoserver_layer_type_postgis_field.inc270N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergeoserver_layer_type_postgis_field.inc276N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergeoserver_layer_type_postgis_field.inc282N/A
Expandpostgis.install1110
MessageGroupFilenameLineFunctionStatus
Function summaries should be one line only. (Drupal Docs)Otherpostgis.install28N/A
Control statements should have one space between the control keyword and opening parenthesisOtherpostgis.install51N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.install51N/A
Control statements should have one space between the control keyword and opening parenthesisOtherpostgis.install60N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.install60N/A
table names should be enclosed in {curly_brackets}Otherpostgis.install62N/A
Control statements should have one space between the control keyword and opening parenthesisOtherpostgis.install63N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.install63N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.install67N/A
There should be no trailing spacesOtherpostgis.install73N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherpostgis.install74N/A
Use the matching Drupal theme functions, not raw HTML.Otherpostgis.install77N/A
Expandpostgis.module005
MessageGroupFilenameLineFunctionStatus
Use the matching Drupal theme functions, not raw HTML.Otherpostgis.module62N/A
Function summaries should be one line only. (Drupal Docs)Otherpostgis.module67N/A
If you define a @param or @return, you should document it as well.Otherpostgis.module70N/A
If you define a @param or @return, you should document it as well.Otherpostgis.module71N/A
If you define a @param or @return, you should document it as well.Otherpostgis.module73N/A
Expandpostgis.test109
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherpostgis.test-1N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.test3N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.test11N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.test27N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.test39N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.test48N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.test62N/A
There should be no trailing spacesOtherpostgis.test69N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.test70N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.test76N/A

All

Test nameMinorCriticalNormal
Expandincludes/postgis.feeds.inc001
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherpostgis.feeds.inc30N/A
Expandincludes/postgis.field.formatter.inc306
MessageGroupFilenameLineFunctionStatus
Function documentation should be less than 80 characters per line.Otherpostgis.field.formatter.inc9N/A
Function summaries should be one line only. (Drupal Docs)Otherpostgis.field.formatter.inc9N/A
There should be no trailing spacesOtherpostgis.field.formatter.inc10N/A
Function documentation should be less than 80 characters per line.Otherpostgis.field.formatter.inc40N/A
Function summaries should be one line only. (Drupal Docs)Otherpostgis.field.formatter.inc40N/A
There should be no trailing spacesOtherpostgis.field.formatter.inc41N/A
Function summaries should be one line only. (Drupal Docs)Otherpostgis.field.formatter.inc77N/A
There should be no trailing spacesOtherpostgis.field.formatter.inc78N/A
Doxygen uses @todo and @bug to markup things to be done.Otherpostgis.field.formatter.inc116N/A
Expandincludes/postgis.field.inc217
MessageGroupFilenameLineFunctionStatus
Function summaries should be one line only. (Drupal Docs)Otherpostgis.field.inc28N/A
There should be no trailing spacesOtherpostgis.field.inc29N/A
do not use mixed case (camelCase), use lower case and _Otherpostgis.field.inc75N/A
do not use mixed case (camelCase), use lower case and _Otherpostgis.field.inc77N/A
do not use mixed case (camelCase), use lower case and _Otherpostgis.field.inc78N/A
Function documentation should be less than 80 characters per line.Otherpostgis.field.inc96N/A
Function summaries should be one line only. (Drupal Docs)Otherpostgis.field.inc96N/A
There should be no trailing spacesOtherpostgis.field.inc97N/A
Remove the empty commented line in your function documentation.Otherpostgis.field.inc119N/A
table names should be enclosed in {curly_brackets}Otherpostgis.field.inc165N/A
Expandincludes/postgis.field.widget.inc124
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherpostgis.field.widget.inc58N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherpostgis.field.widget.inc72N/A
Function summaries should be one line only. (Drupal Docs)Otherpostgis.field.widget.inc84N/A
There should be no trailing spacesOtherpostgis.field.widget.inc85N/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.Otherpostgis.field.widget.inc136N/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.Otherpostgis.field.widget.inc139N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherpostgis.field.widget.inc154N/A
Expandincludes/postgis.geometry.inc1013
MessageGroupFilenameLineFunctionStatus
use a space between the closing parenthesis and the open bracketOtherpostgis.geometry.inc41N/A
There should be no trailing spacesOtherpostgis.geometry.inc58N/A
Control statements should have one space between the control keyword and opening parenthesisOtherpostgis.geometry.inc76N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.geometry.inc76N/A
@param and @return descriptions begin indented on the next line. (Drupal Docs)Otherpostgis.geometry.inc144N/A
Control statements should have one space between the control keyword and opening parenthesisOtherpostgis.geometry.inc220N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.geometry.inc220N/A
Control statements should have one space between the control keyword and opening parenthesisOtherpostgis.geometry.inc226N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.geometry.inc226N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherpostgis.geometry.inc248N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherpostgis.geometry.inc325N/A
@param and @return descriptions begin indented on the next line. (Drupal Docs)Otherpostgis.geometry.inc355N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherpostgis.geometry.inc372N/A
Doxygen uses @todo and @bug to markup things to be done.Otherpostgis.geometry.inc415N/A
Expandincludes/postgis.openlayers.inc002
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherpostgis.openlayers.inc15N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherpostgis.openlayers.inc44N/A
Expandplugins/geoserver/layer_types/geoserver_layer_type_postgis_field.inc005
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergeoserver_layer_type_postgis_field.inc259N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergeoserver_layer_type_postgis_field.inc264N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergeoserver_layer_type_postgis_field.inc270N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergeoserver_layer_type_postgis_field.inc276N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othergeoserver_layer_type_postgis_field.inc282N/A
Expandpostgis.install1110
MessageGroupFilenameLineFunctionStatus
Function summaries should be one line only. (Drupal Docs)Otherpostgis.install28N/A
Control statements should have one space between the control keyword and opening parenthesisOtherpostgis.install51N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.install51N/A
Control statements should have one space between the control keyword and opening parenthesisOtherpostgis.install60N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.install60N/A
table names should be enclosed in {curly_brackets}Otherpostgis.install62N/A
Control statements should have one space between the control keyword and opening parenthesisOtherpostgis.install63N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.install63N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.install67N/A
There should be no trailing spacesOtherpostgis.install73N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherpostgis.install74N/A
Use the matching Drupal theme functions, not raw HTML.Otherpostgis.install77N/A
Expandpostgis.module005
MessageGroupFilenameLineFunctionStatus
Use the matching Drupal theme functions, not raw HTML.Otherpostgis.module62N/A
Function summaries should be one line only. (Drupal Docs)Otherpostgis.module67N/A
If you define a @param or @return, you should document it as well.Otherpostgis.module70N/A
If you define a @param or @return, you should document it as well.Otherpostgis.module71N/A
If you define a @param or @return, you should document it as well.Otherpostgis.module73N/A
Expandpostgis.test109
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherpostgis.test-1N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.test3N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.test11N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.test27N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.test39N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.test48N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.test62N/A
There should be no trailing spacesOtherpostgis.test69N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.test70N/A
use a space between the closing parenthesis and the open bracketOtherpostgis.test76N/A