From f2e7235a0ebe51182fb28c9f4edef57c22687117 Mon Sep 17 00:00:00 2001
From: Diego Pino Garcia <dpino@igalia.com>
Date: Mon, 9 Nov 2009 18:08:09 +0100
Subject: [PATCH]  ItEr34S08ValidacionEProbasFuncionaisItEr33S08: [Bug #76] Add
 confirm deleting WorkReportLine dialog

---
 .../workreports/WorkReportCRUDController.java | 28 +++++++++++++++----
 1 file changed, 23 insertions(+), 5 deletions(-)

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 74a4162c1..26c389e48 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
@@ -553,12 +553,11 @@ public class WorkReportCRUDController extends GenericForwardComposer implements
     }
 
     /**
-     * Append a delete {@link Button} numHours to {@link Row}
+     * Append a delete {@link Button} to {@link Row}
      *
      * @param row
      */
-    private void appendDeleteButton(Row row) {
-        final Row li = row;
+    private void appendDeleteButton(final Row row) {
         Button delete = new Button("", "/common/img/ico_borrar1.png");
         delete.setHoverImage("/common/img/ico_borrar.png");
         delete.setSclass("icono");
@@ -566,13 +565,32 @@ public class WorkReportCRUDController extends GenericForwardComposer implements
         delete.addEventListener(Events.ON_CLICK, new EventListener() {
             @Override
             public void onEvent(Event event) throws Exception {
-                WorkReportLine workReportLine = (WorkReportLine) li.getValue();
-                removeWorkReportLine(workReportLine);
+                confirmRemove((WorkReportLine) row.getValue());
             }
         });
         row.appendChild(delete);
     }
 
+    public void confirmRemove(WorkReportLine workReportLine) {
+        try {
+            int status = Messagebox.show(_("Confirm deleting {0}. Are you sure?", getWorkReportLineName(workReportLine)), _("Delete"),
+                    Messagebox.OK | Messagebox.CANCEL, Messagebox.QUESTION);
+            if (Messagebox.OK == status) {
+                removeWorkReportLine(workReportLine);
+            }
+        } catch (InterruptedException e) {
+            messagesForUser.showMessage(
+                    Level.ERROR, e.getMessage());
+            LOG.error(_("Error on showing removing element: ", workReportLine.getId()), e);
+        }
+    }
+
+    private String getWorkReportLineName(WorkReportLine workReportLine) {
+        final String resourceName = ((Worker) workReportLine.getResource()).getName();
+        final String code = workReportLine.getOrderElement().getCode();
+        return resourceName + " - " + code;
+    }
+
     /**
      * Binds Intbox numHours to a {@link WorkReportLine} numHours
      * @param intNumHours