From 40e42873aeeea56c9371fb5d30fdd8f3e6940bef Mon Sep 17 00:00:00 2001
From: Manuel Rego Casasnovas <mrego@igalia.com>
Date: Thu, 4 Mar 2010 15:44:36 +0100
Subject: [PATCH]  ItEr49S04ValidacionEProbasFuncionaisItEr48S04: [Bug #358]
 Fixed problem saving advances types.

---
 .../web/advance/AdvanceTypeCRUDController.java      | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/navalplanner-webapp/src/main/java/org/navalplanner/web/advance/AdvanceTypeCRUDController.java b/navalplanner-webapp/src/main/java/org/navalplanner/web/advance/AdvanceTypeCRUDController.java
index ec9ded9fd..cc27472f7 100644
--- a/navalplanner-webapp/src/main/java/org/navalplanner/web/advance/AdvanceTypeCRUDController.java
+++ b/navalplanner-webapp/src/main/java/org/navalplanner/web/advance/AdvanceTypeCRUDController.java
@@ -162,12 +162,14 @@ public class AdvanceTypeCRUDController extends GenericForwardComposer {
         return newConstraint;
     }
 
-    private void save() {
+    private boolean save() {
         try {
             advanceTypeModel.save();
             messagesForUser.showMessage(Level.INFO, _("Advance type saved"));
+            return true;
         } catch (ValidationException e) {
             messagesForUser.showInvalidValues(e);
+            return false;
         }
     }
 
@@ -236,12 +238,15 @@ public class AdvanceTypeCRUDController extends GenericForwardComposer {
     }
 
     public void saveAndExit() {
-        save();
-        goToList();
+        if (save()) {
+            goToList();
+        }
     }
 
     public void saveAndContinue() {
-        save();
+        if (save()) {
+            goToEditForm(getAdvanceType());
+        }
     }
 
     public boolean isImmutable() {