Another module to convert.
Nothing special here, just a few conversions...
I may be misreading the patch, but doesn't the first block move the first update query out of the else {} clause? Looks good otherwise visually.
There are two different queries, one inside the else, one outside, the closing bracket is moved a bit down. *Should* be fine...
Ah! So it is. I didn't see it there. :-) Looks good then. Rock on!
Committed to CVS HEAD. Thanks!
Automatically closed -- issue fixed for 2 weeks with no activity.
Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.
Comments
Comment #1
berdirNothing special here, just a few conversions...
Comment #2
Crell commentedI may be misreading the patch, but doesn't the first block move the first update query out of the else {} clause? Looks good otherwise visually.
Comment #3
berdirThere are two different queries, one inside the else, one outside, the closing bracket is moved a bit down. *Should* be fine...
Comment #4
Crell commentedAh! So it is. I didn't see it there. :-) Looks good then. Rock on!
Comment #5
dries commentedCommitted to CVS HEAD. Thanks!