Author: fdesbois Date: 2010-03-30 08:24:27 +0000 (Tue, 30 Mar 2010) New Revision: 398 Modified: trunk/wao-ui/src/main/java/fr/ifremer/wao/ui/pages/Contacts.java Log: Regression on cancel contactEdit (due to contactState refactor) Modified: trunk/wao-ui/src/main/java/fr/ifremer/wao/ui/pages/Contacts.java =================================================================== --- trunk/wao-ui/src/main/java/fr/ifremer/wao/ui/pages/Contacts.java 2010-03-29 16:19:14 UTC (rev 397) +++ trunk/wao-ui/src/main/java/fr/ifremer/wao/ui/pages/Contacts.java 2010-03-30 08:24:27 UTC (rev 398) @@ -294,16 +294,6 @@ return contacts; } -// public Map<String, Contact> getContacts() throws WaoException { -// if (contacts == null) { -// if (log.isInfoEnabled()) { -// log.info("BUSINESS REQUEST [getContactsByFilter]"); -// } -// contacts = serviceContact.getContactsByFilter(getContactFilter()); -// } -// return contacts; -// } - public BeanModel<Contact> getContactModel() { if (contactModel == null) { contactModel = user.isAdmin() ? @@ -562,8 +552,8 @@ contactsForm.clearErrors(); // Validation for saving contact depends on contactState (only edition // form) - ContactState contactState = contactEdited.getContactState(); - if (!edited && contactState != null) { + if (!edited && contactEdited != null) { + ContactState contactState = contactEdited.getContactState(); if (log.isInfoEnabled()) { log.info("For state : " + contactState); } @@ -576,31 +566,39 @@ if (begin != null && !row.isValid(begin)) { contactsForm.recordError(beginDate, - "La date de début de la marée doit correspondre à un mois valide (non vide) de la ligne " + row.getCode()); + "La date de début de la marée doit correspondre à un " + + "mois valide (non vide) de la ligne " + row.getCode()); } if (begin != null && end != null && end.before(begin)) { - contactsForm.recordError(endDate, "La date de fin de la marée ne peut pas être antérieure à celle de début"); + contactsForm.recordError(endDate, "La date de fin de la marée" + + " ne peut pas être antérieure à celle de début"); } Date current = WaoContext.getCurrentDate(); if (end != null && end.after(current)) { - contactsForm.recordError(endDate, "La date de fin de la marée ne peut pas être postérieure à la date du jour"); + contactsForm.recordError(endDate, "La date de fin de la marée" + + " ne peut pas être postérieure à la date du jour"); } if (end != null && input != null && end.after(input)) { - contactsForm.recordError(inputDate, "La date de saisie des données ne peut pas être antérieure à la date de fin de la marée"); + contactsForm.recordError(inputDate, "La date de saisie des" + + " données ne peut pas être antérieure à la date de" + + " fin de la marée"); } if (input != null && input.after(current)) { - contactsForm.recordError(inputDate, "La date de saisie des données ne peut pas être postérieure à la date du jour"); + contactsForm.recordError(inputDate, "La date de saisie des" + + " données ne peut pas être postérieure à la date" + + " du jour"); } // Non abouti, Refus ou Refus Définitif if (contactState.isUnfinishedState()) { String newComment = contactEdited.getComment(); - if (StringUtils.isNotEmpty(newComment) && !newComment.equals(oldComment)) { + if (StringUtils.isNotEmpty(newComment) && + !newComment.equals(oldComment)) { // RAZ des champs contactEdited.setTideBeginDate(null); contactEdited.setTideEndDate(null); @@ -609,18 +607,26 @@ contactEdited.setMammalsObservation(false); contactEdited.setDataInputDate(null); } else { - contactsForm.recordError(comment, "Vous devez ajouter un commentaire pour l'état '" + contactState + "'"); + contactsForm.recordError(comment, "Vous devez ajouter" + + " un commentaire pour l'état" + + " '" + contactState + "'"); } // Embarquement Réalisé } else if (contactState.equals(ContactState.BOARDING_DONE)) { if (begin == null) { - contactsForm.recordError(beginDate, "La date de début de marée est obligatoire pour l'état '" + contactState + "'"); + contactsForm.recordError(beginDate, "La date de début de" + + " marée est obligatoire pour l'état" + + " '" + contactState + "'"); } if (end == null) { - contactsForm.recordError(endDate, "La date de fin de marée est obligatoire pour l'état '" + contactState + "'"); + contactsForm.recordError(endDate, "La date de fin de" + + " marée est obligatoire pour l'état" + + " '" + contactState + "'"); } if (contactEdited.getNbObservants() == 0) { - contactsForm.recordError(nbObservants, "Il ne peut y avoir aucun observateur pour l'état '" + contactState + "'"); + contactsForm.recordError(nbObservants, "Il ne peut y" + + " avoir aucun observateur pour l'état" + + " '" + contactState + "'"); } } }