Closed (fixed)
Project:
Override Node Options
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
28 Jan 2009 at 15:58 UTC
Updated:
3 Jan 2014 at 00:07 UTC
Jump to comment: Most recent
Comments
Comment #1
marcushenningsen commentedComment #2
drewish commentedLooks like an improvement but there's still a ton of formatting issues.
Comment #3
joachim commentedI'd already committed fixes to the spacing of 'if' statements before I saw this patch.
I'm afraid it's probably not going to apply and rerolling this sort of patch is just as tedious if not more so than redoing it from scratch.
I've tackled a few of the indentation problems (until my eyes began to glaze).
The patch above is obsolete; marking this as active again and giving it a better title.
I'm tagging this as Novice, as it's probably a good exercise for someone wanting to try making a patch or learning the Drupal code style. I would suggest only tackling one function at a time though!
Comment #4
dave reidCommitted everything except for the if-statement wrappings since we don't really have a standard on it yet.
http://drupal.org/cvs?commit=307120