> Try adding a department with an ampersand such as "Orange & Purple
> Things

OK, this should be working now.  I wasn't running rawurlencode on values of
GET vars.

Interestingly enough, fixing this turned up another bug involved with
passing values to printEditRow().  The admin_department screen was calling
prepareText() on values passed to printEditRow, which in turn called
prepareText().  :)  That created an endlessly expanding string of &
entities.  We'll have to look for any other cases of this.

> screen/item line 144:

Ignore my previous comment.  You're right, this is just a case of using a
value that might be unset.  I changed the logic to:

  if(($sku["ListPrice"] > 0.00) AND
   ($sku["SalePrice"] < $sku["ListPrice"]))
  {
   $price .= "<br>\nNormally " . formatMoney($sku["ListPrice"]);
  }

If the list price is greater than zero and the sale price is less than the
list price, then show off that the customer is saving money...

Leon


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

Reply via email to