Zen Cart Logo

Zen Colorbox

Views: 108,370

Results 201 to 220 of 350
06 Sep 2017, 13:41
#201
ideasgirl avatar

ideasgirl

Totally Zenned

Join Date:
Aug 2005
Location:
Trujillo Alto, Puerto Rico
Posts:
1,437
Plugin Contributions:
3

Zen Colorbox

mc12345678:

There appears to be something wrong with the zen_colorbox function that is causing the image filename not to be returned, something similar to what was happening before. Would check to validate that the correct file has been uploaded.

As to the image "size", in the html, I don't see a size discrepancy. Would have to look around to see how the image is "appearing" as 50x50, but don't see anything obvious. On my device it takes the full screen when selecting the additional images.
Hi, any progress on this? I won't be around for a while since the emergency of Irma (I'm in Puerto Rico), but I would like to take again on this when I come back.

I figured out the add. images size issue, so the only thing to be solved is the main image not opening.

06 Sep 2017, 13:53
#202
mc12345678 avatar

mc12345678

Totally Zenned

Join Date:
Jul 2012
Posts:
16,908
Plugin Contributions:
2

Re: Zen Colorbox

ideasgirl:

Hi, any progress on this? I won't be around for a while since the emergency of Irma (I'm in Puerto Rico), but I would like to take again on this when I come back.

I figured out the add. images size issue, so the only thing to be solved is the main image not opening.
"Externally" I can't identify why the software on your site is "deleting" the filename that would be expected is not provided back. Either I need to provide you troubleshooting/data collection actions, the software on the live system needs to be verified to be that which is provided in the download/as provided in github, or access provided to do the same (seeing that you are in prepations to be away).

Let me know how you would like to proceed at this point in light of natural events.

06 Sep 2017, 22:46
#203
jpda avatar

jpda

Zen Follower

Join Date:
Jun 2014
Posts:
157
Plugin Contributions:
0

Re: Zen Colorbox

mc12345678:

"Externally" I can't identify why the software on your site is "deleting" the filename that would be expected is not provided back. Either I need to provide you troubleshooting/data collection actions, the software on the live system needs to be verified to be that which is provided in the download/as provided in github, or access provided to do the same (seeing that you are in prepations to be away).

Let me know how you would like to proceed at this point in light of natural events.

ideasgirl:

I just installed it and when main image tries to open, it continues in a loop and doesn't open, on the other hand I can open additional images and I can browse from pic to pic but can't get main image to show. It will stay in a continues loop.

This is the error from the logs:

This is the error from the logs:
[30-Aug-2017 20:00:29] PHP Fatal error: Call to undefined function zen_lightbox() in /.../includes/templates/my_template/templates/tpl_modules_main_product_image.php on line 24

@mc12345678 Did you notice the error in the log file referring to zen_lightbox()?

Cheers,

jpda

07 Sep 2017, 01:06
#204
mc12345678 avatar

mc12345678

Totally Zenned

Join Date:
Jul 2012
Posts:
16,908
Plugin Contributions:
2

Re: Zen Colorbox

jpda:

@mc12345678 Did you notice the error in the log file referring to zen_lightbox()?

Cheers,

jpda

Along with the disconnect of what was contained in the requested file at the same line? Might say so, though I forgot about that. I kind of figured the issue got resolved once the correct line was returned and after the first suggestion of verifying the file(s).

That discrepancy is partly why the suggestion has been to validate the file installation. Though, I haven't asked if the error message has persisted or if it went away once the "correct" file was put into place or the changes successfully merged. Afterall the line provided from the troubled file is/was correct (still a bit of assumptions being made like that the line was provided from the live site and not from a backup or the download itself.)

07 Sep 2017, 01:20
#205
jpda avatar

jpda

Zen Follower

Join Date:
Jun 2014
Posts:
157
Plugin Contributions:
0

Re: Zen Colorbox

mc12345678:

Along with the disconnect of what was contained in the requested file at the same line? Might say so, though I forgot about that. I kind of figured the issue got resolved once the correct line was returned and after the first suggestion of verifying the file(s).

That discrepancy is partly why the suggestion has been to validate the file installation. Though, I haven't asked if the error message has persisted or if it went away once the "correct" file was put into place or the changes successfully merged. Afterall the line provided from the troubled file is/was correct (still a bit of assumptions being made like that the line was provided from the live site and not from a backup or the download itself.)

Thanks, I was just wondering because the contents of line 24 were from zen-colorbox.

Cheers,

jpda

17 Oct 2017, 04:39
#206
barco57 avatar

barco57

Totally Zenned

Join Date:
Apr 2006
Location:
West Salem, IL
Posts:
2,841
Plugin Contributions:
0

Re: Zen Colorbox

ZC version 1.5.4

[16-Oct-2017 20:57:36 America/Los_Angeles] PHP Warning:  constant(): Couldn't find constant ZEN_COLORBOX_PLUGIN_CHECK in /home/heavenandearthde/public_html/wDgyYtZUuSM/includes/init_includes/init_zcb_config.php on line 101

Why? best way to troubleshoot this?

17 Oct 2017, 09:24
#207
mc12345678 avatar

mc12345678

Totally Zenned

Join Date:
Jul 2012
Posts:
16,908
Plugin Contributions:
2

Re: Zen Colorbox

barco57:

ZC version 1.5.4

[16-Oct-2017 20:57:36 America/Los_Angeles] PHP Warning: constant(): Couldn't find constant ZEN_COLORBOX_PLUGIN_CHECK in /home/heavenandearthde/public_html/wDgyYtZUuSM/includes/init_includes/init_zcb_config.php on line 101

> Why? best way to troubleshoot this?

So, to offer more of an answer to the why, I'd probably need more information. My thought though based on a rereview of the current installation package is to execute the uninstall sql found in admin/includes/installers/zen_colorbox and then allow the install to progress naturally. It could be also that the uninstall need be executed a couple of times to truly get all of the snippets depending on historical attempts to have previously attempted to install the plugin.

Now, if through all that, the mydebug log file still gets generated and only one time not with each and every admin page click, then it's likely a strict php related thing where the code is to some extent parsed before executed and the expectation of the value being defined during execution is not recognized.

Obviously to prevent even the first occurrence of that message, line 101 could be modified to include a check for the variable to be defined before testing against the value in the addition of an && at the beginning... such as:

if (defined($module_constant . '_PLUGIN_CHECK') && ...


But by simply adding that test to address a one-time message could also affect future recognition that the plugin has an available update. So perhaps the conditions above could be further explained. One log file or one everytime the admin is navigated? Success of the database related uninstall followed by reinstall? Did that change the system response, etc...
17 Oct 2017, 11:47
#208
mc12345678 avatar

mc12345678

Totally Zenned

Join Date:
Jul 2012
Posts:
16,908
Plugin Contributions:
2

Re: Zen Colorbox

On further review, the installer sql code does not include a configuration setting to ensure that plugin check value is incorporated for an upgrade from a pre-2.1.0 version. The error would continue to occur for such a situation until either the database upgrade (no file change necessary) was performed by either doing a fresh install to a database by either removing the existing settings (uninstall script from previous post) or to a store that has not had this plugin previously installed or by executing a query that would specifically add the setting present currently in the init file. This is something to be addressed in the next version.

Specifically why? It was a mistake/oversight of my doing.

19 Oct 2017, 14:01
#209
mvstudio avatar

mvstudio

Zen Follower

Join Date:
Apr 2008
Posts:
447
Plugin Contributions:
1

Re: Zen Colorbox

Hello all.
I believe I found a bug (((??))).
With Display Image Counter turned off, the image opens up as a separate page (not as an overlay) and an error is generated - Uncaught SyntaxError: Unexpected token }
I was able to pinpoint the error to includes/classes/zen_colorbox/options.php
For some reason the last empty echo is the one causing the error and the image to open in a separate window.

So replacing this

echo 
',current:'; 
if (ZEN_COLORBOX_COUNTER == 'true') 
{
	echo '"{current} of {total}"'; 
}
else 
{
	echo ''; 
}

with this

echo 
',current:'; 
if (ZEN_COLORBOX_COUNTER == 'true') 
{
	echo '"{current} of {total}"'; 
}
else 
{
	echo '"Image {current}"'; 
}

solved the issue for me. The image overlay works now as it should and it displays the current image number only which is what I wanted.

19 Oct 2017, 15:51
#210
mc12345678 avatar

mc12345678

Totally Zenned

Join Date:
Jul 2012
Posts:
16,908
Plugin Contributions:
2

Re: Zen Colorbox

mvstudio:

Hello all.
I believe I found a bug (((??))).
With Display Image Counter turned off, the image opens up as a separate page (not as an overlay) and an error is generated - Uncaught SyntaxError: Unexpected token }
I was able to pinpoint the error to includes/classes/zen_colorbox/options.php
For some reason the last empty echo is the one causing the error and the image to open in a separate window.

So replacing this

echo
',current:';
if (ZEN_COLORBOX_COUNTER == 'true')
{
echo '"{current} of {total}"';
}
else
{
echo '';
}

> 
> with this
> 
> ```php
echo 
',current:'; 
if (ZEN_COLORBOX_COUNTER == 'true') 
{
    echo '"{current} of {total}"'; 
}
else 
{
    echo '"Image {current}"'; 
}

solved the issue for me. The image overlay works now as it should and it displays the current image number only which is what I wanted.

While wasn't able to reproduce the problem, I can see that having nothing (not even empty quotes) could potentially be a problem.

I believe the intent was to not display any information as compared to identifying an image number; however, it can be controlled/edited as desired/shown above.

Because the default setting is: "image {current} of {total}", in order to not display the "current" (counter as called in the settings) the value has to be altered, changed, or set. My recommendation for a non-counter display would be to set it to a pair of double quotes as below. This will provide a true setting/value which will be empty as compared to the "nothingness" that was provided in the distribution.

echo 
',current:'; 
if (ZEN_COLORBOX_COUNTER == 'true') 
{
    echo '"{current} of {total}"'; 
}
else 
{
    echo '""'; 
}
19 Oct 2017, 17:24
#211
mvstudio avatar

mvstudio

Zen Follower

Join Date:
Apr 2008
Posts:
447
Plugin Contributions:
1

Re: Zen Colorbox

Yes, that's why I wrote "solved the issue for me. The image overlay works now as it should and it displays the current image number only which is what I wanted". I should have added it can be left blank too as intended but with the double quotation marks inside the single ones.
I wanted to only display the image number, but nothing is cool too =)

Weird you couldn't replicate the issue. I got an error during validation and the image opened as a regular link, no overlay or thumbnails. =(

23 Oct 2017, 23:12
#212
barco57 avatar

barco57

Totally Zenned

Join Date:
Apr 2006
Location:
West Salem, IL
Posts:
2,841
Plugin Contributions:
0

Re: Zen Colorbox

1.5.5.e
found that includes/modules/pages/checkout_payment/jscript_zen_colorbox.php causes an issue when using the Authorize.net (AIM) module. Seems to pass the CC number to the confim page but when you click the confirm button you get sent back to the paument page with the following error ""Credit card number is required. - Your credit card could not be authorized for this reason. Please correct the information and try again or contact us for further assistance."

I assume the jscript_zen_colorbox.php is on that page to load the cvv popup in colorbox instead, so I solved the problem by removing jscript_zen_colorbox.php from that path so it doesn't load on that page.

24 Oct 2017, 18:45
#213
mc12345678 avatar

mc12345678

Totally Zenned

Join Date:
Jul 2012
Posts:
16,908
Plugin Contributions:
2

Re: Zen Colorbox

barco57:

1.5.5.e
found that includes/modules/pages/checkout_payment/jscript_zen_colorbox.php causes an issue when using the Authorize.net (AIM) module. Seems to pass the CC number to the confim page but when you click the confirm button you get sent back to the paument page with the following error ""Credit card number is required. - Your credit card could not be authorized for this reason. Please correct the information and try again or contact us for further assistance."

I assume the jscript_zen_colorbox.php is on that page to load the cvv popup in colorbox instead, so I solved the problem by removing jscript_zen_colorbox.php from that path so it doesn't load on that page.

Looking back through things a little, the colorbox option is provided on the checkout_payment page to offer an in place popup of the coupon window help and that only (sure it could likely be used to support entry for some other popup as well I would think, but that was not its original consideration for the payment page). That said, there are a few things not quite right here... For one, the link does not appear until a coupon has been entered (either entered without selecting a payment or having returned to the payment page). Another is that if the link is presented it potentially causes mixed content (which then can carry over into the colorbox segment and becomes more "apparent"). Then there is the fact that it is a javascript like link instead of a simple link used say on the product information page. So I'm not entirely sure which of the above played a factor in the problem seen with authorize.net (AIM) because of the additional information below as well. Of course another slightly more "limiting" condition would be to prevent the javascript from loading if the store doesn't offer coupons to begin with which is likely a good course of action but doesn't necessarily solve the issue.

What I'm seeing in the code based on the error message is that the checkout_payment page is submitted and some amount of data is sent to the checkout_process "page". Then within the checkout_process module received data is evaluated for accuracy and sent to authorize.net (AIM). In this scenario, the error is actually coming directly from authorize.net (AIM) which has identified that the credit card number was not provided. So it seems that either colorbox at one point or another is affecting the credit card number collection or there is some validation that has occurred as part of the submission and prior to processing in the checkout_process module that has disturbed the credit card number.

I've made the following changes though to includes/modules/pages/checkout_payment/jscript_zen_colorbox.php having tested this functionality in place on a cart that hasn't had anything changed in this area but still without an authorize.net (AIM) payment module in place. I also modified includes/modules/order_totals/ot_coupon.php function credit_selection() beginning at line 131 from:

  function credit_selection() {
    global $discount_coupon;
    // note the placement of the redeem code can be moved within the array on the instructions or the title
    $selection = array('id' => $this->code,
                       'module' => $this->title,
                       'redeem_instructions' => MODULE_ORDER_TOTAL_COUPON_REDEEM_INSTRUCTIONS . ($discount_coupon->fields['coupon_code'] != '' ? MODULE_ORDER_TOTAL_COUPON_REMOVE_INSTRUCTIONS : '') . ($discount_coupon->fields['coupon_code'] != '' ? MODULE_ORDER_TOTAL_COUPON_TEXT_CURRENT_CODE . '<a href="javascript:couponpopupWindow(\'' . zen_href_link(FILENAME_POPUP_COUPON_HELP, 'cID=' . $_SESSION['cc_id']) . '\')">' . $discount_coupon->fields['coupon_code'] . '</a><br /><br />' : ''),

to:

  function credit_selection() {
    global $discount_coupon, $request_type;
    // note the placement of the redeem code can be moved within the array on the instructions or the title
    $selection = array('id' => $this->code,
                       'module' => $this->title,
                       'redeem_instructions' => MODULE_ORDER_TOTAL_COUPON_REDEEM_INSTRUCTIONS . ($discount_coupon->fields['coupon_code'] != '' ? MODULE_ORDER_TOTAL_COUPON_REMOVE_INSTRUCTIONS : '') . ($discount_coupon->fields['coupon_code'] != '' ? MODULE_ORDER_TOTAL_COUPON_TEXT_CURRENT_CODE . '<a href="javascript:couponpopupWindow(\'' . zen_href_link(FILENAME_POPUP_COUPON_HELP, 'cID=' . $_SESSION['cc_id'], $request_type) . '\')">' . $discount_coupon->fields['coupon_code'] . '</a><br /><br />' : ''),

This ensured that the link would be generated according to the page's load type (https: or http:). In a way this is becoming a moot point because either a store is going completely SSL or doesn't have one... But, regardless the link generated was one that was http: and then when colorbox got involved this caused mixed content.

The changes I made to includes/modules/pages/checkout_payment/jscript_zen_colorbox.php were:

From:

require_once(DIR_FS_CATALOG . DIR_WS_CLASSES . 'zen_colorbox/display_link.php');

to:

require_once(DIR_FS_CATALOG . DIR_WS_CLASSES . 'zen_colorbox/display_js_link.php');

Operation on a responsive template ZC 1.5.5e version was improved. I don't have credentials against which to test authorize.net (AIM) to see what is/was going on, but if you could answer a few questions perhaps this issue can be resolved.

Does the store support coupons?
Does the problem occur before/without any coupon entered?
What browser(s) were used to test this?
What other javascript related items are loaded beyond the default install and jscript_zen_colorbox.php?
If the store uses coupons then please consider the below as well.
Is it possible to enter a coupon without selecting a payment method and selecting the continue button and does the problem then occur if the payment method is selected and continue with process?

What is it about the confirmation page information that indicates that the number (in its entirety) has been successfully passed to the confirmation page?

25 Oct 2017, 06:44
#214
barco57 avatar

barco57

Totally Zenned

Join Date:
Apr 2006
Location:
West Salem, IL
Posts:
2,841
Plugin Contributions:
0

Re: Zen Colorbox

Does the store support coupons? Yes
Does the problem occur before/without any coupon entered? Yes, no coupon was entered
What browser(s) were used to test this? Chrome and firefox
What other javascript related items are loaded beyond the default install and jscript_zen_colorbox.php? jscript_imagehover.js, https://ssl.google-analytics.com/urchin.js

What is it about the confirmation page information that indicates that the number (in its entirety) has been successfully passed to the confirmation page? Well, I don't know that it has been passed in its entirety, what I see in the payment section is the correct card type, card owner, card expiration and card number as "Card Number:4111XXXXXXXX1111".

25 Oct 2017, 15:46
#215
mc12345678 avatar

mc12345678

Totally Zenned

Join Date:
Jul 2012
Posts:
16,908
Plugin Contributions:
2

Re: Zen Colorbox

barco57:

Yes
Yes, no coupon was entered
Chrome and firefox
jscript_imagehover.js, https://ssl.google-analytics.com/urchin.js

Well, I don't know that it has been passed in its entirety, what I see in the payment section is the correct card type, card owner, card expiration and card number as "Card Number:4111XXXXXXXX1111".
And did the problem persist with the above correction applied?

Wondering it it is the colorbox js file that is causing the issue, because the operation for the colorbox is tested for execution and in the above situation would not be executed.

13 Nov 2017, 04:22
#216
mc12345678 avatar

mc12345678

Totally Zenned

Join Date:
Jul 2012
Posts:
16,908
Plugin Contributions:
2

Re: Zen Colorbox

Submitted version 2.1.2 to ZC forum with the following changes:

  • Updated installer/updater to support upgrade checking independent of other plugins.
    • Updated code in support of anticipated changes for PHP 7.2.
    • Corrected reported issue for some environments where ZEN_COLORBOX_COUNTER is not equal to true.
    • Corrected use of zen_colorbox for usage on the checkout_payment page to use a javascript link.
    • Added Configuration key to support Upgrade from pre 2.1.0 as intended.

Once reviewed and posted will be available from here.

22 Nov 2017, 16:22
#217
mc12345678 avatar

mc12345678

Totally Zenned

Join Date:
Jul 2012
Posts:
16,908
Plugin Contributions:
2

Re: Zen Colorbox

mc12345678:

Submitted version 2.1.2 to ZC forum with the following changes:

Once reviewed and posted will be available from here.

Version 2.1.2 has been reviewed and posted. It is available from this zen-cart link.

30 Nov 2017, 20:03
#218
ideasgirl avatar

ideasgirl

Totally Zenned

Join Date:
Aug 2005
Location:
Trujillo Alto, Puerto Rico
Posts:
1,437
Plugin Contributions:
3

Re: Zen Colorbox

mc12345678:

"Externally" I can't identify why the software on your site is "deleting" the filename that would be expected is not provided back. Either I need to provide you troubleshooting/data collection actions, the software on the live system needs to be verified to be that which is provided in the download/as provided in github, or access provided to do the same (seeing that you are in prepations to be away).

Let me know how you would like to proceed at this point in light of natural events.

I have access to computer and Internet now 3 times a week and I'm ready to do this. At this point what do you need from me?

04 Dec 2017, 15:05
#219
himmon avatar

himmon

New Zenner

Join Date:
Oct 2010
Location:
London
Posts:
50
Plugin Contributions:
0

Re: Zen Colorbox

Hello mc12345678,

Thank you for the new update of colorbox however I have the following error when importing the SQL part of your installation instructions.

WARNING: An Error occurred, please refresh the page and try again.

I receive the error above straight after importing SQL script into SQL Query Executor on Zencart 1.5.5e

This is the first time I've used colorbox so no left over previous build however prior to this I installed Image Handler-5 (Zen Cart 1.5.5+)

I have followed your steps as instructed.

Rename the following folders to match your template folder.

1_Installer_Files/includes/modules/YOUR_TEMPLATE
1_Installer_Files/includes/templates/YOUR_TEMPLATE

Rename the following folder to match your admin folder.

1_Installer_Files/YOUR_ADMIN

Using your FTP program; upload the includes and the (renamed) admin folder to your server (Note: All files make use of the overrides system, and should merge with your current files not replace them.)
to select the install sql.

Any ideas would be greatly appreciated?

Thank you

Nathan

04 Dec 2017, 16:42
#220
mc12345678 avatar

mc12345678

Totally Zenned

Join Date:
Jul 2012
Posts:
16,908
Plugin Contributions:
2

Re: Zen Colorbox

himmon:

Hello mc12345678,

Thank you for the new update of colorbox however I have the following error when importing the SQL part of your installation instructions.

WARNING: An Error occurred, please refresh the page and try again.

I receive the error above straight after importing SQL script into SQL Query Executor on Zencart 1.5.5e

This is the first time I've used colorbox so no left over previous build however prior to this I installed Image Handler-5 (Zen Cart 1.5.5+)

I have followed your steps as instructed.

Rename the following folders to match your template folder.

1_Installer_Files/includes/modules/YOUR_TEMPLATE
1_Installer_Files/includes/templates/YOUR_TEMPLATE

Rename the following folder to match your admin folder.

1_Installer_Files/YOUR_ADMIN

Using your FTP program; upload the includes and the (renamed) admin folder to your server (Note: All files make use of the overrides system, and should merge with your current files not replace them.)
to select the install sql.

Any ideas would be greatly appreciated?

Thank you

Nathan

So, the way the code is written, there should be no need to paste the sql information. Yes instruction 6 of the readme.html is a little "sly", in that it says to paste if the auto-installer has been prevented from loading. The description of the above actions does not indicate that the installer was specifically skipped... Therefore there should be no reason during install to paste sql into the install SQL Patches window.

So, then there are other questions to be answered based on the potential installation.... All of which basically are to verify if the files have been placed into the correct location. Has the admin->configuration menu been reviewed for Zen Colorbox to be listed?

Because of the error message there should be at least one error log in the logs directory related to the plugin. Always start with the most recent myDebug- style file, addressing the issue(s) identified first at the top of the file and then possibly working down through the file or just repeating the action/operation that caused the file to exist.

Can post the log file here, but be sure to obscure the admin folder name or other sensitive information when posting. Also further, please use the # button in the Quick Reply toolbar to generate ```
[CODE][/CODE]