Posted by ilo on August 27, 2009 at 12:53am
| Project: | Article |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs review |
Issue Summary
I've included a patch with all "coder" module issues fixed. Despite the size of the patch, no changes were done to the current code. Patch is for DRUPAL-6--1.
| Attachment | Size |
|---|---|
| article_coding_standards.patch | 48.66 KB |
Comments
#1
sorry, previous patch mixed some backup files. Forget it, this one is the right one..
#2
That's not enough to silence coder :(
I'll do some cleanups every now and then until it gets solved...
#3
I used coder module DRUPAL-6--2 dev version, with the following options and didn't return any warning.
Drupal Coding Standards (every developer should use)
Drupal Commenting Standards (every developer should use)
Drupal SQL Standards (new review, so use with caution)
Drupal Security Checks (very basic, needs work, but what it finds is good)
Internationalization
and warning level:
minor (most)