|
Camel case is checked also for class functions |
active |
normal |
bug reports |
6.x-1.x-dev |
Code |
|
|
Add spacing and string checks on comments. |
active |
normal |
feature requests |
7.x-1.x-dev |
Review/Rules |
|
|
Operator spacing |
active |
normal |
feature requests |
6.x-2.x-dev |
Review/Rules |
|
|
Coding standards for file ending newlines |
needs work |
minor |
tasks |
7.x-1.x-dev |
Review/Rules |
alexanderpas |
|
coder.drush.inc should declare coder as dependency |
needs review |
normal |
bug reports |
6.x-2.x-dev |
Code |
|
|
Add Danish translation |
active |
normal |
tasks |
6.x-1.0 |
Miscellaneous |
|
|
Add simpletest coding standards to Coder Module |
active |
normal |
tasks |
7.x-1.x-dev |
SimpleTest |
|
|
Permissions mostly capitalized |
active |
normal |
feature requests |
7.x-1.x-dev |
Review/Rules |
|
|
Theme_pager changes |
active |
normal |
feature requests |
7.x-1.x-dev |
Review/Rules |
|
|
Node_view replaced with node_build/drupal_render |
active |
normal |
feature requests |
7.x-1.x-dev |
Review/Rules |
|
|
drupal_uninstall_modules change |
active |
minor |
feature requests |
7.x-1.x-dev |
Documentation |
|
|
Create coder review rules for D7 changes |
active |
normal |
tasks |
7.x-1.x-dev |
Review/Rules |
|
|
Coder test to catch drupal_execute() -> drupal_form_submit() rename |
needs review |
normal |
tasks |
7.x-1.x-dev |
Review/Rules |
brianV |
|
theme_coder_warning() overwrites $warning array with string, then looks for array key |
needs review |
normal |
bug reports |
6.x-2.x-dev |
Code |
|
|
Detect @ in a[@href*= for jQuery |
needs review |
normal |
bug reports |
6.x-2.x-dev |
Code |
|
|
Add check for incorrect use of 'access callback' |
needs review |
critical |
feature requests |
6.x-1.x-dev |
Review/Rules |
|
|
Coder incorrectly detects hook_access functions |
active |
normal |
bug reports |
6.x-1.x-dev |
Code |
|
|
PHP notice |
active |
normal |
bug reports |
6.x-1.x-dev |
Code |
|
|
Blame gratitious use of heredoc syntax |
active |
normal |
feature requests |
7.x-1.x-dev |
Code |
|
|
Include file exclusions don't work |
needs review |
normal |
bug reports |
6.x-1.x-dev |
Code |
|
|
Wrong line displayed in watchdog() errors. |
active |
normal |
bug reports |
6.x-2.x-dev |
Review/Rules |
|
|
coder should not complain about closing php tag within template files |
needs review |
normal |
bug reports |
6.x-2.x-dev |
Code |
|
|
Coder incorrectly detects validate and submit functions |
active |
normal |
bug reports |
6.x-1.x-dev |
Code |
|
|
Coder issues critical errors for valid drupal_mail calls |
active |
normal |
bug reports |
6.x-1.x-dev |
Code |
|
|
Catch .info file syntax error |
active |
normal |
feature requests |
6.x-1.0 |
Code |
|
|
GHOP #136: Improve coder_format to re-format Drupal 6 core without wrong formattings |
needs review |
normal |
tasks |
6.x-1.x-dev |
Coder Format |
sun |
|
provide javascript review |
active |
normal |
feature requests |
6.x-1.x-dev |
Code |
stella |
|
check for pgsql compatibility - double quotes in SQL queries |
needs work |
normal |
feature requests |
6.x-2.x-dev |
Code |
|
|
Supply a front color with the back color |
needs review |
minor |
bug reports |
6.x-1.0 |
User interface |
|
|
No warning that hook_link_alter() parameters have changed in 6.x. |
active |
minor |
bug reports |
6.x-2.x-dev |
Review/Rules |
|
|
Locale check doesnt consider Unicode strings: period |
needs review |
normal |
bug reports |
6.x-1.0 |
Code |
|
|
Locale check doesnt consider Unicode strings: first letter case |
needs review |
normal |
bug reports |
6.x-1.0 |
Code |
|
|
Empty translation is pointed as an error |
needs review |
normal |
bug reports |
6.x-1.0 |
Code |
|
|
t() called in a global scope |
active |
normal |
feature requests |
6.x-2.x-dev |
Review/Rules |
|
|
Zerod lines are set as -1 |
needs review |
normal |
bug reports |
6.x-1.x-dev |
Code |
|
|
How to check array content from *.install? |
active |
normal |
support requests |
6.x-2.x-dev |
Review/Rules |
|
|
false positive in patch on final ?> |
active |
normal |
bug reports |
6.x-2.x-dev |
Code |
|
|
false positive on non existing code |
needs review |
normal |
bug reports |
6.x-2.x-dev |
Review/Rules |
|
|
check for common XSS mistakes with drupal_set_title and drupal_set_message |
needs review |
normal |
feature requests |
6.x-2.x-dev |
Code |
|
|
Coder concatenates strings in odd ways during parsing |
needs review |
critical |
bug reports |
6.x-2.x-dev |
Code |
douggreen |
|
Add form to test pasted code |
active |
normal |
feature requests |
6.x-2.x-dev |
User interface |
|
|
Review getInfo() in .test files |
active |
normal |
feature requests |
6.x-2.x-dev |
Review/Rules |
|
|
coder_format: switch in a switch? |
active |
normal |
bug reports |
6.x-2.x-dev |
Coder Format |
|
|
Hyperlink the offending line so that it opens in editor |
active |
normal |
feature requests |
6.x-2.x-dev |
Code |
|
|
CVS keyword warning - reduce level |
active |
normal |
bug reports |
6.x-2.x-dev |
Code |
|
|
Write Rule security check for missing db_rewrite_sql on SELECT FROM {node} |
reviewed & tested by the community |
normal |
tasks |
6.x-2.x-dev |
Review/Rules |
douggreen |
|
'_content_admin_field' to 'content_field_edit_form' |
active |
normal |
feature requests |
6.x-2.x-dev |
Code |
|
|
change category of "l() already contains a check_plain() ..." |
active |
normal |
tasks |
6.x-1.x-dev |
Review/Rules |
|
|
5.x-6.x test catches false positive for use of hook_info() |
reviewed & tested by the community |
normal |
bug reports |
6.x-1.x-dev |
Review/Rules |
|
|
false positive on "select this and that from there" strings |
needs review |
normal |
bug reports |
6.x-2.x-dev |
Review/Rules |
|