Quote Originally Posted by Miff View Post
No, as mentioned previously, this is not the default core file.
< chomp > I can however confirm that the error was indeed on line 445.
OK, but since your file has been modified, then referencing line 445 along with "so someone else is bound to have this issue. For that reason Il post my fix" , is actually misleading, and possibly even 'dangerous'.

It may be obvious to both you and I that the *actual* line number will be different in the non modified code, there are some people that will take this literally, and end up commenting out a completely different line, which will cause all sorts of additional problems. (Yes, some people are that silly).

Anyway, that wasn't the point I was trying to make. I was looking at the code trying to determine why your problem existed in the first place, and couldn't see any possible cause for the problem. It was during this time that I noted you'd referenced a different line number than the one I was looking at, and *if* it was a typo (445 vs 415 wouldn't be a difficult error to make) I was going to go back and take an even closer look,

Before I asked you about this though I checked this same file in *all* of the different zencart versions I have on my test bed, expecting to find one that would match what you had referenced. I didn't find one, which is why I probed you for more info. :)

It is now quite apparent that the cause of your problem is indeed related to your code modification. This doesn't make the two possible solutions any different, but I/we are now back to square#1 in not knowing which is going to be the best of the two, because that will depend on what the modified code is actually trying to do.

The *correct* solution would actually be in the code that has been added, because as I stated, both your solution and mine are both nothing more than a 'kludge' that is hiding the *real* cause. I *hope* you understand what I'm saying here?

Cheers
Rod