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.
This morning I just updated the color box module to version: 7.x-2.2
and since then it stopped working, including the links which used to open using colorbox-node
Please check, as I am currently struck on live server.
Comment | File | Size | Author |
---|---|---|---|
#23 | colorbox_triggerfield_class_fix_1894332.patch | 1.32 KB | frjo |
#22 | colorbox-view-export.txt | 6.39 KB | Anandyrh |
Comments
Comment #1
Anandyrh CreditAttribution: Anandyrh commentedI am unable to revert back too :(
Comment #2
frjo CreditAttribution: frjo commentedHave you updated the Colorbox plugin to the latest version as well?
What does the status page at "admin/reports/status" say?
Comment #3
Anandyrh CreditAttribution: Anandyrh commentedI have updated the Colorbox plugin to the latest and the status pages shows [Colorbox plugin 1.3.23] in green,
and still all the pages where I am using the colorbox they all the broken, until and unless I disable this module.
Comment #4
ytorf CreditAttribution: ytorf commentedI ran into the same problem when I upgraded to 7.x-2.2. Looks like the path to the Javascript files has changed. If you download the new Colorbox library and replace the existing one in sites/all/libraries/colorbox it should work. I just moved jquery.colorbox.js and jquery.colorbox-min.js from sites/all/libraries/colorbox/colorbox to sites/all/libraries/colorbox and it started working again.
Comment #5
ytorf CreditAttribution: ytorf commentedAlso you can download the new Colorbox library here: http://www.jacklmoore.com/colorbox
Comment #6
Anandyrh CreditAttribution: Anandyrh commentedThe path on my site is sites/all/libraries/colorbox/jquery.colorbox.js and the same for the others.
but still I have the same problem.
Comment #7
Anandyrh CreditAttribution: Anandyrh commentedI also tried by uninstalling and installing the colorbox and the colorbox node module the issue did persists.
Comment #8
frjo CreditAttribution: frjo commentedCan you post a link to your site here, or send it to me directly if you prefer.
Comment #9
Anandyrh CreditAttribution: Anandyrh commentedhi frijo,
I had a backup of live server so I could restore the old version on live, and moved the unstable d7 to local server so it's not possible to share the link now :(
Comment #10
Anandyrh CreditAttribution: Anandyrh commentedcurrently I am using a views colorbox on ubercart product pages, and when the colorbox module is enabled all the products link give 5
00 Internal server error
. else rest of nodes of other content types are working fine.Comment #11
frjo CreditAttribution: frjo commentedGood to here that you got your site working again.
If you in some way can give me access to the troubled version of the site I would be happy to take a look at it.
Comment #12
Anandyrh CreditAttribution: Anandyrh commentedsure, will try it... but the issue still persist on local site, with 7.x-2.2 version. so I feel the ticket should be kept open/active state.
Comment #13
Anandyrh CreditAttribution: Anandyrh commentedhi @frjo,
This issue is due to field type 'Colorbox: Colorbox trigger' when I remove this trigger field from the view all works fine on node.
There is something seriously wrong with it, as it collapses the entire node.
Comment #14
Anandyrh CreditAttribution: Anandyrh commentedComment #15
frjo CreditAttribution: frjo commentedDoes Colorbox on a standard image field work as it should?
Below is the only small change that has been made to the Colorbox trigger part of the code. Could you try to revert that and see if your problems disappear? Not that it should create any problems.
Comment #16
Anandyrh CreditAttribution: Anandyrh commentedHi Frjo,
Thanks a lot for the quick update, please let me know where should the above code need to be updated? (On which file?)
Comment #17
Anandyrh CreditAttribution: Anandyrh commentedSorry if am wrong.
As per my understanding on the above code shared on #15
on the file located in path:
[sites/all/modules/colorbox/views/colorbox_handler_field_colorbox.inc]
replace the code on line number 193:Replace:
$link_tag = l($link_text, $link_target, $link_options);
With:
$link_tag = l($link_text, '', $link_options);
Solves my problem.
@Frijo, please alert me if I have done anything wrong.
Comment #18
morphay CreditAttribution: morphay commentedhi guys!
I have the same problem with "Colorbox: Colorbox trigger" , remove code from colorbox_handler_field_colorbox.inc not solved the problem. Then I returned the old ColorBox v1.3.20.1 and it worked. Drupal module version 7.x-2.2.
Comment #19
kbrinnerI am running Colorbox 7.x-2.2 on Drupal 7.19 and found that after updating Colorbox to 2.2 it wasn't working correctly. However, updating the Colorbox library solved this issue. Although this doesn't seem to work for others, I wanted to report that it's working for me. It's currently on my local machine only but when I update to prod I can post a link if it would be helpful.
Comment #20
Miao1994 CreditAttribution: Miao1994 commentedI have the same problem, colorbox doesn't work with standard image field anymore after updating to 7.x-2.2. Colorbox plugin 1.3.23.
Comment #21
frjo CreditAttribution: frjo commentedIf the Colorbox does not work for normal image fields then it's almost certain that the Colorbox plugin (library) has not been updated correctly just as @kbrinner explains.
@Anandyrh, You did exactly right and I'm quite surprised that this was causing your problem. If you can not make the site available to me, could you send me and export of the view you are using? That might be enough for me to fins out whats causing this.
Comment #22
Anandyrh CreditAttribution: Anandyrh commentedHi Frjo
I have attached the view. This view was built to open images inside a colorbox, images which are attached to a node.
Comment #23
frjo CreditAttribution: frjo commented@Anandyrh Could you try the attached patch on a 7.x-2.x-dev download of the Colorbox module? I found a bug where a class attribute was a string and not a array.
PS. Why do you use the complicated Colorbox trigger field for this and not simple use the Colorbox formatter on the image field? As far as I can see uou are simply displaying a image in the Colorbox.
Comment #24
frjo CreditAttribution: frjo commentedComment #25
Anandyrh CreditAttribution: Anandyrh commentedThanks a lot Frjo,
Sure I'll try the 7.x-2.x-dev with patch and will let you know the result.
The reason I am using the Colorbox trigger field is because I have 6 images attached to a node and I need to display the first one as a Thumbnail and on click of it I had to show all the 6 images in a popup.
Comment #26
Miao1994 CreditAttribution: Miao1994 commentedI just uploaded to 7.x-2.2-dev from 7.x-2.2 and everything seems to work fine. I tried it on standard image field. Module works also using colorbox-node and views. Drupal 7.18 and Colorbox plugin 1.3.23.
Comment #27
Tino CreditAttribution: Tino commentedSimple imagefield with colorbox display was not displaying clicked image in a colorbox anymore after upgrading to 7.x-2.2, but just loaded it as a seperate page.
Updating to 7.x-2.x-dev of 2013-Jan-23 fixed it for me as well.
Comment #28
frjo CreditAttribution: frjo commented@Tine this issue is only for the Colorbox trigger field. If Colorbox works with 7.x-2.x-dev of 2013-Jan-23 it will work with 7.x-2.2 as well since there are no difference between them.
Comment #29
Tino CreditAttribution: Tino commented@frjo: Sorry for posting in the wrong issue queue. But I can assure you there is a difference on my site. As soon as I completely reinstall 2.2, colorbox images are not opened in a colorbox overlay anymore, but in a new page. Reinstalling 2.x-dev and it works as expected.
Comment #30
frjo CreditAttribution: frjo commentedCommitted to 7-dev.