From 2f0208a0bd5148013b4765f4c06ebd47be4faa68 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jacobo=20Aragunde=20P=C3=A9rez?= <jaragunde@igalia.com>
Date: Mon, 12 Apr 2010 13:59:39 +0200
Subject: [PATCH]  ItEr53S10AdaptacionServiciosRESTItEr52S10: Added the
 checkbox 'generate code' to the Work Report administration page.

Code fields are enabled/disabled when the checkbox changes.
Codes for unsaved entities are re-generated when the checkbox is activated.
The value of the checkbox is persistent (as an attribute of WorkReport).
---
 .../workreports/entities/WorkReport.java      | 10 ++++++++++
 .../workreports/entities/WorkReports.hbm.xml  |  2 ++
 .../workreports/WorkReportCRUDController.java | 19 +++++++++++++++++++
 .../web/workreports/WorkReportModel.java      | 11 +++++++++++
 .../main/webapp/workreports/workReport.zul    | 10 ++++++++--
 5 files changed, 50 insertions(+), 2 deletions(-)

diff --git a/navalplanner-business/src/main/java/org/navalplanner/business/workreports/entities/WorkReport.java b/navalplanner-business/src/main/java/org/navalplanner/business/workreports/entities/WorkReport.java
index 8a5b5fdb5..94e807ee2 100644
--- a/navalplanner-business/src/main/java/org/navalplanner/business/workreports/entities/WorkReport.java
+++ b/navalplanner-business/src/main/java/org/navalplanner/business/workreports/entities/WorkReport.java
@@ -74,6 +74,8 @@ public class WorkReport extends IntegrationEntity {
 
     private OrderElement orderElement;
 
+    private Boolean generateCode = false;
+
     private Set<Label> labels = new HashSet<Label>();
 
     private Set<WorkReportLine> workReportLines = new HashSet<WorkReportLine>();
@@ -197,6 +199,14 @@ public class WorkReport extends IntegrationEntity {
         }
     }
 
+    public Boolean getGenerateCode() {
+        return generateCode;
+    }
+
+    public void setGenerateCode(Boolean generateCode) {
+        this.generateCode = generateCode;
+    }
+
     @SuppressWarnings("unused")
     @AssertTrue(message = "date:the date must be not null if is shared by lines")
     public boolean checkConstraintDateMustBeNotNullIfIsSharedByLines() {
diff --git a/navalplanner-business/src/main/resources/org/navalplanner/business/workreports/entities/WorkReports.hbm.xml b/navalplanner-business/src/main/resources/org/navalplanner/business/workreports/entities/WorkReports.hbm.xml
index 5b19c3f0d..cea8b131b 100644
--- a/navalplanner-business/src/main/resources/org/navalplanner/business/workreports/entities/WorkReports.hbm.xml
+++ b/navalplanner-business/src/main/resources/org/navalplanner/business/workreports/entities/WorkReports.hbm.xml
@@ -61,6 +61,8 @@
 
     <property name="date"/>
 
+    <property name="generateCode" not-null="true" />
+
     <many-to-one name="workReportType" class="WorkReportType" column="WORK_REPORT_TYPE_ID" not-null="true"/>
 
     <many-to-one name="resource" class="org.navalplanner.business.resources.entities.Resource" column="RESOURCE_ID"/>
diff --git a/navalplanner-webapp/src/main/java/org/navalplanner/web/workreports/WorkReportCRUDController.java b/navalplanner-webapp/src/main/java/org/navalplanner/web/workreports/WorkReportCRUDController.java
index a88c70543..96823a25b 100644
--- a/navalplanner-webapp/src/main/java/org/navalplanner/web/workreports/WorkReportCRUDController.java
+++ b/navalplanner-webapp/src/main/java/org/navalplanner/web/workreports/WorkReportCRUDController.java
@@ -57,6 +57,7 @@ import org.navalplanner.web.common.entrypoints.URLHandler;
 import org.zkoss.ganttz.IPredicate;
 import org.zkoss.zk.ui.Component;
 import org.zkoss.zk.ui.WrongValueException;
+import org.zkoss.zk.ui.event.CheckEvent;
 import org.zkoss.zk.ui.event.Event;
 import org.zkoss.zk.ui.event.EventListener;
 import org.zkoss.zk.ui.event.Events;
@@ -1119,6 +1120,7 @@ public class WorkReportCRUDController extends GenericForwardComposer implements
         final WorkReportLine line = (WorkReportLine) row.getValue();
         final Textbox code = new Textbox();
         code.setWidth("150px");
+        code.setDisabled(getWorkReport().getGenerateCode());
         code.applyProperties();
 
          if (line.getCode() != null) {
@@ -1672,4 +1674,21 @@ public class WorkReportCRUDController extends GenericForwardComposer implements
         }
     }
 
+    public void onCheckGenerateCode(Event e) {
+        CheckEvent ce = (CheckEvent) e;
+        if(ce.isChecked()) {
+            //we have to auto-generate the code for new objects
+            if(getWorkReport().isNewObject()) {
+                getWorkReport().setCodeAutogenerated();
+                Util.reloadBindings(createWindow);
+            }
+            for(WorkReportLine line : getWorkReportLines()) {
+                if(line.isNewObject()) {
+                    line.setCodeAutogenerated();
+                }
+            }
+        }
+        reloadWorkReportLines();
+    }
+
 }
diff --git a/navalplanner-webapp/src/main/java/org/navalplanner/web/workreports/WorkReportModel.java b/navalplanner-webapp/src/main/java/org/navalplanner/web/workreports/WorkReportModel.java
index ab4fe41b0..86ff5a51a 100644
--- a/navalplanner-webapp/src/main/java/org/navalplanner/web/workreports/WorkReportModel.java
+++ b/navalplanner-webapp/src/main/java/org/navalplanner/web/workreports/WorkReportModel.java
@@ -28,6 +28,7 @@ import java.util.List;
 import java.util.Map;
 
 import org.apache.commons.lang.Validate;
+import org.navalplanner.business.common.daos.IConfigurationDAO;
 import org.navalplanner.business.common.exceptions.InstanceNotFoundException;
 import org.navalplanner.business.common.exceptions.ValidationException;
 import org.navalplanner.business.labels.daos.ILabelDAO;
@@ -79,6 +80,9 @@ public class WorkReportModel implements IWorkReportModel {
     @Autowired
     private ILabelDAO labelDAO;
 
+    @Autowired
+    private IConfigurationDAO configurationDAO;
+
     private WorkReportType workReportType;
 
     private WorkReport workReport;
@@ -113,6 +117,10 @@ public class WorkReportModel implements IWorkReportModel {
         editing = false;
         forceLoadWorkReportTypeFromDB(workReportType);
         workReport = WorkReport.create(this.workReportType);
+        workReport.setGenerateCode(configurationDAO.getConfiguration().getGenerateCodeForWorkReport());
+        if(!workReport.getGenerateCode()) {
+            workReport.setCode("");
+        }
         loadMaps();
     }
 
@@ -334,6 +342,9 @@ public class WorkReportModel implements IWorkReportModel {
     public WorkReportLine addWorkReportLine() {
         if (workReport != null) {
             WorkReportLine workReportLine = WorkReportLine.create(workReport);
+            if(!workReport.getGenerateCode()) {
+                workReportLine.setCode("");
+            }
             workReport.addWorkReportLine(workReportLine);
             return workReportLine;
         }
diff --git a/navalplanner-webapp/src/main/webapp/workreports/workReport.zul b/navalplanner-webapp/src/main/webapp/workreports/workReport.zul
index 2524b1f16..215c87479 100644
--- a/navalplanner-webapp/src/main/webapp/workreports/workReport.zul
+++ b/navalplanner-webapp/src/main/webapp/workreports/workReport.zul
@@ -128,8 +128,14 @@
                                     </row>
                                     <row>
                                         <label value="${i18n:_('Code')}:" />
-                                        <textbox value="@{controller.workReport.code}" width="500px"
-                                            constraint="no empty:${i18n:_('cannot be null or empty')}" />
+                                        <hbox>
+                                            <textbox width="350px" value="@{controller.workReport.code}"
+                                                constraint="no empty:${i18n:_('cannot be null or empty')}"
+                                                disabled="@{controller.workReport.generateCode}" />
+                                            <checkbox id="generateCode" label="${i18n:_('Generate code')}"
+                                                onCheck="controller.onCheckGenerateCode(event)"
+                                                checked="@{controller.workReport.generateCode}" />
+                                        </hbox>
                                     </row>
                                 </rows>
                             </grid>