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 013606090..985783baf 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 @@ -149,9 +149,9 @@ public class AdvanceTypeCRUDController extends GenericForwardComposer { throw new WrongValueException(comp, _("The name is not valid, the name must not be null ")); } - if (!(advanceTypeModel.distinctNames((String) value))) { + if (!advanceTypeModel.distinctNames((String) value)) { throw new WrongValueException(comp, - _("The name is not valid, there is another advance type with a similar name. ")); + _("The name is not valid, there is another advance type with the same name. ")); } } }; diff --git a/navalplanner-webapp/src/main/java/org/navalplanner/web/advance/AdvanceTypeModel.java b/navalplanner-webapp/src/main/java/org/navalplanner/web/advance/AdvanceTypeModel.java index 7bb6a13fe..a8a9182a8 100644 --- a/navalplanner-webapp/src/main/java/org/navalplanner/web/advance/AdvanceTypeModel.java +++ b/navalplanner-webapp/src/main/java/org/navalplanner/web/advance/AdvanceTypeModel.java @@ -136,10 +136,8 @@ public class AdvanceTypeModel implements IAdvanceTypeModel { } for (AdvanceType advanceType : advanceTypeDAO .list(AdvanceType.class)) { - if ((advanceType.getId() == null) - || (!advanceType.getId().equals(this.advanceType.getId())) - && (advanceType.getUnitName().contains(name) || (name - .contains(advanceType.getUnitName())))) { + if (!advanceType.getId().equals(this.advanceType.getId()) + && advanceType.getUnitName().equalsIgnoreCase(name)) { return false; } }