From cc5c61ed5271761e8158db653c8b335f3daff2cd Mon Sep 17 00:00:00 2001
From: vAugagneur <vincent.augagneur@prestashop.com>
Date: Wed, 5 Sep 2012 09:18:06 +0000
Subject: [PATCH] [-] FO : fixed bug #PSCFV-3853

---
 controllers/front/OrderOpcController.php |  2 +-
 themes/default/order-carrier.tpl         | 47 ++++++++++++++----------
 2 files changed, 29 insertions(+), 20 deletions(-)

diff --git a/controllers/front/OrderOpcController.php b/controllers/front/OrderOpcController.php
index 0dfb052ad..130f531bd 100644
--- a/controllers/front/OrderOpcController.php
+++ b/controllers/front/OrderOpcController.php
@@ -494,7 +494,7 @@ class OrderOpcControllerCore extends ParentOrderController
 			}
 		
 		$carriers = $this->context->cart->simulateCarriersOutput();
-		$delivery_option = $this->context->cart->getDeliveryOption(null, true, false);
+		$delivery_option = $this->context->cart->getDeliveryOption(null, false, false);
 		$wrapping_fees = (float)(Configuration::get('PS_GIFT_WRAPPING_PRICE'));
 		$wrapping_fees_tax = new Tax((int)(Configuration::get('PS_GIFT_WRAPPING_TAX')));
 		$wrapping_fees_tax_inc = $wrapping_fees * (1 + (((float)($wrapping_fees_tax->rate) / 100)));
diff --git a/themes/default/order-carrier.tpl b/themes/default/order-carrier.tpl
index 62adb14ad..6ad17c751 100644
--- a/themes/default/order-carrier.tpl
+++ b/themes/default/order-carrier.tpl
@@ -71,21 +71,25 @@
 {/if}
 
 {if !$opc}
-{capture name=path}{l s='Shipping'}{/capture}
-{include file="$tpl_dir./breadcrumb.tpl"}
+	{capture name=path}{l s='Shipping'}{/capture}
+	{include file="$tpl_dir./breadcrumb.tpl"}
 {/if}
 
-{if !$opc}<h1>{l s='Shipping'}</h1>{else}<h2><span>2</span> {l s='Delivery methods'}</h2>{/if}
+{if !$opc}
+	<h1>{l s='Shipping'}</h1>
+{else}
+	<h2><span>2</span> {l s='Delivery methods'}</h2>
+{/if}
 
 {if !$opc}
-{assign var='current_step' value='shipping'}
-{include file="$tpl_dir./order-steps.tpl"}
-
-{include file="$tpl_dir./errors.tpl"}
-
-<form id="form" action="{$link->getPageLink('order', true, NULL, "multi-shipping={$multi_shipping}")}" method="post" onsubmit="return acceptCGV();">
+	{assign var='current_step' value='shipping'}
+	{include file="$tpl_dir./order-steps.tpl"}
+	
+	{include file="$tpl_dir./errors.tpl"}
+	
+	<form id="form" action="{$link->getPageLink('order', true, NULL, "multi-shipping={$multi_shipping}")}" method="post" onsubmit="return acceptCGV();">
 {else}
-<div id="opc_delivery_methods" class="opc-main-block">
+	<div id="opc_delivery_methods" class="opc-main-block">
 	<div id="opc_delivery_methods-overlay" class="opc-overlay" style="display: none;"></div>
 {/if}
 
@@ -96,16 +100,20 @@
 {else}
 	<h3 class="carrier_title">{l s='Choose your delivery method'}</h3>
 	
-	<div id="HOOK_BEFORECARRIER">{if isset($carriers) && isset($HOOK_BEFORECARRIER)}{$HOOK_BEFORECARRIER}{/if}</div>
+	<div id="HOOK_BEFORECARRIER">
+		{if isset($carriers) && isset($HOOK_BEFORECARRIER)}
+			{$HOOK_BEFORECARRIER}
+		{/if}
+	</div>
 	{if isset($isVirtualCart) && $isVirtualCart}
-	<p class="warning">{l s='No carrier needed for this order'}</p>
+		<p class="warning">{l s='No carrier needed for this order'}</p>
 	{else}
-	{if $recyclablePackAllowed}
-	<p class="checkbox">
-		<input type="checkbox" name="recyclable" id="recyclable" value="1" {if $recyclable == 1}checked="checked"{/if} />
-		<label for="recyclable">{l s='I agree to receive my order in recycled packaging'}.</label>
-	</p>
-	{/if}
+		{if $recyclablePackAllowed}
+			<p class="checkbox">
+				<input type="checkbox" name="recyclable" id="recyclable" value="1" {if $recyclable == 1}checked="checked"{/if} />
+				<label for="recyclable">{l s='I agree to receive my order in recycled packaging'}.</label>
+			</p>
+		{/if}
 	<div class="delivery_options_address">
 	{if isset($delivery_option_list)}
 		{foreach $delivery_option_list as $id_address => $option_list}
@@ -195,7 +203,8 @@
 											({l s='products concerned:'}
 										{/if}
 										{* This foreach is on one line, to avoid tabulation in the title attribute of the acronym *}
-										{foreach $carrier.product_list as $product}{if $product@index == 4}<acronym title="{/if}{if $product@index >= 4}{$product.name}{if !$product@last}, {else}">...</acronym>){/if}{else}{$product.name}{if !$product@last}, {else}){/if}{/if}{/foreach}
+										{foreach $carrier.product_list as $product}
+										{if $product@index == 4}<acronym title="{/if}{if $product@index >= 4}{$product.name}{if !$product@last}, {else}">...</acronym>){/if}{else}{$product.name}{if !$product@last}, {else}){/if}{/if}{/foreach}
 									{/if}
 								</td>
 							</tr>