code style cleanup
TUc - January 28, 2009 - 15:58
| Project: | Override Node Options |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | active |
| Issue tags: | Novice |
Jump to:
Description
While patching some previous issues I came accross some code-style errors.
This is a general patch which solves it all in one (two previous patches + code style).
greetz,
TUc
| Attachment | Size |
|---|---|
| override_node_options.general.patch | 16.86 KB |
#1
#2
Looks like an improvement but there's still a ton of formatting issues.
#3
I'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!