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 got bitten by this bug when enabling the CDN module. It causes a lot of link to be changed to point to an external site, and I believe that breaks the link that colorbox creates with the l() function call in colorbox.theme.inc's theme_colorbox_imagefield().
Patch to follow.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | colorbox-links-in-colorbox-imagefield-are-sometimes-broken-2061207-3.patch | 683 bytes | badrange |
| #1 | colorbox-links-in-colorbox-imagefield-are-sometimes-broken-2061207-1.patch | 637 bytes | badrange |
Comments
Comment #1
badrange CreditAttribution: badrange commentedThis patch seems to fix the problem for any url-parameters, not just the itok one that I tripped over.
Comment #2
frjo CreditAttribution: frjo commentedComment #3
badrange CreditAttribution: badrange commentedWe are running colorbox and cdn on a site with several translations, and noticed that the image links didn't work on the translated pages even with our patch; this one fixes it somehow.
Comment #4
Vacilando CreditAttribution: Vacilando commentedI've run into the same problem here -- and the second patch solved it nicely; thank you, @badrange!
Increasing priority one notch to bring it to the attention of the maintainer.
Comment #5
frjo CreditAttribution: frjo commentedComment #6
frjo CreditAttribution: frjo commentedPart of the just released 7.x-2.5, thanks for the patch!