Author: bleny Date: 2013-08-20 15:25:20 +0200 (Tue, 20 Aug 2013) New Revision: 367 Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revisions... Log: refs #2999 when affectation is removed, show if list is complete or not Added: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/DeliveredRequestedListStatus.java Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/RequestedArticleService.java trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/DeliverRequestedArticleAction.java trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/WithdrawItemAction.java Added: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/DeliveredRequestedListStatus.java =================================================================== --- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/DeliveredRequestedListStatus.java (rev 0) +++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/DeliveredRequestedListStatus.java 2013-08-20 13:25:20 UTC (rev 367) @@ -0,0 +1,19 @@ +package com.franciaflex.magalie.persistence.entity; + +public enum DeliveredRequestedListStatus { + + PENDING, + + AFFECTED, + + COMPLETE; + + public boolean isComplete() { + return this == COMPLETE; + } + + public boolean isAffected() { + return this == AFFECTED; + } + +} //DeliveredRequestedListStatus Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/RequestedArticleService.java =================================================================== --- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/RequestedArticleService.java 2013-08-19 15:43:42 UTC (rev 366) +++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/RequestedArticleService.java 2013-08-20 13:25:20 UTC (rev 367) @@ -201,7 +201,7 @@ + " to " + requestedList.getCode()); } - boolean isFinished = oldAffectation.getStatus().equals(DeliveredRequestedListStatus.COMPLETE); + boolean isFinished = oldAffectation.getStatus().isComplete(); if (!isFinished) { @@ -398,7 +398,7 @@ DeliveredRequestedList deliveredRequestedList = dao.findByAffectedTo(magalieUser); - if (deliveredRequestedList != null && deliveredRequestedList.getStatus() == DeliveredRequestedListStatus.AFFECTED) { + if (deliveredRequestedList != null && deliveredRequestedList.getStatus().isAffected()) { if (log.isInfoEnabled()) { log.info("user " + magalieUser.getLogin() + Modified: trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/DeliverRequestedArticleAction.java =================================================================== --- trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/DeliverRequestedArticleAction.java 2013-08-19 15:43:42 UTC (rev 366) +++ trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/DeliverRequestedArticleAction.java 2013-08-20 13:25:20 UTC (rev 367) @@ -115,8 +115,20 @@ if (oldAffectation != null) { - session.addMessage("Vous n'êtes plus affecté à la liste " + oldAffectation.getRequestedList().getCode()); + String message = "Vous n'êtes plus affecté à la liste " + oldAffectation.getRequestedList().getCode(); + if (oldAffectation.getStatus().isComplete()) { + + message += " : liste terminée"; + + } else { + + message += " : liste non terminée"; + + } + + session.addMessage(message); + } DeliveredRequestedList newAffectation = findOrderToExecuteResult.getNewAffectation(); Modified: trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/WithdrawItemAction.java =================================================================== --- trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/WithdrawItemAction.java 2013-08-19 15:43:42 UTC (rev 366) +++ trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/WithdrawItemAction.java 2013-08-20 13:25:20 UTC (rev 367) @@ -26,7 +26,6 @@ import com.franciaflex.magalie.MagalieTechnicalException; import com.franciaflex.magalie.persistence.entity.Article; import com.franciaflex.magalie.persistence.entity.DeliveredRequestedList; -import com.franciaflex.magalie.persistence.entity.DeliveredRequestedListStatus; import com.franciaflex.magalie.persistence.entity.MagalieUser; import com.franciaflex.magalie.persistence.entity.RequestedArticle; import com.franciaflex.magalie.services.StorageMovementConfirmation; @@ -205,7 +204,7 @@ DeliveredRequestedList deliveredRequestedList = deliveredRequestedListOptional.get(); - if (deliveredRequestedList.getStatus() == DeliveredRequestedListStatus.COMPLETE) { + if (deliveredRequestedList.getStatus().isComplete()) { session.addMessage("La liste " + deliveredRequestedList.getRequestedList().getCode() + " est terminée");