Hi
Hi
After upgrade from 7.x-2.17 to 7.x-2.19
status reports message:
"The version of the Colorbox plugin library could not be detected. You need to download the Colorbox plugin, extract the archive and place the colorbox directory in the sites/all/libraries directory on your server" !?
Still not works even if download the plugin
Colorbox and its plugin library has always been part of Commerce Kickstart. and there was ever no need to download before a plugin library for colorbox.

Best regards
Thanks much for suggestions.

CommentFileSizeAuthor
#9 plugin_version_detection-2360375-9.patch496 bytesrolfmeijer
par défaut 2014-10-20 à 14.29.01.jpg51.23 KBSalif

Comments

Salif’s picture

Salif commented

Colorbox Plugin not being recognized:
Here is a similar but Very old issue discussed on colorbox issue: drupal.org/node/1116672
It also concerns the colorbox plugin in commerce kickstart, from comment #18 going down

lsolesen’s picture

lsolesen commented

#1116672: Colorbox Plugin not being recognized

Salif’s picture

Salif commented

Yes the issue (Colorbox Plugin not being recognized) has been fixed and closed a long time ago.
The solution suggested was to use drush command:
$ drush colorbox-plugin.
But the same issue appears again when updating to commerce kickstart 7.x-2.19
$ drush colorbox-plugin is not the solution as CK package comes with colorbox already integrated in it

tnfno’s picture

tnfno commented

Same problem, have tried all 'old tricks' I could find but still get the error and Colorbox is not working. It worked fine before this Kickstart update.

tnfno’s picture

tnfno commented

A temporary workaround is to update the colorbox module. Download the latest Colorbox 7.28 into profiles/commerce_kickstart/modules/contrib/colorbox and then do a drush colorbox-plugin. Works for me adter that, at least until next kickstart update.

Salif’s picture

Salif commented

Thanks @ tnfno
#5 works

vacho’s picture

vacho commented

#5 don't work for me.

lsolesen’s picture

lsolesen commented
Assigned: Salif » Unassigned
Status: Active » Postponed (maintainer needs more info)

@Salif Does Kickstart bundle the wrong version of Colorbox or what seems to be the problem here?

rolfmeijer’s picture

rolfmeijer commented
StatusFileSize
new plugin_version_detection-2360375-9.patch496 bytes

The difference is the way the version is detected. This is done on line 84 of the colorbox.module.

Instead of:
'pattern' => '@(?i:Colorbox) v([0-9\.a-z]+)@',
you’ll need:
'pattern' => '@(?i:Colorbox)\sv?([0-9\.a-z]+)@',

I have a patch tested on Commerce Kickstart 2.12, which uses colorbox 2.5.

Hope this helps.

lsolesen’s picture

lsolesen commented

@rolfmeijer - so this is a colorbox issue - or a kickstart issue?

scott859’s picture

scott859 commented

@rolfmeijer - Thank you - that patch (#9) worked for me.

lsolesen’s picture

lsolesen commented

Ah, I see now. Colorbox is bundled with CK 2.x. But why is it bundled, when it is not enabled by default. Shouldn't we leave this up to the developers of the sites?

lsolesen’s picture

lsolesen commented

This belongs in a new issue #2426715: Remove Colorbox from build

lsolesen’s picture

lsolesen commented
Status: Postponed (maintainer needs more info) » Needs review

Pull request: https://github.com/commerceguys/commerce_kickstart/pull/92

Waiting for Travis tests results: https://travis-ci.org/lsolesen/commerce_kickstart/builds/50815834

  • bojanz committed 112dac8 on 7.x-2.x 
    Merge pull request #92 from lsolesen/colorbox

Issue #2360375 by Salif,...
  • lsolesen committed 1cdc0fc on 7.x-2.x authored by rolfmeijer
    Issue #2360375 by Salif, rolfmeijer: The version of the Colorbox plugin...
bojanz’s picture

bojanz commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.