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

[Bruce1952]

Image Manageris quite the different animal/plugin from Image Handler. Further discussion of "Image Manager" should be in that add-on's support thread (which may or may not exist).

If it is different then why does it have the same basic layout.

The only difference that I can see is that image handler does not check images for products which would say to me that half the old module was cut out.

Why is so.

[nl2dav]

Ok the old module was called image manager version 1.9 that I know of and this is in

Its some kind of fork from IH2 (!) by some guy called boudewijn => https://www.zen-cart.com/showthread....-Manager-addon

This plugin hasn't been worked on anymore for the last 10+ years.

[HeathenMagic]

Hi there,
I am just eliminating log errors, I noticed the following for IH:-

--> PHP Deprecated: addslashes(): Passing null to parameter #1 ($string) of type string is deprecated in /includes/classes/observers/ImageHandlerObserver.php on line 82.

I know this is because I am on php 8.3, but just thought I would see if there was a possible quick fix? Thanks in advance.

[lat9]

Hi there,
I am just eliminating log errors, I noticed the following for IH:-



I know this is because I am on php 8.3, but just thought I would see if there was a possible quick fix? Thanks in advance.

First, that's not the full log, since the full log would show the page (and product number) being displayed. It would also help to know what version of Zen Cart is in use.

That log's implying that the associated product's $products_name is NULL and I'm surprised that you're not seeing other logs for the page. Let me know the Zen Cart version and I'll get you a correction.

[HeathenMagic]

First, that's not the full log, since the full log would show the page (and product number) being displayed. It would also help to know what version of Zen Cart is in use.

That log's implying that the associated product's $products_name is NULL and I'm surprised that you're not seeing other logs for the page. Let me know the Zen Cart version and I'll get you a correction.

Hi there, sorry I should have mentioned. I am on 2.01 zencart and php 8.3. I will try and find the log again, I clear after a few logs pile up again. But I will get back with a full one, sorry about that. I think as it was from one log with an unrelated warning, I did too small a snippet.

[HeathenMagic]

Am struggling to find that log again, it must be a particular action that triggers it. But I found this one whilst I was waiting for that log to come up:-

25-Jun-2025 06:27:51 Europe/London] Request URI: /products_all?products_id=20890&view=rows, IP address: 207.46.13.17, Language id 1
#0 [internal function]: zen_debug_error_handler()
#1 /includes/classes/observers/ImageHandlerObserver.php(82): addslashes()
#2 /includes/classes/traits/NotifierManager.php(106): ImageHandlerObserver->update()
#3 /includes/functions/extra_functions/wt_wokiee_template_functions.php(597): base->notify()
#4 /includes/functions/extra_functions/wt_wokiee_template_functions.php(819): wt_wokiee_additional_images()
#5 /includes/modules/****/product_listing.php(168): get_wt_wokiee_product_content()
#6 /includes/templates/****/templates/tpl_modules_product_listing.php(10): include('/home/****/...')
#7 /includes/templates/****/templates/tpl_products_all_default.php(40): require('/home/****/...')
#8 /includes/templates/****/common/tpl_main_page.php(167): require('/home/****/...')
#9 /index.php(94): require('/home/*****/...')
--> PHP Deprecated: addslashes(): Passing null to parameter #1 ($string) of type string is deprecated in /includes/classes/observers/ImageHandlerObserver.php on line 82.

[lat9]

Am struggling to find that log again, it must be a particular action that triggers it. But I found this one whilst I was waiting for that log to come up:-

Thanks for that. It appears that /includes/functions/extra_functions/wt_wokiee_template_functions.php, on line 597, is issuing a non-standard notification that IH is watching for. The base code issues that notification out of /includes/modules/additional_images:

        // -----
        // This notifier lets any image-handler know the current image being processed, providing the following parameters:
        //
        // $p1 ... (r/o) ... The current product's name
        // $p2 ... (r/w) ... The (possibly updated) filename (including path) of the current additional image.
        //
        $GLOBALS['zco_notifier']->notify('NOTIFY_MODULES_ADDITIONAL_IMAGES_GET_LARGE', $products_name, $products_image_large);

That log implies that the notification is being issued with a non-string $products_name parameter.

You can update IH's /includes/classes/observers/ImageHandlerObserver.php, starting at line 75, to add the string-casts highlighted below to work-around that issue:

            case 'NOTIFY_MODULES_ADDITIONAL_IMAGES_GET_LARGE':
                $products_name = (string)$p1;
                $products_image_large = (string)$p2;
                if (ih_image_supported($products_image_large) === false) {
                    return;
                }

[lat9]

v5.3.6 of Image Handler 5 is now available for download: https://www.zen-cart.com/downloads.php?do=file&id=2169

This release casts notification variables to prevent unwanted PHP logs due to notification anomalies associated with the wokiee template.

[HeathenMagic]

Thanks for that. It appears that /includes/functions/extra_functions/wt_wokiee_template_functions.php, on line 597, is issuing a non-standard notification that IH is watching for. The base code issues that notification out of /includes/modules/additional_images:

        // -----
        // This notifier lets any image-handler know the current image being processed, providing the following parameters:
        //
        // $p1 ... (r/o) ... The current product's name
        // $p2 ... (r/w) ... The (possibly updated) filename (including path) of the current additional image.
        //
        $GLOBALS['zco_notifier']->notify('NOTIFY_MODULES_ADDITIONAL_IMAGES_GET_LARGE', $products_name, $products_image_large);

That log implies that the notification is being issued with a non-string $products_name parameter.

You can update IH's /includes/classes/observers/ImageHandlerObserver.php, starting at line 75, to add the string-casts highlighted below to work-around that issue:

            case 'NOTIFY_MODULES_ADDITIONAL_IMAGES_GET_LARGE':
                $products_name = (string)$p1;
                $products_image_large = (string)$p2;
                if (ih_image_supported($products_image_large) === false) {
                    return;
                }

Thanks for that, yes I was getting a series of notifications on same log for same error. I have applied fix as you specified :-)