From 15883806f56a77bbb9b94a80796e85e7df5dc01c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Lorenzo=20Tilve=20=C3=81lvaro?= <ltilve@igalia.com>
Date: Thu, 6 Jun 2013 21:18:48 +0200
Subject: [PATCH] Added to configuration class accessors for automatic budget
 properties

FEA: ItEr77S17AutomaticBudgeting
---
 .../common/entities/Configuration.java         | 17 +++++++++++++++--
 .../web/common/ConfigurationController.java    |  8 ++++++++
 .../web/common/ConfigurationModel.java         | 16 ++++++++++++++++
 .../web/common/IConfigurationModel.java        |  4 ++++
 .../src/main/webapp/common/configuration.zul   | 18 +++++++++---------
 5 files changed, 52 insertions(+), 11 deletions(-)

diff --git a/libreplan-business/src/main/java/org/libreplan/business/common/entities/Configuration.java b/libreplan-business/src/main/java/org/libreplan/business/common/entities/Configuration.java
index e3c6e5194..c8a6c69fb 100644
--- a/libreplan-business/src/main/java/org/libreplan/business/common/entities/Configuration.java
+++ b/libreplan-business/src/main/java/org/libreplan/business/common/entities/Configuration.java
@@ -116,6 +116,10 @@ public class Configuration extends BaseEntity {
 
     private Integer secondsPlanningWarning = 30;
 
+    private TypeOfWorkHours budgetDefaultTypeOfWorkHours;
+
+    private Boolean enabledAutomaticBudget = false;
+
     /**
      * Maximum users configurable directly in database for SaaS products. If
      * zero it means that there isn't any limitation.
@@ -128,6 +132,7 @@ public class Configuration extends BaseEntity {
      */
     private Integer maxResources = 0;
 
+
     public void setDefaultCalendar(BaseCalendar defaultCalendar) {
         this.defaultCalendar = defaultCalendar;
     }
@@ -477,11 +482,11 @@ public class Configuration extends BaseEntity {
     }
 
     public TypeOfWorkHours getBudgetDefaultTypeOfWorkHours() {
-        return personalTimesheetsTypeOfWorkHours;
+        return budgetDefaultTypeOfWorkHours;
     }
 
     public void setBudgetDefaultTypeOfWorkHours(TypeOfWorkHours typeOfWorkHours) {
-        personalTimesheetsTypeOfWorkHours = typeOfWorkHours;
+        budgetDefaultTypeOfWorkHours = typeOfWorkHours;
     }
 
     public PersonalTimesheetsPeriodicityEnum getPersonalTimesheetsPeriodicity() {
@@ -511,4 +516,12 @@ public class Configuration extends BaseEntity {
         this.secondsPlanningWarning = secondsPlanningWarning;
     }
 
+    public Boolean isEnabledAutomaticBudget() {
+        return enabledAutomaticBudget;
+    }
+
+    public void setEnabledAutomaticBudget(Boolean enabledAutomaticBudget) {
+        this.enabledAutomaticBudget = enabledAutomaticBudget;
+    }
+
 }
diff --git a/libreplan-webapp/src/main/java/org/libreplan/web/common/ConfigurationController.java b/libreplan-webapp/src/main/java/org/libreplan/web/common/ConfigurationController.java
index 6470b2aa9..a1ef7b11d 100644
--- a/libreplan-webapp/src/main/java/org/libreplan/web/common/ConfigurationController.java
+++ b/libreplan-webapp/src/main/java/org/libreplan/web/common/ConfigurationController.java
@@ -1010,6 +1010,14 @@ public class ConfigurationController extends GenericForwardComposer {
         configurationModel.setBudgetDefaultTypeOfWorkHours(typeOfWorkHours);
     }
 
+    public Boolean getEnabledAutomaticBudget() {
+        return configurationModel.getEnabledAutomaticBudget();
+    }
+
+    public void setEnabledAutomaticBudget(Boolean enabledAutomaticBudget) {
+        configurationModel.setEnabledAutomaticBudget(enabledAutomaticBudget);
+    }
+
     public List<PersonalTimesheetsPeriodicityEnum> getPersonalTimesheetsPeriodicities() {
         return Arrays.asList(PersonalTimesheetsPeriodicityEnum.values());
     }
diff --git a/libreplan-webapp/src/main/java/org/libreplan/web/common/ConfigurationModel.java b/libreplan-webapp/src/main/java/org/libreplan/web/common/ConfigurationModel.java
index a6b04e4c5..c6b0fee6e 100644
--- a/libreplan-webapp/src/main/java/org/libreplan/web/common/ConfigurationModel.java
+++ b/libreplan-webapp/src/main/java/org/libreplan/web/common/ConfigurationModel.java
@@ -733,4 +733,20 @@ public class ConfigurationModel implements IConfigurationModel {
             configuration.setBudgetDefaultTypeOfWorkHours(typeOfWorkHours);
         }
     }
+
+    @Override
+    public Boolean getEnabledAutomaticBudget() {
+        if (configuration == null) {
+            return null;
+        }
+        return (configuration.isEnabledAutomaticBudget());
+    }
+
+    @Override
+    public void setEnabledAutomaticBudget(Boolean enabledAutomaticBudget) {
+        if (configuration != null) {
+            configuration.setEnabledAutomaticBudget(enabledAutomaticBudget);
+        }
+    }
+
 }
diff --git a/libreplan-webapp/src/main/java/org/libreplan/web/common/IConfigurationModel.java b/libreplan-webapp/src/main/java/org/libreplan/web/common/IConfigurationModel.java
index 9c81a1d2d..5a423e31e 100644
--- a/libreplan-webapp/src/main/java/org/libreplan/web/common/IConfigurationModel.java
+++ b/libreplan-webapp/src/main/java/org/libreplan/web/common/IConfigurationModel.java
@@ -196,4 +196,8 @@ public interface IConfigurationModel {
 
     void setBudgetDefaultTypeOfWorkHours(TypeOfWorkHours typeOfWorkHours);
 
+    Boolean getEnabledAutomaticBudget();
+
+    void setEnabledAutomaticBudget(Boolean enabledAutomaticBudget);
+
 }
diff --git a/libreplan-webapp/src/main/webapp/common/configuration.zul b/libreplan-webapp/src/main/webapp/common/configuration.zul
index 0f9effd9f..88638ff9b 100644
--- a/libreplan-webapp/src/main/webapp/common/configuration.zul
+++ b/libreplan-webapp/src/main/webapp/common/configuration.zul
@@ -214,16 +214,16 @@
                             <row>
                                 <label
                                     value="${i18n:_('Automatic budget')}" />
-                                    <hbox>
-                                <label
-                                    value="${i18n:_('Hours type')}" />
+                                <hbox align="pack">
+                                    <label
+                                        value="${i18n:_('Hours type')}" />
                                     <bandboxSearch
-                                    finder="TypeOfWorkHoursBandboxFinder"
-                                    selectedElement="@{configurationController.budgetDefaultTypeOfWorkHours, access='both'}" />
-                                        <checkbox
-                                            label="${i18n:_('calculate based on task criteria and cost categories')}"
-                                            checked="@{configurationController.enabledAutomaticBudget}" />
-                                    </hbox>
+                                        finder="TypeOfWorkHoursBandboxFinder"
+                                        selectedElement="@{configurationController.budgetDefaultTypeOfWorkHours, access='both'}" />
+                                    <checkbox
+                                        label="${i18n:_('calculate based on task criteria and cost categories')}"
+                                        checked="@{configurationController.enabledAutomaticBudget}" />
+                                </hbox>
                             </row>
                             <row>
                                 <label