Minor code change suggestion in the includes/templates/westminster_new/common/tpl_header.php file
The original file looks like this:
Code:
<?php if ($_SESSION['cart']->count_contents() != 0) { ?>
<li><a href="<?php echo zen_href_link(FILENAME_SHOPPING_CART, '', 'NONSSL'); ?>"><?php echo HEADER_TITLE_CART_CONTENTS; ?></a></li>
<li><a href="<?php echo zen_href_link(FILENAME_CHECKOUT_SHIPPING, '', 'SSL'); ?>"><?php echo HEADER_TITLE_CHECKOUT; ?></a></li>
<?php }?>
The Westminster New template looks like this:
<li><a href="<?php echo zen_href_link(FILENAME_SHOPPING_CART, '', 'NONSSL'); ?>"><i class="fa fa-shopping-cart" title="Shopping Cart"></i><?php echo $_SESSION['cart']->count_contents();?> - <?php echo $currencies->format($_SESSION['cart']->show_total());?></a></li>
<li><a href="<?php echo zen_href_link(FILENAME_CHECKOUT_SHIPPING, '', 'SSL'); ?>"><i class="fa fa-check-square-o" title="Checkout"></i><?php echo HEADER_TITLE_CHECKOUT; ?></a></li>
<?php if ($_SESSION['cart']->count_contents() != 0) { ?>
<?php }?>
May I suggest this instead in a future update:
Code:
<?php //if ($_SESSION['cart']->count_contents() != 0) { ?>
<li><a href="<?php echo zen_href_link(FILENAME_SHOPPING_CART, '', 'NONSSL'); ?>"><i class="fa fa-shopping-cart" title="Shopping Cart"></i><?php echo $_SESSION['cart']->count_contents();?> - <?php echo $currencies->format($_SESSION['cart']->show_total());?></a></li>
<li><a href="<?php echo zen_href_link(FILENAME_CHECKOUT_SHIPPING, '', 'SSL'); ?>"><i class="fa fa-check-square-o" title="Checkout"></i><?php echo HEADER_TITLE_CHECKOUT; ?></a></li>
<?php //}?>
Note the inclusion of the double slashes (//) in the opening and closing of this code.. This may be anal, but if one is using a file comparison program this helps the program line up the code to better "see" and highlight the file change diffs since they are in their original positions..
Bookmarks