>> /*
>> ** Workaround for MSIE 5 Macintosh setcookie problem.
>> */
>> if(ereg("MSIE 5.0", $HTTP_USER_AGENT) && ereg("Mac",
> $HTTP_USER_AGENT))
>> {
>> define("USE_FLAT_URLS", FALSE);
>> define("USE_COOKIES", FALSE);
>> define("USE_ONLY_COOKIES", FALSE);
>> }

> It's slightly more efficient to use strstr() instead of ereg, BTW. :)

Ah yes thanks... I was just copying and pasting from initialization and
didn't think about it ...it should therefore be:

if(strstr($HTTP_USER_AGENT, "MSIE 5.0")
&& strstr($HTTP_USER_AGENT, "Mac"))

-----------------------------------------------------
 
>> department; using sampledb.sql data and flattened URL's the
>> links don't take you anywhere as the names contain spaces
>> ...rawurlencode() seems to sort the problem.
> 
> Yep.  IE deals with spaces OK, but Nav doesn't.  I fixed it.

I don't think you've commited those changes to CVS (I'm up to date as of
your last post) so don't know if this is an issue, but here's one to look
out for...Try adding a department with an ampersand such as "Orange & Purple
Things"... in my case IE5 (not using flattened URL's as above) doesn't
encode it (and it doesn't work) but Nav (using flattened URL's ) does:

HTML in IE5 Macintosh:

<a 
href="http://192.168.0.1/freetrade2/index.php?SCREEN=department&amp;sid=BFth
ohgojQzwEHIE&amp;department=Purple &amp; Orange Things"><h2>Purple & Orange
Things</h2>

NN4.77 Macintosh:

<a 
href="http://192.168.0.1/freetrade2/index.php/department/department/Purple%2
0%26%20Orange%20Things.html"><h2>Purple & Orange Things</h2>

---------------------------------------------------
A suggestion:

screen/item line 144:

if($sku["SalePrice"] != $sku["ListPrice"])

Doesn't allow for no entry in ListPrice. How about something like...

if(!($sku["SalePrice"] == $sku["ListPrice"]
|| $sku["ListPrice"] == "0.00"))

...to cover that scenario?

Nick

----------------------
Lab2 Design-Unit
URL: http://www.lab2.com.au
e-mail: [EMAIL PROTECTED]
----------------------


_______________________________________________
FreeTrade-dev mailing list
[EMAIL PROTECTED]
http://share.whichever.com/mailman/listinfo/freetrade-dev

Reply via email to