Author: bleny Date: 2013-08-26 17:58:30 +0200 (Mon, 26 Aug 2013) New Revision: 375 Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revisions... Log: fixes #2998 show listCode on withdraw screen, after confirmation of withdraw for requestedArticle jump to another requestedArticle 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 trunk/magalie-web/src/main/webapp/WEB-INF/content/deliver-requested-article-input.jsp trunk/magalie-web/src/main/webapp/WEB-INF/content/withdraw-item-input.jsp trunk/magalie-web/src/main/webapp/css/magalie-ck3x-reduced.css 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-26 10:09:03 UTC (rev 374) +++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/RequestedArticleService.java 2013-08-26 15:58:30 UTC (rev 375) @@ -467,7 +467,7 @@ } - public RequestedList getAffectationForUser(MagalieUser magalieUser) { + public Optional<RequestedList> getAffectationForUser(MagalieUser magalieUser) { JpaMagaliePersistenceContext persistenceContext = serviceContext.getPersistenceContext(); @@ -475,15 +475,19 @@ DeliveredRequestedList deliveredRequestedList = dao.findByAffectedTo(magalieUser); - RequestedList requestedList = null; + Optional<RequestedList> affectationForUser; - if (deliveredRequestedList != null) { + if (deliveredRequestedList == null) { - requestedList = deliveredRequestedList.getRequestedList(); + affectationForUser = Optional.absent(); + } else { + + affectationForUser = Optional.of(deliveredRequestedList.getRequestedList()); + } - return requestedList; + return affectationForUser; } 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-26 10:09:03 UTC (rev 374) +++ trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/DeliverRequestedArticleAction.java 2013-08-26 15:58:30 UTC (rev 375) @@ -32,7 +32,9 @@ import com.franciaflex.magalie.web.Activity; import com.franciaflex.magalie.web.MagalieActionSupport; import com.franciaflex.magalie.web.MagalieSession; +import com.google.common.base.Optional; import com.google.common.base.Preconditions; +import com.opensymphony.xwork2.Preparable; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.apache.struts2.convention.annotation.Result; @@ -43,7 +45,7 @@ @Results({ @Result(name="success", type="redirectAction", params = { "actionName", "withdraw-item!input", "requestedArticleId", "${requestedArticleId}" }) }) -public class DeliverRequestedArticleAction extends MagalieActionSupport { +public class DeliverRequestedArticleAction extends MagalieActionSupport implements Preparable { private static final Log log = LogFactory.getLog(DeliverRequestedArticleAction.class); @@ -68,16 +70,27 @@ } @Override - public String input() { + public void prepare() { MagalieUser magalieUser = session.getMagalieUser(); - requestedList = service.getAffectationForUser(magalieUser); + Optional<RequestedList> affectationForUser = service.getAffectationForUser(magalieUser); + if (affectationForUser.isPresent()) { + + requestedList = affectationForUser.get(); + + } + Building building = session.getBuilding(); listTypes = service.getListTypes(building); + } + + @Override + public String input() { + return INPUT; } 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-26 10:09:03 UTC (rev 374) +++ trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/WithdrawItemAction.java 2013-08-26 15:58:30 UTC (rev 375) @@ -47,7 +47,7 @@ @Results({ @Result(name="KANBANS", type="redirectAction", params = { "actionName", "fulfil-kanban!input" }), - @Result(name="REQUESTED_LISTS", type="redirectAction", params = { "actionName", "deliver-requested-article!input" }) + @Result(name="REQUESTED_LISTS", type="redirectAction", params = { "actionName", "deliver-requested-article", "listType", "%{listType}" }) }) public class WithdrawItemAction extends MagalieActionSupport implements Preparable { @@ -71,6 +71,10 @@ protected Article article; + protected String requestedListCode; + + protected String listType; + public void setArticleId(String articleId) { this.articleId = articleId; } @@ -120,6 +124,8 @@ articleId = article.getId(); + requestedListCode = requestedArticle.getRequestedList().getCode(); + } Preconditions.checkNotNull(articleId); @@ -144,6 +150,10 @@ return expectedQuantity; } + public String getRequestedListCode() { + return requestedListCode; + } + public String getModelAsJson() throws JSONException { Map<String,Object> model = Maps.newHashMap(); @@ -195,12 +205,25 @@ articleStorageService.confirmStorageMovement(confirmation, magalieUser); + if (confirmation.getRequestedArticleId() != null) { + + RequestedArticle requestedArticle = + requestedArticleService.getRequestedArticle(confirmation.getRequestedArticleId()); + + listType = requestedArticle.getRequestedList().getListType(); + + } + String resultName = session.getActivity().name(); return resultName; } + public String getListType() { + return listType; + } + public String cancel() { MagalieUser magalieUser = session.getMagalieUser(); Modified: trunk/magalie-web/src/main/webapp/WEB-INF/content/deliver-requested-article-input.jsp =================================================================== --- trunk/magalie-web/src/main/webapp/WEB-INF/content/deliver-requested-article-input.jsp 2013-08-26 10:09:03 UTC (rev 374) +++ trunk/magalie-web/src/main/webapp/WEB-INF/content/deliver-requested-article-input.jsp 2013-08-26 15:58:30 UTC (rev 375) @@ -46,6 +46,12 @@ <s:a href="%{removeUserAffectationUrl}" cssClass="btn btn-block" id="removeUserAffectationLink">Abandonner la liste (Esc)</s:a> </s:if> <s:else> + <script type="text/javascript"> + $(document).ready(function () { + bindKey('Esc', function(){ redirectTo($('#removeUserAffectationLink'));}); + }); + </script> + <s:if test="listTypes.empty"> Aucune liste à servir à traiter </s:if> Modified: trunk/magalie-web/src/main/webapp/WEB-INF/content/withdraw-item-input.jsp =================================================================== --- trunk/magalie-web/src/main/webapp/WEB-INF/content/withdraw-item-input.jsp 2013-08-26 10:09:03 UTC (rev 374) +++ trunk/magalie-web/src/main/webapp/WEB-INF/content/withdraw-item-input.jsp 2013-08-26 15:58:30 UTC (rev 375) @@ -38,6 +38,11 @@ <header> <ul class="oneItemPerLine"> + <s:if test="%{requestedListCode != null}"> + <li> + Liste : <s:property value="requestedListCode" /> + </li> + </s:if> <li> Réf. : <s:property value="article.code" /> </li> Modified: trunk/magalie-web/src/main/webapp/css/magalie-ck3x-reduced.css =================================================================== --- trunk/magalie-web/src/main/webapp/css/magalie-ck3x-reduced.css 2013-08-26 10:09:03 UTC (rev 374) +++ trunk/magalie-web/src/main/webapp/css/magalie-ck3x-reduced.css 2013-08-26 15:58:30 UTC (rev 375) @@ -64,11 +64,14 @@ margin-bottom: 0px; } -#storageMovements, #locations { min-height: 100px; /* 5 * 20px for each element */ } +#storageMovements { + min-height: 80px; /* 4 * 20px for each element */ +} + #storageMovements div, #locations dl { margin: 0;