MySQL
- Setup environment
- Detect an invalid patch URL
- Detect invalid repository information
- Detect invalid patch format
- Detect a non-applicable patch
- Detect invalid PHP syntax
- Detect a Drupal installation failure
- Detect a test run failure
- Detect a failing test
- Complete test suite with all tests passing
218 pass(es), 0 fail(s), and 0 exception(s)
| Test name | Pass | Fail | Exception |
Userpoints Node Visits (UserpointsNodeVisitsTestCase) [Userpoints] | 138 | 0 | 0 |
Userpoints Nodes and Comments (UserpointsNodeCommentTestCase) [Userpoints] | 80 | 0 | 0 |
Code review
- Setup environment
- Detect an invalid patch URL
- Detect invalid repository information
- Detect invalid patch format
- Detect a non-applicable patch
- Detect invalid PHP syntax
- Detect a review run failure
- Detect coder flags
Ensure that your code follows the Drupal standard and passes a coder review.
- Complete coder review with all passed
7 minor(s), 2 critical(s), and 5 normal(s)
Non-pass
| Test name | Minor | Critical | Normal |
userpoints_nc.module | 4 | 1 | 5 |
| Message | Group | Filename | Line | Function | Status |
| @file block missing (Drupal Docs) | Other | userpoints_nc.module | -1 | N/A |  |
| Potential problem: "SELECT FROM {node}" statements should probably be wrapped in db_rewrite_sql() and with the alias for {node} table defined (e.g. {node} n) | Other | userpoints_nc.module | 370 | N/A |  |
| Use "elseif" in place of "else if" | Other | userpoints_nc.module | 404 | N/A |  |
| Use an indent of 2 spaces, with no tabs | Other | userpoints_nc.module | 405 | N/A |  |
| Use "elseif" in place of "else if" | Other | userpoints_nc.module | 416 | N/A |  |
| Use an indent of 2 spaces, with no tabs | Other | userpoints_nc.module | 417 | N/A |  |
| Use "elseif" in place of "else if" | Other | userpoints_nc.module | 514 | N/A |  |
| Use an indent of 2 spaces, with no tabs | Other | userpoints_nc.module | 515 | N/A |  |
| Use "elseif" in place of "else if" | Other | userpoints_nc.module | 526 | N/A |  |
| Use an indent of 2 spaces, with no tabs | Other | userpoints_nc.module | 527 | N/A |  |
|
userpoints_nc_visits/userpoints_nc_visits.module | 0 | 1 | 0 |
| Message | Group | Filename | Line | Function | Status |
| table names should be enclosed in {curly_brackets} | Other | userpoints_nc_visits.module | 92 | N/A |  |
|
All
| Test name | Minor | Critical | Normal |
userpoints_nc.install | 0 | 0 | 0 |
userpoints_nc.module | 4 | 1 | 5 |
| Message | Group | Filename | Line | Function | Status |
| @file block missing (Drupal Docs) | Other | userpoints_nc.module | -1 | N/A |  |
| Potential problem: "SELECT FROM {node}" statements should probably be wrapped in db_rewrite_sql() and with the alias for {node} table defined (e.g. {node} n) | Other | userpoints_nc.module | 370 | N/A |  |
| Use "elseif" in place of "else if" | Other | userpoints_nc.module | 404 | N/A |  |
| Use an indent of 2 spaces, with no tabs | Other | userpoints_nc.module | 405 | N/A |  |
| Use "elseif" in place of "else if" | Other | userpoints_nc.module | 416 | N/A |  |
| Use an indent of 2 spaces, with no tabs | Other | userpoints_nc.module | 417 | N/A |  |
| Use "elseif" in place of "else if" | Other | userpoints_nc.module | 514 | N/A |  |
| Use an indent of 2 spaces, with no tabs | Other | userpoints_nc.module | 515 | N/A |  |
| Use "elseif" in place of "else if" | Other | userpoints_nc.module | 526 | N/A |  |
| Use an indent of 2 spaces, with no tabs | Other | userpoints_nc.module | 527 | N/A |  |
|
userpoints_nc.test | 1 | 0 | 0 |
| Message | Group | Filename | Line | Function | Status |
| in most cases, replace the string function with the drupal_ equivalent string functions | Other | userpoints_nc.test | 42 | N/A |  |
|
userpoints_nc_revision/userpoints_nc_revision.module | 0 | 0 | 0 |
userpoints_nc_visits/userpoints_nc_visits.install | 0 | 0 | 0 |
userpoints_nc_visits/userpoints_nc_visits.module | 0 | 1 | 0 |
| Message | Group | Filename | Line | Function | Status |
| table names should be enclosed in {curly_brackets} | Other | userpoints_nc_visits.module | 92 | N/A |  |
|
userpoints_nc_visits/userpoints_nc_visits.test | 2 | 0 | 0 |
| Message | Group | Filename | Line | Function | Status |
| There should be no trailing spaces | Other | userpoints_nc_visits.test | 6 | N/A |  |
| in most cases, replace the string function with the drupal_ equivalent string functions | Other | userpoints_nc_visits.test | 52 | N/A |  |
|