Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.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 CreditAttribution: yuriy.babenko commentedAttaching a patch.
Comment #2
yuriy.babenko CreditAttribution: yuriy.babenko commentedDrupal doesn't like the hash sign in the file name... patch re-attached.
Comment #3
moonray CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: moonray commentedComment #6
moonray CreditAttribution: moonray commentedA working patch to fix this (based on the previous patches).
Comment #8
moonray CreditAttribution: moonray commented