Closed (fixed)
Project:
Diff
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
26 Mar 2008 at 19:18 UTC
Updated:
22 Apr 2008 at 14:21 UTC
Jump to comment: Most recent file
This module needs a patch for code style. See coder.module to help identify errors. Especially needed is diffengine.php
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | patch.patch | 64.25 KB | webernet |
| #1 | patch.patch | 58.39 KB | webernet |
Comments
Comment #1
webernet commentedUntested, and probably incomplete.
Converts tabs to spaces, capitalizes TRUE/FALSE, adds {} to if statements etc.
Comment #2
webernet commentedNo parse errors, all that's left should be CamelCase issues.
Comment #3
moshe weitzman commentedcommitted to 6-2 branch. many thanks. i believe that camelcase is the standard for classes so no changes there. i could be wrong though.
it would be great if folks could review the theme_table() patch at http://drupal.org/node/239071. thats the last remaining hurdle before we write a patch for core.
Comment #4
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.