From 9a8453f5f88c8ba18d28893686a250ec956cd6b4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=93scar=20Gonz=C3=A1lez=20Fern=C3=A1ndez?=
 <ogonzalez@igalia.com>
Date: Tue, 8 Dec 2009 19:35:55 +0100
Subject: [PATCH] ItEr38S05ValidacionEProbasFuncionaisItEr37S06: Caching result
 of getOrders at OrderCrudController because it's called multiple times when
 loading page

---
 .../org/navalplanner/web/orders/OrderCRUDController.java  | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/navalplanner-webapp/src/main/java/org/navalplanner/web/orders/OrderCRUDController.java b/navalplanner-webapp/src/main/java/org/navalplanner/web/orders/OrderCRUDController.java
index 58b760ed6..528798dbd 100644
--- a/navalplanner-webapp/src/main/java/org/navalplanner/web/orders/OrderCRUDController.java
+++ b/navalplanner-webapp/src/main/java/org/navalplanner/web/orders/OrderCRUDController.java
@@ -223,8 +223,13 @@ public class OrderCRUDController extends GenericForwardComposer {
                 .getVariable("assignedMaterialsController", true);
     }
 
+    private List<Order> ordersListCached;
+
     public List<Order> getOrders() {
-        return orderModel.getOrders();
+        if (ordersListCached != null) {
+            return ordersListCached;
+        }
+        return ordersListCached = orderModel.getOrders();
     }
 
     private OnlyOneVisible getVisibility() {
@@ -281,6 +286,7 @@ public class OrderCRUDController extends GenericForwardComposer {
 
     public void goToList() {
         showWindow(listWindow);
+        ordersListCached = null;
     }
 
     public void reloadHoursGroupOrder() {