From ee0d08cb9db2482a787236243ae09dc6e9e6206c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Francisco=20Jim=C3=A9nez=20Cabrera?= <jkfran@gmail.com>
Date: Sat, 25 Nov 2017 02:21:00 +0100
Subject: [PATCH] BO: Better performance in order template

---
 .../controllers/orders/helpers/view/view.tpl  | 20 +++++++++++--------
 1 file changed, 12 insertions(+), 8 deletions(-)

diff --git a/admin-dev/themes/default/template/controllers/orders/helpers/view/view.tpl b/admin-dev/themes/default/template/controllers/orders/helpers/view/view.tpl
index 34d09e852d182..4a8ab67b2c633 100644
--- a/admin-dev/themes/default/template/controllers/orders/helpers/view/view.tpl
+++ b/admin-dev/themes/default/template/controllers/orders/helpers/view/view.tpl
@@ -67,6 +67,10 @@
   <div>{$hook_invoice}</div>
   {/if}
 
+  {assign var="order_documents" value=$order->getDocuments()}
+  {assign var="order_shipping" value=$order->getShipping()}
+  {assign var="order_return" value=$order->getReturn()}
+
   <div class="panel kpi-container">
     <div class="row">
       <div class="col-xs-6 col-sm-3 box-stats color3" >
@@ -184,7 +188,7 @@
           <li>
             <a href="#documents">
               <i class="icon-file-text"></i>
-              {l s='Documents' d='Admin.Orderscustomers.Feature'} <span class="badge">{$order->getDocuments()|@count}</span>
+              {l s='Documents' d='Admin.Orderscustomers.Feature'} <span class="badge">{$order_documents|@count}</span>
             </a>
           </li>
         </ul>
@@ -255,7 +259,7 @@
           </div>
           <!-- Tab documents -->
           <div class="tab-pane" id="documents">
-            <h4 class="visible-print">{l s='Documents' d='Admin.Orderscustomers.Feature'} <span class="badge">({$order->getDocuments()|@count})</span></h4>
+            <h4 class="visible-print">{l s='Documents' d='Admin.Orderscustomers.Feature'} <span class="badge">({$order_documents|@count})</span></h4>
             {* Include document template *}
             {include file='controllers/orders/_documents.tpl'}
           </div>
@@ -273,13 +277,13 @@
           <li class="active">
             <a href="#shipping">
               <i class="icon-truck "></i>
-              {l s='Shipping' d='Admin.Catalog.Feature'} <span class="badge">{$order->getShipping()|@count}</span>
+              {l s='Shipping' d='Admin.Catalog.Feature'} <span class="badge">{$order_shipping|@count}</span>
             </a>
           </li>
           <li>
             <a href="#returns">
               <i class="icon-undo"></i>
-              {l s='Merchandise Returns' d='Admin.Orderscustomers.Feature'} <span class="badge">{$order->getReturn()|@count}</span>
+              {l s='Merchandise Returns' d='Admin.Orderscustomers.Feature'} <span class="badge">{$order_return|@count}</span>
             </a>
           </li>
         </ul>
@@ -288,7 +292,7 @@
         {$HOOK_CONTENT_SHIP}
           <!-- Tab shipping -->
           <div class="tab-pane active" id="shipping">
-            <h4 class="visible-print">{l s='Shipping' d='Admin.Catalog.Feature'} <span class="badge">({$order->getShipping()|@count})</span></h4>
+            <h4 class="visible-print">{l s='Shipping' d='Admin.Catalog.Feature'} <span class="badge">({$order_shipping|@count})</span></h4>
             <!-- Shipping block -->
             {if !$order->isVirtual()}
             <div class="form-horizontal">
@@ -321,10 +325,10 @@
           </div>
           <!-- Tab returns -->
           <div class="tab-pane" id="returns">
-            <h4 class="visible-print">{l s='Merchandise Returns' d='Admin.Orderscustomers.Feature'} <span class="badge">({$order->getReturn()|@count})</span></h4>
+            <h4 class="visible-print">{l s='Merchandise Returns' d='Admin.Orderscustomers.Feature'} <span class="badge">({$order_return|@count})</span></h4>
             {if !$order->isVirtual()}
             <!-- Return block -->
-              {if $order->getReturn()|count > 0}
+              {if $order_return|count > 0}
               <div class="table-responsive">
                 <table class="table">
                   <thead>
@@ -336,7 +340,7 @@
                     </tr>
                   </thead>
                   <tbody>
-                    {foreach from=$order->getReturn() item=line}
+                    {foreach from=$order_return item=line}
                     <tr>
                       <td>{$line.date_add}</td>
                       <td>{$line.type}</td>