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

r356 - in trunk: magalie-persistence/src/main/java/com/franciaflex/magalie/persistence magalie-persistence/src/main/xmi magalie-services/src/main/java/com/franciaflex/magalie/services/service magalie-services/src/main/resources magalie-services/src/test/java/com/franciaflex/magalie/services/service magalie-web/src/main/webapp/WEB-INF/content
by bleny＠users.forge.codelutin.com 01 Aug '13

01 Aug '13

Author: bleny Date: 2013-08-01 14:39:29 +0200 (Thu, 01 Aug 2013) New Revision: 356 Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision… Log: fixes #2990 by adding supplierGroup column Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/Suppliers.java trunk/magalie-persistence/src/main/xmi/magalie.zargo trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/SuppliersToReceive.java trunk/magalie-services/src/main/resources/fixtures.yaml trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/ReceptionServiceTest.java trunk/magalie-web/src/main/webapp/WEB-INF/content/prepare-article-reception-input.jsp Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/Suppliers.java =================================================================== --- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/Suppliers.java 2013-08-01 10:09:09 UTC (rev 355) +++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/Suppliers.java 2013-08-01 12:39:29 UTC (rev 356) @@ -31,6 +31,15 @@ public class Suppliers { + public static Function<Supplier, String> getSupplierGroup() { + return new GetSupplierGroup(); + } + + public static Comparator<Supplier> orderByNamesComparator() { + Comparator<Supplier> orderByNamesComparator = Ordering.natural().onResultOf(new GetName()); + return orderByNamesComparator; + } + protected static class GetName implements Function<Supplier, String> { @Override @@ -39,9 +48,12 @@ } } - public static Comparator<Supplier> orderByNamesComparator() { - Comparator<Supplier> orderByNamesComparator = Ordering.natural().onResultOf(new GetName()); - return orderByNamesComparator; + protected static class GetSupplierGroup implements Function<Supplier, String> { + + @Override + public String apply(Supplier supplier) { + return supplier.getSupplierGroup(); + } } } Modified: trunk/magalie-persistence/src/main/xmi/magalie.zargo =================================================================== (Binary files differ) Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/SuppliersToReceive.java =================================================================== --- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/SuppliersToReceive.java 2013-08-01 10:09:09 UTC (rev 355) +++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/SuppliersToReceive.java 2013-08-01 12:39:29 UTC (rev 356) @@ -26,44 +26,61 @@ import com.franciaflex.magalie.persistence.Suppliers; import com.franciaflex.magalie.persistence.entity.Supplier; import com.google.common.base.Predicates; -import com.google.common.collect.ImmutableList; +import com.google.common.collect.ImmutableListMultimap; +import com.google.common.collect.ImmutableSetMultimap; import com.google.common.collect.Iterables; -import com.google.common.collect.Sets; +import com.google.common.collect.Multimaps; +import com.google.common.collect.Ordering; -import java.util.List; -import java.util.Set; +import java.util.Comparator; public class SuppliersToReceive { - protected List<Supplier> suppliers; - protected boolean articlesWithoutSuppliers; protected boolean somethingToReceive; + protected ImmutableSetMultimap<String,Supplier> suppliersByGroup; + public SuppliersToReceive(Iterable<Supplier> suppliers) { somethingToReceive = ! Iterables.isEmpty(suppliers); articlesWithoutSuppliers = Iterables.contains(suppliers, null); - Set<Supplier> sortedSuppliers = Sets.newTreeSet(Suppliers.orderByNamesComparator()); + // on veut présenter les groupes dans l'ordre alphabétique + Comparator<String> supplierGroupsOrder = Ordering.natural(); - Iterables.addAll(sortedSuppliers, Iterables.filter(suppliers, Predicates.notNull())); + // dans chaque groupe, on veut présenter les fournisseurs triés par nom + Comparator<Supplier> suppliersOrdering = Suppliers.orderByNamesComparator(); - this.suppliers = ImmutableList.copyOf(sortedSuppliers); + ImmutableListMultimap<String, Supplier> suppliersIndexedByGroups = + Multimaps.index( + Iterables.filter( + suppliers, + Predicates.notNull()), + Suppliers.getSupplierGroup() + ); + suppliersByGroup = ImmutableSetMultimap + .<String, Supplier> builder() + .orderKeysBy(supplierGroupsOrder) + .orderValuesBy(suppliersOrdering) + .putAll(suppliersIndexedByGroups) + .build(); + } public boolean isSomethingToReceive() { return somethingToReceive; } - public List<Supplier> getSuppliers() { - return suppliers; - } - public boolean isArticlesWithoutSuppliers() { return articlesWithoutSuppliers; } + + public ImmutableSetMultimap<String, Supplier> getSuppliersByGroup() { + return suppliersByGroup; + } + } Modified: trunk/magalie-services/src/main/resources/fixtures.yaml =================================================================== --- trunk/magalie-services/src/main/resources/fixtures.yaml 2013-08-01 10:09:09 UTC (rev 355) +++ trunk/magalie-services/src/main/resources/fixtures.yaml 2013-08-01 12:39:29 UTC (rev 356) @@ -324,15 +324,38 @@ &supplier1 !supplier id: supplier1 name: Somfy + supplierGroup: Matériel supplier2: &supplier2 !supplier id: supplier2 name: Soliris + supplierGroup: Toiles +supplier3: + &supplier3 !supplier + id: supplier3 + name: MégaToiles + supplierGroup: Toiles + +supplier4: + &supplier4 !supplier + id: supplier4 + name: onatout + supplierGroup: Matériel + +supplier5: + &supplier5 !supplier + id: supplier5 + name: SuperStores + supplierGroup: Toiles + suppliers: - *supplier1 - *supplier2 + - *supplier3 + - *supplier4 + - *supplier5 article1: &article1 !article Modified: trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/ReceptionServiceTest.java =================================================================== --- trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/ReceptionServiceTest.java 2013-08-01 10:09:09 UTC (rev 355) +++ trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/ReceptionServiceTest.java 2013-08-01 12:39:29 UTC (rev 356) @@ -41,7 +41,9 @@ import com.franciaflex.magalie.services.exception.InvalidMagalieBarcodeException; import com.franciaflex.magalie.services.exception.PreparedArticleReceptionAlreadyStoredException; import com.google.common.collect.ImmutableMap; +import com.google.common.collect.ImmutableSetMultimap; import com.google.common.collect.Iterables; +import org.apache.commons.lang3.StringUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.junit.Assert; @@ -51,6 +53,7 @@ import javax.persistence.EntityManager; import javax.persistence.EntityTransaction; import java.text.ParseException; +import java.util.Collection; import java.util.Date; import java.util.List; import java.util.Map; @@ -96,11 +99,11 @@ Assert.assertTrue(receivedSuppliers.isSomethingToReceive()); - List<Supplier> suppliers = receivedSuppliers.getSuppliers(); + ImmutableSetMultimap<String, Supplier> suppliersByGroup = receivedSuppliers.getSuppliersByGroup(); - Assert.assertEquals(1, suppliers.size()); + Assert.assertEquals(1, suppliersByGroup.size()); - Assert.assertEquals(supplier, suppliers.get(0)); + Assert.assertEquals(supplier, Iterables.get(suppliersByGroup.entries(), 0).getValue()); Assert.assertTrue(receivedSuppliers.isArticlesWithoutSuppliers()); @@ -214,7 +217,7 @@ SuppliersToReceive receivedSuppliers = service.getReceivedSuppliers(building); - List<Supplier> suppliers = receivedSuppliers.getSuppliers(); + ImmutableSetMultimap<String, Supplier> suppliers = receivedSuppliers.getSuppliersByGroup(); Assert.assertTrue(receivedSuppliers.isSomethingToReceive()); @@ -370,4 +373,28 @@ } + @Test + public void testSuppliersPresentation() { + + Iterable<Supplier> suppliers = fixture("suppliers"); + + SuppliersToReceive suppliersToReceive = new SuppliersToReceive(suppliers); + + ImmutableSetMultimap<String,Supplier> suppliersByGroup = suppliersToReceive.getSuppliersByGroup(); + + for (Map.Entry<String, Collection<Supplier>> suppliersByGroupEntry : suppliersByGroup.asMap().entrySet()) { + + String supplierGroup = suppliersByGroupEntry.getKey(); + + Collection<Supplier> suppliersInGroup = suppliersByGroupEntry.getValue(); + + if (log.isDebugEnabled()) { + log.debug(supplierGroup + " ↓\n " + StringUtils.join(suppliersInGroup, "\n ")); + } + + } + + // just check that there is no exception, user can check is order is OK + } + } Modified: trunk/magalie-web/src/main/webapp/WEB-INF/content/prepare-article-reception-input.jsp =================================================================== --- trunk/magalie-web/src/main/webapp/WEB-INF/content/prepare-article-reception-input.jsp 2013-08-01 10:09:09 UTC (rev 355) +++ trunk/magalie-web/src/main/webapp/WEB-INF/content/prepare-article-reception-input.jsp 2013-08-01 12:39:29 UTC (rev 356) @@ -47,11 +47,19 @@ </s:a> </li> - <s:iterator value="suppliersToReceive.suppliers" status="stat"> + <s:iterator value="suppliersToReceive.suppliersByGroup.entries()" status="stat"> + + <%-- Afficher le nom du groupe --%> + <s:set var="currentGroup"><s:property value="key" /></s:set> + <s:if test="#lastGroup != #currentGroup"> + <s:set var="lastGroup"><s:property value="key" /></s:set> + Groupe <s:property value="key" /> + </s:if> + <li> <s:set var="prepareArticleReceptionId">prepareArticleReceptionId<s:property value="#stat.index" /></s:set> <s:url action="prepare-article-reception" id="prepareArticleReceptionUrl"> - <s:param name="supplierId" value="%{id}" /> + <s:param name="supplierId" value="%{value.id}" /> </s:url> <s:if test="#stat.index + 1 < shortcuts.size()"> <script type="text/javascript"> @@ -60,13 +68,13 @@ bindKey(<s:property value="shortcuts.get(#stat.index + 1)" />, function(){ redirectTo($('#' + id));}); }); </script> - <s:a href="%{prepareArticleReceptionUrl}" id="%{prepareArticleReceptionId}"> - (<s:property value="shortcuts.get(#stat.index + 1)" />) <s:property value="name" /> + <s:a href="%{prepareArticleReceptionUrl}" id="%{prepareArticleReceptionId}"> + (<s:property value="shortcuts.get(#stat.index + 1)" />) <s:property value="value.name" /> </s:a> </s:if> <s:else> <s:a href="%{prepareArticleReceptionUrl}" > - <s:property value="name" /> + <s:property value="value.name" /> </s:a> </s:else> </li>

1 0

r355 - trunk/magalie-web/src/main/webapp/WEB-INF/content
by bleny＠users.forge.codelutin.com 01 Aug '13

01 Aug '13

Author: bleny Date: 2013-08-01 12:09:09 +0200 (Thu, 01 Aug 2013) New Revision: 355 Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision… Log: facorize document.ready call Modified: trunk/magalie-web/src/main/webapp/WEB-INF/content/deliver-requested-article-input.jsp 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-01 08:27:29 UTC (rev 354) +++ trunk/magalie-web/src/main/webapp/WEB-INF/content/deliver-requested-article-input.jsp 2013-08-01 10:09:09 UTC (rev 355) @@ -28,6 +28,7 @@ $(document).ready(function () { $("#requestedArticle").focus(); bindKey('0', function(){ redirectTo($('#requestedArticle'));}); + bindKey('Esc', function(){ redirectTo($('#removeUserAffectationLink'));}); }); </script> @@ -65,12 +66,5 @@ </s:iterator> </s:else> -<script type="text/javascript"> - $(document).ready(function () { - $("#requestedArticle").focus(); - bindKey('Esc', function(){ redirectTo($('#removeUserAffectationLink'));}); - }); -</script> - <s:url action="remove-user-affectation" id="removeUserAffectationUrl"/> <s:a href="%{removeUserAffectationUrl}" cssClass="btn btn-block" id="removeUserAffectationLink">Changer d'activité (Esc)</s:a>

1 0

r354 - trunk/magalie-web/src/main/webapp/js
by bleny＠users.forge.codelutin.com 01 Aug '13

01 Aug '13

Author: bleny Date: 2013-08-01 10:27:29 +0200 (Thu, 01 Aug 2013) New Revision: 354 Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision… Log: fixes #3006 don't show next location if confirm, if one location is not used or if quantity withdraw > expected 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-07-31 16:15:32 UTC (rev 353) +++ trunk/magalie-web/src/main/webapp/js/withdraw-item-input.js 2013-08-01 08:27:29 UTC (rev 354) @@ -44,31 +44,42 @@ } model.newEmptyStorageMovementForNextLocation = function() { - var storageMovement; - $.ajax({ - dataType: "json", - url: getRealTimeStorageMovementTaskJsonUrl, - data: { articleId: this.articleId }, - async: false, - success: function(data) { - storageMovement = data.realTimeStorageMovementTask; - }, - fail: function(jqXHR, textStatus, errorThrown) { - console.error(jqXHR); - console.error(textStatus); - console.error(errorThrown); - } - }); - if (storageMovement.articleUnavailable) { - alert('Plus de stock'); - } else { - if (this.expectedQuantity) { - storageMovement.expectedQuantity = Math.min(model.remaining(), storageMovement.storedArticle.quantity); + var quantityIsRemaining = ! model.expectedQuantity // pas de quantité demandée précisée au départ donc on ne sait pas s'il faut s'arrêter donc on propose au cas où + || model.withdrawn() < model.expectedQuantity // on a une quantité demandée mais on ne l'a pas encore atteinte + var allIsUsed = true; + for (var i = 0; i < this.storageMovements.length; i++) { + if ( ! this.storageMovements[i].used) { + allIsUsed = false; + } + } + var proposeAnotherLocation = quantityIsRemaining && allIsUsed; + if (proposeAnotherLocation) { + var storageMovement; + $.ajax({ + dataType: "json", + url: getRealTimeStorageMovementTaskJsonUrl, + data: { articleId: this.articleId }, + async: false, + success: function(data) { + storageMovement = data.realTimeStorageMovementTask; + }, + fail: function(jqXHR, textStatus, errorThrown) { + console.error(jqXHR); + console.error(textStatus); + console.error(errorThrown); + } + }); + if (storageMovement.articleUnavailable) { + alert('Plus de stock'); } else { - storageMovement.expectedQuantity = storageMovement.storedArticle.quantity; + if (this.expectedQuantity) { + storageMovement.expectedQuantity = Math.min(model.remaining(), storageMovement.storedArticle.quantity); + } else { + storageMovement.expectedQuantity = storageMovement.storedArticle.quantity; + } + this.storageMovements.push(storageMovement); + this.storageMovementsIndex += 1; } - this.storageMovements.push(storageMovement); - this.storageMovementsIndex += 1; } } @@ -233,14 +244,6 @@ view.refresh(); - var proposeAnotherLocation = ! model.expectedQuantity // pas de quantité demandée précisée au départ donc on ne sait pas s'il faut s'arrêter donc on propose au cas où - || model.withdrawn() < model.expectedQuantity // on a une quantité demandée mais on ne l'a pas encore atteinte - if (proposeAnotherLocation) { - model.newEmptyStorageMovementForNextLocation(); - } - - view.refresh(); - }, onNext : function() { @@ -249,6 +252,10 @@ controller.next(false); + model.newEmptyStorageMovementForNextLocation(); + + view.refresh(); + } catch (ex) { if (ex instanceof ValidationError) { @@ -266,6 +273,10 @@ controller.next(true); + model.newEmptyStorageMovementForNextLocation(); + + view.refresh(); + } catch (ex) { if (ex instanceof ValidationError) { @@ -306,6 +317,12 @@ $('#withdraw-item').submit(); + } else { + + model.newEmptyStorageMovementForNextLocation(); + + view.refresh(); + } }

1 0

r353 - trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service
by bleny＠users.forge.codelutin.com 31 Jul '13

31 Jul '13

Author: bleny Date: 2013-07-31 18:15:32 +0200 (Wed, 31 Jul 2013) New Revision: 353 Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision… Log: refs #3006 prevent creation of duplicate storage movement Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ArticleStorageService.java 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-07-31 15:49:10 UTC (rev 352) +++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ArticleStorageService.java 2013-07-31 16:15:32 UTC (rev 353) @@ -49,6 +49,7 @@ import com.google.common.collect.ImmutableSet; import com.google.common.collect.Iterables; import com.google.common.collect.Lists; +import com.google.common.collect.Maps; import com.google.common.collect.Multimap; import com.google.common.collect.Multimaps; import com.google.common.collect.Ordering; @@ -433,27 +434,46 @@ Location destinationLocation = locationDao.findById(destinationLocationId); - StorageMovement storageMovement = new StorageMovement(); + StorageMovementJpaDao storageMovementDao = + persistenceContext.getStorageMovementDao(); - storageMovement.setOriginLocation(originLocation); + Map<String, Object> storageMovementProperties = Maps.newHashMap(); + storageMovementProperties.put(StorageMovement.PROPERTY_MAGALIE_USER, magalieUser); + storageMovementProperties.put(StorageMovement.PROPERTY_ARTICLE, article); + storageMovementProperties.put(StorageMovement.PROPERTY_ORIGIN_LOCATION, originLocation); + storageMovementProperties.put(StorageMovement.PROPERTY_DESTINATION_LOCATION, destinationLocation); + storageMovementProperties.put(StorageMovement.PROPERTY_CONFIRM_DATE, null); - storageMovement.setDestinationLocation(destinationLocation); + StorageMovement storageMovement = storageMovementDao.findByProperties(storageMovementProperties); - storageMovement.setMagalieUser(magalieUser); + if (storageMovement == null) { - storageMovement.setArticle(article); + storageMovement = new StorageMovement(); + storageMovement.setOriginLocation(originLocation); + + storageMovement.setDestinationLocation(destinationLocation); + + storageMovement.setMagalieUser(magalieUser); + + storageMovement.setArticle(article); + + storageMovementDao.persist(storageMovement); + + } else { + + if (log.isDebugEnabled()) { + log.debug("storage movement already exists, user just want to update quantity " + storageMovement); + } + + } + storageMovement.setActualQuantity(quantity); Date now = serviceContext.getNow(); storageMovement.setMovementDate(now); - StorageMovementJpaDao storageMovementDao = - persistenceContext.getStorageMovementDao(); - - storageMovementDao.persist(storageMovement); - persistenceContext.commit(); if (log.isInfoEnabled()) {

1 0

r352 - in trunk: magalie-persistence/src/main/java/com/franciaflex/magalie/persistence magalie-services/src/main/java/com/franciaflex/magalie/services/service magalie-services/src/test/java/com/franciaflex/magalie/services/service
by bleny＠users.forge.codelutin.com 31 Jul '13

31 Jul '13

Author: bleny Date: 2013-07-31 17:49:10 +0200 (Wed, 31 Jul 2013) New Revision: 352 Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision… Log: fixes #3001 urgent field was considered but shortest list was priorized, now list size is ignored Modified: 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-services/src/main/java/com/franciaflex/magalie/services/service/RequestedArticleService.java trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/RequestedArticleServiceTest.java Modified: 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-07-31 15:31:58 UTC (rev 351) +++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/RequestedArticles.java 2013-07-31 15:49:10 UTC (rev 352) @@ -31,7 +31,6 @@ import org.nuiton.jpa.api.JpaEntities; import java.util.Comparator; -import java.util.List; public class RequestedArticles { @@ -49,12 +48,12 @@ } } - public static Comparator<RequestedArticle> comparator(List<RequestedArticle> requests, RequestedList affectedRequestedList) { + public static Comparator<RequestedArticle> comparator(RequestedList affectedRequestedList) { Comparator<RequestedArticle> comparator = Ordering.compound( Lists.newArrayList( - new RequestedListComparator(RequestedLists.comparator(requests, affectedRequestedList)), + new RequestedListComparator(RequestedLists.comparator(affectedRequestedList)), // we can still have ambiguous or priorities, so to be deterministic, use arbitrary order JpaEntities.<RequestedArticle>arbitraryComparator() Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/RequestedLists.java =================================================================== --- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/RequestedLists.java 2013-07-31 15:31:58 UTC (rev 351) +++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/RequestedLists.java 2013-07-31 15:49:10 UTC (rev 352) @@ -23,21 +23,14 @@ * #L% */ -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 { @@ -77,40 +70,6 @@ } - 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 && ! x.equals(y)) { - 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(); } @@ -119,41 +78,15 @@ 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) { + public static Comparator<RequestedList> comparator(RequestedList affectedRequestedList) { Ordering<RequestedList> comparator = Ordering.compound( Lists.newArrayList( givenRequestListFirst(affectedRequestedList), - finishEarlyFirstComparator(requests), urgentFirstComparator(), requestDateFirstComparator() ) 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-07-31 15:31:58 UTC (rev 351) +++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/RequestedArticleService.java 2013-07-31 15:49:10 UTC (rev 352) @@ -286,7 +286,7 @@ } Comparator<RequestedArticle> priorityComparator = - RequestedArticles.comparator(requests, affectedRequestedList); + RequestedArticles.comparator(affectedRequestedList); Set<RequestedArticle> requestedArticles = Sets.newTreeSet(priorityComparator); 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-07-31 15:31:58 UTC (rev 351) +++ trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/RequestedArticleServiceTest.java 2013-07-31 15:49:10 UTC (rev 352) @@ -93,12 +93,12 @@ Assert.assertEquals(requestedArticles.size(), requestedArticlesByPriority.size()); - Assert.assertEquals("requestedArticle5", Iterables.get(requestedArticlesByPriority, 0).getId()); - Assert.assertEquals("requestedArticle6", Iterables.get(requestedArticlesByPriority, 1).getId()); - Assert.assertEquals("requestedArticle1", Iterables.get(requestedArticlesByPriority, 2).getId()); - Assert.assertEquals("requestedArticle2", Iterables.get(requestedArticlesByPriority, 3).getId()); - Assert.assertEquals("requestedArticle3", Iterables.get(requestedArticlesByPriority, 4).getId()); - Assert.assertEquals("requestedArticle4", Iterables.get(requestedArticlesByPriority, 5).getId()); + Assert.assertEquals("requestedArticle1", Iterables.get(requestedArticlesByPriority, 0).getId()); + Assert.assertEquals("requestedArticle2", Iterables.get(requestedArticlesByPriority, 1).getId()); + Assert.assertEquals("requestedArticle3", Iterables.get(requestedArticlesByPriority, 2).getId()); + Assert.assertEquals("requestedArticle4", Iterables.get(requestedArticlesByPriority, 3).getId()); + Assert.assertEquals("requestedArticle5", Iterables.get(requestedArticlesByPriority, 4).getId()); + Assert.assertEquals("requestedArticle6", Iterables.get(requestedArticlesByPriority, 5).getId()); } @@ -212,6 +212,20 @@ } + // continuing first list, user affectation not changed + { + orderToExecute = service.findOrderToExecute(magalieUser, building, LIST_TYPE); + Preconditions.checkState(orderToExecute.isSuccess()); + + Assert.assertNull(orderToExecute.getOldAffectation()); + Assert.assertNull(orderToExecute.getNewAffectation()); + + StorageMovementConfirmation confirmation = new StorageMovementConfirmation(); + confirmation.setRequestedArticleId(orderToExecute.getRequestedArticle().getId()); + service.onStorageMovementConfirmation(confirmation, emptySet); + + } + // first list is finished, must know old affectation and new affectation { orderToExecute = service.findOrderToExecute(magalieUser, building, LIST_TYPE);

1 0

r351 - in trunk: magalie-services/src/main/java/com/franciaflex/magalie/services/service magalie-services/src/test/java/com/franciaflex/magalie/services/service magalie-web/src/main/java/com/franciaflex/magalie/web/action magalie-web/src/main/webapp/js
by bleny＠users.forge.codelutin.com 31 Jul '13

31 Jul '13

Author: bleny Date: 2013-07-31 17:31:58 +0200 (Wed, 31 Jul 2013) New Revision: 351 Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision… Log: fixes #2995 report error if user scan a location known as full Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ReceptionService.java trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/ReceptionServiceTest.java trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/LocationJsonAction.java trunk/magalie-web/src/main/webapp/js/receive-article-input.js Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ReceptionService.java =================================================================== --- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ReceptionService.java 2013-07-31 14:11:50 UTC (rev 350) +++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ReceptionService.java 2013-07-31 15:31:58 UTC (rev 351) @@ -26,6 +26,7 @@ import com.franciaflex.magalie.persistence.JpaMagaliePersistenceContext; import com.franciaflex.magalie.persistence.Locations; import com.franciaflex.magalie.persistence.StoredArticles; +import com.franciaflex.magalie.persistence.dao.ArticleJpaDao; import com.franciaflex.magalie.persistence.dao.LocationDao; import com.franciaflex.magalie.persistence.dao.LocationJpaDao; import com.franciaflex.magalie.persistence.dao.PreparedArticleReceptionJpaDao; @@ -403,4 +404,32 @@ } + public Location getLocationToStoreSomething(Building building, MagalieUser magalieUser, String locationBarcode, String articleId) throws InvalidMagalieBarcodeException { + + MagalieBarcodeService magalieBarcodeService = + serviceContext.newService(MagalieBarcodeService.class); + + Location location = magalieBarcodeService.getLocation(locationBarcode, building); + + if (location.isFullLocation()) { + + if (log.isInfoEnabled()) { + log.info(magalieUser + " scanned barcode " + locationBarcode + " for reception but location " + location + " is full, reporting error"); + } + + LocationErrorsService locationErrorsService = + serviceContext.newService(LocationErrorsService.class); + + ArticleJpaDao articleDao = serviceContext.getPersistenceContext().getArticleDao(); + + Article article = articleDao.findById(articleId); + + locationErrorsService.reportError(location, article, magalieUser); + + } + + return location; + + } + } Modified: trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/ReceptionServiceTest.java =================================================================== --- trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/ReceptionServiceTest.java 2013-07-31 14:11:50 UTC (rev 350) +++ trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/ReceptionServiceTest.java 2013-07-31 15:31:58 UTC (rev 351) @@ -24,6 +24,7 @@ */ import com.franciaflex.magalie.persistence.Locations; +import com.franciaflex.magalie.persistence.dao.LocationErrorJpaDao; import com.franciaflex.magalie.persistence.dao.ReceivedPreparedArticleReceptionJpaDao; import com.franciaflex.magalie.persistence.dao.StorageMovementJpaDao; import com.franciaflex.magalie.persistence.entity.Article; @@ -354,4 +355,19 @@ } + @Test + public void testGetLocationToStoreSomethingReportErrorIfFullLocation() throws InvalidMagalieBarcodeException { + + Location fullLocation = fixture("locationFull1"); + + service.getLocationToStoreSomething(building, magalieUser, fullLocation.getBarcode(), article.getId()); + + EntityManager entityManager = getJpaEntityManagerRule().newEntityManager(); + + LocationErrorJpaDao locationErrorJpaDao = new LocationErrorJpaDao(entityManager); + + Assert.assertFalse(locationErrorJpaDao.findAll().isEmpty()); + + } + } Modified: trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/LocationJsonAction.java =================================================================== --- trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/LocationJsonAction.java 2013-07-31 14:11:50 UTC (rev 350) +++ trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/LocationJsonAction.java 2013-07-31 15:31:58 UTC (rev 351) @@ -25,10 +25,13 @@ import com.franciaflex.magalie.persistence.entity.Building; import com.franciaflex.magalie.persistence.entity.Location; +import com.franciaflex.magalie.persistence.entity.MagalieUser; import com.franciaflex.magalie.services.exception.InvalidMagalieBarcodeException; -import com.franciaflex.magalie.services.service.MagalieBarcodeService; +import com.franciaflex.magalie.services.service.ReceptionService; import com.franciaflex.magalie.web.MagalieActionSupport; import com.franciaflex.magalie.web.MagalieSession; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import org.apache.struts2.convention.annotation.Result; import org.apache.struts2.convention.annotation.Results; @@ -37,19 +40,23 @@ }) public class LocationJsonAction extends MagalieActionSupport{ - protected MagalieBarcodeService service; + private static final Log log = LogFactory.getLog(LocationJsonAction.class); + protected ReceptionService service; + protected Location location; protected String locationBarcode; protected MagalieSession session; + protected String articleId; + public void setSession(MagalieSession session) { this.session = session; } - public void setService(MagalieBarcodeService service) { + public void setService(ReceptionService service) { this.service = service; } @@ -57,12 +64,22 @@ this.locationBarcode = locationBarcode; } + public void setArticleId(String articleId) { + this.articleId = articleId; + } + @Override public String execute() throws InvalidMagalieBarcodeException { + MagalieUser magalieUser = session.getMagalieUser(); + + if (log.isInfoEnabled()) { + log.info(magalieUser + " scanned " + locationBarcode + " to store something"); + } + Building building = session.getBuilding(); - location = service.getLocation(locationBarcode, building); + location = service.getLocationToStoreSomething(building, magalieUser, locationBarcode, articleId); return SUCCESS; Modified: trunk/magalie-web/src/main/webapp/js/receive-article-input.js =================================================================== --- trunk/magalie-web/src/main/webapp/js/receive-article-input.js 2013-07-31 14:11:50 UTC (rev 350) +++ trunk/magalie-web/src/main/webapp/js/receive-article-input.js 2013-07-31 15:31:58 UTC (rev 351) @@ -55,7 +55,10 @@ $.ajax({ dataType: "json", url: locationJsonUrl, - data: { locationBarcode: barcode }, + data: { + locationBarcode: barcode, + articleId: model.storedArticle.article.id + }, async: false, success: function(data) { location = data.location; @@ -66,6 +69,9 @@ if (location == null) { throw new ValidationError(barcode + " n'est pas le code barre d'un emplacement valide"); } + if (location.fullLocation) { + throw new ValidationError("Emplt " + barcode + " en erreur - choisir un autre"); + } return location; };

1 0

r350 - in trunk/magalie-services/src: main/java/com/franciaflex/magalie/services/service test/java/com/franciaflex/magalie/services/service
by bleny＠users.forge.codelutin.com 31 Jul '13

31 Jul '13

Author: bleny Date: 2013-07-31 16:11:50 +0200 (Wed, 31 Jul 2013) New Revision: 350 Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision… Log: fixes #2989 reported errors was not considered Added: trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/StorageTransferServiceTest.java Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/StorageTransferService.java Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/StorageTransferService.java =================================================================== --- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/StorageTransferService.java 2013-07-31 13:00:16 UTC (rev 349) +++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/StorageTransferService.java 2013-07-31 14:11:50 UTC (rev 350) @@ -104,7 +104,7 @@ StoredArticleDao storedArticleDao = persistenceContext.getStoredArticleDao(); - Iterable<StoredArticle > storedArticles = storedArticleDao.findAllByLocation(location); + Iterable<StoredArticle> storedArticles = storedArticleDao.findAllByLocation(location); ArticleStorageService articleStorageService = serviceContext.newService(ArticleStorageService.class); @@ -118,7 +118,12 @@ storedArticles = articleStorageService.computeActualQuantities(storedArticles, allImpactingStoredArticle); + LocationErrorsService locationErrorsService = serviceContext.newService(LocationErrorsService.class); + + storedArticles = locationErrorsService.filterLocationInError(storedArticles); + return Lists.newArrayList(storedArticles); + } public void confirmStorageTransfer(MagalieUser magalieUser, StoredArticle storedArticle, double quantity, Location destinationLocation) { Added: trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/StorageTransferServiceTest.java =================================================================== --- trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/StorageTransferServiceTest.java (rev 0) +++ trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/StorageTransferServiceTest.java 2013-07-31 14:11:50 UTC (rev 350) @@ -0,0 +1,63 @@ +package com.franciaflex.magalie.services.service; + +import com.franciaflex.magalie.persistence.entity.Building; +import com.franciaflex.magalie.persistence.entity.Location; +import com.franciaflex.magalie.persistence.entity.MagalieUser; +import com.franciaflex.magalie.persistence.entity.StoredArticle; +import com.franciaflex.magalie.services.AbstractMagalieServiceTest; +import org.junit.Assert; +import org.junit.Before; +import org.junit.Test; + +import java.text.ParseException; +import java.util.Date; +import java.util.List; + +public class StorageTransferServiceTest extends AbstractMagalieServiceTest { + + protected StorageTransferService service; + + protected MagalieUser magalieUser; + + protected Building building; + + @Before + public void setUp() throws ParseException { + + loadFixtures("fixtures"); + + service = newService(StorageTransferService.class); + + serviceContext.setDate(new Date(1363948427576l)); + + magalieUser = fixture("bruno"); + + building = fixture("B1"); + + } + + @Test + public void testGetStoredArticlesInLocationIgnoreArticlesReportedInError() { + + Location location = fixture("location3"); + + List<StoredArticle> storedArticlesInLocation = + service.getStoredArticlesInLocation(location); + + Assert.assertEquals(2, storedArticlesInLocation.size()); + + LocationErrorsService locationErrorsService = + newService(LocationErrorsService.class); + + locationErrorsService.reportError(storedArticlesInLocation.get(0).getId(), magalieUser); + + List<StoredArticle> storedArticlesInLocationAfterError = + service.getStoredArticlesInLocation(location); + + Assert.assertEquals( + storedArticlesInLocation.size() - 1, + storedArticlesInLocationAfterError.size()); + + } + +}

1 0

r349 - trunk/magalie-web/src/main/webapp/WEB-INF/content
by bleny＠users.forge.codelutin.com 31 Jul '13

31 Jul '13

Author: bleny Date: 2013-07-31 15:00:16 +0200 (Wed, 31 Jul 2013) New Revision: 349 Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision… Log: fixes #2988 add binding for F5 key Modified: trunk/magalie-web/src/main/webapp/WEB-INF/content/simple-withdraw-item-input.jsp Modified: trunk/magalie-web/src/main/webapp/WEB-INF/content/simple-withdraw-item-input.jsp =================================================================== --- trunk/magalie-web/src/main/webapp/WEB-INF/content/simple-withdraw-item-input.jsp 2013-07-31 12:53:45 UTC (rev 348) +++ trunk/magalie-web/src/main/webapp/WEB-INF/content/simple-withdraw-item-input.jsp 2013-07-31 13:00:16 UTC (rev 349) @@ -29,6 +29,7 @@ <script type="text/javascript"> $(document).ready(function () { bindKey('Esc', function(){ redirectTo($('#cancelLink'));}); + bindKey('F5', function(){ $('#reportError').click(); }); $('input[name="originLocationBarcode"]').focus(); }); </script> @@ -80,7 +81,7 @@ <div class="btn-group"> <s:a href="%{cancelUrl}" cssClass="btn btn-mini" id="cancelLink" >Quitter (Esc)</s:a> <s:submit action="simple-withdraw-item" value="Valider" cssClass="btn btn-mini btn-primary" /> - <s:submit action="simple-withdraw-item!reportError" value="Anomalie (F5)" cssClass="btn btn-mini btn-danger" /> + <s:submit action="simple-withdraw-item!reportError" value="Anomalie (F5)" cssClass="btn btn-mini btn-danger" id="reportError" /> </div> </s:form>

1 0

r348 - in trunk: magalie-services/src/main/java/com/franciaflex/magalie/services/service magalie-web/src/main/java/com/franciaflex/magalie/web/action
by bleny＠users.forge.codelutin.com 31 Jul '13

31 Jul '13

Author: bleny Date: 2013-07-31 14:53:45 +0200 (Wed, 31 Jul 2013) New Revision: 348 Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision… Log: fixes #2995 : the stored article asked was in the result of preferred location Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ReceptionService.java trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ReceptionTask.java trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/ReceiveArticleAction.java Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ReceptionService.java =================================================================== --- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ReceptionService.java 2013-07-31 12:15:10 UTC (rev 347) +++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ReceptionService.java 2013-07-31 12:53:45 UTC (rev 348) @@ -206,24 +206,21 @@ Set<Location> preferredLocations = getPreferredLocations(warehouse, article); - boolean userMustChooseExtraLocation = preferredLocations.isEmpty(); + Iterable<Location> acceptablePreferredLocations = Iterables.filter(preferredLocations, isAcceptableForReception); // second step, add extra locations Set<Location> extraLocations = getExtraLocations(warehouse); - Set<Location> locations = Sets.newLinkedHashSet(); + Iterable<Location> acceptableExtraLocations = Iterables.filter(extraLocations, isAcceptableForReception); - Iterables.addAll(locations, Iterables.filter(preferredLocations, isAcceptableForReception)); + ReceptionTask receptionTask = new ReceptionTask(storedArticle, quantity, acceptablePreferredLocations, acceptableExtraLocations); - Iterables.addAll(locations, Iterables.filter(extraLocations, isAcceptableForReception)); - - ReceptionTask receptionTask = new ReceptionTask(storedArticle, quantity, locations, userMustChooseExtraLocation); - if (log.isDebugEnabled()) { - log.debug(locations.size() + " locations to receive articles for article " - + article + " in warehouse " + warehouse + " are " - + StringUtils.join(locations, ", ")); + log.debug("locations to receive articles for article " + + article + " in warehouse " + warehouse + " are acceptable = " + + StringUtils.join(acceptablePreferredLocations, ", ") + + " extra = " + StringUtils.join(acceptableExtraLocations, ", ")); } return receptionTask; Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ReceptionTask.java =================================================================== --- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ReceptionTask.java 2013-07-31 12:15:10 UTC (rev 347) +++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ReceptionTask.java 2013-07-31 12:53:45 UTC (rev 348) @@ -25,6 +25,9 @@ import com.franciaflex.magalie.persistence.entity.Location; import com.franciaflex.magalie.persistence.entity.StoredArticle; +import com.google.common.collect.Iterables; +import com.google.common.collect.Sets; +import org.apache.commons.lang3.builder.ReflectionToStringBuilder; import java.util.Set; @@ -36,13 +39,17 @@ protected double quantity; + protected Iterable<Location> preferredLocations; + + protected Iterable<Location> extraLocations; + protected Set<Location> locations; - public ReceptionTask(StoredArticle storedArticle, double quantity, Set<Location> locations, boolean userMustChooseExtraLocation) { + public ReceptionTask(StoredArticle storedArticle, double quantity, Iterable<Location> preferredLocations, Iterable<Location> extraLocations) { this.storedArticle = storedArticle; this.quantity = quantity; - this.locations = locations; - this.userMustChooseExtraLocation = userMustChooseExtraLocation; + this.preferredLocations = preferredLocations; + this.extraLocations = extraLocations; } public StoredArticle getStoredArticle() { @@ -50,6 +57,11 @@ } public Set<Location> getLocations() { + if (locations == null) { + locations = Sets.newLinkedHashSet(); + Iterables.addAll(locations, preferredLocations); + Iterables.addAll(locations, extraLocations); + } return locations; } @@ -58,7 +70,12 @@ } public boolean isUserMustChooseExtraLocation() { - return userMustChooseExtraLocation; + return Iterables.isEmpty(preferredLocations); } + @Override + public String toString() { + return ReflectionToStringBuilder.toString(this); + } + } Modified: trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/ReceiveArticleAction.java =================================================================== --- trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/ReceiveArticleAction.java 2013-07-31 12:15:10 UTC (rev 347) +++ trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/ReceiveArticleAction.java 2013-07-31 12:53:45 UTC (rev 348) @@ -98,12 +98,6 @@ receptionTask = service.getReceptionTaskForPreparedArticleReception(building, preparedArticleReceptionBarcode); - if (receptionTask.isUserMustChooseExtraLocation()) { - - addActionMessage("Pas de stock : emplacement libre proposé"); - - } - } catch (PreparedArticleReceptionAlreadyStoredException e) { session.addMessage("Article déjà rangé"); @@ -126,6 +120,12 @@ Preconditions.checkNotNull(receptionTask); + if (receptionTask.isUserMustChooseExtraLocation()) { + + addActionMessage("Pas de stock : emplacement libre proposé"); + + } + return INPUT; }

1 0

r347 - trunk/magalie-web/src/main/webapp/WEB-INF/content
by bleny＠users.forge.codelutin.com 31 Jul '13

31 Jul '13

Author: bleny Date: 2013-07-31 14:15:10 +0200 (Wed, 31 Jul 2013) New Revision: 347 Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision… Log: change label in withdraw-item-input Modified: trunk/magalie-web/src/main/webapp/WEB-INF/content/withdraw-item-input.jsp 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-07-26 11:53:22 UTC (rev 346) +++ trunk/magalie-web/src/main/webapp/WEB-INF/content/withdraw-item-input.jsp 2013-07-31 12:15:10 UTC (rev 347) @@ -45,7 +45,7 @@ Desc. : <s:property value="article.description" /> </li> <li> - Stocké : + Prélevé : <span id="withdrawn">0</span> <s:if test="expectedQuantity"> / <s:property value="expectedQuantity" />

1 0