@@ -, +, @@ - adding a new column for basketgroup, displaying the name and number of the basketgroup - displaying the name of the basket as well as its number - displaying fund in a separate column to make sorting easier, and renaming it from "budget" to "fund" (the confusion can be found elsewhere in Koha...) - displaying branch in a separate column to make sorting easier (this column could be improved later, by getting branch from basketgroup if there is one) - 3 new columns in SELECT, 1 new join - cosmetic : rename "closedate" to "aqbasket.closedate" in 4 occurences : it will be usefull later, because I am envisonning an other patch that will create a "closedate" column in basketgroup table, and I don't want MySQL to crash because of ambiguous column. --- C4/Acquisition.pm | 14 +++++++++----- .../prog/en/modules/acqui/lateorders.tt | 19 +++++++++++++------ 2 files changed, 22 insertions(+), 11 deletions(-) --- a/C4/Acquisition.pm +++ a/C4/Acquisition.pm @@ -1859,7 +1859,7 @@ sub GetParcels { =head3 GetLateOrders - @results = &GetLateOrders; + $results = &GetLateOrders($delay, $supplierid, $branch, $estimateddeliverydatefrom, $estimateddeliverydateto); Searches for bookseller with late orders. @@ -1885,6 +1885,9 @@ sub GetLateOrders { SELECT aqbasket.basketno, aqorders.ordernumber, DATE(aqbasket.closedate) AS orderdate, + aqbasket.basketname AS basketname, + aqbasket.basketgroupid AS basketgroupid, + aqbasketgroups.name AS basketgroupname, aqorders.rrp AS unitpricesupplier, aqorders.ecost AS unitpricelib, aqorders.claims_count AS claims_count, @@ -1905,6 +1908,7 @@ sub GetLateOrders { LEFT JOIN aqbudgets ON aqorders.budget_id = aqbudgets.budget_id, aqbasket LEFT JOIN borrowers ON aqbasket.authorisedby = borrowers.borrowernumber LEFT JOIN aqbooksellers ON aqbasket.booksellerid = aqbooksellers.id + LEFT JOIN aqbasketgroups ON aqbasket.basketgroupid = aqbasketgroups.id WHERE aqorders.basketno = aqbasket.basketno AND ( datereceived = '' OR datereceived IS NULL @@ -1918,10 +1922,10 @@ sub GetLateOrders { $select .= " aqorders.quantity - COALESCE(aqorders.quantityreceived,0) AS quantity, (aqorders.quantity - COALESCE(aqorders.quantityreceived,0)) * aqorders.rrp AS subtotal, - DATEDIFF(CAST(now() AS date),closedate) AS latesince + DATEDIFF(CAST(now() AS date),aqbasket.closedate) AS latesince "; if ( defined $delay ) { - $from .= " AND (closedate <= DATE_SUB(CAST(now() AS date),INTERVAL ? DAY)) " ; + $from .= " AND (aqbasket.closedate <= DATE_SUB(CAST(now() AS date),INTERVAL ? DAY)) " ; push @query_params, $delay; } $having = " @@ -1934,10 +1938,10 @@ sub GetLateOrders { $select .= " aqorders.quantity AS quantity, aqorders.quantity * aqorders.rrp AS subtotal, - (CAST(now() AS date) - closedate) AS latesince + (CAST(now() AS date) - aqbasket.closedate) AS latesince "; if ( defined $delay ) { - $from .= " AND (closedate <= (CAST(now() AS date) -(INTERVAL ? DAY)) "; + $from .= " AND (aqbasket.closedate <= (CAST(now() AS date) -(INTERVAL ? DAY)) "; push @query_params, $delay; } } --- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/lateorders.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/lateorders.tt @@ -117,6 +117,9 @@ $(document).ready(function() { Information Total cost Basket + Basket group + Branch + Fund Claims count Claimed date @@ -154,14 +157,18 @@ $(document).ready(function() { [% lateorder.unitpricesupplier %]x[% lateorder.quantity %] = [% lateorder.subtotal %] -

[% lateorder.budget %]

- [% lateorder.basketno %] - -

-

[% lateorder.branch %]

+ [% lateorder.basketname %] ([% lateorder.basketno %]) + +

+ + [% lateorder.basketgroupname %] ([% lateorder.basketgroupid %]) + + [% lateorder.branch %] + + [% lateorder.budget %] [% lateorder.claims_count %] [% lateorder.claimed_date %] @@ -172,7 +179,7 @@ $(document).ready(function() { Total [% total %] -   +   --