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
0 minor(s), 0 critical(s), and 5 normal(s)
Non-pass
| Test name | Minor | Critical | Normal |
excommunicate.install | 0 | 0 | 1 |
| Message | Group | Filename | Line | Function | Status |
| Use sentence case, not title case, for end-user strings. (Wikipedia) | Other | excommunicate.install | 47 | N/A |  |
|
excommunicate.module | 0 | 0 | 4 |
| Message | Group | Filename | Line | Function | Status |
| Core uses "e-mail" in end-user text and "mail" elsewhere (database, function names, etc.) | Other | excommunicate.module | 138 | N/A |  |
| Core uses "e-mail" in end-user text and "mail" elsewhere (database, function names, etc.) | Other | excommunicate.module | 139 | N/A |  |
| Function summaries should be one line only. (Drupal Docs) | Other | excommunicate.module | 146 | N/A |  |
| Use PHP's master function, not an alias. | Other | excommunicate.module | 190 | N/A |  |
|
All
| Test name | Minor | Critical | Normal |
excommunicate.api.php | 0 | 0 | 0 |
excommunicate.install | 0 | 0 | 1 |
| Message | Group | Filename | Line | Function | Status |
| Use sentence case, not title case, for end-user strings. (Wikipedia) | Other | excommunicate.install | 47 | N/A |  |
|
excommunicate.module | 0 | 0 | 4 |
| Message | Group | Filename | Line | Function | Status |
| Core uses "e-mail" in end-user text and "mail" elsewhere (database, function names, etc.) | Other | excommunicate.module | 138 | N/A |  |
| Core uses "e-mail" in end-user text and "mail" elsewhere (database, function names, etc.) | Other | excommunicate.module | 139 | N/A |  |
| Function summaries should be one line only. (Drupal Docs) | Other | excommunicate.module | 146 | N/A |  |
| Use PHP's master function, not an alias. | Other | excommunicate.module | 190 | N/A |  |
|
excommunicate.smtp_library.inc | 0 | 0 | 0 |