Franciaflex-magalie-commits
Threads by month
- ----- 2026 -----
- June
- May
- April
- March
- February
- January
- ----- 2025 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2024 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2023 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2022 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2021 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2020 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2019 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2018 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2017 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2016 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2015 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2014 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2013 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
April 2013
- 2 participants
- 82 discussions
15 Apr '13
Author: bleny
Date: 2013-04-15 14:51:16 +0200 (Mon, 15 Apr 2013)
New Revision: 96
Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision…
Log:
add B2 building in fixtures
Modified:
trunk/magalie-services/src/main/resources/fixtures.yaml
Modified: trunk/magalie-services/src/main/resources/fixtures.yaml
===================================================================
--- trunk/magalie-services/src/main/resources/fixtures.yaml 2013-04-12 13:54:08 UTC (rev 95)
+++ trunk/magalie-services/src/main/resources/fixtures.yaml 2013-04-15 12:51:16 UTC (rev 96)
@@ -37,8 +37,14 @@
id: building_1
code: B1
+B2:
+ &B2 !building
+ id: building_2
+ code: B2
+
buildings:
- *B1
+ - *B2
U01:
&U01 !warehouse
@@ -56,13 +62,13 @@
&U03 !warehouse
id: warehouse_u03
code: U03
- building: *B1
+ building: *B2
U04:
&U04 !warehouse
id: warehouse_u04
code: U04
- building: *B1
+ building: *B2
SOM:
&SOM !warehouse
1
0
r95 - in trunk: 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-web/src/main/webapp/WEB-INF/content
by bleny@users.forge.codelutin.com 12 Apr '13
by bleny@users.forge.codelutin.com 12 Apr '13
12 Apr '13
Author: bleny
Date: 2013-04-12 15:54:08 +0200 (Fri, 12 Apr 2013)
New Revision: 95
Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision…
Log:
introduce supplier
Added:
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Supplier.java
Modified:
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Article.java
trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ArticleStorageService.java
trunk/magalie-services/src/main/resources/fixtures.yaml
trunk/magalie-web/src/main/webapp/WEB-INF/content/withdraw-item-input.jsp
Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Article.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Article.java 2013-04-12 12:40:47 UTC (rev 94)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Article.java 2013-04-12 13:54:08 UTC (rev 95)
@@ -3,6 +3,7 @@
import javax.persistence.Entity;
import javax.persistence.Id;
import javax.persistence.ManyToMany;
+import javax.persistence.ManyToOne;
import java.util.Set;
@Entity
@@ -14,6 +15,9 @@
@ManyToMany
protected Set<Location> fixedLocations;
+ @ManyToOne
+ protected Supplier supplier;
+
protected String description;
protected String code;
@@ -89,4 +93,12 @@
public void setFixedLocationsFirst(boolean fixedLocationsFirst) {
this.fixedLocationsFirst = fixedLocationsFirst;
}
+
+ public Supplier getSupplier() {
+ return supplier;
+ }
+
+ public void setSupplier(Supplier supplier) {
+ this.supplier = supplier;
+ }
}
Added: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Supplier.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Supplier.java (rev 0)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Supplier.java 2013-04-12 13:54:08 UTC (rev 95)
@@ -0,0 +1,30 @@
+package com.franciaflex.magalie.persistence.entity;
+
+import javax.persistence.Entity;
+import javax.persistence.Id;
+
+@Entity
+public class Supplier extends AbstractEntity {
+
+ @Id
+ protected String id;
+
+ protected String name;
+
+ @Override
+ public String getId() {
+ return id;
+ }
+
+ public void setId(String id) {
+ this.id = id;
+ }
+
+ public String getName() {
+ return name;
+ }
+
+ public void setName(String name) {
+ this.name = name;
+ }
+}
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-12 12:40:47 UTC (rev 94)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ArticleStorageService.java 2013-04-12 13:54:08 UTC (rev 95)
@@ -54,6 +54,8 @@
Preconditions.checkNotNull(bookArticleRequest.getArticle());
+ Preconditions.checkNotNull(bookArticleRequest.getDestinationLocation());
+
Preconditions.checkArgument(bookArticleRequest.getRequestedQuantity() > 0.);
if (log.isInfoEnabled()) {
Modified: trunk/magalie-services/src/main/resources/fixtures.yaml
===================================================================
--- trunk/magalie-services/src/main/resources/fixtures.yaml 2013-04-12 12:40:47 UTC (rev 94)
+++ trunk/magalie-services/src/main/resources/fixtures.yaml 2013-04-12 13:54:08 UTC (rev 95)
@@ -426,6 +426,7 @@
article: *article1
quantity: 80
requestedList: *requestedList1
+ destinationLocation: *location14
requestedArticle2:
&requestedArticle2 !requested-article
@@ -433,6 +434,7 @@
article: *article2
quantity: 11
requestedList: *requestedList1
+ destinationLocation: *location14
requestedArticle3:
&requestedArticle3 !requested-article
@@ -440,6 +442,7 @@
article: *article3
quantity: 111
requestedList: *requestedList1
+ destinationLocation: *location14
requestedArticle4:
&requestedArticle4 !requested-article
@@ -447,6 +450,7 @@
article: *article4
quantity: 2
requestedList: *requestedList1
+ destinationLocation: *location14
requestedArticle5:
&requestedArticle5 !requested-article
@@ -454,6 +458,7 @@
article: *article5
quantity: 5
requestedList: *requestedList2
+ destinationLocation: *location14
requestedArticle6:
&requestedArticle6 !requested-article
@@ -461,6 +466,7 @@
article: *article6
quantity: 27
requestedList: *requestedList2
+ destinationLocation: *location14
requestedArticles:
- *requestedArticle1
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-04-12 12:40:47 UTC (rev 94)
+++ trunk/magalie-web/src/main/webapp/WEB-INF/content/withdraw-item-input.jsp 2013-04-12 13:54:08 UTC (rev 95)
@@ -90,8 +90,6 @@
<dd><s:property value="storageMovementTask.article.code" /></dd>
<dt>Desc.</dt>
<dd><s:property value="storageMovementTask.article.description" /></dd>
- <dt>LàS</dt>
- <dd>123456789</dd>
<dt>Prélev.</dt>
<dd><span id="withdrawn">0</span> / <s:property value="storageMovementTask.quantity" /> <s:property value="storageMovementTask.article.unit" /></dd>
</dl>
1
0
r94 - in trunk: 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
by bleny@users.forge.codelutin.com 12 Apr '13
by bleny@users.forge.codelutin.com 12 Apr '13
12 Apr '13
Author: bleny
Date: 2013-04-12 14:40:47 +0200 (Fri, 12 Apr 2013)
New Revision: 94
Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision…
Log:
introduce requestList.building
Modified:
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/RequestedArticle.java
trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/RequestedArticleService.java
trunk/magalie-services/src/main/resources/fixtures.yaml
Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/RequestedArticle.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/RequestedArticle.java 2013-04-12 10:23:19 UTC (rev 93)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/RequestedArticle.java 2013-04-12 12:40:47 UTC (rev 94)
@@ -3,6 +3,7 @@
import javax.persistence.Entity;
import javax.persistence.Id;
import javax.persistence.ManyToOne;
+import javax.persistence.OneToOne;
@Entity
public class RequestedArticle extends AbstractEntity {
@@ -16,6 +17,9 @@
@ManyToOne
protected RequestedList requestedList;
+ @OneToOne
+ protected Location destinationLocation;
+
protected double quantity;
@Override
@@ -50,4 +54,12 @@
public void setRequestedList(RequestedList requestedList) {
this.requestedList = requestedList;
}
+
+ public Location getDestinationLocation() {
+ return destinationLocation;
+ }
+
+ public void setDestinationLocation(Location destinationLocation) {
+ this.destinationLocation = destinationLocation;
+ }
}
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-12 10:23:19 UTC (rev 93)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/RequestedArticleService.java 2013-04-12 12:40:47 UTC (rev 94)
@@ -30,6 +30,7 @@
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.Location;
import com.franciaflex.magalie.persistence.entity.MagalieUser;
import com.franciaflex.magalie.persistence.entity.RequestedArticle;
import com.franciaflex.magalie.persistence.entity.StorageMovementOrder;
@@ -116,8 +117,10 @@
double quantity = requestedArticle.getQuantity();
- BookArticleRequest bookArticleRequest = new BookArticleRequest(magalieUser, building, article, quantity, null, false);
+ Location destinationLocation = requestedArticle.getDestinationLocation();
+ BookArticleRequest bookArticleRequest = new BookArticleRequest(magalieUser, building, article, quantity, destinationLocation, false);
+
try {
storageMovementOrder = articleStorageService.bookArticle(bookArticleRequest);
@@ -184,22 +187,28 @@
protected Queue<RequestedArticle> getRequestedArticlesByPriority(List<RequestedArticle> requests) {
- Comparator<RequestedArticle> priorityComparator =
- Ordering.compound(
- Lists.newArrayList(
- RequestedArticles.finishEarlyFirstComparator(requests),
- RequestedArticles.urgentFirstComparator(),
- RequestedArticles.requestDateFirstComparator()
- )
- );
+ PriorityQueue<RequestedArticle> requestedArticles = new PriorityQueue<RequestedArticle>();
- PriorityQueue<RequestedArticle> requestedArticles =
- new PriorityQueue<RequestedArticle>(
- requests.size(),
- priorityComparator);
+ if ( ! requests.isEmpty()) {
- requestedArticles.addAll(requests);
+ Comparator<RequestedArticle> priorityComparator =
+ Ordering.compound(
+ Lists.newArrayList(
+ RequestedArticles.finishEarlyFirstComparator(requests),
+ RequestedArticles.urgentFirstComparator(),
+ RequestedArticles.requestDateFirstComparator()
+ )
+ );
+ requestedArticles =
+ new PriorityQueue<RequestedArticle>(
+ requests.size(),
+ priorityComparator);
+
+ requestedArticles.addAll(requests);
+
+ }
+
return requestedArticles;
}
Modified: trunk/magalie-services/src/main/resources/fixtures.yaml
===================================================================
--- trunk/magalie-services/src/main/resources/fixtures.yaml 2013-04-12 10:23:19 UTC (rev 93)
+++ trunk/magalie-services/src/main/resources/fixtures.yaml 2013-04-12 12:40:47 UTC (rev 94)
@@ -70,12 +70,19 @@
code: SOM
building: *B1
+B01:
+ &B01 !warehouse
+ id: warehouse_b01
+ code: B01
+ building: *B1
+
warehouses:
- *U01
- *U02
- *U03
- *U04
- *SOM
+ - *B01
location1:
&location1 !location
@@ -168,6 +175,13 @@
code: REC
requiredAccreditationLevel: 0
+location14:
+ &location14 !location
+ id: location14
+ warehouse: *B01
+ code: SANS
+ requiredAccreditationLevel: 0
+
locations:
- *location1
- *location2
@@ -182,6 +196,7 @@
- *location11
- *location12
- *location13
+ - *location14
article1:
&article1 !article
@@ -391,6 +406,7 @@
code: 123456789
requestDate: 2013-04-08 12:00:00
urgent: true
+ building: *B1
requestedList2:
&requestedList2 !requested-list
@@ -398,6 +414,7 @@
code: 123456789
requestDate: 2013-04-08 12:00:00
urgent: false
+ building: *B1
requestedLists:
- *requestedList1
1
0
r93 - 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-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 12 Apr '13
by bleny@users.forge.codelutin.com 12 Apr '13
12 Apr '13
Author: bleny
Date: 2013-04-12 12:23:19 +0200 (Fri, 12 Apr 2013)
New Revision: 93
Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision…
Log:
introduce notion of warehouses without locations
Modified:
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/Locations.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/LocationDao.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/WarehouseDao.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/LocationJpaDao.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/WarehouseJpaDao.java
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
Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/Locations.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/Locations.java 2013-04-12 10:09:55 UTC (rev 92)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/Locations.java 2013-04-12 10:23:19 UTC (rev 93)
@@ -70,4 +70,9 @@
public static Predicate<Location> locationIsNotReportedInError(Collection<Location> allLocationsInError) {
return Predicates.not(Predicates.in(allLocationsInError));
}
+
+ public static String codeForWarehouseWithoutLocations() {
+ return "SANS";
+ }
+
}
Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/LocationDao.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/LocationDao.java 2013-04-12 10:09:55 UTC (rev 92)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/LocationDao.java 2013-04-12 10:23:19 UTC (rev 93)
@@ -1,7 +1,10 @@
package com.franciaflex.magalie.persistence.dao;
import com.franciaflex.magalie.persistence.entity.Location;
+import com.franciaflex.magalie.persistence.entity.Warehouse;
public interface LocationDao extends Dao<Location> {
+ Location find(String code, Warehouse warehouse);
+
}
Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/WarehouseDao.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/WarehouseDao.java 2013-04-12 10:09:55 UTC (rev 92)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/WarehouseDao.java 2013-04-12 10:23:19 UTC (rev 93)
@@ -1,10 +1,15 @@
package com.franciaflex.magalie.persistence.dao;
+import com.franciaflex.magalie.persistence.entity.Building;
import com.franciaflex.magalie.persistence.entity.Warehouse;
+import java.util.List;
+
/**
* @author bleny
*/
public interface WarehouseDao extends Dao<Warehouse> {
+ List<Warehouse> findAllWithoutLocations(Building building);
+
}
Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/LocationJpaDao.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/LocationJpaDao.java 2013-04-12 10:09:55 UTC (rev 92)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/LocationJpaDao.java 2013-04-12 10:23:19 UTC (rev 93)
@@ -2,10 +2,10 @@
import com.franciaflex.magalie.persistence.dao.LocationDao;
import com.franciaflex.magalie.persistence.entity.Location;
+import com.franciaflex.magalie.persistence.entity.Warehouse;
import javax.persistence.EntityManager;
-import javax.persistence.Query;
-import java.util.List;
+import javax.persistence.TypedQuery;
public class LocationJpaDao extends AbstractJpaDao<Location> implements LocationDao {
@@ -18,4 +18,12 @@
return Location.class;
}
+ @Override
+ public Location find(String code, Warehouse warehouse) {
+ TypedQuery<Location> query = entityManager.createQuery("from Location l where l.code = :code and l.warehouse = :warehouse", getEntityClass());
+ query.setParameter("code", code);
+ query.setParameter("warehouse", warehouse);
+ Location singleResult = query.getSingleResult();
+ return singleResult;
+ }
}
Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/WarehouseJpaDao.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/WarehouseJpaDao.java 2013-04-12 10:09:55 UTC (rev 92)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/WarehouseJpaDao.java 2013-04-12 10:23:19 UTC (rev 93)
@@ -1,9 +1,13 @@
package com.franciaflex.magalie.persistence.dao.jpa;
+import com.franciaflex.magalie.persistence.Locations;
import com.franciaflex.magalie.persistence.dao.WarehouseDao;
+import com.franciaflex.magalie.persistence.entity.Building;
import com.franciaflex.magalie.persistence.entity.Warehouse;
import javax.persistence.EntityManager;
+import javax.persistence.TypedQuery;
+import java.util.List;
public class WarehouseJpaDao extends AbstractJpaDao<Warehouse> implements WarehouseDao {
@@ -15,4 +19,12 @@
protected Class<Warehouse> getEntityClass() {
return Warehouse.class;
}
+
+ @Override
+ public List<Warehouse> findAllWithoutLocations(Building building) {
+ TypedQuery<Warehouse> query = entityManager.createQuery("from Warehouse w where w.building = :building and w in (select l.warehouse from Location l where l.code = :code)", getEntityClass());
+ query.setParameter("building", building);
+ query.setParameter("code", Locations.codeForWarehouseWithoutLocations());
+ return query.getResultList();
+ }
}
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-12 10:09:55 UTC (rev 92)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/FulfilKanbanService.java 2013-04-12 10:23:19 UTC (rev 93)
@@ -1,6 +1,8 @@
package com.franciaflex.magalie.services.service;
+import com.franciaflex.magalie.persistence.Locations;
import com.franciaflex.magalie.persistence.MagaliePersistenceContext;
+import com.franciaflex.magalie.persistence.dao.LocationDao;
import com.franciaflex.magalie.persistence.dao.WarehouseDao;
import com.franciaflex.magalie.persistence.entity.Article;
import com.franciaflex.magalie.persistence.entity.Building;
@@ -84,21 +86,25 @@
protected Location getDestinationLocation(Warehouse destinationWarehouse) {
- // TODO brendan 10/04/13 how to define destination for warehouse, something like "INC" ?
+ MagaliePersistenceContext persistenceContext = serviceContext.getPersistenceContext();
- return null;
+ LocationDao locationDao = persistenceContext.getLocationDao();
+ Location location = locationDao.find(Locations.codeForWarehouseWithoutLocations(), destinationWarehouse);
+
+ return location;
+
}
- public List<Warehouse> getDestinationStores(Building building) {
+ public List<Warehouse> getDestinationWarehouses(Building building) {
MagaliePersistenceContext persistenceContext = serviceContext.getPersistenceContext();
WarehouseDao warehouseDao = persistenceContext.getWarehouseDao();
- // FIXME brendan 10/04/13 consider building
+ List<Warehouse> destinationWarehouses = warehouseDao.findAllWithoutLocations(building);
- return warehouseDao.findAll();
+ return destinationWarehouses;
}
}
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-12 10:09:55 UTC (rev 92)
+++ trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/FulfilKanbanAction.java 2013-04-12 10:23:19 UTC (rev 93)
@@ -50,7 +50,7 @@
Building building = session.getBuilding();
- destinationWarehouses = service.getDestinationStores(building);
+ destinationWarehouses = service.getDestinationWarehouses(building);
// let's help user by selecting by default the previously used destination warehouse
1
0
r92 - in trunk: 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
by bleny@users.forge.codelutin.com 12 Apr '13
by bleny@users.forge.codelutin.com 12 Apr '13
12 Apr '13
Author: bleny
Date: 2013-04-12 12:09:55 +0200 (Fri, 12 Apr 2013)
New Revision: 92
Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision…
Log:
add building in RequestList
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/entity/RequestedList.java
trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/RequestedArticleService.java
trunk/magalie-services/src/main/resources/fixtures2.yaml
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-12 09:24:13 UTC (rev 91)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/RequestedArticleJpaDao.java 2013-04-12 10:09:55 UTC (rev 92)
@@ -21,12 +21,12 @@
@Override
public List<RequestedArticle> findAllUndelivered(Building building) {
- // XXX brendan 10/04/13 what about building ???
TypedQuery<RequestedArticle> query =
entityManager.createQuery(
- "from RequestedArticle ra where ra not in (select dra.requestedArticle from DeliveredRequestedArticle dra)",
+ "from RequestedArticle ra where ra not in (select dra.requestedArticle from DeliveredRequestedArticle dra) and ra.requestedList.building = :building",
getEntityClass()
);
+ query.setParameter("building", building);
return query.getResultList();
}
Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/RequestedList.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/RequestedList.java 2013-04-12 09:24:13 UTC (rev 91)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/RequestedList.java 2013-04-12 10:09:55 UTC (rev 92)
@@ -2,6 +2,7 @@
import javax.persistence.Entity;
import javax.persistence.Id;
+import javax.persistence.ManyToOne;
import java.util.Date;
@Entity
@@ -10,6 +11,9 @@
@Id
protected String id;
+ @ManyToOne
+ protected Building building;
+
protected String code;
protected Date requestDate;
@@ -48,4 +52,12 @@
public void setCode(String code) {
this.code = code;
}
+
+ public Building getBuilding() {
+ return building;
+ }
+
+ public void setBuilding(Building building) {
+ this.building = building;
+ }
}
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-12 09:24:13 UTC (rev 91)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/RequestedArticleService.java 2013-04-12 10:09:55 UTC (rev 92)
@@ -87,8 +87,6 @@
log.info(allRequestedArticles.size() + " articles requested in building " + building.getCode());
}
- // TODO brendan 05/04/13 consider warehouse.code
-
return allRequestedArticles;
}
Modified: trunk/magalie-services/src/main/resources/fixtures2.yaml
===================================================================
--- trunk/magalie-services/src/main/resources/fixtures2.yaml 2013-04-12 09:24:13 UTC (rev 91)
+++ trunk/magalie-services/src/main/resources/fixtures2.yaml 2013-04-12 10:09:55 UTC (rev 92)
@@ -199,6 +199,7 @@
code: 987654321
requestDate: 2013-04-08 12:00:00
urgent: true
+ building: *B1
requestedList2:
&requestedList2 !requested-list
@@ -206,6 +207,7 @@
code: 123456789
requestDate: 2013-04-08 12:00:00
urgent: false
+ building: *B1
requestedLists:
- *requestedList1
1
0
12 Apr '13
Author: bleny
Date: 2013-04-12 11:24:13 +0200 (Fri, 12 Apr 2013)
New Revision: 91
Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision…
Log:
introduce requested lists
Added:
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/RequestedListDao.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/RequestedListJpaDao.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/RequestedList.java
Modified:
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/RequestedArticles.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/RequestedArticle.java
trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/MagalieFixtures.java
trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/FixturesService.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/main/resources/fixtures2.yaml
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-11 21:40:01 UTC (rev 90)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/JpaMagaliePersistenceContext.java 2013-04-12 09:24:13 UTC (rev 91)
@@ -7,6 +7,7 @@
import com.franciaflex.magalie.persistence.dao.RequestedArticleDao;
import com.franciaflex.magalie.persistence.dao.LocationDao;
import com.franciaflex.magalie.persistence.dao.LocationErrorDao;
+import com.franciaflex.magalie.persistence.dao.RequestedListDao;
import com.franciaflex.magalie.persistence.dao.StorageMovementDao;
import com.franciaflex.magalie.persistence.dao.StorageMovementOrderDao;
import com.franciaflex.magalie.persistence.dao.WarehouseDao;
@@ -18,6 +19,7 @@
import com.franciaflex.magalie.persistence.dao.jpa.LocationJpaDao;
import com.franciaflex.magalie.persistence.dao.jpa.MagalieUserJpaDao;
import com.franciaflex.magalie.persistence.dao.jpa.RequestedArticleJpaDao;
+import com.franciaflex.magalie.persistence.dao.jpa.RequestedListJpaDao;
import com.franciaflex.magalie.persistence.dao.jpa.StorageMovementJpaDao;
import com.franciaflex.magalie.persistence.dao.jpa.StorageMovementOrderJpaDao;
import com.franciaflex.magalie.persistence.dao.jpa.WarehouseJpaDao;
@@ -41,6 +43,27 @@
}
@Override
+ public void commit() {
+ entityTransaction.commit();
+ entityTransaction = entityManager.getTransaction();
+ entityTransaction.begin();
+ }
+
+ @Override
+ public void rollback() {
+ entityTransaction.rollback();
+ entityTransaction = entityManager.getTransaction();
+ entityTransaction.begin();
+ }
+
+ @Override
+ public void clearDatabase() {
+ rollback();
+ JpaUtil.cleanDatabase(entityManager);
+ commit();
+ }
+
+ @Override
public MagalieUserDao getMagalieUserDao() {
return new MagalieUserJpaDao(entityManager);
}
@@ -96,23 +119,8 @@
}
@Override
- public void commit() {
- entityTransaction.commit();
- entityTransaction = entityManager.getTransaction();
- entityTransaction.begin();
+ public RequestedListDao getRequestedListDao() {
+ return new RequestedListJpaDao(entityManager);
}
- @Override
- public void rollback() {
- entityTransaction.rollback();
- entityTransaction = entityManager.getTransaction();
- entityTransaction.begin();
- }
-
- @Override
- public void clearDatabase() {
- rollback();
- JpaUtil.cleanDatabase(entityManager);
- commit();
- }
}
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-11 21:40:01 UTC (rev 90)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/MagaliePersistenceContext.java 2013-04-12 09:24:13 UTC (rev 91)
@@ -3,14 +3,15 @@
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.LocationDao;
+import com.franciaflex.magalie.persistence.dao.LocationErrorDao;
import com.franciaflex.magalie.persistence.dao.MagalieUserDao;
import com.franciaflex.magalie.persistence.dao.RequestedArticleDao;
-import com.franciaflex.magalie.persistence.dao.LocationDao;
-import com.franciaflex.magalie.persistence.dao.LocationErrorDao;
+import com.franciaflex.magalie.persistence.dao.RequestedListDao;
import com.franciaflex.magalie.persistence.dao.StorageMovementDao;
import com.franciaflex.magalie.persistence.dao.StorageMovementOrderDao;
+import com.franciaflex.magalie.persistence.dao.StoredArticleDao;
import com.franciaflex.magalie.persistence.dao.WarehouseDao;
-import com.franciaflex.magalie.persistence.dao.StoredArticleDao;
/**
* @author bleny
@@ -44,4 +45,7 @@
BuildingDao getBuildingDao();
WarehouseDao getWarehouseDao();
+
+ RequestedListDao getRequestedListDao();
+
}
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-04-11 21:40:01 UTC (rev 90)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/RequestedArticles.java 2013-04-12 09:24:13 UTC (rev 91)
@@ -1,18 +1,25 @@
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 final class UrgentFirstComparator implements Comparator<RequestedArticle> {
+ protected static class UrgentFirstComparator implements Comparator<RequestedArticle> {
@Override
public int compare(RequestedArticle x, RequestedArticle y) {
- return BooleanComparator.getTrueFirstComparator().compare(x.isUrgent(), y.isUrgent());
+ return BooleanComparator.getTrueFirstComparator().compare(x.getRequestedList().isUrgent(), y.getRequestedList().isUrgent());
}
}
@@ -21,32 +28,41 @@
@Override
public int compare(RequestedArticle x, RequestedArticle y) {
- return x.getRequestDate().compareTo(y.getRequestDate());
+ return x.getRequestedList().getRequestDate().compareTo(y.getRequestedList().getRequestDate());
}
}
protected static class FinishEarlyFirstComparator implements Comparator<RequestedArticle> {
- protected Map<String, Integer> requestListToArticleRemainingCount;
+ protected Map<RequestedList, Integer> requestListToArticleRemainingCount;
- // TODO brendan 05/04/13 something to do here
+ public FinishEarlyFirstComparator(Map<RequestedList, Integer> requestListToArticleRemainingCount) {
+ this.requestListToArticleRemainingCount = requestListToArticleRemainingCount;
+ }
protected int getRemainingCount(RequestedArticle requestedArticle) {
- Integer remainingCount = requestListToArticleRemainingCount.get(requestedArticle.getRequestList());
- return remainingCount == null ? Integer.MAX_VALUE : remainingCount;
+ Integer remainingCount = requestListToArticleRemainingCount.get(requestedArticle.getRequestedList());
+ return remainingCount;
}
@Override
public int compare(RequestedArticle x, RequestedArticle y) {
- return 0;
-// int xRemainingCount = getRemainingCount(x);
-// int yRemainingCount = getRemainingCount(y);
-// return xRemainingCount - yRemainingCount;
+ 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();
}
@@ -55,7 +71,28 @@
return new RequestDateFistComparator();
}
- public static Comparator<RequestedArticle> finishEarlyFirstComparator() {
- return new FinishEarlyFirstComparator();
+ 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/dao/RequestedListDao.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/RequestedListDao.java (rev 0)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/RequestedListDao.java 2013-04-12 09:24:13 UTC (rev 91)
@@ -0,0 +1,10 @@
+package com.franciaflex.magalie.persistence.dao;
+
+import com.franciaflex.magalie.persistence.entity.RequestedList;
+
+/**
+ * @author bleny
+ */
+public interface RequestedListDao extends Dao<RequestedList> {
+
+}
Added: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/RequestedListJpaDao.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/RequestedListJpaDao.java (rev 0)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/RequestedListJpaDao.java 2013-04-12 09:24:13 UTC (rev 91)
@@ -0,0 +1,19 @@
+package com.franciaflex.magalie.persistence.dao.jpa;
+
+import com.franciaflex.magalie.persistence.dao.RequestedListDao;
+import com.franciaflex.magalie.persistence.entity.RequestedList;
+
+import javax.persistence.EntityManager;
+
+public class RequestedListJpaDao extends AbstractJpaDao<RequestedList> implements RequestedListDao {
+
+ public RequestedListJpaDao(EntityManager entityManager) {
+ super(entityManager);
+ }
+
+ @Override
+ protected Class<RequestedList> getEntityClass() {
+ return RequestedList.class;
+ }
+
+}
Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/RequestedArticle.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/RequestedArticle.java 2013-04-11 21:40:01 UTC (rev 90)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/RequestedArticle.java 2013-04-12 09:24:13 UTC (rev 91)
@@ -3,7 +3,6 @@
import javax.persistence.Entity;
import javax.persistence.Id;
import javax.persistence.ManyToOne;
-import java.util.Date;
@Entity
public class RequestedArticle extends AbstractEntity {
@@ -14,12 +13,9 @@
@ManyToOne
protected Article article;
- protected String requestList;
+ @ManyToOne
+ protected RequestedList requestedList;
- protected Date requestDate;
-
- protected boolean urgent;
-
protected double quantity;
@Override
@@ -31,30 +27,6 @@
this.id = id;
}
- public String getRequestList() {
- return requestList;
- }
-
- public void setRequestList(String requestList) {
- this.requestList = requestList;
- }
-
- public Date getRequestDate() {
- return requestDate;
- }
-
- public void setRequestDate(Date requestDate) {
- this.requestDate = requestDate;
- }
-
- public boolean isUrgent() {
- return urgent;
- }
-
- public void setUrgent(boolean urgent) {
- this.urgent = urgent;
- }
-
public Article getArticle() {
return article;
}
@@ -70,4 +42,12 @@
public void setQuantity(double quantity) {
this.quantity = quantity;
}
+
+ public RequestedList getRequestedList() {
+ return requestedList;
+ }
+
+ public void setRequestedList(RequestedList requestedList) {
+ this.requestedList = requestedList;
+ }
}
Added: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/RequestedList.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/RequestedList.java (rev 0)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/RequestedList.java 2013-04-12 09:24:13 UTC (rev 91)
@@ -0,0 +1,51 @@
+package com.franciaflex.magalie.persistence.entity;
+
+import javax.persistence.Entity;
+import javax.persistence.Id;
+import java.util.Date;
+
+@Entity
+public class RequestedList extends AbstractEntity {
+
+ @Id
+ protected String id;
+
+ protected String code;
+
+ protected Date requestDate;
+
+ protected boolean urgent;
+
+ @Override
+ public String getId() {
+ return id;
+ }
+
+ public void setId(String id) {
+ this.id = id;
+ }
+
+ public Date getRequestDate() {
+ return requestDate;
+ }
+
+ public void setRequestDate(Date requestDate) {
+ this.requestDate = requestDate;
+ }
+
+ public boolean isUrgent() {
+ return urgent;
+ }
+
+ public void setUrgent(boolean urgent) {
+ this.urgent = urgent;
+ }
+
+ public String getCode() {
+ return code;
+ }
+
+ public void setCode(String code) {
+ this.code = code;
+ }
+}
Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/MagalieFixtures.java
===================================================================
--- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/MagalieFixtures.java 2013-04-11 21:40:01 UTC (rev 90)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/MagalieFixtures.java 2013-04-12 09:24:13 UTC (rev 91)
@@ -8,6 +8,7 @@
import com.franciaflex.magalie.persistence.entity.MagalieUser;
import com.franciaflex.magalie.persistence.entity.RequestedArticle;
import com.franciaflex.magalie.persistence.entity.Location;
+import com.franciaflex.magalie.persistence.entity.RequestedList;
import com.franciaflex.magalie.persistence.entity.Warehouse;
import com.franciaflex.magalie.persistence.entity.StoredArticle;
import org.apache.commons.io.Charsets;
@@ -37,6 +38,7 @@
reader.getConfig().setClassTag("stored-article", StoredArticle.class);
reader.getConfig().setClassTag("user", MagalieUser.class);
reader.getConfig().setClassTag("requested-article", RequestedArticle.class);
+ reader.getConfig().setClassTag("requested-list", RequestedList.class);
reader.getConfig().setClassTag("location", Location.class);
try {
fixtures = (Map<String, Object>) reader.read();
Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/FixturesService.java
===================================================================
--- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/FixturesService.java 2013-04-11 21:40:01 UTC (rev 90)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/FixturesService.java 2013-04-12 09:24:13 UTC (rev 91)
@@ -3,18 +3,20 @@
import com.franciaflex.magalie.persistence.MagaliePersistenceContext;
import com.franciaflex.magalie.persistence.dao.ArticleDao;
import com.franciaflex.magalie.persistence.dao.BuildingDao;
+import com.franciaflex.magalie.persistence.dao.LocationDao;
import com.franciaflex.magalie.persistence.dao.MagalieUserDao;
import com.franciaflex.magalie.persistence.dao.RequestedArticleDao;
-import com.franciaflex.magalie.persistence.dao.LocationDao;
+import com.franciaflex.magalie.persistence.dao.RequestedListDao;
+import com.franciaflex.magalie.persistence.dao.StoredArticleDao;
import com.franciaflex.magalie.persistence.dao.WarehouseDao;
-import com.franciaflex.magalie.persistence.dao.StoredArticleDao;
import com.franciaflex.magalie.persistence.entity.Article;
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.RequestedArticle;
-import com.franciaflex.magalie.persistence.entity.Location;
+import com.franciaflex.magalie.persistence.entity.RequestedList;
+import com.franciaflex.magalie.persistence.entity.StoredArticle;
import com.franciaflex.magalie.persistence.entity.Warehouse;
-import com.franciaflex.magalie.persistence.entity.StoredArticle;
import com.franciaflex.magalie.services.MagalieFixtures;
import com.franciaflex.magalie.services.MagalieService;
import com.franciaflex.magalie.services.MagalieServiceContext;
@@ -145,6 +147,18 @@
persistenceContext.commit();
+ RequestedListDao requestedListDao = persistenceContext.getRequestedListDao();
+
+ Collection<RequestedList> requestedLists = fixtures.fixture("requestedLists");
+
+ for (RequestedList requestedList : requestedLists) {
+
+ requestedListDao.persist(requestedList);
+
+ }
+
+ persistenceContext.commit();
+
RequestedArticleDao requestedArticleDao = persistenceContext.getRequestedArticleDao();
Collection<RequestedArticle> requestedArticles = fixtures.fixture("requestedArticles");
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-11 21:40:01 UTC (rev 90)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/RequestedArticleService.java 2013-04-12 09:24:13 UTC (rev 91)
@@ -189,7 +189,7 @@
Comparator<RequestedArticle> priorityComparator =
Ordering.compound(
Lists.newArrayList(
- RequestedArticles.finishEarlyFirstComparator(),
+ RequestedArticles.finishEarlyFirstComparator(requests),
RequestedArticles.urgentFirstComparator(),
RequestedArticles.requestDateFirstComparator()
)
Modified: trunk/magalie-services/src/main/resources/fixtures.yaml
===================================================================
--- trunk/magalie-services/src/main/resources/fixtures.yaml 2013-04-11 21:40:01 UTC (rev 90)
+++ trunk/magalie-services/src/main/resources/fixtures.yaml 2013-04-12 09:24:13 UTC (rev 91)
@@ -385,61 +385,66 @@
storedArticlesForArticle6:
- *storedArticle13
+requestedList1:
+ &requestedList1 !requested-list
+ id: requestedList1
+ code: 123456789
+ requestDate: 2013-04-08 12:00:00
+ urgent: true
+
+requestedList2:
+ &requestedList2 !requested-list
+ id: requestedList2
+ code: 123456789
+ requestDate: 2013-04-08 12:00:00
+ urgent: false
+
+requestedLists:
+ - *requestedList1
+ - *requestedList2
+
requestedArticle1:
&requestedArticle1 !requested-article
id: requestedArticle1
article: *article1
quantity: 80
- requestList: 123456789
- requestDate: 2013-04-08 12:00:00
- urgent: true
+ requestedList: *requestedList1
requestedArticle2:
&requestedArticle2 !requested-article
id: requestedArticle2
article: *article2
quantity: 11
- requestList: 123456789
- requestDate: 2013-04-08 12:00:00
- urgent: false
+ requestedList: *requestedList1
requestedArticle3:
&requestedArticle3 !requested-article
id: requestedArticle3
article: *article3
quantity: 111
- requestList: 123456789
- requestDate: 2013-04-08 12:00:00
- urgent: true
+ requestedList: *requestedList1
requestedArticle4:
&requestedArticle4 !requested-article
id: requestedArticle4
article: *article4
quantity: 2
- requestList: 123456789
- requestDate: 2013-04-08 12:00:00
- urgent: true
+ requestedList: *requestedList1
requestedArticle5:
&requestedArticle5 !requested-article
id: requestedArticle5
article: *article5
quantity: 5
- requestList: 987654321
- requestDate: 2013-04-08 12:00:00
- urgent: true
+ requestedList: *requestedList2
requestedArticle6:
&requestedArticle6 !requested-article
id: requestedArticle6
article: *article6
quantity: 27
- requestList: 987654321
- requestDate: 2013-04-08 12:00:00
- urgent: true
+ requestedList: *requestedList2
-
requestedArticles:
- *requestedArticle1
- *requestedArticle2
Modified: trunk/magalie-services/src/main/resources/fixtures2.yaml
===================================================================
--- trunk/magalie-services/src/main/resources/fixtures2.yaml 2013-04-11 21:40:01 UTC (rev 90)
+++ trunk/magalie-services/src/main/resources/fixtures2.yaml 2013-04-12 09:24:13 UTC (rev 91)
@@ -193,23 +193,37 @@
- *storedArticle1
- *storedArticle2
+requestedList1:
+ &requestedList1 !requested-list
+ id: requestedList1
+ code: 987654321
+ requestDate: 2013-04-08 12:00:00
+ urgent: true
+
+requestedList2:
+ &requestedList2 !requested-list
+ id: requestedList2
+ code: 123456789
+ requestDate: 2013-04-08 12:00:00
+ urgent: false
+
+requestedLists:
+ - *requestedList1
+ - *requestedList2
+
requestedArticle1:
&requestedArticle1 !requested-article
id: requestedArticle1
article: *article3
quantity: 100
- requestList: 123456789
- requestDate: 2013-04-08 12:00:00
- urgent: true
+ requestedList: *requestedList1
requestedArticle2:
&requestedArticle2 !requested-article
id: requestedArticle2
article: *article3
quantity: 250
- requestList: 123456789
- requestDate: 2013-04-08 12:00:00
- urgent: false
+ requestedList: *requestedList2
requestedArticles:
- *requestedArticle1
1
0
r90 - 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 11 Apr '13
by bleny@users.forge.codelutin.com 11 Apr '13
11 Apr '13
Author: bleny
Date: 2013-04-11 23:40:01 +0200 (Thu, 11 Apr 2013)
New Revision: 90
Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision…
Log:
introduce BookArticleRequest
Added:
trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/BookArticleRequest.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/FulfilKanbanService.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/ArticleStorageServiceTest.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-04-11 21:07:14 UTC (rev 89)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ArticleStorageService.java 2013-04-11 21:40:01 UTC (rev 90)
@@ -45,52 +45,26 @@
this.serviceContext = serviceContext;
}
- public StorageMovementOrder bookArticle(MagalieUser magalieUser, Building building, Article article, double requestedQuantity) throws UnavailableArticleException, RequiredDriverLicenceException {
-
- return bookArticle(magalieUser, building, article, requestedQuantity, null);
-
- }
-
- public StorageMovementOrder bookArticleWithBestEffortPolicy(
- MagalieUser magalieUser, Building building, Article article, double requestedQuantity, Location destinationLocation)
- throws RequiredDriverLicenceException, UnavailableArticleException {
-
- return bookArticle(magalieUser, building, article ,requestedQuantity, destinationLocation, true);
-
- }
-
- public StorageMovementOrder bookArticle(
- MagalieUser magalieUser, Building building, Article article, double requestedQuantity, Location destinationLocation)
+ public StorageMovementOrder bookArticle(BookArticleRequest bookArticleRequest)
throws UnavailableArticleException, RequiredDriverLicenceException {
- return bookArticle(magalieUser, building, article ,requestedQuantity, destinationLocation, false);
+ Preconditions.checkNotNull(bookArticleRequest.getMagalieUser());
- }
+ Preconditions.checkNotNull(bookArticleRequest.getBuilding());
- protected StorageMovementOrder bookArticle(
- MagalieUser magalieUser, Building building, Article article, double requestedQuantity, Location destinationLocation, boolean bestEffortPolicy)
- throws UnavailableArticleException, RequiredDriverLicenceException {
+ Preconditions.checkNotNull(bookArticleRequest.getArticle());
- Preconditions.checkNotNull(magalieUser);
+ Preconditions.checkArgument(bookArticleRequest.getRequestedQuantity() > 0.);
- Preconditions.checkNotNull(building);
-
- Preconditions.checkNotNull(article);
-
- Preconditions.checkArgument(requestedQuantity > 0.);
-
if (log.isInfoEnabled()) {
- log.info("will process article request"
- + ": requested = " + requestedQuantity
- );
+ log.info("will process article request" + bookArticleRequest);
}
- List<StoredArticle> storedArticles = getStoredArticles(building, article);
+ List<StoredArticle> storedArticles = getStoredArticles(bookArticleRequest.getBuilding(), bookArticleRequest.getArticle());
Queue<StoredArticle> sortedStoredArticles = sortStoredArticlesByPriority(storedArticles);
- StorageMovementOrder storageMovementOrder = buildStorageMovementOrder(
- magalieUser, article, requestedQuantity, destinationLocation, bestEffortPolicy, sortedStoredArticles);
+ StorageMovementOrder storageMovementOrder = buildStorageMovementOrder(bookArticleRequest, sortedStoredArticles);
MagaliePersistenceContext persistenceContext = serviceContext.getPersistenceContext();
@@ -104,8 +78,14 @@
}
- protected StorageMovementOrder buildStorageMovementOrder(MagalieUser magalieUser, Article article, double requestedQuantity, Location destinationLocation, boolean bestEffortPolicy, Queue<StoredArticle> sortedStoredArticles) throws RequiredDriverLicenceException, UnavailableArticleException {
+ protected StorageMovementOrder buildStorageMovementOrder(BookArticleRequest bookArticleRequest, Queue<StoredArticle> sortedStoredArticles) throws RequiredDriverLicenceException, UnavailableArticleException {
+ Article article = bookArticleRequest.getArticle();
+
+ MagalieUser magalieUser = bookArticleRequest.getMagalieUser();
+
+ double requestedQuantity = bookArticleRequest.getRequestedQuantity();
+
StorageMovementOrder storageMovementOrder = new StorageMovementOrder();
double quantity = 0;
@@ -138,7 +118,7 @@
newStorageMovement.setOriginLocation(originLocation);
- newStorageMovement.setDestinationLocation(destinationLocation);
+ newStorageMovement.setDestinationLocation(bookArticleRequest.getDestinationLocation());
newStorageMovement.setExpectedQuantity(quantityWithdrawn);
@@ -147,7 +127,7 @@
}
}
- if (bestEffortPolicy) {
+ if (bookArticleRequest.isBestEffortPolicy()) {
log.debug("best effort policy, requested quantity was "
+ requestedQuantity + " but only " + quantity +
Added: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/BookArticleRequest.java
===================================================================
--- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/BookArticleRequest.java (rev 0)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/BookArticleRequest.java 2013-04-11 21:40:01 UTC (rev 90)
@@ -0,0 +1,60 @@
+package com.franciaflex.magalie.services.service;
+
+import com.franciaflex.magalie.persistence.entity.Article;
+import com.franciaflex.magalie.persistence.entity.Building;
+import com.franciaflex.magalie.persistence.entity.Location;
+import com.franciaflex.magalie.persistence.entity.MagalieUser;
+import org.apache.commons.lang3.builder.ReflectionToStringBuilder;
+
+public class BookArticleRequest {
+
+ protected MagalieUser magalieUser;
+
+ protected Building building;
+
+ protected Article article;
+
+ protected double requestedQuantity;
+
+ protected Location destinationLocation;
+
+ protected boolean bestEffortPolicy;
+
+ public BookArticleRequest(MagalieUser magalieUser, Building building, Article article, double requestedQuantity, Location destinationLocation, boolean bestEffortPolicy) {
+ this.magalieUser = magalieUser;
+ this.building = building;
+ this.article = article;
+ this.requestedQuantity = requestedQuantity;
+ this.destinationLocation = destinationLocation;
+ this.bestEffortPolicy = bestEffortPolicy;
+ }
+
+ public MagalieUser getMagalieUser() {
+ return magalieUser;
+ }
+
+ public Building getBuilding() {
+ return building;
+ }
+
+ public Article getArticle() {
+ return article;
+ }
+
+ public double getRequestedQuantity() {
+ return requestedQuantity;
+ }
+
+ public Location getDestinationLocation() {
+ return destinationLocation;
+ }
+
+ public boolean isBestEffortPolicy() {
+ return bestEffortPolicy;
+ }
+
+ @Override
+ public String toString() {
+ return ReflectionToStringBuilder.reflectionToString(this);
+ }
+}
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-11 21:07:14 UTC (rev 89)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/FulfilKanbanService.java 2013-04-11 21:40:01 UTC (rev 90)
@@ -70,13 +70,13 @@
Location destinationLocation = getDestinationLocation(destinationWarehouse);
+ BookArticleRequest bookArticleRequest =
+ new BookArticleRequest(
+ magalieUser, building, article,
+ quantityInKanban, destinationLocation, true);
+
StorageMovementOrder storageMovementOrder =
- articleStorageService.bookArticleWithBestEffortPolicy(
- magalieUser,
- building,
- article,
- quantityInKanban,
- destinationLocation);
+ articleStorageService.bookArticle(bookArticleRequest);
return storageMovementOrder;
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-11 21:07:14 UTC (rev 89)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/RequestedArticleService.java 2013-04-11 21:40:01 UTC (rev 90)
@@ -118,9 +118,11 @@
double quantity = requestedArticle.getQuantity();
+ BookArticleRequest bookArticleRequest = new BookArticleRequest(magalieUser, building, article, quantity, null, false);
+
try {
- storageMovementOrder = articleStorageService.bookArticle(magalieUser, building, article, quantity, null);
+ storageMovementOrder = articleStorageService.bookArticle(bookArticleRequest);
everythingIsUnavailable = false;
Modified: trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/ArticleStorageServiceTest.java
===================================================================
--- trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/ArticleStorageServiceTest.java 2013-04-11 21:07:14 UTC (rev 89)
+++ trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/ArticleStorageServiceTest.java 2013-04-11 21:40:01 UTC (rev 90)
@@ -40,6 +40,24 @@
}
+ protected StorageMovementOrder bookArticle(MagalieUser magalieUser, Building building, Article article, double requestedQuantity) throws UnavailableArticleException, RequiredDriverLicenceException {
+
+ BookArticleRequest bookArticleRequest = new BookArticleRequest(magalieUser, building, article, requestedQuantity, null, false);
+
+ return service.bookArticle(bookArticleRequest);
+
+ }
+
+ public StorageMovementOrder bookArticleWithBestEffortPolicy(
+ MagalieUser magalieUser, Building building, Article article, double requestedQuantity, Location destinationLocation)
+ throws RequiredDriverLicenceException, UnavailableArticleException {
+
+ BookArticleRequest bookArticleRequest = new BookArticleRequest(magalieUser, building, article, requestedQuantity, null, true);
+
+ return service.bookArticle(bookArticleRequest);
+
+ }
+
@Test
public void testThrowUnavailableArticle() throws Exception {
@@ -48,7 +66,7 @@
StorageMovementOrder storageMovementOrder;
try {
- storageMovementOrder = service.bookArticle(alexandre, building, article, 100.);
+ storageMovementOrder = bookArticle(alexandre, building, article, 100.);
if (log.isDebugEnabled()) {
log.debug("storageMovementOrder = " + storageMovementOrder);
}
@@ -70,7 +88,7 @@
StorageMovementOrder storageMovementOrder;
try {
- storageMovementOrder = service.bookArticle(basile, building, article, 70.);
+ storageMovementOrder = bookArticle(basile, building, article, 70.);
if (log.isDebugEnabled()) {
log.debug("storageMovementOrder = " + storageMovementOrder);
}
@@ -91,7 +109,7 @@
StorageMovementOrder storageMovementOrder = null;
try {
- storageMovementOrder = service.bookArticle(basile, building, article, 5.);
+ storageMovementOrder = bookArticle(basile, building, article, 5.);
if (log.isDebugEnabled()) {
log.debug("storageMovementOrder = " + storageMovementOrder);
}
@@ -118,7 +136,7 @@
StorageMovementOrder storageMovementOrder = null;
try {
- storageMovementOrder = service.bookArticle(alexandre, building, article, 50.);
+ storageMovementOrder = bookArticle(alexandre, building, article, 50.);
if (log.isDebugEnabled()) {
log.debug("storageMovementOrder = " + storageMovementOrder);
}
@@ -152,7 +170,7 @@
StorageMovementOrder storageMovementOrder = null;
try {
- storageMovementOrder = service.bookArticle(alexandre, building, article, 30.);
+ storageMovementOrder = bookArticle(alexandre, building, article, 30.);
if (log.isDebugEnabled()) {
log.debug("storageMovementOrder = " + storageMovementOrder);
}
@@ -176,7 +194,7 @@
}
try {
- storageMovementOrder = service.bookArticle(cathy, building, article, 30.);
+ storageMovementOrder = bookArticle(cathy, building, article, 30.);
if (log.isDebugEnabled()) {
log.debug("storageMovementOrder = " + storageMovementOrder);
}
@@ -211,7 +229,7 @@
locationErrorsService.reportError(location2, alexandre);
try {
- StorageMovementOrder storageMovementOrder = service.bookArticle(cathy, building, article, 30.);
+ StorageMovementOrder storageMovementOrder = bookArticle(cathy, building, article, 30.);
if (log.isDebugEnabled()) {
log.debug("storageMovementOrder = " + storageMovementOrder);
}
@@ -242,7 +260,7 @@
Location location1 = fixture("location1");
try {
- StorageMovementOrder storageMovementOrder = service.bookArticleWithBestEffortPolicy(alexandre, building, article, 30., null);
+ StorageMovementOrder storageMovementOrder = bookArticleWithBestEffortPolicy(alexandre, building, article, 30., null);
if (log.isDebugEnabled()) {
log.debug("storageMovementOrder = " + storageMovementOrder);
}
1
0
r89 - 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 magalie-services/src/main/resources
by bleny@users.forge.codelutin.com 11 Apr '13
by bleny@users.forge.codelutin.com 11 Apr '13
11 Apr '13
Author: bleny
Date: 2013-04-11 23:07:14 +0200 (Thu, 11 Apr 2013)
New Revision: 89
Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision…
Log:
introduce fixedLocationsFirst in Article
Modified:
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/StoredArticles.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Article.java
trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ArticleStorageService.java
trunk/magalie-services/src/main/resources/fixtures.yaml
trunk/magalie-services/src/main/resources/fixtures2.yaml
Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/StoredArticles.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/StoredArticles.java 2013-04-11 20:24:53 UTC (rev 88)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/StoredArticles.java 2013-04-11 21:07:14 UTC (rev 89)
@@ -5,6 +5,7 @@
import com.franciaflex.magalie.persistence.entity.Location;
import com.franciaflex.magalie.persistence.entity.StoredArticle;
import com.google.common.base.Function;
+import com.google.common.base.Preconditions;
import com.google.common.base.Predicate;
import com.google.common.base.Predicates;
import com.google.common.collect.Ordering;
@@ -15,19 +16,19 @@
public class StoredArticles {
- protected static class FixedLocationForArticleFirst implements Comparator<StoredArticle> {
+ /**
+ * Put fixed sites for given article first, or last according to the value of
+ * {@link com.franciaflex.magalie.persistence.entity.Article#isFixedLocationsFirst()}
+ */
+ protected static class FixedLocationForArticle implements Comparator<StoredArticle> {
- protected Article article;
-
- public FixedLocationForArticleFirst(Article article) {
- this.article = article;
- }
-
@Override
public int compare(StoredArticle x, StoredArticle y) {
+ Article article = x.getArticle();
+ Preconditions.checkArgument(article.equals(y.getArticle()));
boolean xIsInFixedLocation = article.isFixedLocation(x.getLocation());
boolean yIsInFixedLocation = article.isFixedLocation(y.getLocation());
- return BooleanComparator.getTrueFirstComparator().compare(xIsInFixedLocation, yIsInFixedLocation);
+ return BooleanComparator.getBooleanComparator( ! article.isFixedLocationsFirst()).compare(xIsInFixedLocation, yIsInFixedLocation);
}
}
@@ -94,14 +95,10 @@
return new AccessibleLocationFirst(magalieUser);
}
- public static Comparator<StoredArticle> fixedLocationForArticleFirstComparator(Article article) {
- return new FixedLocationForArticleFirst(article);
+ public static Comparator<StoredArticle> fixedLocationForArticleComparator() {
+ return Ordering.from(new FixedLocationForArticle()).reverse();
}
- public static Comparator<StoredArticle> fixedLocationForArticleLastComparator(Article article) {
- return Ordering.from(new FixedLocationForArticleFirst(article)).reverse();
- }
-
public static Function<StoredArticle, Location> getLocationFunction() {
return new GetLocationFunction();
}
Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Article.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Article.java 2013-04-11 20:24:53 UTC (rev 88)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Article.java 2013-04-11 21:07:14 UTC (rev 89)
@@ -22,6 +22,8 @@
protected String unit;
+ protected boolean fixedLocationsFirst;
+
@Override
public String getId() {
return id;
@@ -79,4 +81,12 @@
}
return isFixedLocation;
}
+
+ public boolean isFixedLocationsFirst() {
+ return fixedLocationsFirst;
+ }
+
+ public void setFixedLocationsFirst(boolean fixedLocationsFirst) {
+ this.fixedLocationsFirst = fixedLocationsFirst;
+ }
}
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-11 20:24:53 UTC (rev 88)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ArticleStorageService.java 2013-04-11 21:07:14 UTC (rev 89)
@@ -25,13 +25,14 @@
import com.google.common.collect.Lists;
import com.google.common.collect.Maps;
import com.google.common.collect.Ordering;
-import com.google.common.collect.Sets;
import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
import java.util.Date;
import java.util.List;
import java.util.Map;
+import java.util.PriorityQueue;
+import java.util.Queue;
import java.util.Set;
public class ArticleStorageService implements MagalieService {
@@ -86,20 +87,25 @@
List<StoredArticle> storedArticles = getStoredArticles(building, article);
- Ordering<StoredArticle> storedArticlesOrdering =
- Ordering.compound(
- Lists.newArrayList(
- StoredArticles.fixedLocationForArticleLastComparator(article),
- StoredArticles.locationWithLowestQuantityFirstComparator(),
- StoredArticles.articleStoredInLocationsRequiringDriverLicenseFirstComparator()
- )
- );
+ Queue<StoredArticle> sortedStoredArticles = sortStoredArticlesByPriority(storedArticles);
- Set<StoredArticle> sortedStoredArticles =
- Sets.newTreeSet(storedArticlesOrdering);
+ StorageMovementOrder storageMovementOrder = buildStorageMovementOrder(
+ magalieUser, article, requestedQuantity, destinationLocation, bestEffortPolicy, sortedStoredArticles);
- Iterables.addAll(sortedStoredArticles, storedArticles);
+ MagaliePersistenceContext persistenceContext = serviceContext.getPersistenceContext();
+ StorageMovementOrderDao dao = persistenceContext.getStorageMovementOrderDao();
+
+ dao.persist(storageMovementOrder);
+
+ persistenceContext.commit();
+
+ return storageMovementOrder;
+
+ }
+
+ protected StorageMovementOrder buildStorageMovementOrder(MagalieUser magalieUser, Article article, double requestedQuantity, Location destinationLocation, boolean bestEffortPolicy, Queue<StoredArticle> sortedStoredArticles) throws RequiredDriverLicenceException, UnavailableArticleException {
+
StorageMovementOrder storageMovementOrder = new StorageMovementOrder();
double quantity = 0;
@@ -161,16 +167,30 @@
}
- MagaliePersistenceContext persistenceContext = serviceContext.getPersistenceContext();
+ return storageMovementOrder;
- StorageMovementOrderDao dao = persistenceContext.getStorageMovementOrderDao();
+ }
- dao.persist(storageMovementOrder);
+ protected Queue<StoredArticle> sortStoredArticlesByPriority(List<StoredArticle> storedArticles) {
- persistenceContext.commit();
+ Ordering<StoredArticle> orderingByPriority = Ordering.compound(
+ Lists.newArrayList(
+ StoredArticles.fixedLocationForArticleComparator(),
+ StoredArticles.locationWithLowestQuantityFirstComparator(),
+ StoredArticles.articleStoredInLocationsRequiringDriverLicenseFirstComparator()
+ )
+ );
- return storageMovementOrder;
+ Queue<StoredArticle> storedArticlesByPriority =
+ new PriorityQueue<StoredArticle>(
+ storedArticles.size(),
+ orderingByPriority
+ );
+ storedArticlesByPriority.addAll(storedArticles);
+
+ return storedArticlesByPriority;
+
}
public StorageMovementTask getStorageMovementTask(String storageMovementOrderId) {
Modified: trunk/magalie-services/src/main/resources/fixtures.yaml
===================================================================
--- trunk/magalie-services/src/main/resources/fixtures.yaml 2013-04-11 20:24:53 UTC (rev 88)
+++ trunk/magalie-services/src/main/resources/fixtures.yaml 2013-04-11 21:07:14 UTC (rev 89)
@@ -193,6 +193,7 @@
quantityInKanban: 50
unit: M
description: Jonc Souple D5
+ fixedLocationsFirst: false
article2:
&article2 !article
@@ -203,6 +204,7 @@
quantityInKanban: 3
unit: PCE
description: Moteur Oréa Slt 50 RTS 50/12
+ fixedLocationsFirst: false
article3:
&article3 !article
@@ -213,6 +215,7 @@
quantityInKanban: 50
unit: PCE
description: Interface ASA/SOMFY + clip
+ fixedLocationsFirst: false
article4:
&article4 !article
@@ -221,6 +224,7 @@
quantityInKanban: 20
unit: PCE
description: Kit Tr.PM Fixe 122G D40Plat 10
+ fixedLocationsFirst: false
article5:
&article5 !article
@@ -231,6 +235,7 @@
quantityInKanban: 10
unit: B60
description: Caisson 2005 3F 6000 Brun
+ fixedLocationsFirst: false
article6:
&article6 !article
@@ -241,6 +246,7 @@
quantityInKanban: 9
unit: PCE
description: Ensemble SOLIRIS UNO
+ fixedLocationsFirst: false
articles:
- *article1
Modified: trunk/magalie-services/src/main/resources/fixtures2.yaml
===================================================================
--- trunk/magalie-services/src/main/resources/fixtures2.yaml 2013-04-11 20:24:53 UTC (rev 88)
+++ trunk/magalie-services/src/main/resources/fixtures2.yaml 2013-04-11 21:07:14 UTC (rev 89)
@@ -128,6 +128,7 @@
quantityInKanban: 10
unit: kg
description: very big screws
+ fixedLocationsFirst: false
article2:
&article2 !article
@@ -136,6 +137,7 @@
quantityInKanban: 2
unit: box
description: little buttons
+ fixedLocationsFirst: false
article3:
&article3 !article
@@ -146,6 +148,7 @@
quantityInKanban: 50
unit: g
description: magic powder
+ fixedLocationsFirst: false
articles:
- *article1
1
0
11 Apr '13
Author: bleny
Date: 2013-04-11 22:24:53 +0200 (Thu, 11 Apr 2013)
New Revision: 88
Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision…
Log:
replace Building.name by Building.code
Added:
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/WarehouseDao.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/WarehouseJpaDao.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Warehouse.java
trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/BuildingsService.java
Removed:
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/StoreDao.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/StoreJpaDao.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Store.java
trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/StoreService.java
Modified:
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/LocationDao.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/LocationJpaDao.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/StoredArticleJpaDao.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Building.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Location.java
trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/MagalieFixtures.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/FixturesService.java
trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/FulfilKanbanService.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/main/resources/fixtures2.yaml
trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/ArticleStorageServiceTest.java
trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/MagalieSession.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/LoginAction.java
trunk/magalie-web/src/main/webapp/WEB-INF/content/choose-activity.jsp
trunk/magalie-web/src/main/webapp/WEB-INF/content/fulfil-kanban-input.jsp
trunk/magalie-web/src/main/webapp/WEB-INF/content/login-input.jsp
trunk/magalie-web/src/main/webapp/WEB-INF/content/report.jsp
trunk/magalie-web/src/main/webapp/WEB-INF/content/withdraw-item-input.jsp
trunk/magalie-web/src/main/webapp/js/withdraw-item-input.js
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-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/JpaMagaliePersistenceContext.java 2013-04-11 20:24:53 UTC (rev 88)
@@ -9,7 +9,7 @@
import com.franciaflex.magalie.persistence.dao.LocationErrorDao;
import com.franciaflex.magalie.persistence.dao.StorageMovementDao;
import com.franciaflex.magalie.persistence.dao.StorageMovementOrderDao;
-import com.franciaflex.magalie.persistence.dao.StoreDao;
+import com.franciaflex.magalie.persistence.dao.WarehouseDao;
import com.franciaflex.magalie.persistence.dao.StoredArticleDao;
import com.franciaflex.magalie.persistence.dao.jpa.ArticleJpaDao;
import com.franciaflex.magalie.persistence.dao.jpa.BuildingJpaDao;
@@ -20,7 +20,7 @@
import com.franciaflex.magalie.persistence.dao.jpa.RequestedArticleJpaDao;
import com.franciaflex.magalie.persistence.dao.jpa.StorageMovementJpaDao;
import com.franciaflex.magalie.persistence.dao.jpa.StorageMovementOrderJpaDao;
-import com.franciaflex.magalie.persistence.dao.jpa.StoreJpaDao;
+import com.franciaflex.magalie.persistence.dao.jpa.WarehouseJpaDao;
import com.franciaflex.magalie.persistence.dao.jpa.StoredArticleJpaDao;
import javax.persistence.EntityManager;
@@ -86,8 +86,8 @@
}
@Override
- public StoreDao getStoreDao() {
- return new StoreJpaDao(entityManager);
+ public WarehouseDao getWarehouseDao() {
+ return new WarehouseJpaDao(entityManager);
}
@Override
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-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/MagaliePersistenceContext.java 2013-04-11 20:24:53 UTC (rev 88)
@@ -9,7 +9,7 @@
import com.franciaflex.magalie.persistence.dao.LocationErrorDao;
import com.franciaflex.magalie.persistence.dao.StorageMovementDao;
import com.franciaflex.magalie.persistence.dao.StorageMovementOrderDao;
-import com.franciaflex.magalie.persistence.dao.StoreDao;
+import com.franciaflex.magalie.persistence.dao.WarehouseDao;
import com.franciaflex.magalie.persistence.dao.StoredArticleDao;
/**
@@ -43,5 +43,5 @@
BuildingDao getBuildingDao();
- StoreDao getStoreDao();
+ WarehouseDao getWarehouseDao();
}
Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/LocationDao.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/LocationDao.java 2013-04-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/LocationDao.java 2013-04-11 20:24:53 UTC (rev 88)
@@ -2,10 +2,6 @@
import com.franciaflex.magalie.persistence.entity.Location;
-import java.util.List;
-
public interface LocationDao extends Dao<Location> {
- List<String> getAllStoreCodes();
-
}
Deleted: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/StoreDao.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/StoreDao.java 2013-04-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/StoreDao.java 2013-04-11 20:24:53 UTC (rev 88)
@@ -1,10 +0,0 @@
-package com.franciaflex.magalie.persistence.dao;
-
-import com.franciaflex.magalie.persistence.entity.Store;
-
-/**
- * @author bleny
- */
-public interface StoreDao extends Dao<Store> {
-
-}
Copied: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/WarehouseDao.java (from rev 86, trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/StoreDao.java)
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/WarehouseDao.java (rev 0)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/WarehouseDao.java 2013-04-11 20:24:53 UTC (rev 88)
@@ -0,0 +1,10 @@
+package com.franciaflex.magalie.persistence.dao;
+
+import com.franciaflex.magalie.persistence.entity.Warehouse;
+
+/**
+ * @author bleny
+ */
+public interface WarehouseDao extends Dao<Warehouse> {
+
+}
Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/LocationJpaDao.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/LocationJpaDao.java 2013-04-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/LocationJpaDao.java 2013-04-11 20:24:53 UTC (rev 88)
@@ -18,10 +18,4 @@
return Location.class;
}
- @Override
- public List<String> getAllStoreCodes() {
- Query query = entityManager.createQuery("select distinct s.storeCode from Location s order by s.storeCode");
- List<String> resultList = query.getResultList();
- return resultList;
- }
}
Deleted: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/StoreJpaDao.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/StoreJpaDao.java 2013-04-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/StoreJpaDao.java 2013-04-11 20:24:53 UTC (rev 88)
@@ -1,18 +0,0 @@
-package com.franciaflex.magalie.persistence.dao.jpa;
-
-import com.franciaflex.magalie.persistence.dao.StoreDao;
-import com.franciaflex.magalie.persistence.entity.Store;
-
-import javax.persistence.EntityManager;
-
-public class StoreJpaDao extends AbstractJpaDao<Store> implements StoreDao {
-
- public StoreJpaDao(EntityManager entityManager) {
- super(entityManager);
- }
-
- @Override
- protected Class<Store> getEntityClass() {
- return Store.class;
- }
-}
Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/StoredArticleJpaDao.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/StoredArticleJpaDao.java 2013-04-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/StoredArticleJpaDao.java 2013-04-11 20:24:53 UTC (rev 88)
@@ -22,7 +22,7 @@
@Override
public List<StoredArticle> findAllForArticleInBuilding(Article article, Building building) {
- Query query = entityManager.createQuery("from StoredArticle sa where sa.article = :article and sa.location.store.building = :building");
+ Query query = entityManager.createQuery("from StoredArticle sa where sa.article = :article and sa.location.warehouse.building = :building");
query.setParameter("article", article);
query.setParameter("building", building);
List<StoredArticle> resultList = query.getResultList();
Copied: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/WarehouseJpaDao.java (from rev 86, trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/StoreJpaDao.java)
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/WarehouseJpaDao.java (rev 0)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/WarehouseJpaDao.java 2013-04-11 20:24:53 UTC (rev 88)
@@ -0,0 +1,18 @@
+package com.franciaflex.magalie.persistence.dao.jpa;
+
+import com.franciaflex.magalie.persistence.dao.WarehouseDao;
+import com.franciaflex.magalie.persistence.entity.Warehouse;
+
+import javax.persistence.EntityManager;
+
+public class WarehouseJpaDao extends AbstractJpaDao<Warehouse> implements WarehouseDao {
+
+ public WarehouseJpaDao(EntityManager entityManager) {
+ super(entityManager);
+ }
+
+ @Override
+ protected Class<Warehouse> getEntityClass() {
+ return Warehouse.class;
+ }
+}
Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Building.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Building.java 2013-04-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Building.java 2013-04-11 20:24:53 UTC (rev 88)
@@ -9,7 +9,7 @@
@Id
protected String id;
- protected String name;
+ protected String code;
@Override
public String getId() {
@@ -20,11 +20,11 @@
this.id = id;
}
- public String getName() {
- return name;
+ public String getCode() {
+ return code;
}
- public void setName(String name) {
- this.name = name;
+ public void setCode(String code) {
+ this.code = code;
}
}
Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Location.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Location.java 2013-04-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Location.java 2013-04-11 20:24:53 UTC (rev 88)
@@ -11,9 +11,9 @@
protected String id;
@ManyToOne
- protected Store store;
+ protected Warehouse warehouse;
- protected String locationCode;
+ protected String code;
protected int requiredAccreditationLevel;
@@ -26,20 +26,20 @@
this.id = id;
}
- public Store getStore() {
- return store;
+ public Warehouse getWarehouse() {
+ return warehouse;
}
- public void setStore(Store store) {
- this.store = store;
+ public void setWarehouse(Warehouse warehouse) {
+ this.warehouse = warehouse;
}
- public String getLocationCode() {
- return locationCode;
+ public String getCode() {
+ return code;
}
- public void setLocationCode(String locationCode) {
- this.locationCode = locationCode;
+ public void setCode(String code) {
+ this.code = code;
}
public int getRequiredAccreditationLevel() {
@@ -51,7 +51,7 @@
}
public String getBarcode() {
- String barcode = getStoreCode() + getLocationCode();
+ String barcode = getWarehouse().getCode() + getCode();
return barcode;
}
@@ -59,7 +59,4 @@
return requiredAccreditationLevel == 9;
}
- public String getStoreCode() {
- return getStore().getName();
- }
}
Deleted: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Store.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Store.java 2013-04-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Store.java 2013-04-11 20:24:53 UTC (rev 88)
@@ -1,42 +0,0 @@
-package com.franciaflex.magalie.persistence.entity;
-
-import javax.persistence.Entity;
-import javax.persistence.Id;
-import javax.persistence.ManyToOne;
-
-@Entity
-public class Store extends AbstractEntity {
-
- @Id
- protected String id;
-
- protected String name;
-
- @ManyToOne
- protected Building building;
-
- @Override
- public String getId() {
- return id;
- }
-
- public void setId(String id) {
- this.id = id;
- }
-
- public Building getBuilding() {
- return building;
- }
-
- public void setBuilding(Building building) {
- this.building = building;
- }
-
- public String getName() {
- return name;
- }
-
- public void setName(String name) {
- this.name = name;
- }
-}
Copied: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Warehouse.java (from rev 86, trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Store.java)
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Warehouse.java (rev 0)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Warehouse.java 2013-04-11 20:24:53 UTC (rev 88)
@@ -0,0 +1,42 @@
+package com.franciaflex.magalie.persistence.entity;
+
+import javax.persistence.Entity;
+import javax.persistence.Id;
+import javax.persistence.ManyToOne;
+
+@Entity
+public class Warehouse extends AbstractEntity {
+
+ @Id
+ protected String id;
+
+ protected String code;
+
+ @ManyToOne
+ protected Building building;
+
+ @Override
+ public String getId() {
+ return id;
+ }
+
+ public void setId(String id) {
+ this.id = id;
+ }
+
+ public Building getBuilding() {
+ return building;
+ }
+
+ public void setBuilding(Building building) {
+ this.building = building;
+ }
+
+ public String getCode() {
+ return code;
+ }
+
+ public void setCode(String code) {
+ this.code = code;
+ }
+}
Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/MagalieFixtures.java
===================================================================
--- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/MagalieFixtures.java 2013-04-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/MagalieFixtures.java 2013-04-11 20:24:53 UTC (rev 88)
@@ -8,7 +8,7 @@
import com.franciaflex.magalie.persistence.entity.MagalieUser;
import com.franciaflex.magalie.persistence.entity.RequestedArticle;
import com.franciaflex.magalie.persistence.entity.Location;
-import com.franciaflex.magalie.persistence.entity.Store;
+import com.franciaflex.magalie.persistence.entity.Warehouse;
import com.franciaflex.magalie.persistence.entity.StoredArticle;
import org.apache.commons.io.Charsets;
import org.apache.commons.io.IOUtils;
@@ -32,7 +32,7 @@
}
YamlReader reader = new YamlReader(yaml);
reader.getConfig().setClassTag("building", Building.class);
- reader.getConfig().setClassTag("store", Store.class);
+ reader.getConfig().setClassTag("warehouse", Warehouse.class);
reader.getConfig().setClassTag("article", Article.class);
reader.getConfig().setClassTag("stored-article", StoredArticle.class);
reader.getConfig().setClassTag("user", MagalieUser.class);
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-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ArticleStorageService.java 2013-04-11 20:24:53 UTC (rev 88)
@@ -190,7 +190,7 @@
Set<Location> locationsInOrder = storageMovementOrder.getLocations();
// we suppose the order concerns movement in the same building
- Building building = Iterables.get(locationsInOrder, 0).getStore().getBuilding();
+ Building building = Iterables.get(locationsInOrder, 0).getWarehouse().getBuilding();
List<StoredArticle> storedArticles = getStoredArticles(building, article);
Copied: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/BuildingsService.java (from rev 86, trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/StoreService.java)
===================================================================
--- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/BuildingsService.java (rev 0)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/BuildingsService.java 2013-04-11 20:24:53 UTC (rev 88)
@@ -0,0 +1,35 @@
+package com.franciaflex.magalie.services.service;
+
+import com.franciaflex.magalie.persistence.dao.BuildingDao;
+import com.franciaflex.magalie.persistence.entity.Building;
+import com.franciaflex.magalie.services.MagalieService;
+import com.franciaflex.magalie.services.MagalieServiceContext;
+
+import java.util.List;
+
+public class BuildingsService implements MagalieService {
+
+ protected MagalieServiceContext serviceContext;
+
+ @Override
+ public void setServiceContext(MagalieServiceContext serviceContext) {
+ this.serviceContext = serviceContext;
+ }
+
+ public List<Building> getAllBuildings() {
+
+ BuildingDao dao = serviceContext.getPersistenceContext().getBuildingDao();
+
+ return dao.findAll();
+
+ }
+
+ public Building getBuilding(String buildingId) {
+
+ BuildingDao dao = serviceContext.getPersistenceContext().getBuildingDao();
+
+ return dao.findById(buildingId);
+
+ }
+
+}
Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/FixturesService.java
===================================================================
--- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/FixturesService.java 2013-04-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/FixturesService.java 2013-04-11 20:24:53 UTC (rev 88)
@@ -6,14 +6,14 @@
import com.franciaflex.magalie.persistence.dao.MagalieUserDao;
import com.franciaflex.magalie.persistence.dao.RequestedArticleDao;
import com.franciaflex.magalie.persistence.dao.LocationDao;
-import com.franciaflex.magalie.persistence.dao.StoreDao;
+import com.franciaflex.magalie.persistence.dao.WarehouseDao;
import com.franciaflex.magalie.persistence.dao.StoredArticleDao;
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.RequestedArticle;
import com.franciaflex.magalie.persistence.entity.Location;
-import com.franciaflex.magalie.persistence.entity.Store;
+import com.franciaflex.magalie.persistence.entity.Warehouse;
import com.franciaflex.magalie.persistence.entity.StoredArticle;
import com.franciaflex.magalie.services.MagalieFixtures;
import com.franciaflex.magalie.services.MagalieService;
@@ -97,13 +97,13 @@
}
- StoreDao storeDao = persistenceContext.getStoreDao();
+ WarehouseDao warehouseDao = persistenceContext.getWarehouseDao();
- Collection<Store> stores = fixtures.fixture("stores");
+ Collection<Warehouse> warehouses = fixtures.fixture("warehouses");
- for (Store store : stores) {
+ for (Warehouse warehouse : warehouses) {
- storeDao.persist(store);
+ warehouseDao.persist(warehouse);
}
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-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/FulfilKanbanService.java 2013-04-11 20:24:53 UTC (rev 88)
@@ -1,13 +1,13 @@
package com.franciaflex.magalie.services.service;
import com.franciaflex.magalie.persistence.MagaliePersistenceContext;
-import com.franciaflex.magalie.persistence.dao.StoreDao;
+import com.franciaflex.magalie.persistence.dao.WarehouseDao;
import com.franciaflex.magalie.persistence.entity.Article;
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.StorageMovementOrder;
-import com.franciaflex.magalie.persistence.entity.Store;
+import com.franciaflex.magalie.persistence.entity.Warehouse;
import com.franciaflex.magalie.services.MagalieService;
import com.franciaflex.magalie.services.MagalieServiceContext;
import com.franciaflex.magalie.services.exception.InvalidMagalieBarcodeException;
@@ -29,15 +29,15 @@
this.serviceContext = serviceContext;
}
- public Store getStore(String storeId) {
+ public Warehouse getStore(String storeId) {
MagaliePersistenceContext persistenceContext = serviceContext.getPersistenceContext();
- StoreDao storeDao = persistenceContext.getStoreDao();
+ WarehouseDao warehouseDao = persistenceContext.getWarehouseDao();
- Store store = storeDao.findById(storeId);
+ Warehouse warehouse = warehouseDao.findById(storeId);
- return store;
+ return warehouse;
}
@@ -62,13 +62,13 @@
ArticleStorageService articleStorageService =
serviceContext.newService(ArticleStorageService.class);
- StoreDao storeDao = serviceContext.getPersistenceContext().getStoreDao();
+ WarehouseDao warehouseDao = serviceContext.getPersistenceContext().getWarehouseDao();
- Store destinationStore = storeDao.findById(destinationStoreId);
+ Warehouse destinationWarehouse = warehouseDao.findById(destinationStoreId);
- Building building = destinationStore.getBuilding();
+ Building building = destinationWarehouse.getBuilding();
- Location destinationLocation = getDestinationLocation(destinationStore);
+ Location destinationLocation = getDestinationLocation(destinationWarehouse);
StorageMovementOrder storageMovementOrder =
articleStorageService.bookArticleWithBestEffortPolicy(
@@ -82,23 +82,23 @@
}
- protected Location getDestinationLocation(Store destinationStore) {
+ protected Location getDestinationLocation(Warehouse destinationWarehouse) {
- // TODO brendan 10/04/13 how to define destination for store, something like "INC" ?
+ // TODO brendan 10/04/13 how to define destination for warehouse, something like "INC" ?
return null;
}
- public List<Store> getDestinationStores(Building building) {
+ public List<Warehouse> getDestinationStores(Building building) {
MagaliePersistenceContext persistenceContext = serviceContext.getPersistenceContext();
- StoreDao storeDao = persistenceContext.getStoreDao();
+ WarehouseDao warehouseDao = persistenceContext.getWarehouseDao();
// FIXME brendan 10/04/13 consider building
- return storeDao.findAll();
+ return warehouseDao.findAll();
}
}
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-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/RequestedArticleService.java 2013-04-11 20:24:53 UTC (rev 88)
@@ -84,10 +84,10 @@
List<RequestedArticle> allRequestedArticles = requestedArticleDao.findAllUndelivered(building);
if (log.isInfoEnabled()) {
- log.info(allRequestedArticles.size() + " articles requested in building " + building.getName());
+ log.info(allRequestedArticles.size() + " articles requested in building " + building.getCode());
}
- // TODO brendan 05/04/13 consider storeCode
+ // TODO brendan 05/04/13 consider warehouse.code
return allRequestedArticles;
Deleted: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/StoreService.java
===================================================================
--- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/StoreService.java 2013-04-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/StoreService.java 2013-04-11 20:24:53 UTC (rev 88)
@@ -1,35 +0,0 @@
-package com.franciaflex.magalie.services.service;
-
-import com.franciaflex.magalie.persistence.dao.BuildingDao;
-import com.franciaflex.magalie.persistence.entity.Building;
-import com.franciaflex.magalie.services.MagalieService;
-import com.franciaflex.magalie.services.MagalieServiceContext;
-
-import java.util.List;
-
-public class StoreService implements MagalieService {
-
- protected MagalieServiceContext serviceContext;
-
- @Override
- public void setServiceContext(MagalieServiceContext serviceContext) {
- this.serviceContext = serviceContext;
- }
-
- public List<Building> getAllBuildings() {
-
- BuildingDao dao = serviceContext.getPersistenceContext().getBuildingDao();
-
- return dao.findAll();
-
- }
-
- public Building getBuilding(String buildingId) {
-
- BuildingDao dao = serviceContext.getPersistenceContext().getBuildingDao();
-
- return dao.findById(buildingId);
-
- }
-
-}
Modified: trunk/magalie-services/src/main/resources/fixtures.yaml
===================================================================
--- trunk/magalie-services/src/main/resources/fixtures.yaml 2013-04-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-services/src/main/resources/fixtures.yaml 2013-04-11 20:24:53 UTC (rev 88)
@@ -35,42 +35,42 @@
B1:
&B1 !building
id: building_1
- name: B1
+ code: B1
buildings:
- *B1
U01:
- &U01 !store
- id: store_u01
- name: U01
+ &U01 !warehouse
+ id: warehouse_u01
+ code: U01
building: *B1
U02:
- &U02 !store
- id: store_u02
- name: U02
+ &U02 !warehouse
+ id: warehouse_u02
+ code: U02
building: *B1
U03:
- &U03 !store
- id: store_u03
- name: U03
+ &U03 !warehouse
+ id: warehouse_u03
+ code: U03
building: *B1
U04:
- &U04 !store
- id: store_u04
- name: U04
+ &U04 !warehouse
+ id: warehouse_u04
+ code: U04
building: *B1
SOM:
- &SOM !store
- id: store_som
- name: SOM
+ &SOM !warehouse
+ id: warehouse_som
+ code: SOM
building: *B1
-stores:
+warehouses:
- *U01
- *U02
- *U03
@@ -80,92 +80,92 @@
location1:
&location1 !location
id: location1
- store: *U01
- locationCode: A01
+ warehouse: *U01
+ code: A01
requiredAccreditationLevel: 0
location2:
&location2 !location
id: location2
- store: *U01
- locationCode: B02
+ warehouse: *U01
+ code: B02
requiredAccreditationLevel: 9
location3:
&location3 !location
id: location3
- store: *U01
- locationCode: C03
+ warehouse: *U01
+ code: C03
requiredAccreditationLevel: 0
location4:
&location4 !location
id: location4
- store: *U01
- locationCode: D04
+ warehouse: *U01
+ code: D04
requiredAccreditationLevel: 9
location5:
&location5 !location
id: location5
- store: *U02
- locationCode: F01
+ warehouse: *U02
+ code: F01
requiredAccreditationLevel: 0
location6:
&location6 !location
id: location6
- store: *U02
- locationCode: G02
+ warehouse: *U02
+ code: G02
requiredAccreditationLevel: 0
location7:
&location7 !location
id: location7
- store: *U02
- locationCode: H03
+ warehouse: *U02
+ code: H03
requiredAccreditationLevel: 9
location8:
&location8 !location
id: location8
- store: *U04
- locationCode: A010
+ warehouse: *U04
+ code: A010
requiredAccreditationLevel: 0
location9:
&location9 !location
id: location9
- store: *SOM
- locationCode: B10
+ warehouse: *SOM
+ code: B10
requiredAccreditationLevel: 0
location10:
&location10 !location
id: location10
- store: *SOM
- locationCode: B24
+ warehouse: *SOM
+ code: B24
requiredAccreditationLevel: 0
location11:
&location11 !location
id: location11
- store: *SOM
- locationCode: B77
+ warehouse: *SOM
+ code: B77
requiredAccreditationLevel: 0
location12:
&location12 !location
id: location12
- store: *SOM
- locationCode: C12
+ warehouse: *SOM
+ code: C12
requiredAccreditationLevel: 0
location13:
&location13 !location
id: location13
- store: *U01
- locationCode: REC
+ warehouse: *U01
+ code: REC
requiredAccreditationLevel: 0
locations:
Modified: trunk/magalie-services/src/main/resources/fixtures2.yaml
===================================================================
--- trunk/magalie-services/src/main/resources/fixtures2.yaml 2013-04-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-services/src/main/resources/fixtures2.yaml 2013-04-11 20:24:53 UTC (rev 88)
@@ -35,42 +35,42 @@
B1:
&B1 !building
id: building_1
- name: B1
+ code: B1
buildings:
- *B1
U01:
- &U01 !store
- id: store_u01
- name: U01
+ &U01 !warehouse
+ id: warehouse_u01
+ code: U01
building: *B1
U02:
- &U02 !store
- id: store_u02
- name: U02
+ &U02 !warehouse
+ id: warehouse_u02
+ code: U02
building: *B1
U03:
- &U03 !store
- id: store_u03
- name: U03
+ &U03 !warehouse
+ id: warehouse_u03
+ code: U03
building: *B1
U04:
- &U04 !store
- id: store_u04
- name: U04
+ &U04 !warehouse
+ id: warehouse_u04
+ code: U04
building: *B1
SOM:
- &SOM !store
- id: store_som
- name: SOM
+ &SOM !warehouse
+ id: warehouse_som
+ code: SOM
building: *B1
-stores:
+warehouses:
- *U01
- *U02
- *U03
@@ -80,36 +80,36 @@
location1:
&location1 !location
id: location1
- store: *U01
- locationCode: A1
+ warehouse: *U01
+ code: A1
requiredAccreditationLevel: 0
location2:
&location2 !location
id: location2
- store: *U01
- locationCode: B2
+ warehouse: *U01
+ code: B2
requiredAccreditationLevel: 9
location3:
&location3 !location
id: location3
- store: *U01
- locationCode: C3
+ warehouse: *U01
+ code: C3
requiredAccreditationLevel: 0
location4:
&location4 !location
id: location4
- store: *U01
- locationCode: D4
+ warehouse: *U01
+ code: D4
requiredAccreditationLevel: 9
location5:
&location5 !location
id: location5
- store: *U02
- locationCode: A1
+ warehouse: *U02
+ code: A1
requiredAccreditationLevel: 0
locations:
Modified: trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/ArticleStorageServiceTest.java
===================================================================
--- trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/ArticleStorageServiceTest.java 2013-04-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/ArticleStorageServiceTest.java 2013-04-11 20:24:53 UTC (rev 88)
@@ -159,10 +159,10 @@
Assert.assertEquals(2, storageMovementOrder.getStorageMovements().size());
StorageMovement storageMovement1 = Iterables.get(storageMovementOrder.getStorageMovements(), 0, null);
Assert.assertEquals(20., storageMovement1.getExpectedQuantity(), DELTA);
- Assert.assertEquals("B2", storageMovement1.getOriginLocation().getLocationCode());
+ Assert.assertEquals("B2", storageMovement1.getOriginLocation().getCode());
StorageMovement storageMovement2 = Iterables.get(storageMovementOrder.getStorageMovements(), 1, null);
Assert.assertEquals(10., storageMovement2.getExpectedQuantity(), DELTA);
- Assert.assertEquals("A1", storageMovement2.getOriginLocation().getLocationCode());
+ Assert.assertEquals("A1", storageMovement2.getOriginLocation().getCode());
} catch (RequiredDriverLicenceException e) {
if (log.isDebugEnabled()) {
log.debug("exception raised = " + e);
@@ -183,7 +183,7 @@
Assert.assertEquals(1, storageMovementOrder.getStorageMovements().size());
StorageMovement storageMovement = Iterables.get(storageMovementOrder.getStorageMovements(), 0, null);
Assert.assertEquals(30., storageMovement.getExpectedQuantity(), DELTA);
- Assert.assertEquals("A1", storageMovement.getOriginLocation().getLocationCode());
+ Assert.assertEquals("A1", storageMovement.getOriginLocation().getCode());
} catch (RequiredDriverLicenceException e) {
if (log.isDebugEnabled()) {
log.debug("exception raised = " + e);
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-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/MagalieSession.java 2013-04-11 20:24:53 UTC (rev 88)
@@ -2,7 +2,7 @@
import com.franciaflex.magalie.persistence.entity.Building;
import com.franciaflex.magalie.persistence.entity.MagalieUser;
-import com.franciaflex.magalie.persistence.entity.Store;
+import com.franciaflex.magalie.persistence.entity.Warehouse;
import java.io.Serializable;
@@ -21,7 +21,7 @@
protected MagalieUser magalieUser;
protected Building building;
- private Store lastUsedDestinationStoreForKanbans;
+ private Warehouse lastUsedDestinationWarehouseForKanbans;
public MagalieUser getMagalieUser() {
return magalieUser;
@@ -39,11 +39,11 @@
this.building = building;
}
- public Store getLastUsedDestinationStoreForKanbans() {
- return lastUsedDestinationStoreForKanbans;
+ public Warehouse getLastUsedDestinationWarehouseForKanbans() {
+ return lastUsedDestinationWarehouseForKanbans;
}
- public void setLastUsedDestinationStoreForKanbans(Store lastUsedDestinationStoreForKanbans) {
- this.lastUsedDestinationStoreForKanbans = lastUsedDestinationStoreForKanbans;
+ public void setLastUsedDestinationWarehouseForKanbans(Warehouse lastUsedDestinationWarehouseForKanbans) {
+ this.lastUsedDestinationWarehouseForKanbans = lastUsedDestinationWarehouseForKanbans;
}
}
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-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/FulfilKanbanAction.java 2013-04-11 20:24:53 UTC (rev 88)
@@ -3,7 +3,7 @@
import com.franciaflex.magalie.persistence.entity.Building;
import com.franciaflex.magalie.persistence.entity.MagalieUser;
import com.franciaflex.magalie.persistence.entity.StorageMovementOrder;
-import com.franciaflex.magalie.persistence.entity.Store;
+import com.franciaflex.magalie.persistence.entity.Warehouse;
import com.franciaflex.magalie.services.exception.InvalidMagalieBarcodeException;
import com.franciaflex.magalie.services.exception.RequiredDriverLicenceException;
import com.franciaflex.magalie.services.exception.UnavailableArticleException;
@@ -33,9 +33,9 @@
protected StorageMovementOrder storageMovementOrder;
- protected List<Store> destinationStores;
+ protected List<Warehouse> destinationWarehouses;
- protected String destinationStoreId;
+ protected String destinationWarehouseId;
public void setService(FulfilKanbanService service) {
this.service = service;
@@ -50,15 +50,15 @@
Building building = session.getBuilding();
- destinationStores = service.getDestinationStores(building);
+ destinationWarehouses = service.getDestinationStores(building);
- // let's help user by selecting by default the previously used destination store
+ // let's help user by selecting by default the previously used destination warehouse
- Store lastUsedDestinationStoreForKanbans = session.getLastUsedDestinationStoreForKanbans();
+ Warehouse lastUsedDestinationWarehouseForKanbans = session.getLastUsedDestinationWarehouseForKanbans();
- if (lastUsedDestinationStoreForKanbans != null) {
+ if (lastUsedDestinationWarehouseForKanbans != null) {
- destinationStoreId = lastUsedDestinationStoreForKanbans.getId();
+ destinationWarehouseId = lastUsedDestinationWarehouseForKanbans.getId();
}
@@ -66,20 +66,20 @@
}
- public String getDestinationStoreId() {
- return destinationStoreId;
+ public String getDestinationWarehouseId() {
+ return destinationWarehouseId;
}
- public List<Store> getDestinationStores() {
- return destinationStores;
+ public List<Warehouse> getDestinationWarehouses() {
+ return destinationWarehouses;
}
public void setArticleBarcode(String articleBarcode) {
this.articleBarcode = articleBarcode;
}
- public void setDestinationStoreId(String destinationStoreId) {
- this.destinationStoreId = destinationStoreId;
+ public void setDestinationWarehouseId(String destinationWarehouseId) {
+ this.destinationWarehouseId = destinationWarehouseId;
}
@Override
@@ -91,14 +91,14 @@
MagalieUser magalieUser = session.getMagalieUser();
- Store destinationStore = service.getStore(destinationStoreId);
+ Warehouse destinationWarehouse = service.getStore(destinationWarehouseId);
- // save used destination store to propose it by default on next kanban
- session.setLastUsedDestinationStoreForKanbans(destinationStore);
+ // save used destination warehouse to propose it by default on next kanban
+ session.setLastUsedDestinationWarehouseForKanbans(destinationWarehouse);
try {
- storageMovementOrder = service.fulfilKanbanTask(articleBarcode, magalieUser, destinationStoreId);
+ storageMovementOrder = service.fulfilKanbanTask(articleBarcode, magalieUser, destinationWarehouseId);
} catch (InvalidMagalieBarcodeException e) {
Modified: trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/LoginAction.java
===================================================================
--- trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/LoginAction.java 2013-04-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-web/src/main/java/com/franciaflex/magalie/web/action/LoginAction.java 2013-04-11 20:24:53 UTC (rev 88)
@@ -2,8 +2,8 @@
import com.franciaflex.magalie.persistence.entity.Building;
import com.franciaflex.magalie.persistence.entity.MagalieUser;
+import com.franciaflex.magalie.services.service.BuildingsService;
import com.franciaflex.magalie.services.service.MagalieUsersService;
-import com.franciaflex.magalie.services.service.StoreService;
import com.franciaflex.magalie.web.MagalieActionSupport;
import com.franciaflex.magalie.web.MagalieSession;
import org.apache.struts2.convention.annotation.Result;
@@ -18,7 +18,7 @@
protected MagalieUsersService magalieUsersService;
- protected StoreService storeService;
+ protected BuildingsService buildingsService;
protected List<MagalieUser> allMagalieUsers;
@@ -34,8 +34,8 @@
this.magalieUsersService = magalieUsersService;
}
- public void setStoreService(StoreService storeService) {
- this.storeService = storeService;
+ public void setBuildingsService(BuildingsService buildingsService) {
+ this.buildingsService = buildingsService;
}
public void setSession(MagalieSession session) {
@@ -55,7 +55,7 @@
allMagalieUsers = magalieUsersService.getAllMagalieUsers();
- allBuildings = storeService.getAllBuildings();
+ allBuildings = buildingsService.getAllBuildings();
return INPUT;
@@ -82,7 +82,7 @@
MagalieUser magalieUser = magalieUsersService.getMagalieUser(login);
- Building building = storeService.getBuilding(buildingId);
+ Building building = buildingsService.getBuilding(buildingId);
session.setMagalieUser(magalieUser);
Modified: trunk/magalie-web/src/main/webapp/WEB-INF/content/choose-activity.jsp
===================================================================
--- trunk/magalie-web/src/main/webapp/WEB-INF/content/choose-activity.jsp 2013-04-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-web/src/main/webapp/WEB-INF/content/choose-activity.jsp 2013-04-11 20:24:53 UTC (rev 88)
@@ -35,7 +35,7 @@
<dt>Identifiant</dt>
<dd><s:property value="magalieUser.name" /> (<s:property value="magalieUser.login" />)</dd>
<dt>Bâtiment</dt>
- <dd><s:property value="building.name" /></dd>
+ <dd><s:property value="building.code" /></dd>
</dl>
</header>
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-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-web/src/main/webapp/WEB-INF/content/fulfil-kanban-input.jsp 2013-04-11 20:24:53 UTC (rev 88)
@@ -9,10 +9,10 @@
<s:textfield name="articleBarcode" label="Article" inputAppendIcon="barcode" cssClass="input-medium" />
<s:select label="Magasin destination"
- key="destinationStoreId"
- list="destinationStores"
+ key="destinationWarehouseId"
+ list="destinationWarehouses"
listKey="id"
- listValue="name"
+ listValue="code"
required="true"
/>
Modified: trunk/magalie-web/src/main/webapp/WEB-INF/content/login-input.jsp
===================================================================
--- trunk/magalie-web/src/main/webapp/WEB-INF/content/login-input.jsp 2013-04-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-web/src/main/webapp/WEB-INF/content/login-input.jsp 2013-04-11 20:24:53 UTC (rev 88)
@@ -19,7 +19,7 @@
key="buildingId"
list="allBuildings"
listKey="id"
- listValue="name"
+ listValue="code"
required="true"
/>
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-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-web/src/main/webapp/WEB-INF/content/report.jsp 2013-04-11 20:24:53 UTC (rev 88)
@@ -48,15 +48,15 @@
<s:property value="id" />
</td>
<td>
- <s:property value="originLocation.storeCode" />
+ <s:property value="originLocation.warehouse.code" />
-
- <s:property value="originLocation.locationCode" />
+ <s:property value="originLocation.code" />
</td>
<td>
<s:if test="destinationLocation">
- <s:property value="destinationLocation.storeCode" />
+ <s:property value="destinationLocation.warehouse.code" />
-
- <s:property value="destinationLocation.locationCode" />
+ <s:property value="destinationLocation.code" />
</s:if>
</td>
<td>
@@ -120,15 +120,15 @@
<s:property value="id" />
</td>
<td>
- <s:property value="originLocation.storeCode" />
+ <s:property value="originLocation.warehouse.code" />
-
- <s:property value="originLocation.locationCode" />
+ <s:property value="originLocation.code" />
</td>
<td>
<s:if test="destinationLocation">
- <s:property value="destinationLocation.storeCode" />
+ <s:property value="destinationLocation.warehouse.code" />
-
- <s:property value="destinationLocation.locationCode" />
+ <s:property value="destinationLocation.code" />
</s:if>
</td>
<td>
@@ -186,10 +186,10 @@
<s:property value="id" />
</td>
<td>
- <s:property value="location.storeCode" />
+ <s:property value="location.warehouse.code" />
</td>
<td>
- <s:property value="location.locationCode" />
+ <s:property value="location.code" />
</td>
<td>
<s:property value="magalieUser.name" />
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-04-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-web/src/main/webapp/WEB-INF/content/withdraw-item-input.jsp 2013-04-11 20:24:53 UTC (rev 88)
@@ -111,7 +111,7 @@
<div id="storageMovementTemplate" class="template">
<dl class="dl-horizontal">
<dt>Empl.</dt>
- <dd><span data="originLocation.store.name"></span> <span data="originLocation.locationCode"></span></dd>
+ <dd><span data="originLocation.warehouse.code"></span> <span data="originLocation.code"></span></dd>
<dt>Prélev.</dt>
<dd><span data="withdrawn">0</span> / <span data="expectedQuantity"></span> <s:property value="storageMovementTask.article.unit" /></dd>
</dl>
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-11 19:44:51 UTC (rev 87)
+++ trunk/magalie-web/src/main/webapp/js/withdraw-item-input.js 2013-04-11 20:24:53 UTC (rev 88)
@@ -111,7 +111,7 @@
} else {
var storageMovement = this.storageMovements[this.storageMovementsIndex];
if (storageMovement.expectedQuantity == 0.) {
- console.debug("will use extra location " + storageMovement.originLocation.locationCode);
+ console.debug("will use extra location " + storageMovement.originLocation.code);
var remainingQuantity = this.quantity - this.withdrawn();
storageMovement.expectedQuantity = remainingQuantity;
// var availableQuantityInLocation = storageMovement.availableQuantity;
1
0
11 Apr '13
Author: bleny
Date: 2013-04-11 21:44:51 +0200 (Thu, 11 Apr 2013)
New Revision: 87
Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revision…
Log:
rename site to location
Added:
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/Locations.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/LocationDao.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/LocationErrorDao.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/LocationErrorJpaDao.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/LocationJpaDao.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Location.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/LocationError.java
trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/LocationErrorsService.java
Removed:
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/Sites.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/SiteDao.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/SiteErrorDao.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/SiteErrorJpaDao.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/SiteJpaDao.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Site.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/SiteError.java
trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/SiteErrorsService.java
Modified:
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/StorageMovements.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/StoredArticles.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/StoredArticleJpaDao.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Article.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/StorageMovement.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/StorageMovementOrder.java
trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/StoredArticle.java
trunk/magalie-persistence/src/test/java/com/franciaflex/magalie/PersistenceTest.java
trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/MagalieFixtures.java
trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/MagalieReport.java
trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/StorageMovementConfirmation.java
trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/StorageMovementTask.java
trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/exception/RequiredDriverLicenceException.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/FixturesService.java
trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/FulfilKanbanService.java
trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ReportService.java
trunk/magalie-services/src/main/resources/fixtures.yaml
trunk/magalie-services/src/main/resources/fixtures2.yaml
trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/ArticleStorageServiceTest.java
trunk/magalie-web/src/main/webapp/WEB-INF/content/report.jsp
trunk/magalie-web/src/main/webapp/WEB-INF/content/withdraw-item-input.jsp
trunk/magalie-web/src/main/webapp/js/withdraw-item-input.js
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-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/JpaMagaliePersistenceContext.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -5,8 +5,8 @@
import com.franciaflex.magalie.persistence.dao.DeliveredRequestedArticleDao;
import com.franciaflex.magalie.persistence.dao.MagalieUserDao;
import com.franciaflex.magalie.persistence.dao.RequestedArticleDao;
-import com.franciaflex.magalie.persistence.dao.SiteDao;
-import com.franciaflex.magalie.persistence.dao.SiteErrorDao;
+import com.franciaflex.magalie.persistence.dao.LocationDao;
+import com.franciaflex.magalie.persistence.dao.LocationErrorDao;
import com.franciaflex.magalie.persistence.dao.StorageMovementDao;
import com.franciaflex.magalie.persistence.dao.StorageMovementOrderDao;
import com.franciaflex.magalie.persistence.dao.StoreDao;
@@ -14,10 +14,10 @@
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.LocationErrorJpaDao;
+import com.franciaflex.magalie.persistence.dao.jpa.LocationJpaDao;
import com.franciaflex.magalie.persistence.dao.jpa.MagalieUserJpaDao;
import com.franciaflex.magalie.persistence.dao.jpa.RequestedArticleJpaDao;
-import com.franciaflex.magalie.persistence.dao.jpa.SiteErrorJpaDao;
-import com.franciaflex.magalie.persistence.dao.jpa.SiteJpaDao;
import com.franciaflex.magalie.persistence.dao.jpa.StorageMovementJpaDao;
import com.franciaflex.magalie.persistence.dao.jpa.StorageMovementOrderJpaDao;
import com.franciaflex.magalie.persistence.dao.jpa.StoreJpaDao;
@@ -66,8 +66,8 @@
}
@Override
- public SiteErrorDao getSiteErrorDao() {
- return new SiteErrorJpaDao(entityManager);
+ public LocationErrorDao getLocationErrorDao() {
+ return new LocationErrorJpaDao(entityManager);
}
@Override
@@ -76,8 +76,8 @@
}
@Override
- public SiteDao getSiteDao() {
- return new SiteJpaDao(entityManager);
+ public LocationDao getLocationDao() {
+ return new LocationJpaDao(entityManager);
}
@Override
Copied: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/Locations.java (from rev 83, trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/Sites.java)
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/Locations.java (rev 0)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/Locations.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -0,0 +1,73 @@
+package com.franciaflex.magalie.persistence;
+
+import com.franciaflex.magalie.persistence.entity.MagalieUser;
+import com.franciaflex.magalie.persistence.entity.Location;
+import com.google.common.base.Predicate;
+import com.google.common.base.Predicates;
+
+import java.util.Collection;
+import java.util.Comparator;
+
+public class Locations {
+
+ protected static class LocationRequireDriverLicensePredicate implements Predicate<Location> {
+
+ @Override
+ public boolean apply(Location location) {
+ return location.isDriverLicenseRequired();
+ }
+ }
+
+ protected static class AccessibleLocationPredicate implements Predicate<Location> {
+
+ protected MagalieUser magalieUser;
+
+ public AccessibleLocationPredicate(MagalieUser magalieUser) {
+ this.magalieUser = magalieUser;
+ }
+
+ @Override
+ public boolean apply(Location location) {
+ boolean isLocationAccessible = magalieUser.isDriverLicenseOwner()
+ || ! location.isDriverLicenseRequired();
+ return isLocationAccessible;
+ }
+
+ }
+
+ protected static class LocationRequiringDriverLicenseFirstComparator implements Comparator<Location> {
+
+ @Override
+ public int compare(Location location1, Location location2) {
+ boolean location1RequireDriverLicense = locationRequireDriverLicensePredicate().apply(location1);
+ boolean location2RequireDriverLicense = locationRequireDriverLicensePredicate().apply(location2);
+ if (location1RequireDriverLicense && ! location2RequireDriverLicense) {
+ return -1;
+ } else if ( ! location1RequireDriverLicense && location2RequireDriverLicense) {
+ return +1;
+ } else {
+ return 0;
+ }
+ }
+ }
+
+ public static Predicate<Location> accessibleLocationPredicate(MagalieUser magalieUser) {
+ return new AccessibleLocationPredicate(magalieUser);
+ }
+
+ public static Predicate<Location> inaccessibleLocationPredicate(MagalieUser magalieUser) {
+ return Predicates.not(accessibleLocationPredicate(magalieUser));
+ }
+
+ public static Predicate<Location> locationRequireDriverLicensePredicate() {
+ return new LocationRequireDriverLicensePredicate();
+ }
+
+ public static Comparator<Location> locationRequiringDriverLicenseFirstComparator() {
+ return new LocationRequiringDriverLicenseFirstComparator();
+ }
+
+ public static Predicate<Location> locationIsNotReportedInError(Collection<Location> allLocationsInError) {
+ return Predicates.not(Predicates.in(allLocationsInError));
+ }
+}
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-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/MagaliePersistenceContext.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -5,8 +5,8 @@
import com.franciaflex.magalie.persistence.dao.DeliveredRequestedArticleDao;
import com.franciaflex.magalie.persistence.dao.MagalieUserDao;
import com.franciaflex.magalie.persistence.dao.RequestedArticleDao;
-import com.franciaflex.magalie.persistence.dao.SiteDao;
-import com.franciaflex.magalie.persistence.dao.SiteErrorDao;
+import com.franciaflex.magalie.persistence.dao.LocationDao;
+import com.franciaflex.magalie.persistence.dao.LocationErrorDao;
import com.franciaflex.magalie.persistence.dao.StorageMovementDao;
import com.franciaflex.magalie.persistence.dao.StorageMovementOrderDao;
import com.franciaflex.magalie.persistence.dao.StoreDao;
@@ -33,11 +33,11 @@
StorageMovementDao getStorageMovementDao();
- SiteErrorDao getSiteErrorDao();
+ LocationErrorDao getLocationErrorDao();
StorageMovementOrderDao getStorageMovementOrderDao();
- SiteDao getSiteDao();
+ LocationDao getLocationDao();
DeliveredRequestedArticleDao getDeliveredRequestedArticleDao();
Deleted: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/Sites.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/Sites.java 2013-04-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/Sites.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -1,73 +0,0 @@
-package com.franciaflex.magalie.persistence;
-
-import com.franciaflex.magalie.persistence.entity.MagalieUser;
-import com.franciaflex.magalie.persistence.entity.Site;
-import com.google.common.base.Predicate;
-import com.google.common.base.Predicates;
-
-import java.util.Collection;
-import java.util.Comparator;
-
-public class Sites {
-
- protected static class SiteRequireDriverLicensePredicate implements Predicate<Site> {
-
- @Override
- public boolean apply(Site site) {
- return site.isDriverLicenseRequired();
- }
- }
-
- protected static class AccessibleSitePredicate implements Predicate<Site> {
-
- protected MagalieUser magalieUser;
-
- public AccessibleSitePredicate(MagalieUser magalieUser) {
- this.magalieUser = magalieUser;
- }
-
- @Override
- public boolean apply(Site site) {
- boolean isSiteAccessible = magalieUser.isDriverLicenseOwner()
- || ! site.isDriverLicenseRequired();
- return isSiteAccessible;
- }
-
- }
-
- protected static class SiteRequiringDriverLicenseFirstComparator implements Comparator<Site> {
-
- @Override
- public int compare(Site site1, Site site2) {
- boolean site1RequireDriverLicense = siteRequireDriverLicensePredicate().apply(site1);
- boolean site2RequireDriverLicense = siteRequireDriverLicensePredicate().apply(site2);
- if (site1RequireDriverLicense && ! site2RequireDriverLicense) {
- return -1;
- } else if ( ! site1RequireDriverLicense && site2RequireDriverLicense) {
- return +1;
- } else {
- return 0;
- }
- }
- }
-
- public static Predicate<Site> accessibleSitePredicate(MagalieUser magalieUser) {
- return new AccessibleSitePredicate(magalieUser);
- }
-
- public static Predicate<Site> inaccessibleSitePredicate(MagalieUser magalieUser) {
- return Predicates.not(accessibleSitePredicate(magalieUser));
- }
-
- public static Predicate<Site> siteRequireDriverLicensePredicate() {
- return new SiteRequireDriverLicensePredicate();
- }
-
- public static Comparator<Site> siteRequiringDriverLicenseFirstComparator() {
- return new SiteRequiringDriverLicenseFirstComparator();
- }
-
- public static Predicate<Site> siteIsNotReportedInError(Collection<Site> allSitesInError) {
- return Predicates.not(Predicates.in(allSitesInError));
- }
-}
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-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/StorageMovements.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -1,7 +1,7 @@
package com.franciaflex.magalie.persistence;
import com.franciaflex.magalie.persistence.entity.Article;
-import com.franciaflex.magalie.persistence.entity.Site;
+import com.franciaflex.magalie.persistence.entity.Location;
import com.franciaflex.magalie.persistence.entity.StorageMovement;
import com.google.common.base.Function;
import com.google.common.base.Predicate;
@@ -17,11 +17,11 @@
}
}
- protected static class GetOriginSiteFunction implements Function<StorageMovement, Site> {
+ protected static class GetOriginFunction implements Function<StorageMovement, Location> {
@Override
- public Site apply(StorageMovement storageMovement) {
- return storageMovement.getOriginSite();
+ public Location apply(StorageMovement storageMovement) {
+ return storageMovement.getOriginLocation();
}
}
@@ -37,8 +37,8 @@
return new GetArticleFunction();
}
- public static Function<StorageMovement, Site> getOriginSiteFunction() {
- return new GetOriginSiteFunction();
+ public static Function<StorageMovement, Location> getOriginFunction() {
+ return new GetOriginFunction();
}
public static Predicate<StorageMovement> storageMovementIsConfirmed() {
Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/StoredArticles.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/StoredArticles.java 2013-04-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/StoredArticles.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -2,7 +2,7 @@
import com.franciaflex.magalie.persistence.entity.Article;
import com.franciaflex.magalie.persistence.entity.MagalieUser;
-import com.franciaflex.magalie.persistence.entity.Site;
+import com.franciaflex.magalie.persistence.entity.Location;
import com.franciaflex.magalie.persistence.entity.StoredArticle;
import com.google.common.base.Function;
import com.google.common.base.Predicate;
@@ -15,24 +15,24 @@
public class StoredArticles {
- protected static class FixedSiteForArticleFirst implements Comparator<StoredArticle> {
+ protected static class FixedLocationForArticleFirst implements Comparator<StoredArticle> {
protected Article article;
- public FixedSiteForArticleFirst(Article article) {
+ public FixedLocationForArticleFirst(Article article) {
this.article = article;
}
@Override
public int compare(StoredArticle x, StoredArticle y) {
- boolean xIsInFixedSite = article.isFixedSite(x.getSite());
- boolean yIsInFixedSite = article.isFixedSite(y.getSite());
- return BooleanComparator.getTrueFirstComparator().compare(xIsInFixedSite, yIsInFixedSite);
+ boolean xIsInFixedLocation = article.isFixedLocation(x.getLocation());
+ boolean yIsInFixedLocation = article.isFixedLocation(y.getLocation());
+ return BooleanComparator.getTrueFirstComparator().compare(xIsInFixedLocation, yIsInFixedLocation);
}
}
- protected static class SiteWithHighestQuantityFirst implements Comparator<StoredArticle> {
+ protected static class LocationWithHighestQuantityFirst implements Comparator<StoredArticle> {
@Override
public int compare(StoredArticle o1, StoredArticle o2) {
@@ -41,22 +41,22 @@
}
- protected static class AccessibleSiteFirst implements Comparator<StoredArticle> {
+ protected static class AccessibleLocationFirst implements Comparator<StoredArticle> {
- protected Predicate<StoredArticle> articleStoredInAccessibleSitePredicate;
+ protected Predicate<StoredArticle> articleStoredInAccessibleLocationPredicate;
- public AccessibleSiteFirst(MagalieUser magalieUser) {
- articleStoredInAccessibleSitePredicate =
- articleStoredInAccessibleSitePredicate(magalieUser);
+ public AccessibleLocationFirst(MagalieUser magalieUser) {
+ articleStoredInAccessibleLocationPredicate =
+ articleStoredInAccessibleLocationPredicate(magalieUser);
}
@Override
public int compare(StoredArticle o1, StoredArticle o2) {
- boolean isSite1Accessible = articleStoredInAccessibleSitePredicate.apply(o1);
- boolean isSite2Accessible = articleStoredInAccessibleSitePredicate.apply(o2);
- if (isSite1Accessible && ! isSite2Accessible) {
+ boolean isLocation1Accessible = articleStoredInAccessibleLocationPredicate.apply(o1);
+ boolean isLocation2Accessible = articleStoredInAccessibleLocationPredicate.apply(o2);
+ if (isLocation1Accessible && ! isLocation2Accessible) {
return -1;
- } else if ( ! isSite1Accessible && isSite2Accessible) {
+ } else if ( ! isLocation1Accessible && isLocation2Accessible) {
return 1;
} else {
return 0;
@@ -65,59 +65,59 @@
}
- protected static class GetSiteFunction implements Function<StoredArticle, Site> {
+ protected static class GetLocationFunction implements Function<StoredArticle, Location> {
@Override
- public Site apply(StoredArticle storedArticle) {
- return storedArticle.getSite();
+ public Location apply(StoredArticle storedArticle) {
+ return storedArticle.getLocation();
}
}
- protected static class ArticleStoredInSitesRequiringDriverLicenseFirstComparator implements Comparator<StoredArticle> {
+ protected static class ArticleStoredInLocationsRequiringDriverLicenseFirstComparator implements Comparator<StoredArticle> {
@Override
public int compare(StoredArticle o1, StoredArticle o2) {
- return Sites.siteRequiringDriverLicenseFirstComparator().compare(o1.getSite(), o2.getSite());
+ return Locations.locationRequiringDriverLicenseFirstComparator().compare(o1.getLocation(), o2.getLocation());
}
}
- public static Comparator<StoredArticle> siteWithHighestQuantityFirst() {
- return new SiteWithHighestQuantityFirst();
+ public static Comparator<StoredArticle> locationWithHighestQuantityFirst() {
+ return new LocationWithHighestQuantityFirst();
}
- public static Comparator<StoredArticle> siteWithLowestQuantityFirstComparator() {
- return Ordering.from(new SiteWithHighestQuantityFirst()).reverse();
+ public static Comparator<StoredArticle> locationWithLowestQuantityFirstComparator() {
+ return Ordering.from(new LocationWithHighestQuantityFirst()).reverse();
}
- public static Comparator<StoredArticle> accessibleSiteFirstComparator(MagalieUser magalieUser) {
- return new AccessibleSiteFirst(magalieUser);
+ public static Comparator<StoredArticle> accessibleLocationFirstComparator(MagalieUser magalieUser) {
+ return new AccessibleLocationFirst(magalieUser);
}
- public static Comparator<StoredArticle> fixedSiteForArticleFirstComparator(Article article) {
- return new FixedSiteForArticleFirst(article);
+ public static Comparator<StoredArticle> fixedLocationForArticleFirstComparator(Article article) {
+ return new FixedLocationForArticleFirst(article);
}
- public static Comparator<StoredArticle> fixedSiteForArticleLastComparator(Article article) {
- return Ordering.from(new FixedSiteForArticleFirst(article)).reverse();
+ public static Comparator<StoredArticle> fixedLocationForArticleLastComparator(Article article) {
+ return Ordering.from(new FixedLocationForArticleFirst(article)).reverse();
}
- public static Function<StoredArticle, Site> getSiteFunction() {
- return new GetSiteFunction();
+ public static Function<StoredArticle, Location> getLocationFunction() {
+ return new GetLocationFunction();
}
- public static Predicate<StoredArticle> articleStoredInAccessibleSitePredicate(MagalieUser magalieUser) {
+ public static Predicate<StoredArticle> articleStoredInAccessibleLocationPredicate(MagalieUser magalieUser) {
Predicate<StoredArticle> predicate = Predicates.compose(
- Sites.accessibleSitePredicate(magalieUser),
- getSiteFunction());
+ Locations.accessibleLocationPredicate(magalieUser),
+ getLocationFunction());
return predicate;
}
- public static Comparator<StoredArticle> articleStoredInSitesRequiringDriverLicenseFirstComparator() {
- return new ArticleStoredInSitesRequiringDriverLicenseFirstComparator();
+ public static Comparator<StoredArticle> articleStoredInLocationsRequiringDriverLicenseFirstComparator() {
+ return new ArticleStoredInLocationsRequiringDriverLicenseFirstComparator();
}
- public static Predicate<StoredArticle> articleNotStoredInSiteReportedInError(Collection<Site> allSitesInError) {
- return Predicates.compose(Sites.siteIsNotReportedInError(allSitesInError), getSiteFunction());
+ public static Predicate<StoredArticle> articleNotStoredInLocationReportedInError(Collection<Location> allLocationsInError) {
+ return Predicates.compose(Locations.locationIsNotReportedInError(allLocationsInError), getLocationFunction());
}
}
Copied: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/LocationDao.java (from rev 83, trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/SiteDao.java)
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/LocationDao.java (rev 0)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/LocationDao.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -0,0 +1,11 @@
+package com.franciaflex.magalie.persistence.dao;
+
+import com.franciaflex.magalie.persistence.entity.Location;
+
+import java.util.List;
+
+public interface LocationDao extends Dao<Location> {
+
+ List<String> getAllStoreCodes();
+
+}
Copied: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/LocationErrorDao.java (from rev 83, trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/SiteErrorDao.java)
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/LocationErrorDao.java (rev 0)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/LocationErrorDao.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -0,0 +1,19 @@
+package com.franciaflex.magalie.persistence.dao;
+
+import com.franciaflex.magalie.persistence.entity.Location;
+import com.franciaflex.magalie.persistence.entity.LocationError;
+
+import java.util.List;
+
+/**
+ * @author bleny
+ */
+public interface LocationErrorDao extends Dao<LocationError> {
+
+ LocationError findByLocation(Location location);
+
+ List<Location> getAllLocationsInError();
+
+ List<LocationError> findAll();
+
+}
Deleted: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/SiteDao.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/SiteDao.java 2013-04-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/SiteDao.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -1,11 +0,0 @@
-package com.franciaflex.magalie.persistence.dao;
-
-import com.franciaflex.magalie.persistence.entity.Site;
-
-import java.util.List;
-
-public interface SiteDao extends Dao<Site> {
-
- List<String> getAllStoreCodes();
-
-}
Deleted: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/SiteErrorDao.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/SiteErrorDao.java 2013-04-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/SiteErrorDao.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -1,19 +0,0 @@
-package com.franciaflex.magalie.persistence.dao;
-
-import com.franciaflex.magalie.persistence.entity.Site;
-import com.franciaflex.magalie.persistence.entity.SiteError;
-
-import java.util.List;
-
-/**
- * @author bleny
- */
-public interface SiteErrorDao extends Dao<SiteError> {
-
- SiteError findBySite(Site site);
-
- List<Site> getAllSitesInError();
-
- List<SiteError> findAll();
-
-}
Copied: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/LocationErrorJpaDao.java (from rev 83, trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/SiteErrorJpaDao.java)
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/LocationErrorJpaDao.java (rev 0)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/LocationErrorJpaDao.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -0,0 +1,47 @@
+package com.franciaflex.magalie.persistence.dao.jpa;
+
+import com.franciaflex.magalie.persistence.dao.LocationErrorDao;
+import com.franciaflex.magalie.persistence.entity.Location;
+import com.franciaflex.magalie.persistence.entity.LocationError;
+import com.google.common.collect.Iterables;
+
+import javax.persistence.EntityManager;
+import javax.persistence.Query;
+import java.util.List;
+
+public class LocationErrorJpaDao extends AbstractJpaDao<LocationError> implements LocationErrorDao {
+
+ public LocationErrorJpaDao(EntityManager entityManager) {
+ super(entityManager);
+ }
+
+ @Override
+ protected Class<LocationError> getEntityClass() {
+ return LocationError.class;
+ }
+
+ @Override
+ public LocationError findByLocation(Location location) {
+ Query query = entityManager.createQuery("from LocationError se where se.location = :location");
+ query.setParameter("location", location);
+ LocationError locationError = (LocationError)
+ Iterables.getOnlyElement(
+ query.getResultList(),
+ null);
+ return locationError;
+ }
+
+ @Override
+ public List<Location> getAllLocationsInError() {
+ Query query = entityManager.createQuery("select se.location from LocationError se");
+ List<Location> allLocationsInError = query.getResultList();
+ return allLocationsInError;
+ }
+
+ @Override
+ public List<LocationError> findAll() {
+ Query query = entityManager.createQuery("from LocationError se order by se.reportDate");
+ List<LocationError> all = query.getResultList();
+ return all;
+ }
+}
Copied: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/LocationJpaDao.java (from rev 83, trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/SiteJpaDao.java)
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/LocationJpaDao.java (rev 0)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/LocationJpaDao.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -0,0 +1,27 @@
+package com.franciaflex.magalie.persistence.dao.jpa;
+
+import com.franciaflex.magalie.persistence.dao.LocationDao;
+import com.franciaflex.magalie.persistence.entity.Location;
+
+import javax.persistence.EntityManager;
+import javax.persistence.Query;
+import java.util.List;
+
+public class LocationJpaDao extends AbstractJpaDao<Location> implements LocationDao {
+
+ public LocationJpaDao(EntityManager entityManager) {
+ super(entityManager);
+ }
+
+ @Override
+ protected Class<Location> getEntityClass() {
+ return Location.class;
+ }
+
+ @Override
+ public List<String> getAllStoreCodes() {
+ Query query = entityManager.createQuery("select distinct s.storeCode from Location s order by s.storeCode");
+ List<String> resultList = query.getResultList();
+ return resultList;
+ }
+}
Deleted: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/SiteErrorJpaDao.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/SiteErrorJpaDao.java 2013-04-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/SiteErrorJpaDao.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -1,47 +0,0 @@
-package com.franciaflex.magalie.persistence.dao.jpa;
-
-import com.franciaflex.magalie.persistence.dao.SiteErrorDao;
-import com.franciaflex.magalie.persistence.entity.Site;
-import com.franciaflex.magalie.persistence.entity.SiteError;
-import com.google.common.collect.Iterables;
-
-import javax.persistence.EntityManager;
-import javax.persistence.Query;
-import java.util.List;
-
-public class SiteErrorJpaDao extends AbstractJpaDao<SiteError> implements SiteErrorDao {
-
- public SiteErrorJpaDao(EntityManager entityManager) {
- super(entityManager);
- }
-
- @Override
- protected Class<SiteError> getEntityClass() {
- return SiteError.class;
- }
-
- @Override
- public SiteError findBySite(Site site) {
- Query query = entityManager.createQuery("from SiteError se where se.site = :site");
- query.setParameter("site", site);
- SiteError siteError = (SiteError)
- Iterables.getOnlyElement(
- query.getResultList(),
- null);
- return siteError;
- }
-
- @Override
- public List<Site> getAllSitesInError() {
- Query query = entityManager.createQuery("select se.site from SiteError se");
- List<Site> allSitesInError = query.getResultList();
- return allSitesInError;
- }
-
- @Override
- public List<SiteError> findAll() {
- Query query = entityManager.createQuery("from SiteError se order by se.reportDate");
- List<SiteError> all = query.getResultList();
- return all;
- }
-}
Deleted: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/SiteJpaDao.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/SiteJpaDao.java 2013-04-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/SiteJpaDao.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -1,27 +0,0 @@
-package com.franciaflex.magalie.persistence.dao.jpa;
-
-import com.franciaflex.magalie.persistence.dao.SiteDao;
-import com.franciaflex.magalie.persistence.entity.Site;
-
-import javax.persistence.EntityManager;
-import javax.persistence.Query;
-import java.util.List;
-
-public class SiteJpaDao extends AbstractJpaDao<Site> implements SiteDao {
-
- public SiteJpaDao(EntityManager entityManager) {
- super(entityManager);
- }
-
- @Override
- protected Class<Site> getEntityClass() {
- return Site.class;
- }
-
- @Override
- public List<String> getAllStoreCodes() {
- Query query = entityManager.createQuery("select distinct s.storeCode from Site s order by s.storeCode");
- List<String> resultList = query.getResultList();
- return resultList;
- }
-}
Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/StoredArticleJpaDao.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/StoredArticleJpaDao.java 2013-04-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/dao/jpa/StoredArticleJpaDao.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -22,7 +22,7 @@
@Override
public List<StoredArticle> findAllForArticleInBuilding(Article article, Building building) {
- Query query = entityManager.createQuery("from StoredArticle sa where sa.article = :article and sa.site.store.building = :building");
+ Query query = entityManager.createQuery("from StoredArticle sa where sa.article = :article and sa.location.store.building = :building");
query.setParameter("article", article);
query.setParameter("building", building);
List<StoredArticle> resultList = query.getResultList();
Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Article.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Article.java 2013-04-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Article.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -12,7 +12,7 @@
protected String id;
@ManyToMany
- protected Set<Site> fixedSites;
+ protected Set<Location> fixedLocations;
protected String description;
@@ -31,12 +31,12 @@
this.id = id;
}
- public Set<Site> getFixedSites() {
- return fixedSites;
+ public Set<Location> getFixedLocations() {
+ return fixedLocations;
}
- public void setFixedSites(Set<Site> fixedSites) {
- this.fixedSites = fixedSites;
+ public void setFixedLocations(Set<Location> fixedLocations) {
+ this.fixedLocations = fixedLocations;
}
public String getCode() {
@@ -71,12 +71,12 @@
this.description = description;
}
- public boolean isFixedSite(Site site) {
- boolean isFixedSite = false;
- Set<Site> fixedSites = getFixedSites();
- if (fixedSites != null) {
- isFixedSite = fixedSites.contains(site);
+ public boolean isFixedLocation(Location location) {
+ boolean isFixedLocation = false;
+ Set<Location> fixedLocations = getFixedLocations();
+ if (fixedLocations != null) {
+ isFixedLocation = fixedLocations.contains(location);
}
- return isFixedSite;
+ return isFixedLocation;
}
}
Copied: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Location.java (from rev 83, trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Site.java)
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Location.java (rev 0)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Location.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -0,0 +1,65 @@
+package com.franciaflex.magalie.persistence.entity;
+
+import javax.persistence.Entity;
+import javax.persistence.Id;
+import javax.persistence.ManyToOne;
+
+@Entity
+public class Location extends AbstractEntity {
+
+ @Id
+ protected String id;
+
+ @ManyToOne
+ protected Store store;
+
+ protected String locationCode;
+
+ protected int requiredAccreditationLevel;
+
+ @Override
+ public String getId() {
+ return id;
+ }
+
+ public void setId(String id) {
+ this.id = id;
+ }
+
+ public Store getStore() {
+ return store;
+ }
+
+ public void setStore(Store store) {
+ this.store = store;
+ }
+
+ public String getLocationCode() {
+ return locationCode;
+ }
+
+ public void setLocationCode(String locationCode) {
+ this.locationCode = locationCode;
+ }
+
+ public int getRequiredAccreditationLevel() {
+ return requiredAccreditationLevel;
+ }
+
+ public void setRequiredAccreditationLevel(int requiredAccreditationLevel) {
+ this.requiredAccreditationLevel = requiredAccreditationLevel;
+ }
+
+ public String getBarcode() {
+ String barcode = getStoreCode() + getLocationCode();
+ return barcode;
+ }
+
+ public boolean isDriverLicenseRequired() {
+ return requiredAccreditationLevel == 9;
+ }
+
+ public String getStoreCode() {
+ return getStore().getName();
+ }
+}
Copied: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/LocationError.java (from rev 83, trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/SiteError.java)
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/LocationError.java (rev 0)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/LocationError.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -0,0 +1,57 @@
+package com.franciaflex.magalie.persistence.entity;
+
+import javax.persistence.Entity;
+import javax.persistence.GeneratedValue;
+import javax.persistence.Id;
+import javax.persistence.ManyToOne;
+import javax.persistence.OneToOne;
+import java.util.Date;
+
+@Entity
+public class LocationError extends AbstractEntity {
+
+ @Id
+ @GeneratedValue
+ protected String id;
+
+ @OneToOne
+ protected Location location;
+
+ @ManyToOne
+ protected MagalieUser magalieUser;
+
+ protected Date reportDate;
+
+ @Override
+ public String getId() {
+ return id;
+ }
+
+ public void setId(String id) {
+ this.id = id;
+ }
+
+ public Location getLocation() {
+ return location;
+ }
+
+ public void setLocation(Location location) {
+ this.location = location;
+ }
+
+ public MagalieUser getMagalieUser() {
+ return magalieUser;
+ }
+
+ public void setMagalieUser(MagalieUser magalieUser) {
+ this.magalieUser = magalieUser;
+ }
+
+ public Date getReportDate() {
+ return reportDate;
+ }
+
+ public void setReportDate(Date reportDate) {
+ this.reportDate = reportDate;
+ }
+}
Deleted: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Site.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Site.java 2013-04-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/Site.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -1,65 +0,0 @@
-package com.franciaflex.magalie.persistence.entity;
-
-import javax.persistence.Entity;
-import javax.persistence.Id;
-import javax.persistence.ManyToOne;
-
-@Entity
-public class Site extends AbstractEntity {
-
- @Id
- protected String id;
-
- @ManyToOne
- protected Store store;
-
- protected String locationCode;
-
- protected int requiredAccreditationLevel;
-
- @Override
- public String getId() {
- return id;
- }
-
- public void setId(String id) {
- this.id = id;
- }
-
- public Store getStore() {
- return store;
- }
-
- public void setStore(Store store) {
- this.store = store;
- }
-
- public String getLocationCode() {
- return locationCode;
- }
-
- public void setLocationCode(String locationCode) {
- this.locationCode = locationCode;
- }
-
- public int getRequiredAccreditationLevel() {
- return requiredAccreditationLevel;
- }
-
- public void setRequiredAccreditationLevel(int requiredAccreditationLevel) {
- this.requiredAccreditationLevel = requiredAccreditationLevel;
- }
-
- public String getBarcode() {
- String barcode = getStoreCode() + getLocationCode();
- return barcode;
- }
-
- public boolean isDriverLicenseRequired() {
- return requiredAccreditationLevel == 9;
- }
-
- public String getStoreCode() {
- return getStore().getName();
- }
-}
Deleted: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/SiteError.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/SiteError.java 2013-04-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/SiteError.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -1,57 +0,0 @@
-package com.franciaflex.magalie.persistence.entity;
-
-import javax.persistence.Entity;
-import javax.persistence.GeneratedValue;
-import javax.persistence.Id;
-import javax.persistence.ManyToOne;
-import javax.persistence.OneToOne;
-import java.util.Date;
-
-@Entity
-public class SiteError extends AbstractEntity {
-
- @Id
- @GeneratedValue
- protected String id;
-
- @OneToOne
- protected Site site;
-
- @ManyToOne
- protected MagalieUser magalieUser;
-
- protected Date reportDate;
-
- @Override
- public String getId() {
- return id;
- }
-
- public void setId(String id) {
- this.id = id;
- }
-
- public Site getSite() {
- return site;
- }
-
- public void setSite(Site site) {
- this.site = site;
- }
-
- public MagalieUser getMagalieUser() {
- return magalieUser;
- }
-
- public void setMagalieUser(MagalieUser magalieUser) {
- this.magalieUser = magalieUser;
- }
-
- public Date getReportDate() {
- return reportDate;
- }
-
- public void setReportDate(Date reportDate) {
- this.reportDate = reportDate;
- }
-}
Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/StorageMovement.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/StorageMovement.java 2013-04-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/StorageMovement.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -16,10 +16,10 @@
protected String id;
@ManyToOne
- protected Site originSite;
+ protected Location originLocation;
@ManyToOne
- protected Site destinationSite;
+ protected Location destinationLocation;
@ManyToOne
protected Article article;
@@ -44,20 +44,20 @@
this.id = id;
}
- public Site getOriginSite() {
- return originSite;
+ public Location getOriginLocation() {
+ return originLocation;
}
- public void setOriginSite(Site originSite) {
- this.originSite = originSite;
+ public void setOriginLocation(Location originLocation) {
+ this.originLocation = originLocation;
}
- public Site getDestinationSite() {
- return destinationSite;
+ public Location getDestinationLocation() {
+ return destinationLocation;
}
- public void setDestinationSite(Site destinationSite) {
- this.destinationSite = destinationSite;
+ public void setDestinationLocation(Location destinationLocation) {
+ this.destinationLocation = destinationLocation;
}
public Article getArticle() {
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-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/StorageMovementOrder.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -99,24 +99,24 @@
storageMovements.add(storageMovement);
}
- public Set<Site> getSites() {
- Set<Site> sites =
+ public Set<Location> getLocations() {
+ Set<Location> locations =
Sets.newHashSet(
Iterables.transform(
getStorageMovements(),
- StorageMovements.getOriginSiteFunction()
+ StorageMovements.getOriginFunction()
)
);
- return sites;
+ return locations;
}
- public StorageMovement findStorageMovementBySiteId(String siteId) {
+ public StorageMovement findStorageMovementByLocationId(String locationId) {
StorageMovement storageMovement =
Iterables.find(
getStorageMovements(),
Predicates.compose(
- Entities.entityHasId(siteId),
- StorageMovements.getOriginSiteFunction()
+ Entities.entityHasId(locationId),
+ StorageMovements.getOriginFunction()
),
null
);
Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/StoredArticle.java
===================================================================
--- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/StoredArticle.java 2013-04-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/StoredArticle.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -14,7 +14,7 @@
protected Article article;
@ManyToOne
- protected Site site;
+ protected Location location;
protected double quantity;
@@ -27,12 +27,12 @@
this.id = id;
}
- public Site getSite() {
- return site;
+ public Location getLocation() {
+ return location;
}
- public void setSite(Site site) {
- this.site = site;
+ public void setLocation(Location location) {
+ this.location = location;
}
public Article getArticle() {
Modified: trunk/magalie-persistence/src/test/java/com/franciaflex/magalie/PersistenceTest.java
===================================================================
--- trunk/magalie-persistence/src/test/java/com/franciaflex/magalie/PersistenceTest.java 2013-04-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-persistence/src/test/java/com/franciaflex/magalie/PersistenceTest.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -26,17 +26,17 @@
//// entityManager.persist(new MagalieUser());
//// entityManager.persist(new Article());
//// entityManager.persist(new StoredArticle());
-//// entityManager.persist(new Site());
+//// entityManager.persist(new Location());
//// entityManager.persist(new RequestedArticle());
//
-//// entityManager.persist(new SiteError());
+//// entityManager.persist(new LocationError());
//// entityManager.persist(new StorageMovement());
//
-// List<Site> storageMovements = fixtures.fixture("storageMovements");
+// List<Location> storageMovements = fixtures.fixture("storageMovements");
//
-// for (Site site : storageMovements) {
+// for (Location location : storageMovements) {
//
-// entityManager.persist(site);
+// entityManager.persist(location);
//
// }
//
Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/MagalieFixtures.java
===================================================================
--- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/MagalieFixtures.java 2013-04-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/MagalieFixtures.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -7,7 +7,7 @@
import com.franciaflex.magalie.persistence.entity.Building;
import com.franciaflex.magalie.persistence.entity.MagalieUser;
import com.franciaflex.magalie.persistence.entity.RequestedArticle;
-import com.franciaflex.magalie.persistence.entity.Site;
+import com.franciaflex.magalie.persistence.entity.Location;
import com.franciaflex.magalie.persistence.entity.Store;
import com.franciaflex.magalie.persistence.entity.StoredArticle;
import org.apache.commons.io.Charsets;
@@ -37,7 +37,7 @@
reader.getConfig().setClassTag("stored-article", StoredArticle.class);
reader.getConfig().setClassTag("user", MagalieUser.class);
reader.getConfig().setClassTag("requested-article", RequestedArticle.class);
- reader.getConfig().setClassTag("site", Site.class);
+ reader.getConfig().setClassTag("location", Location.class);
try {
fixtures = (Map<String, Object>) reader.read();
} catch (YamlException e) {
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-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/MagalieReport.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -1,8 +1,8 @@
package com.franciaflex.magalie.services;
import com.franciaflex.magalie.persistence.StorageMovements;
+import com.franciaflex.magalie.persistence.entity.LocationError;
import com.franciaflex.magalie.persistence.entity.MagalieUser;
-import com.franciaflex.magalie.persistence.entity.SiteError;
import com.franciaflex.magalie.persistence.entity.StorageMovement;
import com.google.common.collect.ImmutableList;
import com.google.common.collect.Iterables;
@@ -18,7 +18,7 @@
protected List<StorageMovement> allStorageMovements;
- protected List<SiteError> allSiteErrors;
+ protected List<LocationError> allLocationErrors;
public void setReportDate(Date reportDate) {
this.reportDate = reportDate;
@@ -62,11 +62,11 @@
return confirmedStorageMovements;
}
- public List<SiteError> getAllSiteErrors() {
- return allSiteErrors;
+ public List<LocationError> getAllLocationErrors() {
+ return allLocationErrors;
}
- public void setAllSiteErrors(List<SiteError> allSiteErrors) {
- this.allSiteErrors = allSiteErrors;
+ public void setAllLocationErrors(List<LocationError> allLocationErrors) {
+ this.allLocationErrors = allLocationErrors;
}
}
Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/StorageMovementConfirmation.java
===================================================================
--- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/StorageMovementConfirmation.java 2013-04-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/StorageMovementConfirmation.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -7,9 +7,9 @@
protected String storageMovementOrderId;
- protected Map<String, Double> sitesIdsToActualQuantities;
+ protected Map<String, Double> locationsIdsToActualQuantities;
- protected Set<String> siteInErrorIds;
+ protected Set<String> locationInErrorIds;
public String getStorageMovementOrderId() {
return storageMovementOrderId;
@@ -19,19 +19,19 @@
this.storageMovementOrderId = storageMovementOrderId;
}
- public Map<String, Double> getSitesIdsToActualQuantities() {
- return sitesIdsToActualQuantities;
+ public Map<String, Double> getLocationsIdsToActualQuantities() {
+ return locationsIdsToActualQuantities;
}
- public void setSitesIdsToActualQuantities(Map<String, Double> sitesIdsToActualQuantities) {
- this.sitesIdsToActualQuantities = sitesIdsToActualQuantities;
+ public void setLocationsIdsToActualQuantities(Map<String, Double> locationsIdsToActualQuantities) {
+ this.locationsIdsToActualQuantities = locationsIdsToActualQuantities;
}
- public Set<String> getSiteInErrorIds() {
- return siteInErrorIds;
+ public Set<String> getLocationInErrorIds() {
+ return locationInErrorIds;
}
- public void setSiteInErrorIds(Set<String> siteInErrorIds) {
- this.siteInErrorIds = siteInErrorIds;
+ public void setLocationInErrorIds(Set<String> locationInErrorIds) {
+ this.locationInErrorIds = locationInErrorIds;
}
}
Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/StorageMovementTask.java
===================================================================
--- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/StorageMovementTask.java 2013-04-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/StorageMovementTask.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -32,13 +32,13 @@
public List<StorageMovement> getStorageMovements() {
- List<StorageMovement> sites = Lists.newLinkedList();
+ List<StorageMovement> storageMovements = Lists.newLinkedList();
- sites.addAll(storageMovementOrder.getStorageMovements());
+ storageMovements.addAll(storageMovementOrder.getStorageMovements());
- sites.addAll(extraStorageMovements);
+ storageMovements.addAll(extraStorageMovements);
- return sites;
+ return storageMovements;
}
Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/exception/RequiredDriverLicenceException.java
===================================================================
--- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/exception/RequiredDriverLicenceException.java 2013-04-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/exception/RequiredDriverLicenceException.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -1,9 +1,9 @@
package com.franciaflex.magalie.services.exception;
-import com.franciaflex.magalie.persistence.Sites;
+import com.franciaflex.magalie.persistence.Locations;
import com.franciaflex.magalie.persistence.entity.Article;
+import com.franciaflex.magalie.persistence.entity.Location;
import com.franciaflex.magalie.persistence.entity.MagalieUser;
-import com.franciaflex.magalie.persistence.entity.Site;
import com.google.common.base.Predicate;
public class RequiredDriverLicenceException extends MagalieException {
@@ -12,7 +12,7 @@
protected MagalieUser magalieUser;
- protected Site site;
+ protected Location location;
protected RequiredDriverLicenceException(String message) {
super(message);
@@ -26,15 +26,15 @@
return magalieUser;
}
- public Site getSite() {
- return site;
+ public Location getLocation() {
+ return location;
}
- public static void throwIfUserCannotAccessSite(MagalieUser magalieUser, Site site, Article article) throws RequiredDriverLicenceException {
+ public static void throwIfUserCannotAccessLocation(MagalieUser magalieUser, Location location, Article article) throws RequiredDriverLicenceException {
- Predicate<Site> isSiteInaccessiblePredicate = Sites.inaccessibleSitePredicate(magalieUser);
+ Predicate<Location> isLocationInaccessiblePredicate = Locations.inaccessibleLocationPredicate(magalieUser);
- if (isSiteInaccessiblePredicate.apply(site)) {
+ if (isLocationInaccessiblePredicate.apply(location)) {
String message = "requested article " + article
+ " can not be retrieved since user " + magalieUser
@@ -44,7 +44,7 @@
newException.magalieUser = magalieUser;
- newException.site = site;
+ newException.location = location;
newException.article = article;
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-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ArticleStorageService.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -2,14 +2,14 @@
import com.franciaflex.magalie.persistence.MagaliePersistenceContext;
import com.franciaflex.magalie.persistence.StoredArticles;
-import com.franciaflex.magalie.persistence.dao.SiteDao;
+import com.franciaflex.magalie.persistence.dao.LocationDao;
import com.franciaflex.magalie.persistence.dao.StorageMovementDao;
import com.franciaflex.magalie.persistence.dao.StorageMovementOrderDao;
import com.franciaflex.magalie.persistence.dao.StoredArticleDao;
import com.franciaflex.magalie.persistence.entity.Article;
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.Site;
import com.franciaflex.magalie.persistence.entity.StorageMovement;
import com.franciaflex.magalie.persistence.entity.StorageMovementOrder;
import com.franciaflex.magalie.persistence.entity.StoredArticle;
@@ -51,23 +51,23 @@
}
public StorageMovementOrder bookArticleWithBestEffortPolicy(
- MagalieUser magalieUser, Building building, Article article, double requestedQuantity, Site destinationSite)
+ MagalieUser magalieUser, Building building, Article article, double requestedQuantity, Location destinationLocation)
throws RequiredDriverLicenceException, UnavailableArticleException {
- return bookArticle(magalieUser, building, article ,requestedQuantity, destinationSite, true);
+ return bookArticle(magalieUser, building, article ,requestedQuantity, destinationLocation, true);
}
public StorageMovementOrder bookArticle(
- MagalieUser magalieUser, Building building, Article article, double requestedQuantity, Site destinationSite)
+ MagalieUser magalieUser, Building building, Article article, double requestedQuantity, Location destinationLocation)
throws UnavailableArticleException, RequiredDriverLicenceException {
- return bookArticle(magalieUser, building, article ,requestedQuantity, destinationSite, false);
+ return bookArticle(magalieUser, building, article ,requestedQuantity, destinationLocation, false);
}
protected StorageMovementOrder bookArticle(
- MagalieUser magalieUser, Building building, Article article, double requestedQuantity, Site destinationSite, boolean bestEffortPolicy)
+ MagalieUser magalieUser, Building building, Article article, double requestedQuantity, Location destinationLocation, boolean bestEffortPolicy)
throws UnavailableArticleException, RequiredDriverLicenceException {
Preconditions.checkNotNull(magalieUser);
@@ -89,9 +89,9 @@
Ordering<StoredArticle> storedArticlesOrdering =
Ordering.compound(
Lists.newArrayList(
- StoredArticles.fixedSiteForArticleLastComparator(article),
- StoredArticles.siteWithLowestQuantityFirstComparator(),
- StoredArticles.articleStoredInSitesRequiringDriverLicenseFirstComparator()
+ StoredArticles.fixedLocationForArticleLastComparator(article),
+ StoredArticles.locationWithLowestQuantityFirstComparator(),
+ StoredArticles.articleStoredInLocationsRequiringDriverLicenseFirstComparator()
)
);
@@ -110,15 +110,15 @@
double quantityLeft = requestedQuantity - quantity;
- double quantityInSite = storedArticle.getQuantity();
+ double quantityInLocation = storedArticle.getQuantity();
- double quantityWithdrawn = Math.min(quantityLeft, quantityInSite);
+ double quantityWithdrawn = Math.min(quantityLeft, quantityInLocation);
if (quantityWithdrawn > 0.) {
- Site originSite = storedArticle.getSite();
+ Location originLocation = storedArticle.getLocation();
- RequiredDriverLicenceException.throwIfUserCannotAccessSite(magalieUser, originSite, article);
+ RequiredDriverLicenceException.throwIfUserCannotAccessLocation(magalieUser, originLocation, article);
quantity += quantityWithdrawn;
@@ -130,9 +130,9 @@
newStorageMovement.setMagalieUser(magalieUser);
- newStorageMovement.setOriginSite(originSite);
+ newStorageMovement.setOriginLocation(originLocation);
- newStorageMovement.setDestinationSite(destinationSite);
+ newStorageMovement.setDestinationLocation(destinationLocation);
newStorageMovement.setExpectedQuantity(quantityWithdrawn);
@@ -187,24 +187,24 @@
Article article = storageMovementOrder.getArticle();
- Set<Site> sitesInOrder = storageMovementOrder.getSites();
+ Set<Location> locationsInOrder = storageMovementOrder.getLocations();
// we suppose the order concerns movement in the same building
- Building building = Iterables.get(sitesInOrder, 0).getStore().getBuilding();
+ Building building = Iterables.get(locationsInOrder, 0).getStore().getBuilding();
List<StoredArticle> storedArticles = getStoredArticles(building, article);
for (StoredArticle storedArticle : storedArticles) {
- Site site = storedArticle.getSite();
+ Location location = storedArticle.getLocation();
- if ( ! sitesInOrder.contains(site)) {
+ if ( ! locationsInOrder.contains(location)) {
StorageMovement newStorageMovement = new StorageMovement();
newStorageMovement.setArticle(article);
- newStorageMovement.setOriginSite(site);
+ newStorageMovement.setOriginLocation(location);
newStorageMovement.setExpectedQuantity(0.);
@@ -231,17 +231,17 @@
List<StorageMovement> storageMovements = storageMovementDao.findAllByArticle(article);
- ImmutableMap<Site, StoredArticle> storedArticlesBySite =
+ ImmutableMap<Location, StoredArticle> storedArticlesByLocation =
Maps.uniqueIndex(
storedArticles,
- StoredArticles.getSiteFunction()
+ StoredArticles.getLocationFunction()
);
for (StorageMovement storageMovement : storageMovements) {
- Site originSite = storageMovement.getOriginSite();
+ Location originLocation = storageMovement.getOriginLocation();
- StoredArticle storedArticle = storedArticlesBySite.get(originSite);
+ StoredArticle storedArticle = storedArticlesByLocation.get(originLocation);
double actualQuantity = storedArticle.getQuantity() - storageMovement.getQuantity();
@@ -251,7 +251,7 @@
} else {
- // site is empty, do not include it as a site where you can take an article
+ // location is empty, do not include it as a location where you can take an article
storedArticles.remove(storedArticle);
}
@@ -260,15 +260,15 @@
// exclude storageMovements reported in error
- SiteErrorsService siteErrorsService = serviceContext.newService(SiteErrorsService.class);
+ LocationErrorsService locationErrorsService = serviceContext.newService(LocationErrorsService.class);
- List<Site> allSitesInError = siteErrorsService.getAllSitesInError();
+ List<Location> allLocationsInError = locationErrorsService.getAllLocationsInError();
storedArticles =
Lists.newArrayList(
Iterables.filter(
storedArticles,
- StoredArticles.articleNotStoredInSiteReportedInError(allSitesInError)
+ StoredArticles.articleNotStoredInLocationReportedInError(allLocationsInError)
)
);
@@ -283,35 +283,35 @@
StorageMovementOrderDao storageMovementOrderDao =
serviceContext.getPersistenceContext().getStorageMovementOrderDao();
- SiteDao siteDao = serviceContext.getPersistenceContext().getSiteDao();
+ LocationDao locationDao = serviceContext.getPersistenceContext().getLocationDao();
StorageMovementOrder storageMovementOrder =
storageMovementOrderDao.findById(storageMovementOrderId);
Date confirmDate = serviceContext.getNow();
- SiteErrorsService siteErrorsService =
- serviceContext.newService(SiteErrorsService.class);
+ LocationErrorsService locationErrorsService =
+ serviceContext.newService(LocationErrorsService.class);
- for (Map.Entry<String, Double> siteIdToActualQuantity : confirmation.getSitesIdsToActualQuantities().entrySet()) {
+ for (Map.Entry<String, Double> locationIdToActualQuantity : confirmation.getLocationsIdsToActualQuantities().entrySet()) {
- String siteId = siteIdToActualQuantity.getKey();
+ String locationId = locationIdToActualQuantity.getKey();
- double actualQuantity = siteIdToActualQuantity.getValue();
+ double actualQuantity = locationIdToActualQuantity.getValue();
StorageMovement storageMovement =
- storageMovementOrder.findStorageMovementBySiteId(siteId);
+ storageMovementOrder.findStorageMovementByLocationId(locationId);
if (storageMovement == null) {
// user made a movement out of the order from its own initiative
storageMovement = new StorageMovement();
- Site originSite = siteDao.findById(siteId);
+ Location originLocation = locationDao.findById(locationId);
- storageMovement.setOriginSite(originSite);
+ storageMovement.setOriginLocation(originLocation);
- // TODO brendan 28/03/13 what about destination site
+ // TODO brendan 28/03/13 what about destination location
storageMovement.setMagalieUser(magalieUser);
@@ -329,11 +329,11 @@
}
- for (String siteInErrorId : confirmation.getSiteInErrorIds()) {
+ for (String locationInErrorId : confirmation.getLocationInErrorIds()) {
- Site siteInError = siteDao.findById(siteInErrorId);
+ Location locationInError = locationDao.findById(locationInErrorId);
- siteErrorsService.reportError(siteInError, magalieUser, confirmDate);
+ locationErrorsService.reportError(locationInError, magalieUser, confirmDate);
}
Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/FixturesService.java
===================================================================
--- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/FixturesService.java 2013-04-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/FixturesService.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -5,14 +5,14 @@
import com.franciaflex.magalie.persistence.dao.BuildingDao;
import com.franciaflex.magalie.persistence.dao.MagalieUserDao;
import com.franciaflex.magalie.persistence.dao.RequestedArticleDao;
-import com.franciaflex.magalie.persistence.dao.SiteDao;
+import com.franciaflex.magalie.persistence.dao.LocationDao;
import com.franciaflex.magalie.persistence.dao.StoreDao;
import com.franciaflex.magalie.persistence.dao.StoredArticleDao;
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.RequestedArticle;
-import com.franciaflex.magalie.persistence.entity.Site;
+import com.franciaflex.magalie.persistence.entity.Location;
import com.franciaflex.magalie.persistence.entity.Store;
import com.franciaflex.magalie.persistence.entity.StoredArticle;
import com.franciaflex.magalie.services.MagalieFixtures;
@@ -109,13 +109,13 @@
persistenceContext.commit();
- SiteDao siteDao = persistenceContext.getSiteDao();
+ LocationDao locationDao = persistenceContext.getLocationDao();
- Collection<Site> sites = fixtures.fixture("sites");
+ Collection<Location> locations = fixtures.fixture("locations");
- for (Site site : sites) {
+ for (Location location : locations) {
- siteDao.persist(site);
+ locationDao.persist(location);
}
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-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/FulfilKanbanService.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -4,8 +4,8 @@
import com.franciaflex.magalie.persistence.dao.StoreDao;
import com.franciaflex.magalie.persistence.entity.Article;
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.Site;
import com.franciaflex.magalie.persistence.entity.StorageMovementOrder;
import com.franciaflex.magalie.persistence.entity.Store;
import com.franciaflex.magalie.services.MagalieService;
@@ -68,7 +68,7 @@
Building building = destinationStore.getBuilding();
- Site destinationSite = getDestinationSite(destinationStore);
+ Location destinationLocation = getDestinationLocation(destinationStore);
StorageMovementOrder storageMovementOrder =
articleStorageService.bookArticleWithBestEffortPolicy(
@@ -76,13 +76,13 @@
building,
article,
quantityInKanban,
- destinationSite);
+ destinationLocation);
return storageMovementOrder;
}
- protected Site getDestinationSite(Store destinationStore) {
+ protected Location getDestinationLocation(Store destinationStore) {
// TODO brendan 10/04/13 how to define destination for store, something like "INC" ?
Copied: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/LocationErrorsService.java (from rev 83, trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/SiteErrorsService.java)
===================================================================
--- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/LocationErrorsService.java (rev 0)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/LocationErrorsService.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -0,0 +1,82 @@
+package com.franciaflex.magalie.services.service;
+
+import com.franciaflex.magalie.persistence.dao.LocationErrorDao;
+import com.franciaflex.magalie.persistence.entity.Location;
+import com.franciaflex.magalie.persistence.entity.LocationError;
+import com.franciaflex.magalie.persistence.entity.MagalieUser;
+import com.franciaflex.magalie.services.MagalieService;
+import com.franciaflex.magalie.services.MagalieServiceContext;
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+
+import java.util.Date;
+import java.util.List;
+
+public class LocationErrorsService implements MagalieService {
+
+ private static final Log log = LogFactory.getLog(LocationErrorsService.class);
+
+ protected MagalieServiceContext serviceContext;
+
+ @Override
+ public void setServiceContext(MagalieServiceContext serviceContext) {
+ this.serviceContext = serviceContext;
+ }
+
+ public void reportError(Location location, MagalieUser magalieUser) {
+
+ Date reportDate = serviceContext.getNow();
+
+ reportError(location, magalieUser, reportDate);
+
+ }
+
+ public void reportError(Location location, MagalieUser magalieUser, Date reportDate) {
+
+ LocationErrorDao locationErrorDao = serviceContext.getPersistenceContext().getLocationErrorDao();
+
+ LocationError existingLocationError = locationErrorDao.findByLocation(location);
+
+ if (existingLocationError == null) {
+
+ LocationError newLocationError = new LocationError();
+
+ newLocationError.setMagalieUser(magalieUser);
+
+ newLocationError.setLocation(location);
+
+ newLocationError.setReportDate(reportDate);
+
+ locationErrorDao.persist(newLocationError);
+
+ serviceContext.getPersistenceContext().commit();
+
+ } else {
+
+ if (log.isInfoEnabled()) {
+ log.info("error on location " + location + " already reported, nothing saved");
+ }
+
+ }
+ }
+
+ public List<Location> getAllLocationsInError() {
+
+ LocationErrorDao locationErrorDao = serviceContext.getPersistenceContext().getLocationErrorDao();
+
+ List<Location> allLocationsInError = locationErrorDao.getAllLocationsInError();
+
+ return allLocationsInError;
+
+ }
+
+ public List<LocationError> getAllLocationErrors() {
+
+ LocationErrorDao locationErrorDao = serviceContext.getPersistenceContext().getLocationErrorDao();
+
+ List<LocationError> allLocationErrors = locationErrorDao.findAll();
+
+ return allLocationErrors;
+
+ }
+}
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-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ReportService.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -1,8 +1,8 @@
package com.franciaflex.magalie.services.service;
import com.franciaflex.magalie.persistence.dao.StorageMovementDao;
+import com.franciaflex.magalie.persistence.entity.LocationError;
import com.franciaflex.magalie.persistence.entity.MagalieUser;
-import com.franciaflex.magalie.persistence.entity.SiteError;
import com.franciaflex.magalie.persistence.entity.StorageMovement;
import com.franciaflex.magalie.services.MagalieReport;
import com.franciaflex.magalie.services.MagalieService;
@@ -40,11 +40,11 @@
magalieReport.setAllStorageMovements(allStorageMovements);
- SiteErrorsService siteErrorsService = serviceContext.newService(SiteErrorsService.class);
+ LocationErrorsService locationErrorsService = serviceContext.newService(LocationErrorsService.class);
- List<SiteError> allSitesInError = siteErrorsService.getAllSiteErrors();
+ List<LocationError> allLocationsInError = locationErrorsService.getAllLocationErrors();
- magalieReport.setAllSiteErrors(allSitesInError);
+ magalieReport.setAllLocationErrors(allLocationsInError);
return magalieReport;
Deleted: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/SiteErrorsService.java
===================================================================
--- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/SiteErrorsService.java 2013-04-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/SiteErrorsService.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -1,82 +0,0 @@
-package com.franciaflex.magalie.services.service;
-
-import com.franciaflex.magalie.persistence.dao.SiteErrorDao;
-import com.franciaflex.magalie.persistence.entity.MagalieUser;
-import com.franciaflex.magalie.persistence.entity.Site;
-import com.franciaflex.magalie.persistence.entity.SiteError;
-import com.franciaflex.magalie.services.MagalieService;
-import com.franciaflex.magalie.services.MagalieServiceContext;
-import org.apache.commons.logging.Log;
-import org.apache.commons.logging.LogFactory;
-
-import java.util.Date;
-import java.util.List;
-
-public class SiteErrorsService implements MagalieService {
-
- private static final Log log = LogFactory.getLog(SiteErrorsService.class);
-
- protected MagalieServiceContext serviceContext;
-
- @Override
- public void setServiceContext(MagalieServiceContext serviceContext) {
- this.serviceContext = serviceContext;
- }
-
- public void reportError(Site site, MagalieUser magalieUser) {
-
- Date reportDate = serviceContext.getNow();
-
- reportError(site, magalieUser, reportDate);
-
- }
-
- public void reportError(Site site, MagalieUser magalieUser, Date reportDate) {
-
- SiteErrorDao siteErrorDao = serviceContext.getPersistenceContext().getSiteErrorDao();
-
- SiteError existingSiteError = siteErrorDao.findBySite(site);
-
- if (existingSiteError == null) {
-
- SiteError newSiteError = new SiteError();
-
- newSiteError.setMagalieUser(magalieUser);
-
- newSiteError.setSite(site);
-
- newSiteError.setReportDate(reportDate);
-
- siteErrorDao.persist(newSiteError);
-
- serviceContext.getPersistenceContext().commit();
-
- } else {
-
- if (log.isInfoEnabled()) {
- log.info("error on site " + site + " already reported, nothing saved");
- }
-
- }
- }
-
- public List<Site> getAllSitesInError() {
-
- SiteErrorDao siteErrorDao = serviceContext.getPersistenceContext().getSiteErrorDao();
-
- List<Site> allSitesInError = siteErrorDao.getAllSitesInError();
-
- return allSitesInError;
-
- }
-
- public List<SiteError> getAllSiteErrors() {
-
- SiteErrorDao siteErrorDao = serviceContext.getPersistenceContext().getSiteErrorDao();
-
- List<SiteError> allSiteErrors = siteErrorDao.findAll();
-
- return allSiteErrors;
-
- }
-}
Modified: trunk/magalie-services/src/main/resources/fixtures.yaml
===================================================================
--- trunk/magalie-services/src/main/resources/fixtures.yaml 2013-04-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-services/src/main/resources/fixtures.yaml 2013-04-11 19:44:51 UTC (rev 87)
@@ -77,118 +77,118 @@
- *U04
- *SOM
-site1:
- &site1 !site
- id: site1
+location1:
+ &location1 !location
+ id: location1
store: *U01
locationCode: A01
requiredAccreditationLevel: 0
-site2:
- &site2 !site
- id: site2
+location2:
+ &location2 !location
+ id: location2
store: *U01
locationCode: B02
requiredAccreditationLevel: 9
-site3:
- &site3 !site
- id: site3
+location3:
+ &location3 !location
+ id: location3
store: *U01
locationCode: C03
requiredAccreditationLevel: 0
-site4:
- &site4 !site
- id: site4
+location4:
+ &location4 !location
+ id: location4
store: *U01
locationCode: D04
requiredAccreditationLevel: 9
-site5:
- &site5 !site
- id: site5
+location5:
+ &location5 !location
+ id: location5
store: *U02
locationCode: F01
requiredAccreditationLevel: 0
-site6:
- &site6 !site
- id: site6
+location6:
+ &location6 !location
+ id: location6
store: *U02
locationCode: G02
requiredAccreditationLevel: 0
-site7:
- &site7 !site
- id: site7
+location7:
+ &location7 !location
+ id: location7
store: *U02
locationCode: H03
requiredAccreditationLevel: 9
-site8:
- &site8 !site
- id: site8
+location8:
+ &location8 !location
+ id: location8
store: *U04
locationCode: A010
requiredAccreditationLevel: 0
-site9:
- &site9 !site
- id: site9
+location9:
+ &location9 !location
+ id: location9
store: *SOM
locationCode: B10
requiredAccreditationLevel: 0
-site10:
- &site10 !site
- id: site10
+location10:
+ &location10 !location
+ id: location10
store: *SOM
locationCode: B24
requiredAccreditationLevel: 0
-site11:
- &site11 !site
- id: site11
+location11:
+ &location11 !location
+ id: location11
store: *SOM
locationCode: B77
requiredAccreditationLevel: 0
-site12:
- &site12 !site
- id: site12
+location12:
+ &location12 !location
+ id: location12
store: *SOM
locationCode: C12
requiredAccreditationLevel: 0
-site13:
- &site13 !site
- id: site13
+location13:
+ &location13 !location
+ id: location13
store: *U01
locationCode: REC
requiredAccreditationLevel: 0
-sites:
- - *site1
- - *site2
- - *site3
- - *site4
- - *site5
- - *site6
- - *site7
- - *site8
- - *site9
- - *site10
- - *site11
- - *site12
- - *site13
+locations:
+ - *location1
+ - *location2
+ - *location3
+ - *location4
+ - *location5
+ - *location6
+ - *location7
+ - *location8
+ - *location9
+ - *location10
+ - *location11
+ - *location12
+ - *location13
article1:
&article1 !article
id: article1
- fixedSites:
- - *site4
- - *site8
+ fixedLocations:
+ - *location4
+ - *location8
code: 4034108
quantityInKanban: 50
unit: M
@@ -197,8 +197,8 @@
article2:
&article2 !article
id: article2
- fixedSites:
- - *site11
+ fixedLocations:
+ - *location11
code: 4033079
quantityInKanban: 3
unit: PCE
@@ -207,8 +207,8 @@
article3:
&article3 !article
id: article3
- fixedSites:
- - *site1
+ fixedLocations:
+ - *location1
code: 4092459
quantityInKanban: 50
unit: PCE
@@ -225,8 +225,8 @@
article5:
&article5 !article
id: article5
- fixedSites:
- - *site5
+ fixedLocations:
+ - *location5
code: 4020012
quantityInKanban: 10
unit: B60
@@ -235,8 +235,8 @@
article6:
&article6 !article
id: article6
- fixedSites:
- - *site3
+ fixedLocations:
+ - *location3
code: 1421812
quantityInKanban: 9
unit: PCE
@@ -254,91 +254,91 @@
&storedArticle1 !stored-article
id: storedArticle1
article: *article1
- site: *site4
+ location: *location4
quantity: 40
storedArticle2:
&storedArticle2 !stored-article
id: storedArticle2
article: *article1
- site: *site8
+ location: *location8
quantity: 20
storedArticle3:
&storedArticle3 !stored-article
id: storedArticle3
article: *article2
- site: *site10
+ location: *location10
quantity: 2
storedArticle4:
&storedArticle4 !stored-article
id: storedArticle4
article: *article3
- site: *site1
+ location: *location1
quantity: 90
storedArticle5:
&storedArticle5 !stored-article
id: storedArticle5
article: *article1
- site: *site13
+ location: *location13
quantity: 30
storedArticle6:
&storedArticle6 !stored-article
id: storedArticle6
article: *article2
- site: *site11
+ location: *location11
quantity: 100
storedArticle7:
&storedArticle7 !stored-article
id: storedArticle7
article: *article2
- site: *site12
+ location: *location12
quantity: 10
storedArticle8:
&storedArticle8 !stored-article
id: storedArticle8
article: *article3
- site: *site2
+ location: *location2
quantity: 500
storedArticle9:
&storedArticle9 !stored-article
id: storedArticle9
article: *article3
- site: *site3
+ location: *location3
quantity: 50
storedArticle10:
&storedArticle10 !stored-article
id: storedArticle10
article: *article5
- site: *site5
+ location: *location5
quantity: 5
storedArticle11:
&storedArticle11 !stored-article
id: storedArticle11
article: *article5
- site: *site6
+ location: *location6
quantity: 2
storedArticle12:
&storedArticle12 !stored-article
id: storedArticle12
article: *article5
- site: *site7
+ location: *location7
quantity: 25
storedArticle13:
&storedArticle13 !stored-article
id: storedArticle13
article: *article6
- site: *site3
+ location: *location3
quantity: 70
storedArticles:
Modified: trunk/magalie-services/src/main/resources/fixtures2.yaml
===================================================================
--- trunk/magalie-services/src/main/resources/fixtures2.yaml 2013-04-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-services/src/main/resources/fixtures2.yaml 2013-04-11 19:44:51 UTC (rev 87)
@@ -77,53 +77,53 @@
- *U04
- *SOM
-site1:
- &site1 !site
- id: site1
+location1:
+ &location1 !location
+ id: location1
store: *U01
locationCode: A1
requiredAccreditationLevel: 0
-site2:
- &site2 !site
- id: site2
+location2:
+ &location2 !location
+ id: location2
store: *U01
locationCode: B2
requiredAccreditationLevel: 9
-site3:
- &site3 !site
- id: site3
+location3:
+ &location3 !location
+ id: location3
store: *U01
locationCode: C3
requiredAccreditationLevel: 0
-site4:
- &site4 !site
- id: site4
+location4:
+ &location4 !location
+ id: location4
store: *U01
locationCode: D4
requiredAccreditationLevel: 9
-site5:
- &site5 !site
- id: site5
+location5:
+ &location5 !location
+ id: location5
store: *U02
locationCode: A1
requiredAccreditationLevel: 0
-sites:
- - *site1
- - *site2
- - *site3
- - *site4
- - *site5
+locations:
+ - *location1
+ - *location2
+ - *location3
+ - *location4
+ - *location5
article1:
&article1 !article
id: article1
- fixedSites:
- - *site1
+ fixedLocations:
+ - *location1
code: 111111111
quantityInKanban: 10
unit: kg
@@ -140,8 +140,8 @@
article3:
&article3 !article
id: article3
- fixedSites:
- - *site2
+ fixedLocations:
+ - *location2
code: 333333333
quantityInKanban: 50
unit: g
@@ -156,28 +156,28 @@
&storedArticle1 !stored-article
id: storedArticle1
article: *article1
- site: *site1
+ location: *location1
quantity: 50
storedArticle2:
&storedArticle2 !stored-article
id: storedArticle2
article: *article1
- site: *site2
+ location: *location2
quantity: 20
storedArticle3:
&storedArticle3 !stored-article
id: storedArticle3
article: *article2
- site: *site1
+ location: *location1
quantity: 10
storedArticle4:
&storedArticle4 !stored-article
id: storedArticle4
article: *article3
- site: *site4
+ location: *location4
quantity: 1000
storedArticles:
Modified: trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/ArticleStorageServiceTest.java
===================================================================
--- trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/ArticleStorageServiceTest.java 2013-04-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/ArticleStorageServiceTest.java 2013-04-11 19:44:51 UTC (rev 87)
@@ -2,8 +2,8 @@
import com.franciaflex.magalie.persistence.entity.Article;
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.Site;
import com.franciaflex.magalie.persistence.entity.StorageMovement;
import com.franciaflex.magalie.persistence.entity.StorageMovementOrder;
import com.franciaflex.magalie.services.AbstractMagalieServiceTest;
@@ -66,7 +66,7 @@
MagalieUser basile = fixture("basile");
Article article = fixture("article1");
- Site site2 = fixture("site2");
+ Location location2 = fixture("location2");
StorageMovementOrder storageMovementOrder;
try {
@@ -79,7 +79,7 @@
if (log.isDebugEnabled()) {
log.debug("exception raised = " + e);
}
- Assert.assertEquals(site2.getId(), e.getSite().getId());
+ Assert.assertEquals(location2.getId(), e.getLocation().getId());
}
}
@@ -135,10 +135,10 @@
}
Assert.assertEquals(2, storageMovementOrder.getStorageMovements().size());
- StorageMovement site1 = Iterables.get(storageMovementOrder.getStorageMovements(), 0, null);
- Assert.assertEquals(20., site1.getExpectedQuantity(), DELTA);
- StorageMovement site2 = Iterables.get(storageMovementOrder.getStorageMovements(), 1, null);
- Assert.assertEquals(30., site2.getExpectedQuantity(), DELTA);
+ StorageMovement location1 = Iterables.get(storageMovementOrder.getStorageMovements(), 0, null);
+ Assert.assertEquals(20., location1.getExpectedQuantity(), DELTA);
+ StorageMovement location2 = Iterables.get(storageMovementOrder.getStorageMovements(), 1, null);
+ Assert.assertEquals(30., location2.getExpectedQuantity(), DELTA);
}
@Test
@@ -159,10 +159,10 @@
Assert.assertEquals(2, storageMovementOrder.getStorageMovements().size());
StorageMovement storageMovement1 = Iterables.get(storageMovementOrder.getStorageMovements(), 0, null);
Assert.assertEquals(20., storageMovement1.getExpectedQuantity(), DELTA);
- Assert.assertEquals("B2", storageMovement1.getOriginSite().getLocationCode());
+ Assert.assertEquals("B2", storageMovement1.getOriginLocation().getLocationCode());
StorageMovement storageMovement2 = Iterables.get(storageMovementOrder.getStorageMovements(), 1, null);
Assert.assertEquals(10., storageMovement2.getExpectedQuantity(), DELTA);
- Assert.assertEquals("A1", storageMovement2.getOriginSite().getLocationCode());
+ Assert.assertEquals("A1", storageMovement2.getOriginLocation().getLocationCode());
} catch (RequiredDriverLicenceException e) {
if (log.isDebugEnabled()) {
log.debug("exception raised = " + e);
@@ -183,7 +183,7 @@
Assert.assertEquals(1, storageMovementOrder.getStorageMovements().size());
StorageMovement storageMovement = Iterables.get(storageMovementOrder.getStorageMovements(), 0, null);
Assert.assertEquals(30., storageMovement.getExpectedQuantity(), DELTA);
- Assert.assertEquals("A1", storageMovement.getOriginSite().getLocationCode());
+ Assert.assertEquals("A1", storageMovement.getOriginLocation().getLocationCode());
} catch (RequiredDriverLicenceException e) {
if (log.isDebugEnabled()) {
log.debug("exception raised = " + e);
@@ -198,17 +198,17 @@
}
@Test
- public void testWithdrawArticleConsiderSitesInError() throws Exception {
+ public void testWithdrawArticleConsiderLocationsInError() throws Exception {
MagalieUser alexandre = fixture("alexandre");
MagalieUser cathy = fixture("cathy");
Article article = fixture("article1");
- // report site 2 in error
- Site site2 = fixture("site2");
- SiteErrorsService siteErrorsService = newService(SiteErrorsService.class);
- siteErrorsService.reportError(site2, alexandre);
+ // report location 2 in error
+ Location location2 = fixture("location2");
+ LocationErrorsService locationErrorsService = newService(LocationErrorsService.class);
+ locationErrorsService.reportError(location2, alexandre);
try {
StorageMovementOrder storageMovementOrder = service.bookArticle(cathy, building, article, 30.);
@@ -218,9 +218,9 @@
Assert.assertEquals(1, storageMovementOrder.getStorageMovements().size());
StorageMovement storageMovement = Iterables.get(storageMovementOrder.getStorageMovements(), 0, null);
Assert.assertNotEquals(
- "site2 must not be returned because it was reported in error",
- site2.getId(),
- storageMovement.getOriginSite().getId());
+ "location2 must not be returned because it was reported in error",
+ location2.getId(),
+ storageMovement.getOriginLocation().getId());
} catch (RequiredDriverLicenceException e) {
if (log.isDebugEnabled()) {
log.debug("exception raised = " + e);
@@ -239,7 +239,7 @@
MagalieUser alexandre = fixture("alexandre");
Article article = fixture("article2");
- Site site1 = fixture("site1");
+ Location location1 = fixture("location1");
try {
StorageMovementOrder storageMovementOrder = service.bookArticleWithBestEffortPolicy(alexandre, building, article, 30., null);
@@ -249,7 +249,7 @@
Assert.assertEquals(1, storageMovementOrder.getStorageMovements().size());
StorageMovement storageMovement = Iterables.get(storageMovementOrder.getStorageMovements(), 0, null);
Assert.assertEquals("only 10 is available, so don't propose more", 10., storageMovement.getQuantity(), DELTA);
- Assert.assertEquals(site1.getId(), storageMovement.getOriginSite().getId());
+ Assert.assertEquals(location1.getId(), storageMovement.getOriginLocation().getId());
} catch (RequiredDriverLicenceException e) {
if (log.isDebugEnabled()) {
log.debug("exception raised = " + e);
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-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-web/src/main/webapp/WEB-INF/content/report.jsp 2013-04-11 19:44:51 UTC (rev 87)
@@ -48,15 +48,15 @@
<s:property value="id" />
</td>
<td>
- <s:property value="originSite.storeCode" />
+ <s:property value="originLocation.storeCode" />
-
- <s:property value="originSite.locationCode" />
+ <s:property value="originLocation.locationCode" />
</td>
<td>
- <s:if test="destinationSite">
- <s:property value="destinationSite.storeCode" />
+ <s:if test="destinationLocation">
+ <s:property value="destinationLocation.storeCode" />
-
- <s:property value="destinationSite.locationCode" />
+ <s:property value="destinationLocation.locationCode" />
</s:if>
</td>
<td>
@@ -120,15 +120,15 @@
<s:property value="id" />
</td>
<td>
- <s:property value="originSite.storeCode" />
+ <s:property value="originLocation.storeCode" />
-
- <s:property value="originSite.locationCode" />
+ <s:property value="originLocation.locationCode" />
</td>
<td>
- <s:if test="destinationSite">
- <s:property value="destinationSite.storeCode" />
+ <s:if test="destinationLocation">
+ <s:property value="destinationLocation.storeCode" />
-
- <s:property value="destinationSite.locationCode" />
+ <s:property value="destinationLocation.locationCode" />
</s:if>
</td>
<td>
@@ -180,16 +180,16 @@
</thead>
<tbody>
- <s:iterator value="report.allSiteErrors">
+ <s:iterator value="report.allLocationErrors">
<tr>
<td>
<s:property value="id" />
</td>
<td>
- <s:property value="site.storeCode" />
+ <s:property value="location.storeCode" />
</td>
<td>
- <s:property value="site.locationCode" />
+ <s:property value="location.locationCode" />
</td>
<td>
<s:property value="magalieUser.name" />
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-04-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-web/src/main/webapp/WEB-INF/content/withdraw-item-input.jsp 2013-04-11 19:44:51 UTC (rev 87)
@@ -99,7 +99,7 @@
<section id="storageMovements">
</section>
<s:form cssClass="form-horizontal">
- <s:textfield name="siteBarcode" label="Empl." inputAppendIcon="barcode" cssClass="input-small" />
+ <s:textfield name="locationBarcode" label="Empl." inputAppendIcon="barcode" cssClass="input-small" />
<s:textfield name="quantity" label="Qté" inputAppend="%{storageMovementTask.article.unit}" cssClass="input-mini" />
<s:hidden name="confirmation" />
@@ -111,7 +111,7 @@
<div id="storageMovementTemplate" class="template">
<dl class="dl-horizontal">
<dt>Empl.</dt>
- <dd><span data="originSite.store.name"></span> <span data="originSite.locationCode"></span></dd>
+ <dd><span data="originLocation.store.name"></span> <span data="originLocation.locationCode"></span></dd>
<dt>Prélev.</dt>
<dd><span data="withdrawn">0</span> / <span data="expectedQuantity"></span> <s:property value="storageMovementTask.article.unit" /></dd>
</dl>
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-11 15:30:47 UTC (rev 86)
+++ trunk/magalie-web/src/main/webapp/js/withdraw-item-input.js 2013-04-11 19:44:51 UTC (rev 87)
@@ -75,7 +75,7 @@
model.getStorageMovement = function(barcode) {
var storageMovement;
for (var i = 0; i < this.storageMovements.length; i++) {
- if (barcode == this.storageMovements[i].originSite.barcode) {
+ if (barcode == this.storageMovements[i].originLocation.barcode) {
storageMovement = this.storageMovements[i];
}
}
@@ -92,7 +92,7 @@
var storageMovement = this.getStorageMovement(barcode);
storageMovement.used = true;
storageMovement.defect =
- storageMovement.expectedQuantity != 0. // don't mark a site in error just because we wasn't supposed to withdraw item from it in the order
+ storageMovement.expectedQuantity != 0. // don't mark a location in error just because we wasn't supposed to withdraw item from it in the order
&& quantity != storageMovement.expectedQuantity;
storageMovement.withdrawn = quantity;
};
@@ -104,38 +104,38 @@
storageMovement.withdrawn = 0;
};
-model.nextSite = function() {
+model.nextLocation = function() {
this.storageMovementsIndex += 1;
if (this.storageMovementsIndex >= this.storageMovements.length) {
this.storageMovementsIndex = null;
} else {
var storageMovement = this.storageMovements[this.storageMovementsIndex];
if (storageMovement.expectedQuantity == 0.) {
- console.debug("will use extra site " + storageMovement.originSite.locationCode);
+ console.debug("will use extra location " + storageMovement.originLocation.locationCode);
var remainingQuantity = this.quantity - this.withdrawn();
storageMovement.expectedQuantity = remainingQuantity;
-// var availableQuantityInSite = storageMovement.availableQuantity;
-// storageMovement.expectedQuantity = Math.min(remainingQuantity, availableQuantityInSite);
+// var availableQuantityInLocation = storageMovement.availableQuantity;
+// storageMovement.expectedQuantity = Math.min(remainingQuantity, availableQuantityInLocation);
console.debug("pushing new value to withdraw: " + storageMovement.expectedQuantity);
}
}
};
model.getConfirmation = function() {
- var sitesIdsToActualQuantities = {};
- var siteInErrorIds = [];
+ var locationsIdsToActualQuantities = {};
+ var locationInErrorIds = [];
$.each(model.storageMovements, function(key, storageMovement) {
if (storageMovement.used) {
- sitesIdsToActualQuantities[storageMovement.originSite.id] = storageMovement.withdrawn;
+ locationsIdsToActualQuantities[storageMovement.originLocation.id] = storageMovement.withdrawn;
if (storageMovement.defect) {
- siteInErrorIds.push(storageMovement.originSite.id);
+ locationInErrorIds.push(storageMovement.originLocation.id);
}
}
});
var confirmation = {
storageMovementOrderId: this.storageMovementOrderId,
- sitesIdsToActualQuantities: sitesIdsToActualQuantities,
- siteInErrorIds: siteInErrorIds
+ locationsIdsToActualQuantities: locationsIdsToActualQuantities,
+ locationInErrorIds: locationInErrorIds
};
return confirmation;
}
@@ -151,7 +151,7 @@
}
},
- refreshSites : function() {
+ refreshLocations : function() {
var storageMovementsElement = $('#storageMovements');
@@ -169,7 +169,7 @@
} else {
$(storageMovementElement).addClass('success');
}
- } else if (storageMovement.originSite.id === model.storageMovements[model.storageMovementsIndex].originSite.id) {
+ } else if (storageMovement.originLocation.id === model.storageMovements[model.storageMovementsIndex].originLocation.id) {
$(storageMovementElement).addClass('pending');
storageMovementsElement.scrollTop(storageMovementElement[0].offsetTop);
} else {
@@ -181,8 +181,8 @@
refreshForm : function() {
// reset field to prevent reuse of the value by mistake
- $('#withdraw-item_siteBarcode').val('');
- // set default value of quantity to what should be withdrawn on the current site
+ $('#withdraw-item_locationBarcode').val('');
+ // set default value of quantity to what should be withdrawn on the current location
if (model.storageMovementsIndex != null) {
$('#withdraw-item_quantity').val(model.storageMovements[model.storageMovementsIndex].expectedQuantity);
}
@@ -198,7 +198,7 @@
refresh : function() {
this.refreshSummary();
- this.refreshSites();
+ this.refreshLocations();
this.refreshForm();
this.refreshButtons();
}
@@ -215,14 +215,14 @@
} else {
- var barcode = $('#withdraw-item_siteBarcode').val();
+ var barcode = $('#withdraw-item_locationBarcode').val();
var quantity = parseFloat($('#withdraw-item_quantity').val());
model.withdraw(barcode, quantity);
- // if used site focused as current, highlight next site to go
- if (barcode == model.storageMovements[model.storageMovementsIndex].originSite.barcode) {
- model.nextSite();
+ // if used location focused as current, highlight next location to go
+ if (barcode == model.storageMovements[model.storageMovementsIndex].originLocation.barcode) {
+ model.nextLocation();
}
}
@@ -245,13 +245,13 @@
} else {
- var barcode = $('#withdraw-item_siteBarcode').val();
+ var barcode = $('#withdraw-item_locationBarcode').val();
model.reportError(barcode);
- // if used site focused as current, highlight next site to go
- if (barcode == model.storageMovements[model.storageMovementsIndex].originSite.barcode) {
- model.nextSite();
+ // if used location focused as current, highlight next location to go
+ if (barcode == model.storageMovements[model.storageMovementsIndex].originLocation.barcode) {
+ model.nextLocation();
}
}
1
0