Closed (fixed)
Project:
Localization server
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
15 Aug 2009 at 23:09 UTC
Updated:
31 Aug 2009 at 17:10 UTC
Jump to comment: Most recent file
Comments
Comment #1
gábor hojtsyGood suggestions!
I've committed an all-whitespace cleanup for the module, so I'm hopeful that you can reroll this patch with only the concrete changes. Please do.
Comment #2
tobiasbReroll
Comment #3
gábor hojtsy- There are whitespace issues in the first hunk of the patch.
- In the second one, you query the blocks table, but you do not include the current theme key. May be a nitpick, but a block can theoretically be setup different ways in different themes.
Comment #4
tobiasbComment #5
gábor hojtsyThank you! Renamed $blockcustom to $customizable and committed that. Keep these nice improvements coming!