- Franciaflex-magalie-commits - mailman.cloud.codelutin.com

r105 - in trunk: magalie-services/src/main/java/com/franciaflex/magalie/services/service magalie-web/src/main/java/com/franciaflex/magalie/web/action magalie-web/src/main/webapp/WEB-INF/content
by bleny＠users.forge.codelutin.com April 17, 2013

April 17, 2013

Author: bleny Date: 2013-04-17 15:27:17 +0200 (Wed, 17 Apr 2013) New Revision: 105 Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision… Log: ask for quantity in kanban when necessary Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/FulfilKanbanService.java trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/FulfilKanbanAction.java trunk/magalie-web/src/main/webapp/WEB-INF/content/fulfil-kanban-input.jsp Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/FulfilKanbanService.java =================================================================== --- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/FulfilKanbanService.java 2013-04-17 10:35:44 UTC (rev 104) +++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/FulfilKanbanService.java 2013-04-17 13:27:17 UTC (rev 105) @@ -12,6 +12,7 @@ import com.franciaflex.magalie.services.MagalieService; import com.franciaflex.magalie.services.MagalieServiceContext; import com.franciaflex.magalie.services.exception.InvalidMagalieBarcodeException; +import com.google.common.base.Preconditions; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; @@ -40,29 +41,43 @@ } + public double getDefinedQuantity( + Article article, + Warehouse destinationStore) { + + double quantityInKanban = article.getQuantityInKanban(); + + // TODO brendan 17/04/13 consider destination store + + return quantityInKanban; + + } + public BookArticleResult fulfilKanbanTask( - String articleBarcode, + Article article, MagalieUser magalieUser, - String destinationStoreId) throws InvalidMagalieBarcodeException { + Warehouse destinationStore) { - if (log.isInfoEnabled()) { - log.info(magalieUser + " requested fulfil kanban with barcode " + articleBarcode); - } - MagalieBarcodeService magalieBarcodeService = serviceContext.newService(MagalieBarcodeService.class); - Article article = magalieBarcodeService.getArticle(articleBarcode); - double quantityInKanban = article.getQuantityInKanban(); + Preconditions.checkArgument(quantityInKanban > 0); + + return fulfilKanbanTask(article, magalieUser, destinationStore, quantityInKanban); + + } + + public BookArticleResult fulfilKanbanTask( + Article article, + MagalieUser magalieUser, + Warehouse destinationWarehouse, + double quantityInKanban) { + ArticleStorageService articleStorageService = serviceContext.newService(ArticleStorageService.class); - WarehouseDao warehouseDao = serviceContext.getPersistenceContext().getWarehouseDao(); - - Warehouse destinationWarehouse = warehouseDao.findById(destinationStoreId); - Building building = destinationWarehouse.getBuilding(); Location destinationLocation = getDestinationLocation(destinationWarehouse); @@ -102,4 +117,15 @@ return destinationWarehouses; } + + public Article getArticle(String articleBarcode) throws InvalidMagalieBarcodeException { + + MagalieBarcodeService magalieBarcodeService = + serviceContext.newService(MagalieBarcodeService.class); + + Article article = magalieBarcodeService.getArticle(articleBarcode); + + return article; + + } } Modified: trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/FulfilKanbanAction.java =================================================================== --- trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/FulfilKanbanAction.java 2013-04-17 10:35:44 UTC (rev 104) +++ trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/FulfilKanbanAction.java 2013-04-17 13:27:17 UTC (rev 105) @@ -1,5 +1,6 @@ package com.franciaflex.magalie.web.action; +import com.franciaflex.magalie.persistence.entity.Article; import com.franciaflex.magalie.persistence.entity.Building; import com.franciaflex.magalie.persistence.entity.MagalieUser; import com.franciaflex.magalie.persistence.entity.StorageMovementOrder; @@ -10,6 +11,7 @@ import com.franciaflex.magalie.web.Activity; import com.franciaflex.magalie.web.MagalieActionSupport; import com.franciaflex.magalie.web.MagalieSession; +import com.opensymphony.xwork2.Preparable; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.apache.struts2.convention.annotation.Result; @@ -21,7 +23,7 @@ @Results({ @Result(name="success", type="redirectAction", params = { "actionName", "withdraw-item!input", "storageMovementOrderId", "${storageMovementOrderId}" }) }) -public class FulfilKanbanAction extends MagalieActionSupport { +public class FulfilKanbanAction extends MagalieActionSupport implements Preparable { private static final Log log = LogFactory.getLog(FulfilKanbanAction.class); @@ -37,6 +39,14 @@ protected String destinationWarehouseId; + protected Double quantity; + + protected boolean askQuantity; + + protected Article article; + + protected Warehouse destinationWarehouse; + public void setService(FulfilKanbanService service) { this.service = service; } @@ -46,7 +56,7 @@ } @Override - public String input() { + public void prepare() { session.setActivity(Activity.KANBANS); @@ -64,8 +74,10 @@ } - return INPUT; + } + public boolean isAskQuantity() { + return askQuantity; } public String getDestinationWarehouseId() { @@ -84,6 +96,10 @@ this.destinationWarehouseId = destinationWarehouseId; } + public void setQuantity(Double quantity) { + this.quantity = quantity; + } + @Override public String execute() { @@ -95,16 +111,14 @@ MagalieUser magalieUser = session.getMagalieUser(); - Warehouse destinationWarehouse = service.getStore(destinationWarehouseId); + destinationWarehouse = service.getStore(destinationWarehouseId); // save used destination warehouse to propose it by default on next kanban session.setLastUsedDestinationWarehouseForKanbans(destinationWarehouse); - BookArticleResult bookArticleResult; - try { - bookArticleResult = service.fulfilKanbanTask(articleBarcode, magalieUser, destinationWarehouseId); + article = service.getArticle(articleBarcode); } catch (InvalidMagalieBarcodeException e) { @@ -114,6 +128,31 @@ } + if (quantity == null) { + + double quantityDefined = service.getDefinedQuantity(article, destinationWarehouse); + + if (quantityDefined > 0) { + + quantity = quantityDefined; + + } + + } + + if (quantity == null || ! (quantity > 0.)) { + + askQuantity = true; + + addFieldError("quantity", "Il faut préciser une quantité pour cet article"); + + return INPUT; + + } + + BookArticleResult bookArticleResult = + service.fulfilKanbanTask(article, magalieUser, destinationWarehouse, quantity); + if (bookArticleResult.isArticleUnavailable()) { addActionError("Cet article n'est pas disponible"); @@ -136,6 +175,14 @@ } + public Article getArticle() { + return article; + } + + public String getArticleBarcode() { + return articleBarcode; + } + public String getStorageMovementOrderId() { return storageMovementOrder.getId(); } Modified: trunk/magalie-web/src/main/webapp/WEB-INF/content/fulfil-kanban-input.jsp =================================================================== --- trunk/magalie-web/src/main/webapp/WEB-INF/content/fulfil-kanban-input.jsp 2013-04-17 10:35:44 UTC (rev 104) +++ trunk/magalie-web/src/main/webapp/WEB-INF/content/fulfil-kanban-input.jsp 2013-04-17 13:27:17 UTC (rev 105) @@ -6,7 +6,7 @@ </head> <s:form> - <s:textfield name="articleBarcode" label="Article" inputAppendIcon="barcode" cssClass="input-medium" /> + <s:textfield key="articleBarcode" label="Article" inputAppendIcon="barcode" cssClass="input-medium" /> <s:select label="Magasin destination" key="destinationWarehouseId" @@ -16,6 +16,10 @@ required="true" /> + <s:if test="askQuantity"> + <s:textfield name="quantity" label="Qté" inputAppend="%{article.unit}" cssClass="input-mini" /> + </s:if> + <div class="btn-group"> <s:submit name="next" value="Suivant" cssClass="btn btn-primary" /> </div>

1 0

r104 - in trunk/magalie-web/src/main/java/com/franciaflex/magalie/web: . action
by bleny＠users.forge.codelutin.com April 17, 2013

April 17, 2013

Author: bleny Date: 2013-04-17 12:35:44 +0200 (Wed, 17 Apr 2013) New Revision: 104 Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision… Log: introduce current activity in session to redirect user Added: trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/Activity.java Modified: trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/MagalieSession.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/FulfilKanbanAction.java trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/WithdrawItemAction.java Added: trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/Activity.java =================================================================== --- trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/Activity.java (rev 0) +++ trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/Activity.java 2013-04-17 10:35:44 UTC (rev 104) @@ -0,0 +1,14 @@ +package com.franciaflex.magalie.web; + +/** + * Current activity of the user. May be used for redirection since a user + * keep doing the same thing until manually declare otherwise. + * + * @author bleny + */ +public enum Activity { + + REQUESTED_LISTS, + KANBANS + +} Modified: trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/MagalieSession.java =================================================================== --- trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/MagalieSession.java 2013-04-17 10:27:04 UTC (rev 103) +++ trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/MagalieSession.java 2013-04-17 10:35:44 UTC (rev 104) @@ -26,6 +26,8 @@ protected Warehouse lastUsedDestinationWarehouseForKanbans; + protected Activity activity; + protected Collection<String> messages; public MagalieUser getMagalieUser() { @@ -63,4 +65,12 @@ getMessages().add(message); } + public Activity getActivity() { + return activity; + } + + public void setActivity(Activity activity) { + this.activity = activity; + } + } 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-04-17 10:27:04 UTC (rev 103) +++ trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/DeliverRequestedArticleAction.java 2013-04-17 10:35:44 UTC (rev 104) @@ -6,6 +6,7 @@ import com.franciaflex.magalie.persistence.entity.StorageMovementOrder; import com.franciaflex.magalie.services.service.FindOrderToExecuteResult; import com.franciaflex.magalie.services.service.RequestedArticleService; +import com.franciaflex.magalie.web.Activity; import com.franciaflex.magalie.web.MagalieActionSupport; import com.franciaflex.magalie.web.MagalieSession; import org.apache.commons.logging.Log; @@ -37,6 +38,8 @@ @Override public String execute() { + session.setActivity(Activity.REQUESTED_LISTS); + MagalieUser magalieUser = session.getMagalieUser(); Building building = session.getBuilding(); Modified: trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/FulfilKanbanAction.java =================================================================== --- trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/FulfilKanbanAction.java 2013-04-17 10:27:04 UTC (rev 103) +++ trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/FulfilKanbanAction.java 2013-04-17 10:35:44 UTC (rev 104) @@ -7,6 +7,7 @@ import com.franciaflex.magalie.services.exception.InvalidMagalieBarcodeException; import com.franciaflex.magalie.services.service.BookArticleResult; import com.franciaflex.magalie.services.service.FulfilKanbanService; +import com.franciaflex.magalie.web.Activity; import com.franciaflex.magalie.web.MagalieActionSupport; import com.franciaflex.magalie.web.MagalieSession; import org.apache.commons.logging.Log; @@ -45,8 +46,10 @@ } @Override - public String input() throws Exception { + public String input() { + session.setActivity(Activity.KANBANS); + Building building = session.getBuilding(); destinationWarehouses = service.getDestinationWarehouses(building); @@ -84,6 +87,8 @@ @Override public String execute() { + session.setActivity(Activity.KANBANS); + if (log.isDebugEnabled()) { log.debug("article barcode is " + articleBarcode); } 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-04-17 10:27:04 UTC (rev 103) +++ trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/WithdrawItemAction.java 2013-04-17 10:35:44 UTC (rev 104) @@ -42,7 +42,8 @@ import java.util.Map; @Results({ - @Result(name="success", type="redirectAction", params = { "actionName", "report" }) + @Result(name="KANBANS", type="redirectAction", params = { "actionName", "fulfil-kanban!input" }), + @Result(name="REQUESTED_LISTS", type="redirectAction", params = { "actionName", "deliver-requested-article!input" }) }) public class WithdrawItemAction extends MagalieActionSupport { @@ -119,7 +120,9 @@ service.confirmStorageMovement(confirmation, magalieUser); - return SUCCESS; + String resultName = session.getActivity().name(); + return resultName; + } }

1 0

r103 - in trunk: magalie-services/src/main/java/com/franciaflex/magalie/services magalie-services/src/main/java/com/franciaflex/magalie/services/service magalie-web/src/main/webapp/WEB-INF/content
by bleny＠users.forge.codelutin.com April 17, 2013

April 17, 2013

Author: bleny Date: 2013-04-17 12:27:04 +0200 (Wed, 17 Apr 2013) New Revision: 103 Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision… Log: add delivered requested lists in report Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/MagalieReport.java trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ReportService.java trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/RequestedArticleService.java trunk/magalie-web/src/main/webapp/WEB-INF/content/report.jsp Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/MagalieReport.java =================================================================== --- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/MagalieReport.java 2013-04-17 09:46:14 UTC (rev 102) +++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/MagalieReport.java 2013-04-17 10:27:04 UTC (rev 103) @@ -1,6 +1,7 @@ package com.franciaflex.magalie.services; import com.franciaflex.magalie.persistence.StorageMovements; +import com.franciaflex.magalie.persistence.entity.DeliveredRequestedList; import com.franciaflex.magalie.persistence.entity.LocationError; import com.franciaflex.magalie.persistence.entity.MagalieUser; import com.franciaflex.magalie.persistence.entity.StorageMovement; @@ -20,6 +21,8 @@ protected List<LocationError> allLocationErrors; + protected List<DeliveredRequestedList> allDeliveredRequestedLists; + public void setReportDate(Date reportDate) { this.reportDate = reportDate; } @@ -69,4 +72,12 @@ public void setAllLocationErrors(List<LocationError> allLocationErrors) { this.allLocationErrors = allLocationErrors; } + + public void setAllDeliveredRequestedLists(List<DeliveredRequestedList> allDeliveredRequestedLists) { + this.allDeliveredRequestedLists = allDeliveredRequestedLists; + } + + public List<DeliveredRequestedList> getAllDeliveredRequestedLists() { + return allDeliveredRequestedLists; + } } Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ReportService.java =================================================================== --- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ReportService.java 2013-04-17 09:46:14 UTC (rev 102) +++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ReportService.java 2013-04-17 10:27:04 UTC (rev 103) @@ -1,6 +1,7 @@ package com.franciaflex.magalie.services.service; import com.franciaflex.magalie.persistence.dao.StorageMovementDao; +import com.franciaflex.magalie.persistence.entity.DeliveredRequestedList; import com.franciaflex.magalie.persistence.entity.LocationError; import com.franciaflex.magalie.persistence.entity.MagalieUser; import com.franciaflex.magalie.persistence.entity.StorageMovement; @@ -46,6 +47,12 @@ magalieReport.setAllLocationErrors(allLocationsInError); + RequestedArticleService requestedArticleService = serviceContext.newService(RequestedArticleService.class); + + List<DeliveredRequestedList> allDeliveredRequestedLists = requestedArticleService.getAllDeliveredRequestedLists(); + + magalieReport.setAllDeliveredRequestedLists(allDeliveredRequestedLists); + return magalieReport; } 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-04-17 09:46:14 UTC (rev 102) +++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/RequestedArticleService.java 2013-04-17 10:27:04 UTC (rev 103) @@ -305,4 +305,16 @@ return requestedArticles; } + + public List<DeliveredRequestedList> getAllDeliveredRequestedLists() { + + MagaliePersistenceContext persistenceContext = serviceContext.getPersistenceContext(); + + DeliveredRequestedListDao dao = persistenceContext.getDeliveredRequestedListDao(); + + List<DeliveredRequestedList> all = dao.findAll(); + + return all; + + } } Modified: trunk/magalie-web/src/main/webapp/WEB-INF/content/report.jsp =================================================================== --- trunk/magalie-web/src/main/webapp/WEB-INF/content/report.jsp 2013-04-17 09:46:14 UTC (rev 102) +++ trunk/magalie-web/src/main/webapp/WEB-INF/content/report.jsp 2013-04-17 10:27:04 UTC (rev 103) @@ -205,6 +205,51 @@ </section> <section> + <h2>Traitement des listes</h2> + + <table class="table"> + <thead> + <tr> + <th> + Id + </th> + <th> + Liste + </th> + <th> + Statut + </th> + <th> + Affectation + </th> + </tr> + </thead> + + <tbody> + <s:iterator value="report.allDeliveredRequestedLists"> + <tr> + <td> + <s:property value="id" /> + </td> + <td> + <s:property value="requestedList.code" /> + </td> + <td> + <s:property value="status" /> + </td> + <td> + <s:if test="affectedTo"> + <a href="#<s:property value="affectedTo.id" />"><s:property value="affectedTo.login" /></a> + </s:if> + </td> + </tr> + </s:iterator> + </tbody> + + </table> +</section> + +<section> <h2>Utilisateurs connus</h2> <table class="table">

1 0

r102 - in trunk: magalie-persistence/src/main/java/com/franciaflex/magalie/persistence magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity magalie-services/src/main/java/com/franciaflex/magalie/services/service
by bleny＠users.forge.codelutin.com April 17, 2013

April 17, 2013

Author: bleny Date: 2013-04-17 11:46:14 +0200 (Wed, 17 Apr 2013) New Revision: 102 Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision… Log: on confirmation, when adding a storage movement set destination location Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/StorageMovements.java trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/StorageMovementOrder.java trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ArticleStorageService.java Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/StorageMovements.java =================================================================== --- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/StorageMovements.java 2013-04-17 09:37:50 UTC (rev 101) +++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/StorageMovements.java 2013-04-17 09:46:14 UTC (rev 102) @@ -25,6 +25,14 @@ } } + protected static class GetDestinationFunction implements Function<StorageMovement, Location> { + + @Override + public Location apply(StorageMovement storageMovement) { + return storageMovement.getDestinationLocation(); + } + } + protected static class StorageMovementIsConfirmedPredicate implements Predicate<StorageMovement> { @Override @@ -41,6 +49,10 @@ return new GetOriginFunction(); } + public static Function<StorageMovement, Location> getDestinationFunction() { + return new GetDestinationFunction(); + } + public static Predicate<StorageMovement> storageMovementIsConfirmed() { return new StorageMovementIsConfirmedPredicate(); } Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/StorageMovementOrder.java =================================================================== --- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/StorageMovementOrder.java 2013-04-17 09:37:50 UTC (rev 101) +++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/StorageMovementOrder.java 2013-04-17 09:46:14 UTC (rev 102) @@ -127,4 +127,20 @@ public String toString() { return ToStringBuilder.reflectionToString(this); } + + protected Set<Location> getDestinationLocations() { + Set<Location> locations = + Sets.newHashSet( + Iterables.transform( + getStorageMovements(), + StorageMovements.getDestinationFunction() + ) + ); + return locations; + } + + public Location getDestinationLocation() { + Location onlyElement = Iterables.getOnlyElement(getDestinationLocations()); + return onlyElement; + } } Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ArticleStorageService.java =================================================================== --- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ArticleStorageService.java 2013-04-17 09:37:50 UTC (rev 101) +++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ArticleStorageService.java 2013-04-17 09:46:14 UTC (rev 102) @@ -306,21 +306,24 @@ public void confirmStorageMovement(StorageMovementConfirmation confirmation, MagalieUser magalieUser) { - String storageMovementOrderId = confirmation.getStorageMovementOrderId(); + MagaliePersistenceContext persistenceContext = serviceContext.getPersistenceContext(); StorageMovementOrderDao storageMovementOrderDao = - serviceContext.getPersistenceContext().getStorageMovementOrderDao(); + persistenceContext.getStorageMovementOrderDao(); - LocationDao locationDao = serviceContext.getPersistenceContext().getLocationDao(); + LocationDao locationDao = persistenceContext.getLocationDao(); StorageMovementOrder storageMovementOrder = - storageMovementOrderDao.findById(storageMovementOrderId); + storageMovementOrderDao.findById( + confirmation.getStorageMovementOrderId()); Date confirmDate = serviceContext.getNow(); LocationErrorsService locationErrorsService = serviceContext.newService(LocationErrorsService.class); + Location destinationLocation = storageMovementOrder.getDestinationLocation(); + for (Map.Entry<String, Double> locationIdToActualQuantity : confirmation.getLocationsIdsToActualQuantities().entrySet()) { String locationId = locationIdToActualQuantity.getKey(); @@ -339,7 +342,7 @@ storageMovement.setOriginLocation(originLocation); - // TODO brendan 28/03/13 what about destination location + storageMovement.setDestinationLocation(destinationLocation); storageMovement.setMagalieUser(magalieUser); @@ -367,7 +370,7 @@ storageMovementOrderDao.merge(storageMovementOrder); - serviceContext.getPersistenceContext().commit(); + persistenceContext.commit(); } }

1 0

r101 - trunk/magalie-web/src/main/webapp/js
by bleny＠users.forge.codelutin.com April 17, 2013

April 17, 2013

Author: bleny Date: 2013-04-17 11:37:50 +0200 (Wed, 17 Apr 2013) New Revision: 101 Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision… Log: save quantity when reporting error in location Modified: trunk/magalie-web/src/main/webapp/js/withdraw-item-input.js Modified: trunk/magalie-web/src/main/webapp/js/withdraw-item-input.js =================================================================== --- trunk/magalie-web/src/main/webapp/js/withdraw-item-input.js 2013-04-17 08:28:26 UTC (rev 100) +++ trunk/magalie-web/src/main/webapp/js/withdraw-item-input.js 2013-04-17 09:37:50 UTC (rev 101) @@ -101,7 +101,6 @@ var storageMovement = this.getStorageMovement(barcode); storageMovement.used = true; storageMovement.defect = true; - storageMovement.withdrawn = 0; }; model.nextLocation = function() { @@ -246,7 +245,10 @@ } else { var barcode = $('#withdraw-item_locationBarcode').val(); + var quantity = parseFloat($('#withdraw-item_quantity').val()); + model.withdraw(barcode, quantity); + model.reportError(barcode); // if used location focused as current, highlight next location to go

1 0

r100 - trunk/magalie-web/src/main/webapp/js
by bleny＠users.forge.codelutin.com April 17, 2013

April 17, 2013

Author: bleny Date: 2013-04-17 10:28:26 +0200 (Wed, 17 Apr 2013) New Revision: 100 Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision… Log: do not show next location when there is nothing more to withdraw Modified: trunk/magalie-web/src/main/webapp/js/withdraw-item-input.js Modified: trunk/magalie-web/src/main/webapp/js/withdraw-item-input.js =================================================================== --- trunk/magalie-web/src/main/webapp/js/withdraw-item-input.js 2013-04-16 17:37:40 UTC (rev 99) +++ trunk/magalie-web/src/main/webapp/js/withdraw-item-input.js 2013-04-17 08:28:26 UTC (rev 100) @@ -169,7 +169,7 @@ } else { $(storageMovementElement).addClass('success'); } - } else if (storageMovement.originLocation.id === model.storageMovements[model.storageMovementsIndex].originLocation.id) { + } else if (storageMovement.originLocation.id === model.storageMovements[model.storageMovementsIndex].originLocation.id && ! model.canTerminate()) { $(storageMovementElement).addClass('pending'); storageMovementsElement.scrollTop(storageMovementElement[0].offsetTop); } else {

1 0

r99 - in trunk: magalie-persistence/src/main/java/com/franciaflex/magalie/persistence magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity magalie-services/src/main/java/com/franciaflex/magalie/services/service magalie-services/src/main/resources magalie-services/src/test/java/com/franciaflex/magalie/serv
by bleny＠users.forge.codelutin.com April 16, 2013

April 16, 2013

Author: bleny Date: 2013-04-16 19:37:40 +0200 (Tue, 16 Apr 2013) New Revision: 99 Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision… Log: add affectation management Added: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/RequestedArticles.java trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/RequestedLists.java trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/DeliveredRequestedListDao.java trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/DeliveredRequestedListJpaDao.java trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/DeliveredRequestedList.java trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/DeliveredRequestedListStatus.java Removed: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/RequestedArticles.java Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/Entities.java trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/JpaMagaliePersistenceContext.java trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/MagaliePersistenceContext.java trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/RequestedArticleDao.java trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/RequestedArticleJpaDao.java trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ArticleStorageService.java trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/FindOrderToExecuteResult.java trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/RequestedArticleService.java trunk/magalie-services/src/main/resources/fixtures.yaml trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/RequestedArticleServiceTest.java trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/MagalieSession.java trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/DeliverRequestedArticleAction.java trunk/magalie-web/src/main/resources/log4j.properties Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/Entities.java =================================================================== --- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/Entities.java 2013-04-15 16:18:12 UTC (rev 98) +++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/Entities.java 2013-04-16 17:37:40 UTC (rev 99) @@ -4,7 +4,10 @@ import com.google.common.base.Function; import com.google.common.base.Predicate; import com.google.common.base.Predicates; +import org.apache.commons.lang3.ObjectUtils; +import java.util.Comparator; + public class Entities { private Entities() {} @@ -17,6 +20,15 @@ } } + protected static class ArbitraryComparator<E extends AbstractEntity> implements Comparator<E> { + + @Override + public int compare(E x, E y) { + return ObjectUtils.compare(x.getId(), y.getId()); + } + + } + public static Function<AbstractEntity, String> getIdFunction() { return new GetIdFunction(); } @@ -25,4 +37,7 @@ return Predicates.compose(Predicates.equalTo(id), getIdFunction()); } + public static <E extends AbstractEntity> Comparator<E> arbitraryComparator() { + return new ArbitraryComparator(); + } } Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/JpaMagaliePersistenceContext.java =================================================================== --- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/JpaMagaliePersistenceContext.java 2013-04-15 16:18:12 UTC (rev 98) +++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/JpaMagaliePersistenceContext.java 2013-04-16 17:37:40 UTC (rev 99) @@ -3,6 +3,7 @@ import com.franciaflex.magalie.persistence.dao.ArticleDao; import com.franciaflex.magalie.persistence.dao.BuildingDao; import com.franciaflex.magalie.persistence.dao.DeliveredRequestedArticleDao; +import com.franciaflex.magalie.persistence.dao.DeliveredRequestedListDao; import com.franciaflex.magalie.persistence.dao.MagalieUserDao; import com.franciaflex.magalie.persistence.dao.RequestedArticleDao; import com.franciaflex.magalie.persistence.dao.LocationDao; @@ -15,6 +16,7 @@ import com.franciaflex.magalie.persistence.dao.jpa.ArticleJpaDao; import com.franciaflex.magalie.persistence.dao.jpa.BuildingJpaDao; import com.franciaflex.magalie.persistence.dao.jpa.DeliveredRequestedArticleJpaDao; +import com.franciaflex.magalie.persistence.dao.jpa.DeliveredRequestedListJpaDao; import com.franciaflex.magalie.persistence.dao.jpa.LocationErrorJpaDao; import com.franciaflex.magalie.persistence.dao.jpa.LocationJpaDao; import com.franciaflex.magalie.persistence.dao.jpa.MagalieUserJpaDao; @@ -123,4 +125,8 @@ return new RequestedListJpaDao(entityManager); } + @Override + public DeliveredRequestedListDao getDeliveredRequestedListDao() { + return new DeliveredRequestedListJpaDao(entityManager); + } } Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/MagaliePersistenceContext.java =================================================================== --- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/MagaliePersistenceContext.java 2013-04-15 16:18:12 UTC (rev 98) +++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/MagaliePersistenceContext.java 2013-04-16 17:37:40 UTC (rev 99) @@ -3,6 +3,7 @@ import com.franciaflex.magalie.persistence.dao.ArticleDao; import com.franciaflex.magalie.persistence.dao.BuildingDao; import com.franciaflex.magalie.persistence.dao.DeliveredRequestedArticleDao; +import com.franciaflex.magalie.persistence.dao.DeliveredRequestedListDao; import com.franciaflex.magalie.persistence.dao.LocationDao; import com.franciaflex.magalie.persistence.dao.LocationErrorDao; import com.franciaflex.magalie.persistence.dao.MagalieUserDao; @@ -48,4 +49,6 @@ RequestedListDao getRequestedListDao(); + DeliveredRequestedListDao getDeliveredRequestedListDao(); + } Deleted: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/RequestedArticles.java =================================================================== --- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/RequestedArticles.java 2013-04-15 16:18:12 UTC (rev 98) +++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/RequestedArticles.java 2013-04-16 17:37:40 UTC (rev 99) @@ -1,98 +0,0 @@ -package com.franciaflex.magalie.persistence; - -import com.franciaflex.magalie.persistence.entity.RequestedArticle; -import com.franciaflex.magalie.persistence.entity.RequestedList; -import com.google.common.base.Function; -import com.google.common.collect.ImmutableListMultimap; -import com.google.common.collect.Maps; -import com.google.common.collect.Multimaps; -import org.apache.commons.collections.comparators.BooleanComparator; - -import java.util.Collection; -import java.util.Comparator; -import java.util.List; -import java.util.Map; - -public class RequestedArticles { - - protected static class UrgentFirstComparator implements Comparator<RequestedArticle> { - - @Override - public int compare(RequestedArticle x, RequestedArticle y) { - return BooleanComparator.getTrueFirstComparator().compare(x.getRequestedList().isUrgent(), y.getRequestedList().isUrgent()); - } - - } - - protected static class RequestDateFistComparator implements Comparator<RequestedArticle> { - - @Override - public int compare(RequestedArticle x, RequestedArticle y) { - return x.getRequestedList().getRequestDate().compareTo(y.getRequestedList().getRequestDate()); - } - - } - - protected static class FinishEarlyFirstComparator implements Comparator<RequestedArticle> { - - protected Map<RequestedList, Integer> requestListToArticleRemainingCount; - - public FinishEarlyFirstComparator(Map<RequestedList, Integer> requestListToArticleRemainingCount) { - this.requestListToArticleRemainingCount = requestListToArticleRemainingCount; - } - - protected int getRemainingCount(RequestedArticle requestedArticle) { - Integer remainingCount = requestListToArticleRemainingCount.get(requestedArticle.getRequestedList()); - return remainingCount; - } - - @Override - public int compare(RequestedArticle x, RequestedArticle y) { - int xRemainingCount = getRemainingCount(x); - int yRemainingCount = getRemainingCount(y); - return xRemainingCount - yRemainingCount; - } - - } - - protected static class GetRequestList implements Function<RequestedArticle, RequestedList> { - - @Override - public RequestedList apply(RequestedArticle requestedArticle) { - return requestedArticle.getRequestedList(); - } - } - - public static Comparator<RequestedArticle> urgentFirstComparator() { - return new UrgentFirstComparator(); - } - - public static Comparator<RequestedArticle> requestDateFirstComparator() { - return new RequestDateFistComparator(); - } - - public static Comparator<RequestedArticle> finishEarlyFirstComparator(List<RequestedArticle> requests) { - - ImmutableListMultimap<RequestedList, RequestedArticle> index = - Multimaps.index(requests, RequestedArticles.getRequestList()); - - Map<RequestedList, Integer> requestListToArticleRemainingCount = Maps.newHashMap(); - - for (Map.Entry<RequestedList, Collection<RequestedArticle>> requestedListCollectionEntry : index.asMap().entrySet()) { - - RequestedList requestedList = requestedListCollectionEntry.getKey(); - - int remainingCount = requestedListCollectionEntry.getValue().size(); - - requestListToArticleRemainingCount.put(requestedList, remainingCount); - - } - - return new FinishEarlyFirstComparator(requestListToArticleRemainingCount); - - } - - public static Function<RequestedArticle, RequestedList> getRequestList() { - return new GetRequestList(); - } -} Added: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/RequestedArticles.java =================================================================== --- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/RequestedArticles.java (rev 0) +++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/RequestedArticles.java 2013-04-16 17:37:40 UTC (rev 99) @@ -0,0 +1,53 @@ +package com.franciaflex.magalie.persistence; + +import com.franciaflex.magalie.persistence.entity.RequestedArticle; +import com.franciaflex.magalie.persistence.entity.RequestedList; +import com.google.common.base.Function; +import com.google.common.collect.Lists; +import com.google.common.collect.Ordering; + +import java.util.Comparator; +import java.util.List; + +public class RequestedArticles { + + protected static class RequestedListComparator implements Comparator<RequestedArticle> { + + public RequestedListComparator(Comparator<RequestedList> requestedListComparator) { + this.requestedListComparator = requestedListComparator; + } + + protected Comparator<RequestedList> requestedListComparator; + + @Override + public int compare(RequestedArticle x, RequestedArticle y) { + return requestedListComparator.compare(x.getRequestedList(), y.getRequestedList()); + } + } + + public static Comparator<RequestedArticle> comparator(List<RequestedArticle> requests, RequestedList affectedRequestedList) { + Comparator<RequestedArticle> comparator = + Ordering.compound( + Lists.newArrayList( + + new RequestedListComparator(RequestedLists.comparator(requests, affectedRequestedList)), + + // we can still have ambiguous or priorities, so to be deterministic, use arbitrary order + Entities.<RequestedArticle>arbitraryComparator() + ) + ); + return comparator; + } + + protected static class GetRequestList implements Function<RequestedArticle, RequestedList> { + + @Override + public RequestedList apply(RequestedArticle requestedArticle) { + return requestedArticle.getRequestedList(); + } + } + + public static Function<RequestedArticle, RequestedList> getRequestList() { + return new GetRequestList(); + } +} Copied: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/RequestedLists.java (from rev 98, trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/RequestedArticles.java) =================================================================== --- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/RequestedLists.java (rev 0) +++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/RequestedLists.java 2013-04-16 17:37:40 UTC (rev 99) @@ -0,0 +1,134 @@ +package com.franciaflex.magalie.persistence; + +import com.franciaflex.magalie.persistence.entity.RequestedArticle; +import com.franciaflex.magalie.persistence.entity.RequestedList; +import com.google.common.collect.ImmutableListMultimap; +import com.google.common.collect.Lists; +import com.google.common.collect.Maps; +import com.google.common.collect.Multimaps; +import com.google.common.collect.Ordering; +import org.apache.commons.collections.comparators.BooleanComparator; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; + +import java.util.Collection; +import java.util.Comparator; +import java.util.List; +import java.util.Map; + +public class RequestedLists { + + private static final Log log = LogFactory.getLog(RequestedLists.class); + + protected static class GivenRequestListFirst implements Comparator<RequestedList> { + + protected RequestedList requestedList; + + public GivenRequestListFirst(RequestedList requestedList) { + this.requestedList = requestedList; + } + + @Override + public int compare(RequestedList x, RequestedList y) { + return BooleanComparator.getTrueFirstComparator().compare(x.equals(requestedList), y.equals(requestedList)); + } + } + + protected static class UrgentFirstComparator implements Comparator<RequestedList> { + + @Override + public int compare(RequestedList x, RequestedList y) { + int compare = BooleanComparator.getTrueFirstComparator().compare(x.isUrgent(), y.isUrgent()); + return compare; + } + + } + + protected static class RequestDateFistComparator implements Comparator<RequestedList> { + + @Override + public int compare(RequestedList x, RequestedList y) { + int compare = x.getRequestDate().compareTo(y.getRequestDate()); + return compare; + } + + } + + protected static class FinishEarlyFirstComparator implements Comparator<RequestedList> { + + protected Map<RequestedList, Integer> requestListToArticleRemainingCount; + + public FinishEarlyFirstComparator(Map<RequestedList, Integer> requestListToArticleRemainingCount) { + this.requestListToArticleRemainingCount = requestListToArticleRemainingCount; + } + + protected int getRemainingCount(RequestedList requestedList) { + Integer remainingCount = requestListToArticleRemainingCount.get(requestedList); + return remainingCount; + } + + @Override + public int compare(RequestedList x, RequestedList y) { + int xRemainingCount = getRemainingCount(x); + int yRemainingCount = getRemainingCount(y); + int compare = xRemainingCount - yRemainingCount; + if (log.isTraceEnabled()) { + if (compare < 0) log.trace(x.getId() + " is before " + y.getId()); + if (compare == 0) log.trace(x.getId() + " is same position as " + y.getId()); + if (compare > 0) log.trace(y.getId() + " is before " + x.getId()); + } + return compare; + } + + } + + public static Comparator<RequestedList> urgentFirstComparator() { + return new UrgentFirstComparator(); + } + + public static Comparator<RequestedList> requestDateFirstComparator() { + return new RequestDateFistComparator(); + } + + public static Comparator<RequestedList> finishEarlyFirstComparator(List<RequestedArticle> requests) { + + ImmutableListMultimap<RequestedList, RequestedArticle> index = + Multimaps.index(requests, RequestedArticles.getRequestList()); + + Map<RequestedList, Integer> requestListToArticleRemainingCount = Maps.newHashMap(); + + for (Map.Entry<RequestedList, Collection<RequestedArticle>> requestedListCollectionEntry : index.asMap().entrySet()) { + + RequestedList requestedList = requestedListCollectionEntry.getKey(); + + int remainingCount = requestedListCollectionEntry.getValue().size(); + + requestListToArticleRemainingCount.put(requestedList, remainingCount); + + } + + if (log.isDebugEnabled()) { + log.debug("number of remaining articles per list: " + requestListToArticleRemainingCount); + } + + return new FinishEarlyFirstComparator(requestListToArticleRemainingCount); + + } + + public static Comparator<RequestedList> givenRequestListFirst(RequestedList requestedList) { + return new GivenRequestListFirst(requestedList); + } + + public static Comparator<RequestedList> comparator(List<RequestedArticle> requests, RequestedList affectedRequestedList) { + Ordering<RequestedList> comparator = + Ordering.compound( + Lists.newArrayList( + givenRequestListFirst(affectedRequestedList), + finishEarlyFirstComparator(requests), + urgentFirstComparator(), + requestDateFirstComparator() + ) + ); + return comparator; + } +} Added: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/DeliveredRequestedListDao.java =================================================================== --- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/DeliveredRequestedListDao.java (rev 0) +++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/DeliveredRequestedListDao.java 2013-04-16 17:37:40 UTC (rev 99) @@ -0,0 +1,16 @@ +package com.franciaflex.magalie.persistence.dao; + +import com.franciaflex.magalie.persistence.entity.DeliveredRequestedList; +import com.franciaflex.magalie.persistence.entity.MagalieUser; +import com.franciaflex.magalie.persistence.entity.RequestedList; + +/** + * @author bleny + */ +public interface DeliveredRequestedListDao extends Dao<DeliveredRequestedList> { + + DeliveredRequestedList findByAffectedTo(MagalieUser affectedTo); + + DeliveredRequestedList findByRequestedList(RequestedList requestedList); + +} Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/RequestedArticleDao.java =================================================================== --- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/RequestedArticleDao.java 2013-04-15 16:18:12 UTC (rev 98) +++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/RequestedArticleDao.java 2013-04-16 17:37:40 UTC (rev 99) @@ -1,6 +1,7 @@ package com.franciaflex.magalie.persistence.dao; import com.franciaflex.magalie.persistence.entity.Building; +import com.franciaflex.magalie.persistence.entity.MagalieUser; import com.franciaflex.magalie.persistence.entity.RequestedArticle; import java.util.List; @@ -10,6 +11,6 @@ */ public interface RequestedArticleDao extends Dao<RequestedArticle> { - List<RequestedArticle> findAllUndelivered(Building building); + List<RequestedArticle> findAllUndelivered(Building building, MagalieUser affectedTo); } Added: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/DeliveredRequestedListJpaDao.java =================================================================== --- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/DeliveredRequestedListJpaDao.java (rev 0) +++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/DeliveredRequestedListJpaDao.java 2013-04-16 17:37:40 UTC (rev 99) @@ -0,0 +1,62 @@ +package com.franciaflex.magalie.persistence.dao.jpa; + +import com.franciaflex.magalie.persistence.dao.DeliveredRequestedListDao; +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.RequestedList; +import com.google.common.collect.Iterables; + +import javax.persistence.EntityManager; +import javax.persistence.TypedQuery; +import java.util.List; + +public class DeliveredRequestedListJpaDao extends AbstractJpaDao<DeliveredRequestedList> implements DeliveredRequestedListDao { + + public DeliveredRequestedListJpaDao(EntityManager entityManager) { + super(entityManager); + } + + @Override + protected Class<DeliveredRequestedList> getEntityClass() { + return DeliveredRequestedList.class; + } + + @Override + public DeliveredRequestedList findByAffectedTo(MagalieUser affectedTo) { + + TypedQuery<DeliveredRequestedList> query = + entityManager.createQuery( + "from DeliveredRequestedList drl where drl.status = :affected and drl.affectedTo = :affectedTo", + getEntityClass()); + + query.setParameter("affectedTo", affectedTo); + + query.setParameter("affected", DeliveredRequestedListStatus.AFFECTED); + + List<DeliveredRequestedList> resultList = query.getResultList(); + + DeliveredRequestedList onlyElement = Iterables.getOnlyElement(resultList, null); + + return onlyElement; + + } + + @Override + public DeliveredRequestedList findByRequestedList(RequestedList requestedList) { + + TypedQuery<DeliveredRequestedList> query = + entityManager.createQuery( + "from DeliveredRequestedList drl where drl.requestedList = :requestedList", + getEntityClass()); + + query.setParameter("requestedList", requestedList); + + List<DeliveredRequestedList> resultList = query.getResultList(); + + DeliveredRequestedList onlyElement = Iterables.getOnlyElement(resultList, null); + + return onlyElement; + + } +} Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/RequestedArticleJpaDao.java =================================================================== --- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/RequestedArticleJpaDao.java 2013-04-15 16:18:12 UTC (rev 98) +++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/RequestedArticleJpaDao.java 2013-04-16 17:37:40 UTC (rev 99) @@ -2,6 +2,8 @@ import com.franciaflex.magalie.persistence.dao.RequestedArticleDao; import com.franciaflex.magalie.persistence.entity.Building; +import com.franciaflex.magalie.persistence.entity.DeliveredRequestedListStatus; +import com.franciaflex.magalie.persistence.entity.MagalieUser; import com.franciaflex.magalie.persistence.entity.RequestedArticle; import javax.persistence.EntityManager; @@ -20,13 +22,19 @@ } @Override - public List<RequestedArticle> findAllUndelivered(Building building) { + public List<RequestedArticle> findAllUndelivered(Building building, MagalieUser affectedTo) { TypedQuery<RequestedArticle> query = entityManager.createQuery( - "from RequestedArticle ra where ra not in (select dra.requestedArticle from DeliveredRequestedArticle dra) and ra.requestedList.building = :building", + "from RequestedArticle ra where " + + " ra not in (select dra.requestedArticle from DeliveredRequestedArticle dra) " + + " and ra.requestedList.building = :building " + + " and ra.requestedList not in (select drl.requestedList from DeliveredRequestedList drl where drl.status = :complete or drl.status = :affected and drl.affectedTo != :affectedTo)", getEntityClass() ); query.setParameter("building", building); + query.setParameter("complete", DeliveredRequestedListStatus.COMPLETE); + query.setParameter("affected", DeliveredRequestedListStatus.AFFECTED); + query.setParameter("affectedTo", affectedTo); return query.getResultList(); } Added: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/DeliveredRequestedList.java =================================================================== --- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/DeliveredRequestedList.java (rev 0) +++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/DeliveredRequestedList.java 2013-04-16 17:37:40 UTC (rev 99) @@ -0,0 +1,56 @@ +package com.franciaflex.magalie.persistence.entity; + + +import javax.persistence.Entity; +import javax.persistence.GeneratedValue; +import javax.persistence.Id; +import javax.persistence.OneToOne; + +@Entity +public class DeliveredRequestedList extends AbstractEntity { + + @Id + @GeneratedValue + protected String id; + + @OneToOne + protected RequestedList requestedList; + + @OneToOne + protected MagalieUser affectedTo; + + protected DeliveredRequestedListStatus status; + + @Override + public String getId() { + return id; + } + + public void setId(String id) { + this.id = id; + } + + public RequestedList getRequestedList() { + return requestedList; + } + + public void setRequestedList(RequestedList requestedList) { + this.requestedList = requestedList; + } + + public MagalieUser getAffectedTo() { + return affectedTo; + } + + public void setAffectedTo(MagalieUser affectedTo) { + this.affectedTo = affectedTo; + } + + public DeliveredRequestedListStatus getStatus() { + return status; + } + + public void setStatus(DeliveredRequestedListStatus status) { + this.status = status; + } +} 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-04-16 17:37:40 UTC (rev 99) @@ -0,0 +1,9 @@ +package com.franciaflex.magalie.persistence.entity; + +public enum DeliveredRequestedListStatus { + + PENDING, + AFFECTED, + COMPLETE + +} Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ArticleStorageService.java =================================================================== --- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ArticleStorageService.java 2013-04-15 16:18:12 UTC (rev 98) +++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ArticleStorageService.java 2013-04-16 17:37:40 UTC (rev 99) @@ -174,20 +174,27 @@ protected Queue<StoredArticle> sortStoredArticlesByPriority(List<StoredArticle> storedArticles) { - Ordering<StoredArticle> orderingByPriority = Ordering.compound( - Lists.newArrayList( - StoredArticles.fixedLocationForArticleComparator(), - StoredArticles.locationWithLowestQuantityFirstComparator(), - StoredArticles.articleStoredInLocationsRequiringDriverLicenseFirstComparator() - ) - ); + Queue<StoredArticle> storedArticlesByPriority = Lists.newLinkedList(); - Queue<StoredArticle> storedArticlesByPriority = - new PriorityQueue<StoredArticle>( - storedArticles.size(), - orderingByPriority - ); + if ( ! storedArticles.isEmpty()) { + Ordering<StoredArticle> orderingByPriority = + Ordering.compound( + Lists.newArrayList( + StoredArticles.fixedLocationForArticleComparator(), + StoredArticles.locationWithLowestQuantityFirstComparator(), + StoredArticles.articleStoredInLocationsRequiringDriverLicenseFirstComparator() + ) + ); + + storedArticlesByPriority = + new PriorityQueue<StoredArticle>( + storedArticles.size(), + orderingByPriority + ); + + } + storedArticlesByPriority.addAll(storedArticles); return storedArticlesByPriority; Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/FindOrderToExecuteResult.java =================================================================== --- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/FindOrderToExecuteResult.java 2013-04-15 16:18:12 UTC (rev 98) +++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/FindOrderToExecuteResult.java 2013-04-16 17:37:40 UTC (rev 99) @@ -1,5 +1,6 @@ package com.franciaflex.magalie.services.service; +import com.franciaflex.magalie.persistence.entity.DeliveredRequestedList; import com.franciaflex.magalie.persistence.entity.StorageMovementOrder; public class FindOrderToExecuteResult { @@ -10,6 +11,12 @@ protected boolean driverLicenseRequired; + protected boolean everythingUnavailable; + + protected DeliveredRequestedList oldAffectation; + + protected DeliveredRequestedList newAffectation; + public boolean isSuccess() { return storageMovementOrder != null; } @@ -38,7 +45,27 @@ return driverLicenseRequired; } + public void setEverythingUnavailable(boolean everythingUnavailable) { + this.everythingUnavailable = everythingUnavailable; + } + public boolean isEverythingUnavailable() { - return ! driverLicenseRequired && ! nothingToDo; + return everythingUnavailable; } + + public void setOldAffectation(DeliveredRequestedList oldAffectation) { + this.oldAffectation = oldAffectation; + } + + public DeliveredRequestedList getOldAffectation() { + return oldAffectation; + } + + public void setNewAffectation(DeliveredRequestedList newAffectation) { + this.newAffectation = newAffectation; + } + + public DeliveredRequestedList getNewAffectation() { + return newAffectation; + } } 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-04-15 16:18:12 UTC (rev 98) +++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/RequestedArticleService.java 2013-04-16 17:37:40 UTC (rev 99) @@ -26,19 +26,21 @@ import com.franciaflex.magalie.persistence.MagaliePersistenceContext; import com.franciaflex.magalie.persistence.RequestedArticles; import com.franciaflex.magalie.persistence.dao.DeliveredRequestedArticleDao; +import com.franciaflex.magalie.persistence.dao.DeliveredRequestedListDao; import com.franciaflex.magalie.persistence.dao.RequestedArticleDao; import com.franciaflex.magalie.persistence.entity.Article; import com.franciaflex.magalie.persistence.entity.Building; import com.franciaflex.magalie.persistence.entity.DeliveredRequestedArticle; +import com.franciaflex.magalie.persistence.entity.DeliveredRequestedList; +import com.franciaflex.magalie.persistence.entity.DeliveredRequestedListStatus; import com.franciaflex.magalie.persistence.entity.Location; import com.franciaflex.magalie.persistence.entity.MagalieUser; import com.franciaflex.magalie.persistence.entity.RequestedArticle; +import com.franciaflex.magalie.persistence.entity.RequestedList; import com.franciaflex.magalie.persistence.entity.StorageMovementOrder; import com.franciaflex.magalie.services.MagalieService; import com.franciaflex.magalie.services.MagalieServiceContext; import com.google.common.base.Preconditions; -import com.google.common.collect.Lists; -import com.google.common.collect.Ordering; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; @@ -58,15 +60,12 @@ this.serviceContext = serviceContext; } - public FindOrderToExecuteResult getStorageMovementOrder( - MagalieUser magalieUser, - Building building) { + public FindOrderToExecuteResult findOrderToExecute(MagalieUser magalieUser, Building building) { - List<RequestedArticle> requestedArticles = getRequestedArticles(building); + List<RequestedArticle> requestedArticles = getRequestedArticles(building, magalieUser); - Queue<RequestedArticle> requestedArticlesByPriority = getRequestedArticlesByPriority(requestedArticles); + Queue<RequestedArticle> requestedArticlesByPriority = getRequestedArticlesByPriority(requestedArticles, magalieUser); - FindOrderToExecuteResult findOrderToExecuteResult = findOrderToExecute( requestedArticlesByPriority, @@ -77,11 +76,11 @@ } - protected List<RequestedArticle> getRequestedArticles(Building building) { + protected List<RequestedArticle> getRequestedArticles(Building building, MagalieUser magalieUser) { RequestedArticleDao requestedArticleDao = serviceContext.getPersistenceContext().getRequestedArticleDao(); - List<RequestedArticle> allRequestedArticles = requestedArticleDao.findAllUndelivered(building); + List<RequestedArticle> allRequestedArticles = requestedArticleDao.findAllUndelivered(building, magalieUser); if (log.isInfoEnabled()) { log.info(allRequestedArticles.size() + " articles requested in building " + building.getCode()); @@ -106,6 +105,8 @@ boolean driverLicenseRequired = false; + boolean somethingIsAvailable = false; + while ( ! success && requestedArticleIterator.hasNext()) { requestedArticle = requestedArticleIterator.next(); @@ -122,6 +123,8 @@ driverLicenseRequired |= bookArticleResult.isArticleInaccessible() && ! bookArticleResult.isArticleUnavailable(); + somethingIsAvailable |= ! bookArticleResult.isArticleUnavailable(); + success = bookArticleResult.isSuccess(); } @@ -130,6 +133,10 @@ if (success) { + if (log.isInfoEnabled()) { + log.info("order to execute found: " + requestedArticle); + } + Preconditions.checkState(requestedArticle != null); MagaliePersistenceContext persistenceContext = serviceContext.getPersistenceContext(); @@ -146,22 +153,114 @@ dao.persist(newDeliveredRequestedArticle); - persistenceContext.commit(); + findOrderToExecuteResult.setStorageMovementOrder(storageMovementOrder); - findOrderToExecuteResult = new FindOrderToExecuteResult(); + // deal with affectations + DeliveredRequestedListDao deliveredRequestedListDao = + persistenceContext.getDeliveredRequestedListDao(); - findOrderToExecuteResult.setStorageMovementOrder(storageMovementOrder); + DeliveredRequestedList oldAffectation = + deliveredRequestedListDao.findByAffectedTo(magalieUser); + RequestedList requestedList = requestedArticle.getRequestedList(); + + DeliveredRequestedList newAffectation = + deliveredRequestedListDao.findByRequestedList(requestedList); + + if (newAffectation == null) { + + newAffectation = new DeliveredRequestedList(); + + deliveredRequestedListDao.persist(newAffectation); + + } + + newAffectation.setAffectedTo(magalieUser); + + newAffectation.setRequestedList(requestedList); + + newAffectation.setStatus(DeliveredRequestedListStatus.AFFECTED); + + deliveredRequestedListDao.merge(newAffectation); + + if (oldAffectation == null) { + + if (log.isInfoEnabled()) { + log.info("user has no previous affectation"); + } + + findOrderToExecuteResult.setNewAffectation(newAffectation); + + } else { + + if (log.isInfoEnabled()) { + log.info("user is affected to " + oldAffectation.getRequestedList().getCode()); + } + + boolean affectationChanged = ! oldAffectation.getRequestedList().equals(requestedList); + + if (affectationChanged) { + + if (log.isInfoEnabled()) { + log.info("user changed affectation from " + + oldAffectation.getRequestedList().getCode() + + " to " + requestedList.getCode()); + } + + boolean isFinished = false; // TODO brendan 16/04/13 really ? + + if (isFinished) { + + if (log.isInfoEnabled()) { + log.info("request list " + oldAffectation.getRequestedList() + + " is complete, save status"); + } + + oldAffectation.setStatus(DeliveredRequestedListStatus.COMPLETE); + + } else { + + if (log.isInfoEnabled()) { + log.info("request list " + oldAffectation.getRequestedList() + + " is not complete, putting it back to pending"); + } + + oldAffectation.setStatus(DeliveredRequestedListStatus.PENDING); + + oldAffectation.setAffectedTo(null); + + } + + findOrderToExecuteResult.setOldAffectation(oldAffectation); + + findOrderToExecuteResult.setNewAffectation(newAffectation); + + } + + } + + persistenceContext.commit(); + } else { if (requestedArticlesByPriority.isEmpty()) { + if (log.isInfoEnabled()) { + log.info("no order to execute found: nothing to do"); + } + findOrderToExecuteResult.setNothingToDo(true); } else { + if (log.isInfoEnabled()) { + log.info("no order to execute found: there is something to do but driver license is required"); + } + findOrderToExecuteResult.setDriverLicenseRequired(driverLicenseRequired); + findOrderToExecuteResult.setEverythingUnavailable( ! somethingIsAvailable); + } } @@ -170,20 +269,29 @@ } - protected Queue<RequestedArticle> getRequestedArticlesByPriority(List<RequestedArticle> requests) { + protected Queue<RequestedArticle> getRequestedArticlesByPriority(List<RequestedArticle> requests, MagalieUser magalieUser) { PriorityQueue<RequestedArticle> requestedArticles = new PriorityQueue<RequestedArticle>(); if ( ! requests.isEmpty()) { + MagaliePersistenceContext persistenceContext = serviceContext.getPersistenceContext(); + + DeliveredRequestedListDao deliveredRequestedListDao = persistenceContext.getDeliveredRequestedListDao(); + + DeliveredRequestedList affectedDeliveredRequestedList = + deliveredRequestedListDao.findByAffectedTo(magalieUser); + + RequestedList affectedRequestedList = null; + + if (affectedDeliveredRequestedList != null) { + + affectedRequestedList = affectedDeliveredRequestedList.getRequestedList(); + + } + Comparator<RequestedArticle> priorityComparator = - Ordering.compound( - Lists.newArrayList( - RequestedArticles.finishEarlyFirstComparator(requests), - RequestedArticles.urgentFirstComparator(), - RequestedArticles.requestDateFirstComparator() - ) - ); + RequestedArticles.comparator(requests, affectedRequestedList); requestedArticles = new PriorityQueue<RequestedArticle>( Modified: trunk/magalie-services/src/main/resources/fixtures.yaml =================================================================== --- trunk/magalie-services/src/main/resources/fixtures.yaml 2013-04-15 16:18:12 UTC (rev 98) +++ trunk/magalie-services/src/main/resources/fixtures.yaml 2013-04-16 17:37:40 UTC (rev 99) @@ -417,7 +417,7 @@ requestedList2: &requestedList2 !requested-list id: requestedList2 - code: 123456789 + code: 987654321 requestDate: 2013-04-08 12:00:00 urgent: false building: *B1 Modified: trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/RequestedArticleServiceTest.java =================================================================== --- trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/RequestedArticleServiceTest.java 2013-04-15 16:18:12 UTC (rev 98) +++ trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/RequestedArticleServiceTest.java 2013-04-16 17:37:40 UTC (rev 99) @@ -22,6 +22,10 @@ protected RequestedArticleService service; + protected MagalieUser magalieUser; + + protected Building building; + @Before public void setUp() throws ParseException { @@ -31,15 +35,16 @@ serviceContext.setDate(new Date(1363948427576l)); + magalieUser = fixture("bruno"); + + building = fixture("B1"); } @Test public void testGetRequestedArticles() { - Building building = fixture("B1"); + List<RequestedArticle> requestedArticles = service.getRequestedArticles(building, magalieUser); - List<RequestedArticle> requestedArticles = service.getRequestedArticles(building); - for (RequestedArticle requestedArticle : requestedArticles) { Assert.assertEquals(building, requestedArticle.getRequestedList().getBuilding()); @@ -54,19 +59,22 @@ List<RequestedArticle> requestedArticles = fixture("requestedArticles"); Queue<RequestedArticle> requestedArticlesByPriority = - service.getRequestedArticlesByPriority(requestedArticles); + service.getRequestedArticlesByPriority(requestedArticles, magalieUser); Assert.assertEquals(requestedArticles.size(), requestedArticlesByPriority.size()); + Assert.assertEquals("requestedArticle5", requestedArticlesByPriority.poll().getId()); + Assert.assertEquals("requestedArticle6", requestedArticlesByPriority.poll().getId()); + Assert.assertEquals("requestedArticle1", requestedArticlesByPriority.poll().getId()); + Assert.assertEquals("requestedArticle2", requestedArticlesByPriority.poll().getId()); + Assert.assertEquals("requestedArticle3", requestedArticlesByPriority.poll().getId()); + Assert.assertEquals("requestedArticle4", requestedArticlesByPriority.poll().getId()); + } @Test public void testFindOrderWhenNoRequestedArticles() { - MagalieUser magalieUser = fixture("bruno"); - - Building building = fixture("B1"); - Queue<RequestedArticle> emptyQueue = Lists.newLinkedList(); FindOrderToExecuteResult orderToExecute = service.findOrderToExecute(emptyQueue, magalieUser, building); @@ -78,4 +86,19 @@ } + @Test + public void testFindOrderRequestedArticles() { + + FindOrderToExecuteResult orderToExecute = service.findOrderToExecute(magalieUser, building); + + Assert.assertFalse(orderToExecute.isNothingToDo()); + Assert.assertFalse(orderToExecute.isEverythingUnavailable()); + Assert.assertFalse(orderToExecute.isDriverLicenseRequired()); + Assert.assertTrue(orderToExecute.isSuccess()); + + Assert.assertNull(orderToExecute.getOldAffectation()); + Assert.assertNotNull(orderToExecute.getNewAffectation()); + + } + } Modified: trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/MagalieSession.java =================================================================== --- trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/MagalieSession.java 2013-04-15 16:18:12 UTC (rev 98) +++ trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/MagalieSession.java 2013-04-16 17:37:40 UTC (rev 99) @@ -3,8 +3,10 @@ import com.franciaflex.magalie.persistence.entity.Building; import com.franciaflex.magalie.persistence.entity.MagalieUser; import com.franciaflex.magalie.persistence.entity.Warehouse; +import com.google.common.collect.Lists; import java.io.Serializable; +import java.util.Collection; /** * The single object every user will have in its session. @@ -21,8 +23,11 @@ protected MagalieUser magalieUser; protected Building building; - private Warehouse lastUsedDestinationWarehouseForKanbans; + protected Warehouse lastUsedDestinationWarehouseForKanbans; + + protected Collection<String> messages; + public MagalieUser getMagalieUser() { return magalieUser; } @@ -46,4 +51,16 @@ public void setLastUsedDestinationWarehouseForKanbans(Warehouse lastUsedDestinationWarehouseForKanbans) { this.lastUsedDestinationWarehouseForKanbans = lastUsedDestinationWarehouseForKanbans; } + + public Collection<String> getMessages() { + if (messages == null) { + messages = Lists.newLinkedList(); + } + return messages; + } + + public void addMessage(String message) { + getMessages().add(message); + } + } 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-04-15 16:18:12 UTC (rev 98) +++ trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/DeliverRequestedArticleAction.java 2013-04-16 17:37:40 UTC (rev 99) @@ -1,6 +1,7 @@ package com.franciaflex.magalie.web.action; import com.franciaflex.magalie.persistence.entity.Building; +import com.franciaflex.magalie.persistence.entity.DeliveredRequestedList; import com.franciaflex.magalie.persistence.entity.MagalieUser; import com.franciaflex.magalie.persistence.entity.StorageMovementOrder; import com.franciaflex.magalie.services.service.FindOrderToExecuteResult; @@ -41,35 +42,59 @@ Building building = session.getBuilding(); FindOrderToExecuteResult findOrderToExecuteResult = - service.getStorageMovementOrder(magalieUser, building); + service.findOrderToExecute(magalieUser, building); + String result; + + addActionMessage("test"); + if (findOrderToExecuteResult.isSuccess()) { + DeliveredRequestedList oldAffectation = findOrderToExecuteResult.getOldAffectation(); + + if (oldAffectation != null) { + + session.addMessage("Vous n'êtes plus affecté à la liste " + oldAffectation.getRequestedList().getCode()); + + } + + DeliveredRequestedList newAffectation = findOrderToExecuteResult.getNewAffectation(); + + if (newAffectation != null) { + + session.addMessage("Vous venez d'être affecté à la liste " + newAffectation.getRequestedList().getCode()); + + } + storageMovementOrder = findOrderToExecuteResult.getStorageMovementOrder(); - return SUCCESS; + result = SUCCESS; - } + } else { - if (findOrderToExecuteResult.isNothingToDo()) { + if (findOrderToExecuteResult.isNothingToDo()) { - addActionMessage("Il n'y a aucune demande. Vous n'avez rien à faire"); + addActionMessage("Il n'y a aucune demande. Vous n'avez rien à faire"); - } + } - if (findOrderToExecuteResult.isDriverLicenseRequired()) { + if (findOrderToExecuteResult.isDriverLicenseRequired()) { - addActionMessage("Il y a des demandes en cours, mais un permis est requis"); + addActionMessage("Il y a des demandes en cours, mais un permis est requis"); - } + } - if (findOrderToExecuteResult.isEverythingUnavailable()) { + if (findOrderToExecuteResult.isEverythingUnavailable()) { - addActionMessage("Toutes les demandes en cours concernent des articles indisponibles"); + addActionMessage("Toutes les demandes en cours concernent des articles indisponibles"); + } + + result = INPUT; + } - return INPUT; + return result; } Modified: trunk/magalie-web/src/main/resources/log4j.properties =================================================================== --- trunk/magalie-web/src/main/resources/log4j.properties 2013-04-15 16:18:12 UTC (rev 98) +++ trunk/magalie-web/src/main/resources/log4j.properties 2013-04-16 17:37:40 UTC (rev 99) @@ -34,6 +34,9 @@ log4j.appender.file.layout=org.apache.log4j.PatternLayout log4j.appender.file.layout.ConversionPattern=%d{yyyy/MM/dd hh:mm:ss} %5p (%F:%L) %m%n -log4j.logger.com.franciaflex.magalie=TRACE -log4j.logger.org.nuiton.web.filter.JpaTransactionFilter=TRACE +# log4j.logger.org.nuiton.web.filter.JpaTransactionFilter=TRACE +# log4j.logger.com.franciaflex.magalie=TRACE + +log4j.logger.com.franciaflex.magalie.services.service.RequestedArticleService=TRACE +log4j.logger.com.franciaflex.magalie.services.service.ArticleStorageService=TRACE

1 0

Jenkins build became unstable: franciaflex-magalie #30
by admin+hudson＠codelutin.com April 16, 2013

April 16, 2013

See <http://ci.nuiton.org/jenkins/job/franciaflex-magalie/30/changes>

1 1

Jenkins build became unstable: franciaflex-magalie » MagaLiE :: Services #30
by admin+hudson＠codelutin.com April 16, 2013

April 16, 2013

See <http://ci.nuiton.org/jenkins/job/franciaflex-magalie/com.franciaflex$magali…>

1 1

r98 - trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service
by bleny＠users.forge.codelutin.com April 15, 2013

April 15, 2013

Author: bleny Date: 2013-04-15 18:18:12 +0200 (Mon, 15 Apr 2013) New Revision: 98 Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision… Log: add test for RequestedArticleService Modified: trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/RequestedArticleServiceTest.java Modified: trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/RequestedArticleServiceTest.java =================================================================== --- trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/RequestedArticleServiceTest.java 2013-04-15 15:37:46 UTC (rev 97) +++ trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/RequestedArticleServiceTest.java 2013-04-15 16:18:12 UTC (rev 98) @@ -34,12 +34,18 @@ } @Test - public void testGetRequestedArticles() throws Exception { + public void testGetRequestedArticles() { Building building = fixture("B1"); List<RequestedArticle> requestedArticles = service.getRequestedArticles(building); + for (RequestedArticle requestedArticle : requestedArticles) { + + Assert.assertEquals(building, requestedArticle.getRequestedList().getBuilding()); + + } + } @Test

1 0