Image Handler 2 (for ZC v1.3.8 ONLY) Support

**mzimmers** · 29 Apr 2010, 12:55 AM

I don't think that my trying to improve documentation on an add-on I don't fully understand is a good idea. Once I get my arms around IH2 a bit better, I'd be happy to try, though.

What I was getting at is, it's quite possible -- in fact, probable -- that IH2 is quite fine just the way it is, and that my troubles with it are more due to my lack of thorough understanding of it. My inability to get the watermarks working, for example, is almost certainly not a problem with the add-on itself, since lots of other people have found that it works fine. The problem is mine. I was only trying to suggest that better documentation (in general) might help to obviate problems like mine. I didn't mean to offend anyone with that remark.

Also: am I correct in assuming that we're talking about this version of IH2?

[URL="http://www.zen-cart.com/index.php?main_page=product_contrib_info&products_id=117"]http://www.zen-cart.com/index.php?main_page=product_contrib_info&products_id=117[/URL

**DivaVocals** · 29 Apr 2010, 01:03 AM

Originally Posted by mzimmers

I don't think that my trying to improve documentation on an add-on I don't fully understand is a good idea. Once I get my arms around IH2 a bit better, I'd be happy to try, though.

What I was getting at is, it's quite possible -- in fact, probable -- that IH2 is quite fine just the way it is, and that my troubles with it are more due to my lack of thorough understanding of it. My inability to get the watermarks working, for example, is almost certainly not a problem with the add-on itself, since lots of other people have found that it works fine. The problem is mine. I was only trying to suggest that better documentation (in general) might help to obviate problems like mine. I didn't mean to offend anyone with that remark.

I wasn't offended, I was merely pointing out that as the co-author of the current readme document I was under the impression that we already HAD clearly explained things.. So if that is not true, then someone will have to provide me with copy to include in the readme file that is improved based on community needs.. If someone wants to do this, I will be more than happy to include it in the next release..

Originally Posted by mzimmers

Also: am I correct in assuming that we're talking about this version of IH2?

[url="http://www.zen-cart.com/index.php?main_page=product_contrib_info&products_id=117"]http://www.zen-cart.com/index.php?main_page=product_contrib_info&products_id=117[/URL

Yes

**mzimmers** · 29 Apr 2010, 01:54 AM

I'd like to return to the subject of why my watermarks don't seem to be working correctly. I've created a watermark file in images, images/large and images/medium. In Configuration->Images, I have watermarks set for medium and large images.

The ONLY place where I get watermarks is when I rollover an image in the "New Products" area.

Any suggestions? Thanks. Other than this, IH2 seems to be working marvelously for me.

**Shane78** · 29 Apr 2010, 07:39 AM

I havent paid for anything! i found this in the downloads section!

Originally Posted by DivaVocals

I just don't understand why you are paying for a version of IH2 when it's not CLEAR what is different about this version than the original add-on..

i said, i would be willing to chip in for the future of this mod.

and wouldn't a rewrite of IH"2 to bring it up to PHP5.3 and to fix the things that dont work be a considerable task.. thus warranting a new version number?

and if i wanted a feature coding and required coughing up some cash for it, i wouldn't mind having this available for free in the downloads... what's a few quid to help others?

**DivaVocals** · 29 Apr 2010, 01:40 PM

Originally Posted by Shane78

I havent paid for anything! i found this in the downloads section!

i said, i would be willing to chip in for the future of this mod.

Have you taken a close look at the the version of IH2 Judy Gunderson has submitted. Her version doesn't MAJORLY change ANYTHING in IH2 so why she submitted it as a NEW and separate module is beyond me..

After comparing her version last night to Rev 7 of IH2 it seems clear that she should have submitted it as an UPDATE with a MUCH BETTER explanation of what she "did" in her submission notes.. Also her "version" implies that it fixes a problem that was never a problem (that I was a aware of) even with the new version of Zen Cart.

My point was I would not pay ANYONE to make this particular version (the Judy Gunderson version) of IH2 PHP 5.3 compatible since with the exception of a VERY MINOR change I do not see ANYTHING ELSE different in her version. (and since I'm not a coder, I will be asking a friend or two to help me interpret the code that Judy did add to IH2 to determine WHAT it is supposed to do)

Originally Posted by Shane78

and wouldn't a rewrite of IH"2 to bring it up to PHP5.3 and to fix the things that dont work be a considerable task.. thus warranting a new version number?

Other than the hover on medium images (which has NEVER been coded into IH2 and which I imagine WOULD be a fairly complicated undertaking if compatibility with lightbox mods is to be maintained..) what features don't work???

I posted yesterday that I will be submitting a PHP 5.3 compatible version of the ORIGINAL IH2. The changes to make IH2 PHP 5.3 compatible are MINOR at best and do not warrant a major version number change.

I am of the opinion that updating the Judy Gunderson version with no real understanding of WHAT SHE CHANGED is not a good idea.. Which is why I PERSONALLY would not pay to fix THAT version..

Originally Posted by Shane78

and if i wanted a feature coding and required coughing up some cash for it, i wouldn't mind having this available for free in the downloads... what's a few quid to help others?

Nothing.. I have paid for feature coding before, but my point was that I personally wouldn't spend ANY money on a version of IH2 which is VAGUELY documented, and as far as I can tell brings NOTHING new to the table.

She (Judy Gunderson) in essence took the Rev 7 version of IH2, add a LITTLE bit of code to it, made some VAGUE reference to some "broken" functionality (that has NEVER been broken or even reported broken), changed NONE of the supporting documentation, and then submitted it as a NEW mod (instead of an update to the existing mod) Now if YOU want to pay to have THAT version updated to make it PHP 5.3 compatible, knock yourself out..

I will submit Rev 8 (as an UPDATE to Rev 7) to the downloads section, and it won't cost me or anyone a dime to use it..

**Shane78** · 29 Apr 2010, 03:49 PM

wow, that's a lot of words to read...

i shall be brief..

a) not even downloaded to look at the differences.. i simply found it last night because i see mods are being updated to 139 without us knowing about it to add to the list.

b) Give the lass a break, good on her for trying. while you might think her efforts arent good or confusing, to be listed it has to get approval... have a rant at who ever approved it. (wait till you see what i do to Login Spuce Up)

c) i dont care if its IH2, IH2a, IH3, or a totally new mod.. so long as it' works and is 139a and 139B (yes, there's another one coming) compatible, PHP5.3 compatible, zen lightbox compatible, isnt buggy or a security risk and easy to install! (having a feature to make me a brew would be good too)

Right, cant be on the ZC forums all day, some of us have proper jobs to do, lol

**mzimmers** · 29 Apr 2010, 06:11 PM

Doesn't anyone want to talk about why I suck at watermarks?

**Shane78** · 29 Apr 2010, 06:41 PM

Originally Posted by mzimmers

Doesn't anyone want to talk about why I suck at watermarks?

dunno.. are all your settings right?
have you got the images you want to use as a watermark in the right place in the right format?
are all the files uploaded...?... for some reason filezilla doesnt always notify you when a file failed to upload... dead annoying!!!!

**DivaVocals** · 29 Apr 2010, 07:29 PM

Originally Posted by Shane78

wow, that's a lot of words to read...

i shall be brief..

a) not even downloaded to look at the differences.. i simply found it last night because i see mods are being updated to 139 without us knowing about it to add to the list.

And this is precisely why I posted what I did.. The Judy Gunderson version is going to cause CONFUSION.. IMO it should NOT even be included on your list of "mods that work with Zen Cart".

It has already been established that IH2 Rev 7 is Zen Cart 1.3.9a compatible, and I will be submitting an update which will ALSO make it PHP 5.3 compatible.

Before we go calling for folks to pool their resources on an update of this "new" IH2 version, we should at least know what it does that is so all fired different from the current version of IH2. (I cannot see what BIG change she made, and the feature she claimed she "fixed" was -to my knowledge- never broken)

Originally Posted by Shane78

b) Give the lass a break, good on her for trying. while you might think her efforts arent good or confusing, to be listed it has to get approval... have a rant at who ever approved it. (wait till you see what i do to Login Spuce Up)

I wasn't giving her a hard time, merely stating what I see to be the current state of affairs. It's not about me thinking her efforts aren't good enough. It's about her not fully qualifying or documenting her changes. It's about her creating a NEW version of IH2, and in doing so creating what appears to be a FORK in this application (whether that was by design or by accident) with no major code changes or documentation to back up the need for a NEW SEPERATE version.. I'll say it again, her "version" of Zen Cart is going to create issues in terms of support because of this. (surely you can see how this is true right??)

Originally Posted by Shane78

c) i dont care if its IH2, IH2a, IH3, or a totally new mod.. so long as it' works and is 139a and 139B (yes, there's another one coming) compatible, PHP5.3 compatible, zen lightbox compatible, isnt buggy or a security risk and easy to install! (having a feature to make me a brew would be good too)

Got it, but if you're going to ask for a new major version number, you should know that it USUALLY means a major code change. (hence the explanation)

It's already been established that IH2 works with the latest and greatest version of Zen Cart, and AGAIN I will be submitting a PHP 5.3 compatible version soon (having it tested now to ensure it really IS PHP 5.3 compatible)

IH2 is stable (ie - not buggy), and compatible with most lightbox mods...

Originally Posted by Shane78

Right, cant be on the ZC forums all day, some of us have proper jobs to do, lol

Right you are.. I have requirements documents to refine..

**Shane78** · 29 Apr 2010, 08:07 PM

Originally Posted by DivaVocals

And this is precisely why I posted what I did.. The Judy Gunderson version is going to cause CONFUSION.. IMO it should NOT even be included on your list of "mods that work with Zen Cart".

It's not actually been added to the list that i have created... dunno where you got the impression it was,,, Less speed and more haste maybe! You're getting confused with the mod test submitted by Clyde Jones. When I add a mod to the list I add the whole title copy and pasted from the listing. Note it says 'zc137' and has been there since the 2nd list I made.

Originally Posted by DivaVocals

Before we go calling for folks to pool their resources on an update of this "new" IH2 version, we should at least know what it does that is so all fired different from the current version of IH2. (I cannot see what BIG change she made, and the feature she claimed she "fixed" was -to my knowledge- never broken)
I wasn't giving her a hard time, merely stating what I see to be the current state of affairs. It's not about me thinking her efforts aren't good enough. It's about her not fully qualifying or documenting her changes. It's about her creating a NEW version of IH2, and in doing so creating what appears to be a FORK in this application (whether that was by design or by accident) with no major code changes or documentation to back up the need for a NEW SEPERATE version.. I'll say it again, her "version" of Zen Cart is going to create issues in terms of support because of this. (surely you can see how this is true right??)

Then make an official complaint/query with the four ZC developers / moderators of ZC that authorised its entry.

Originally Posted by DivaVocals

It's already been established that IH2 works with the latest and greatest version of Zen Cart, and AGAIN I will be submitting a PHP 5.3 compatible version soon (having it tested now to ensure it really IS PHP 5.3 compatible)

excellent, i cannot wait.. will it be zc139b compatible? lol..
im not joking btw... lol, seriously.. it's in the bugs threads!

Whether your intention is to give this lass a hard time or not.. it is coming across that way... agree or disagree with me, that is your prerogative, but that is the perception your posts have given regarding this subject, because they have such gusto about them.