Topic: [Resolved] Products that are on sale are showing prices incorrectly after application addon |

I am noticing a rather large issue with your code at the moment. I placed all of the changes in the readme.txt file into the system, and now products that I have marked to be on sale are showing up as having their prices doubled BEFORE I log in as a wholesaler (so if you go in and just browse as a raw user, the pricing is double). Observe:

Product display before your plug-in:

http://www.mikearsenault.com/images/ProdBeforeYourCode.gif

Product display after your plug-in:

http://www.mikearsenault.com/images/ProdAfterYourCode.gif

My brief investigation has lead me to believe that this function in your code is the culpret:

/* @mod – 200510 – vl_wholesalepricing */
function markup_product_price($price) {
if( current_user_can(‘wholesale_pricing’) ) {
$shop_markup = 0.5;
} else {
$shop_markup = 1.0;
}
if( $shop_markup ) {
$price = ($price * $shop_markup) + $price;
}
return $price;
}

Specifically, the line here:

if( $shop_markup ) {
$price = ($price * $shop_markup) + $price;
}

We installed this add-on after we had entered in a bunch of products. What it seems to be saying is that all prices for products that we have entered need to be at cost for this to work, because you are taking the original price, and then adding on the price * the markup. So when you say in the instructions to make the $shop_markup = 0.75 to offer a 25% discount, what is really happening is that you are increasing the existing list price by 75%. All of the prices we will be entering into our e-commerce install will be the retail price. I can’t expect my client to remember to enter in the products at cost so that this code works as is.

Which puts me in a very awkward position. Changing the code above is minor, I would just have to remove the ‘+ $price’ and that would work as I expect it to. But you charged me $40.00 for the “first working version”. Can you see how this is sort of very bad? And more importantly, what about other users who haven’t caught this error? I am eager to hear what you are going to do to resolve this, because so far, there is no way in hell the $40.00 is worth this level of aggrivation.