coding standards in article module.

ilo - August 27, 2009 - 00:53
Project:Article
Version:6.x-1.x-dev
Component:Code
Category:task
Priority:normal
Assigned:Unassigned
Status:needs review
Description

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.

AttachmentSize
article_coding_standards.patch48.66 KB

#1

ilo - August 27, 2009 - 01:01
Status:active» needs review

sorry, previous patch mixed some backup files. Forget it, this one is the right one..

AttachmentSize
article_coding_standards.patch 11.04 KB

#2

msameer - August 29, 2009 - 11:01

That's not enough to silence coder :(

I'll do some cleanups every now and then until it gets solved...

#3

ilo - August 29, 2009 - 17:51

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)

 
 

Drupal is a registered trademark of Dries Buytaert.