RenJuan/TASKPM
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Bug #1534: Fix query to get info about expenses associated to an order

Browse source 
The query was just getting the tasks with parent the order, but not all the
sub-tasks.

A similar code that for the case of imputed hours has been used for the expenses
case fixing the issue.

FEA: ItEr77S04BugFixing
This commit is contained in:
Manuel Rego Casasnovas 2012-09-19 12:38:58 +02:00
parent 7e0295166c
commit 1d0d40f449
1 changed files with 2 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
libreplan-business/src/main/java/org/libreplan/business/orders/daos/OrderDAO.java
View file

@ -445,9 +445,6 @@ public class OrderDAO extends IntegrationEntityDAO<Order> implements
+ "LEFT OUTER JOIN expense.orderElement exp_ord "
+ "WHERE orderElement.id = exp_ord.id ";
if (!orders.isEmpty()) {
strQuery += "AND orderElement.parent IN (:orders) ";
}
if (startingDate != null && endingDate != null) {
strQuery += "AND expense.date BETWEEN :startingDate AND :endingDate ";
}
@ -464,9 +461,6 @@ public class OrderDAO extends IntegrationEntityDAO<Order> implements
Query query = getSession().createQuery(strQuery);
// Set parameters
if (!orders.isEmpty()) {
query.setParameterList("orders", orders);
}
if (startingDate != null) {
query.setParameter("startingDate", new LocalDate(startingDate));
}
@ -480,7 +474,8 @@ public class OrderDAO extends IntegrationEntityDAO<Order> implements
for (CostExpenseSheetDTO each : list) {
Order order = loadOrderAvoidingProxyFor(each.getOrderElement());
// Apply filtering
if (matchFilterCriterion(each.getOrderElement(), criterions)) {
if (matchFilterCriterion(each.getOrderElement(), criterions)
&& isOrderContained(order, orders)) {
each.setOrder(order);
filteredList.add(each);
}