Image Handler 5 (for v1.5.5 - v1.5.8) Support Thread

Documentation says "For additional information, please refer to the documentation on the plugin's GitHub repository; see above for the link." - but there is no link to Github on this page.

swguy:

Documentation says "For additional information, please refer to the documentation on the plugin's GitHub repository; see above for the link." - but there is no link to Github on this page.
That link is present in the first note in the overview.

Sure enough. Might be worth adding text-decoration:underline to make that a bit more obvious, and making the link span both words (Github repository). It's really hard to spot right now.

It would appear that the problem with Chrome is that Google is attempting to become the new IE. They played nice for some time and now that they have their 'customer' base, they are starting to ignore the browser standards in favor of their own extensions. They have their own 'popup' extension. I guess it's time for me to drop Chrome into the same bit bucket as IE. Haven't tried out Edge yet.

ZC 1.5.6a, clean install of IH5.1.1 and I'm getting these errors logged
Image Handler: Invalid 'small' background specified (-transparent-), using default.
This, even though the admin config section clearly states "Set to -transparent- to keep transparency."

I see that the notes on version 5.1.0 said "#101: Correct incorrect warning about background definitions."
Was this the same issue, and has 5.1.1 broken/overwritten some previous bug fixes?

And what is the recommended fix for the warnings logged re LARGE_IMAGE_HEIGHT and LARGE_IMAGE_WIDTH, as mentioned in post #148
Should I just create those two defines in the Images config page so that I can assign values to them?

strelitzia:

ZC 1.5.6a, clean install of IH5.1.1 and I'm getting these errors logged
Image Handler: Invalid 'small' background specified (-transparent-), using default.
This, even though the admin config section clearly states "Set to -transparent- to keep transparency."

I see that the notes on version 5.1.0 said "#101: Correct incorrect warning about background definitions."
Was this the same issue, and has 5.1.1 broken/overwritten some previous bug fixes?

And what is the recommended fix for the warnings logged re LARGE_IMAGE_HEIGHT and LARGE_IMAGE_WIDTH, as mentioned in post #148
Should I just create those two defines in the Images config page so that I can assign values to them?
What is your setting for Images->IH small images background?

It does look like I missed the incorporation of some of the zc155-related changes (like #101) on the transition to v5.1.1. I'll update GitHub as I know more.

lat9:

What is your setting for Images->IH small images background?

It's set to -transparent-

I'll keep an eye on github for any updates

strelitzia:

It's set to -transparent-

I'll keep an eye on github for any updates
I'll need to review that documentation, as the value should be set to **transparent **(without the leading/trailing minus signs).

strelitzia:

ZC 1.5.6a, clean install of IH5.1.1 and I'm getting these errors logged
Image Handler: Invalid 'small' background specified (-transparent-), using default.
This, even though the admin config section clearly states "Set to -transparent- to keep transparency."

I see that the notes on version 5.1.0 said "#101: Correct incorrect warning about background definitions."
Was this the same issue, and has 5.1.1 broken/overwritten some previous bug fixes?

And what is the recommended fix for the warnings logged re LARGE_IMAGE_HEIGHT and LARGE_IMAGE_WIDTH, as mentioned in post #148
Should I just create those two defines in the Images config page so that I can assign values to them?
Regarding those two constants, it looks like the changes got into the base IH class (/includes/classes/observers/ImageHandlerObserver.php) but didn't make it into the FualSlimBoxObserver.php file in the same directory.

I'll be changing the constants there (LARGE_IMAGE_HEIGHT => LARGE_IMAGE_MAX_HEIGHT, LARGE_IMAGE_WIDTH => LARGE_IMAGE_MAX_WIDTH) to align with the IH-provided configuration values.

lat9:

What is your setting for Images->IH small images background?

It does look like I missed the incorporation of some of the zc155-related changes (like #101) on the transition to v5.1.1. I'll update GitHub as I know more.
After further review, all changes were incorporated on the transition to v5.1.1.

I've just submitted v5.1.2 to the Zen Cart plugins for review. Once approved, it can be downloaded from this link: https://www.zen-cart.com/downloads.php?do=file&id=2169

 This release contains the changes associated with the following GitHub issues:

#140: Suppress errors from imagecreatefrom PHP functions.
#141: Correct PHP 7.2+ invalid numeric value errors.
#142: Correct IH log contents when DIR_FS_CATALOG is set to /.
#144: Additional image "add" button can inhibit workflow.
#146: Correct handling for missing LARGE_IMAGE_WIDTH/_HEIGHT constants.
#147: Correct configuration description for s/m/l image-background settings.

I'll post back here once the Zen Cart download is available; you can also download from the plugin's GitHub repository.

lat9:

I've just submitted v5.1.2 to the Zen Cart plugins for review. Once approved, it can be downloaded from this link: https://www.zen-cart.com/downloads.php?do=file&id=2169

 This release contains the changes associated with the following GitHub issues:

#140: Suppress errors from imagecreatefrom PHP functions.
#141: Correct PHP 7.2+ invalid numeric value errors.
#142: Correct IH log contents when DIR_FS_CATALOG is set to /.
#144: Additional image "add" button can inhibit workflow.
#146: Correct handling for missing LARGE_IMAGE_WIDTH/_HEIGHT constants.
#147: Correct configuration description for s/m/l image-background settings.

I'll post back here once the Zen Cart download is available; you can also download from the plugin's GitHub repository.
Now available for download!

Hello there,
Thanks for the update. I added some additional images to some items, but they are not showing on the actual site as additional images. But I can see some previous items that already had additional images added prior to updating image handler do show additional images. So I am not sure if it is Image Handler cache or some new rewriting rule I need to adhere to? I don't have a cache system on the item pages so I know its not that.

Regards,
James

HeathenMagic:

Hello there,
Thanks for the update. I added some additional images to some items, but they are not showing on the actual site as additional images. But I can see some previous items that already had additional images added prior to updating image handler do show additional images. So I am not sure if it is Image Handler cache or some new rewriting rule I need to adhere to? I don't have a cache system on the item pages so I know its not that.

Regards,
James
Could you point me to a product that does work ... and one that doesn't?

Thanks for your reply, I p'med you.

The issue that HeathenMagic has run into is mismatched image extensions. IH does not consider an extension of .jpg to be the same as .JPG (see this documentation link for additional information). That's why some products' additional images aren't being displayed.

We've found a little gem that might be of help to those with the mismatched extensions AND the fact our suppliers like a, b, c instead of 01, 02, 03. It also helps fight the leading caps problem.

It's a program called Spacetornado Renamer. http://www.spacetornado.com/Renamer/

lat9:

The issue that HeathenMagic has run into is mismatched image extensions. IH does not consider an extension of .jpg to be the same as .JPG (see this documentation link for additional information). That's why some products' additional images aren't being displayed.

My apologies for that, totally my mistake not noticing the upper case and lower case filetype differences. I was able to rename and reupload accordingly. Was due to using different camera for first image a long time ago.

Hi
For zc156 new install this (documentation) line a bit confusing ( for me ):

2 -For Zen Cart versions prior to v1.5.6, rename the two (2) YOUR_TEMPLATE directories to match the name of your currently-active Zen Cart template

The files are still necessary for zc156?
Thanks

ok, ignore that. There are observers now.