shawnm Posted October 18, 2006 Posted October 18, 2006 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
digbydo Posted October 18, 2006 Posted October 18, 2006 Your errant { is on line 172 I think :) one-click installation.. Dave's info
ComicWisdom Posted October 18, 2006 Posted October 18, 2006 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!
Recommended Posts
Archived
This topic is now archived and is closed to further replies.