Author: bleny Date: 2013-08-29 13:18:09 +0200 (Thu, 29 Aug 2013) New Revision: 393 Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revisions... Log: fixes #3112 hide messages that duplicate the message in service notifications 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/WithdrawItemAction.java 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-29 10:48:01 UTC (rev 392) +++ trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/DeliverRequestedArticleAction.java 2013-08-29 11:18:09 UTC (rev 393) @@ -43,7 +43,8 @@ import java.util.List; @Results({ - @Result(name="success", type="redirectAction", params = { "actionName", "withdraw-item!input", "requestedArticleId", "${requestedArticleId}" }) + @Result(name="success", type="redirectAction", params = { "actionName", "withdraw-item!input", "requestedArticleId", "${requestedArticleId}" }), + @Result(name="choose-list-type", type="redirectAction", params = { "actionName", "deliver-requested-article!input" }) }) public class DeliverRequestedArticleAction extends MagalieActionSupport implements Preparable { @@ -107,6 +108,12 @@ this.listType = listType; } + /** + * On arrive ici quand l'utilisateur a sélectionné un type de liste. On cherche + * quelque-chose à faire et si on trouve ou redirige vers l'interface de prélèvement + * sinon on reste sur la liste des types de listes en affichant un message expliquant + * pourquoi on est pas allé plus loin + */ @Override public String execute() { @@ -157,6 +164,44 @@ } + /** + * On arrive ici après avoir terminer de prélever un article. On redirige vers l'article + * suivant s'il y en a un sinon on propose directement de changer le type de liste. + * + * On affiche pas de message particulier, l'utilisateur a déjà été notifié qu'il n'est + * plus affecté à la liste + */ + public String next() { + + Preconditions.checkState(listType != null); + + session.setActivity(Activity.REQUESTED_LISTS); + + MagalieUser magalieUser = session.getMagalieUser(); + + Building building = session.getBuilding(); + + FindOrderToExecuteResult findOrderToExecuteResult = + service.findOrderToExecute(magalieUser, building, listType); + + String result; + + if (findOrderToExecuteResult.isSuccess()) { + + requestedArticle = findOrderToExecuteResult.getRequestedArticle(); + + result = SUCCESS; + + } else { + + result = "choose-list-type"; + + } + + return result; + + } + public String getRequestedArticleId() { return requestedArticle.getId(); } 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-29 10:48:01 UTC (rev 392) +++ trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/WithdrawItemAction.java 2013-08-29 11:18:09 UTC (rev 393) @@ -48,7 +48,7 @@ @Results({ @Result(name="KANBANS", type="redirectAction", params = { "actionName", "fulfil-kanban!input" }), - @Result(name="REQUESTED_LISTS", type="redirectAction", params = { "actionName", "deliver-requested-article", "listType", "%{listType}" }), + @Result(name="REQUESTED_LISTS", type="redirectAction", params = { "actionName", "deliver-requested-article!next", "listType", "%{listType}" }), @Result(name="REQUESTED_LISTS_CANCEL", type="redirectAction", params = { "actionName", "deliver-requested-article!input" }) }) public class WithdrawItemAction extends MagalieActionSupport implements Preparable {