Closed (duplicate)
Project:
User Relationships
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
30 May 2012 at 04:02 UTC
Updated:
1 Jun 2012 at 20:35 UTC
Jump to comment: Most recent file
Comments
Comment #2
berdirThanks!
The problem with fixing coding style is that it often conflicts with other patches but there aren't many big ones needs review currently. Let's fix a few additional things while we're at it (see below) and then I'll commit it.
Let's fix this as well while you're at it. Add a @todo stating that this only exists to define test dependencies.
While you're at it, can you fix the comments of *changed* (not everything, that would be too much) lines as well? This isn't reported by coder.module but comments should start with a space, then an uppercase character and end with a ".".
Haven't seen this before, interesting...
Same here, let's fix stuff that we touch.
"iff" => "if"
The correct thing to do here is actually to move the comment above and format properly.
Comment #3
hefox commentedthe first patch was mostly automated via http://upgrade.boombatower.com/tools, then ran it through coder locally to fix what it couldn't fix, then fixed some additional items like the hook_menu.
Anyhow updated patch with above.
Comment #4
berdirThanks, haven't done a second review so this might not be perfect but it's a huge step in the right direction. Commited.