I actually think this should be fixed in core, and already posted #1072322: Figure out what to do with string concats. It would also be possible to fix this in skinr by splitting the parameter in two via php and using a two-term where clause.
This issue is mostly for documentation, in case someone else finds the postgres problem with skinr and searches here, and as a fallback should the aforementioned issue be rejected for core.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | skinr-removed_concat-replace_system_theme_data-1073108-6.patch | 1.53 KB | moonray |
| #2 | block.skinr_.inc_.1073108.patch | 884 bytes | yuriy.babenko |
| #1 | block.skinr_.inc_.#1073108.patch | 884 bytes | yuriy.babenko |
Comments
Comment #1
yuriy.babenko commentedAttaching a patch.
Comment #2
yuriy.babenko commentedDrupal doesn't like the hash sign in the file name... patch re-attached.
Comment #3
moonray commentedThe explode function call need to ignore any dashes past the first. If delta contained a dash, we'd get errors...
EDIT: e.g. menu-secondary-links
Comment #4
moonray commentedThe patch at #1044222: Remove skin configuration functionality from third-party forms should fix this problem.
Once committed we need to write this patch for the 6.x-2.x branch of Skinr.
Comment #5
moonray commentedComment #6
moonray commentedA working patch to fix this (based on the previous patches).
Comment #8
moonray commented