Jump to content
  • Checkout
  • Login
  • Get in touch

osCommerce

The e-commerce.

Is this an error in the original code?


shawnm

Recommended Posts

Posted

I found this code in the original code... in the file /catalog/checkout_confirmation.php. I was wondering if this code is correct?

 

Line 189:

 

if (sizeof($order->info['tax_groups']) > 1) echo ' <td class="main" valign="top" align="right">' . tep_display_tax_value($order->products[$i]['tax']) . '%</td>' . "\n";

 

echo ' <td class="main" align="right" valign="top">' . $currencies->display_price($order->products[$i]['final_price'], $order->products[$i]['tax'], $order->products[$i]['qty']) . '</td>' . "\n" .

' </tr>' . "\n";

}

 

 

Seems like the If statement needs an "{". I am not a professional coder, but this just didn't seem right. If this is correct coding, I do apologize.

 

Thanks,

Shawn

Posted
I found this code in the original code... in the file /catalog/checkout_confirmation.php. I was wondering if this code is correct?

 

Line 189:

 

if (sizeof($order->info['tax_groups']) > 1) echo ' <td class="main" valign="top" align="right">' . tep_display_tax_value($order->products[$i]['tax']) . '%</td>' . "\n";

 

echo ' <td class="main" align="right" valign="top">' . $currencies->display_price($order->products[$i]['final_price'], $order->products[$i]['tax'], $order->products[$i]['qty']) . '</td>' . "\n" .

' </tr>' . "\n";

}

Seems like the If statement needs an "{". I am not a professional coder, but this just didn't seem right. If this is correct coding, I do apologize.

 

Thanks,

Shawn

 

 

You've got to watch out for those nested statements!!! :D

Just between us, remember there are only 10 kinds of people in the world; those who understand binary and those who don't!!

 

Remember, learning is a "do-it-yourself" experience; although, not necessarily a "do-it-BY-yourself" experience.

 

The quickest way to learn is to forget to BACKUP!

Archived

This topic is now archived and is closed to further replies.

×
×
  • Create New...