From f2e07cb07eede6c27e8d00e6130e38bf465fdd44 Mon Sep 17 00:00:00 2001
From: Manuel Rego Casasnovas <mrego@igalia.com>
Date: Mon, 20 Sep 2010 20:29:26 +0200
Subject: [PATCH] Fixed bug disabling create buttons if user does not have the
 proper permissions.

FEA: ItEr60S04ValidacionEProbasFuncionaisItEr59S04
---
 .../web/orders/OrderCRUDController.java             | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

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 088d2487c..a01126fb8 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
@@ -243,6 +243,8 @@ public class OrderCRUDController extends GenericForwardComposer {
                 .getFellow("bdFilters");
         checkIncludeOrderElements = (Checkbox) filterComponent
                 .getFellow("checkIncludeOrderElements");
+
+        checkCreationPermissions();
     }
 
     private void initEditOrderElementWindow() {
@@ -895,7 +897,6 @@ public class OrderCRUDController extends GenericForwardComposer {
             orderModel.prepareForCreate();
             prepareEditWindow();
             showEditWindow(_("Create order"));
-            checkCreationPermissions();
         } catch (ConcurrentModificationException e) {
             messagesForUser.showMessage(Level.ERROR, e.getMessage());
         }
@@ -1223,14 +1224,12 @@ public class OrderCRUDController extends GenericForwardComposer {
 
     /**
      * Checks the creation permissions of the current user and enables/disables
-     * the save buttons accordingly.
+     * the create buttons accordingly.
      */
     private void checkCreationPermissions() {
-        if(SecurityUtils.isUserInRole(UserRole.ROLE_CREATE_ORDER)) {
-            saveOrderAndContinueButton.setDisabled(false);
-        }
-        else {
-            saveOrderAndContinueButton.setDisabled(true);
+        if (!SecurityUtils.isUserInRole(UserRole.ROLE_CREATE_ORDER)) {
+            createOrderButton.setDisabled(true);
+            createOrderFromTemplateButton.setDisabled(true);
         }
     }