Image Handler 4 (for v1.5.x) Support Thread

[Design75]

Not sure that this is aused by Image Handler.. Likely caused by whatever other image module you have installed or that came bundled with Tableau..

Personally I wish template authors would NOT bundle in things like lightboxes and Image Handler in with their templates.. they are GOTCHAS waiting to happen IMHO..

Yep, Tableau is one of the worst, the only thing that is not included in the package is Zen Cart

[DivaVocals]

Yep, Tableau is one of the worst, the only thing that is not included in the package is Zen Cart

**snickers**

The problem with all this bundling besides the fact that I don't always LIKE the bundling choices the author makes in choosing the bundled modules (ie: I prefer Zen Colorbox, & the author may bundle in Zen Lightbox for example) is that it sometimes makes troubleshooting issues a bit tricky.. Can't tell what's caused by the template, and what's caused by the module, or what's being caused by something else bundled in.. If sunshinelock's issues started RIGHT after installing the template, and the template was installed over a VANILLA Zen Cart install, I'd say the best bet is to go to the author of the template and ask them for assistance to resolve.. I am personally not going to be able to spend a lot of time trying to GUESS what MIGHT be the solution as my free time is limited.. Don't know what effect some of the modules bundled in with this particular template might have..

[dbltoe]

Had this little glitch before on another site with IH3 and wound up dropping it as we found no answer. Just had this (and two others show up on a recent upgrade to 1.5.4.

[04-Feb-2015 16:25:51 America/Chicago] #1 strstr() called at [/home/OURSITE/public_html/includes/classes/bmz_image_handler.class.php:129]
#2 ih_image->determine_image_sizetype() called at [/home/OURSITE/public_html/includes/classes/bmz_image_handler.class.php:55]
#3 ih_image->ih_image() called at [/home/OURSITE/public_html/includes/functions/extra_functions/functions_bmz_image_handler.php:55]
#4 handle_image() called at [/home/OURSITE/public_html/includes/functions/html_output.php:187]
#5 zen_image() called at [/home/OURSITE/public_html/includes/templates/template_default/popup_image/tpl_main_page.php:17]
#6 require(/home/OURSITE/public_html/includes/templates/template_default/popup_image/tpl_main_page.php) called at [/home/OURSITE/public_html/index.php:97]

[04-Feb-2015 16:25:51 America/Chicago] PHP Warning: strstr(): Empty needle in /home/enignet/public_html/includes/classes/bmz_image_handler.class.php on line 129

Please note that this was NOT caused by either a faulty installation or the mod itself.
We found the problem absolutely by accident.
SOLUTION:
Somehow, the menu entry for Configuration >> Images >> Product Info - Image Large Suffix was BLANK The default is _LRG. Adding this back into the menu item cured the problem.

The symptom that led us to the _LRG setting was that IH4 was working fine EXCEPT when looking at the large image on a product's page. Also, when viewing a product in the IH4 Image Manager, there was no Large size image shown.

Again, this was not a fault of the installation or IH4 itself.

Might be something to consider having the setup look for when auto-loading. It could say something like, "Your Images >> Product Info - Image Suffixes cannot be blank. IH4 does not care if the suffix is _LRG or _GRL BUT it does need something there in both the Medium and Large or it will throw the error above.

[dbltoe]

Time out got me. Meant to add

BTW: Everything is working well with 1.5.4, Apache, 5.4.36 PHP, 5.5.40-cll MySQL

[DivaVocals]

Had this little glitch before on another site with IH3 and wound up dropping it as we found no answer. Just had this (and two others show up on a recent upgrade to 1.5.4.

Please note that this was NOT caused by either a faulty installation or the mod itself.
We found the problem absolutely by accident.
SOLUTION:
Somehow, the menu entry for Configuration >> Images >> Product Info - Image Large Suffix was BLANK The default is _LRG. Adding this back into the menu item cured the problem.

The symptom that led us to the _LRG setting was that IH4 was working fine EXCEPT when looking at the large image on a product's page. Also, when viewing a product in the IH4 Image Manager, there was no Large size image shown.

Again, this was not a fault of the installation or IH4 itself.

Might be something to consider having the setup look for when auto-loading. It could say something like, "Your Images >> Product Info - Image Suffixes cannot be blank. IH4 does not care if the suffix is _LRG or _GRL BUT it does need something there in both the Medium and Large or it will throw the error above.

Wow!!! Nice catch!!! Don't cha just LOVE la9's backtrace plugin!!!! Handy as HE!! for tracing strange issues..

Now how to fix/prevent from happening again..

Hmmmm..

Thinking that adding a warning to the auto-installer is absolutely correct (and I'll see if I can figure out how to do that), but I'm wondering if in addition to that perhaps the auto-installer should just update the value to _LRG if this is found to contain a blank or NULL value (something I KNOW I can manage to update)..

OR..

Maybe the auto installer should update the value to DON'T_DELETE_THIS_OR_IMAGE_HANDLER_WILL_BREAK

[dbltoe]

backtrace is good but it wasn't used. Just happened to have two admin menus up from different sites and saw the missing _LRG. When you click to edit, it tells you that the default is _LRG.
Same thing happens with _MED just LOTS MORE WORSER.
Perhaps the installer should change them both to _BOO

[DivaVocals]

backtrace is good but it wasn't used.

Oh I realized you found the problem without backtrace.. I just recognized the backtrace output from your error log..

Just happened to have two admin menus up from different sites and saw the missing _LRG. When you click to edit, it tells you that the default is _LRG.
Same thing happens with _MED just LOTS MORE WORSER.

GADS!!! Found a similar kind of weird one off issue just two days ago with COWOA..

Thanks for sharing this.. I was gonna pull the trigger on the new IH4 this weekend.. Hoping that this is a QUICK fix for me to make happen..

Perhaps the installer should change them both to _BOO

**snickers**

[niccol]

Quote Originally Posted by dbltoe View Post
Perhaps the installer should change them both to _BOO

that would not be so hard. But it would be much better to mend the mod itself so that if the setting was blank it didn't create an error. Judging from the errors what is happening is that IH is checking to see if a large image exists. It is matching the suffix, _LRG (or whatever).

It shouldn't be too hard to put a conditional around all that code so that if the suffix is blank it doesn't even bother looking for a large image.

If not having any suffix messes up any other aspects of images implicit in Zen, which is possible, then it would be better to put code in Zen that assured that the setting could never be blank.

Relying on the auto-installer only to ensure that the setting is not blank seems like a band-aid.

[DivaVocals]

Relying on the auto-installer only to ensure that the setting is not blank seems like a band-aid.

Of course it is.. But it's a bandaid that I KNOW how to create.. You know I'm not REALLY a developer.. I only play one on TV..

[niccol]

Of course it is.. But it's a bandaid that I KNOW how to create.. You know I'm not REALLY a developer.. I only play one on TV..

Yeah, but it is probably easier to change the module than change the auto-installer :-) You've got the line number form the errors

[04-Feb-2015 16:25:51 America/Chicago] PHP Warning: strstr(): Empty needle in /home/enignet/public_html/includes/classes/bmz_image_handler.class.php on line 129

You are probably going to do something like this

	function determine_image_sizetype() {
		global $ihConf;
		
		if (strstr($this->src, $ihConf['large']['suffix'])) {
			$this->sizetype = 'large';
		} elseif (strstr($this->src, $ihConf['medium']['suffix'])) {
			$this->sizetype = 'medium';
		} elseif ((intval($this->width) == intval($ihConf['small']['width'])) && (intval($this->height) == intval($ihConf['small']['height']))) {
			$this->sizetype = 'small';
		} else
		$this->sizetype = 'generic';
	}

changed to:

	function determine_image_sizetype() {
		global $ihConf;
		
		$this->sizetype = 'generic';

		if($ihConf['large']['suffix'] != '')
		{
			if (strstr($this->src, $ihConf['large']['suffix'])) {
				$this->sizetype = 'large';
			}			
		}

		if($ihConf['medium']['suffix'] != '')
		{
			if (strstr($this->src, $ihConf['medium']['suffix'])) {
				$this->sizetype = 'medium';
			}			
		}		
		
		if($ihConf['large']['small'] != '')
		{
			if (strstr($this->src, $ihConf['small']['suffix'])) {
				$this->sizetype = 'small';
			}			
		}
}

Off the top of my head and without looking at any of the other areas or consequences.