Bugzilla – Attachment 21709 Details for
Bug 5336
Acq history search
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 5336: Multiple QA fixes
Bug-5336-Multiple-QA-fixes.patch (text/plain), 6.52 KB, created by
Jonathan Druart
on 2013-10-02 08:51:42 UTC
(
hide
)
Description:
Bug 5336: Multiple QA fixes
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2013-10-02 08:51:42 UTC
Size:
6.52 KB
patch
obsolete
>From 3acc55c5288b0bd7375a7b97a67f5037ca79b78d Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@biblibre.com> >Date: Wed, 2 Oct 2013 09:58:14 +0200 >Subject: [PATCH] Bug 5336: Multiple QA fixes > >- add a message if the search returns no result >- keep selected values if the search returns no result (for fund and > order status) >- remove plurals in order status >- move the order status column in the search results table >--- > acqui/histsearch.pl | 3 + > .../prog/en/modules/acqui/histsearch.tt | 67 ++++++++++++++------ > 2 files changed, 52 insertions(+), 18 deletions(-) > >diff --git a/acqui/histsearch.pl b/acqui/histsearch.pl >index e9e4009..4e152e9 100755 >--- a/acqui/histsearch.pl >+++ b/acqui/histsearch.pl >@@ -149,7 +149,10 @@ $template->param( > basketgroupname => $basketgroupname, > from_placed_on => $from_date, > to_placed_on => $to_date, >+ orderstatus => $orderstatus, >+ budget_id => $budget, > bp_loop => $bp_loop, >+ search_done => $do_search, > debug => $debug || $input->param('debug') || 0, > uc(C4::Context->preference("marcflavour")) => 1 > ); >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/histsearch.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/histsearch.tt >index fe6623b..54d19d0 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/histsearch.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/histsearch.tt >@@ -51,12 +51,36 @@ > <li> > <label for="orderstatus">Order status: </label> > <select name="orderstatus" id="orderstatus"> >- <option value=""></option> >- <option value="new">New orders</option> >- <option value="ordered">Pending orders</option> >- <option value="partial">Partially received orders</option> >- <option value="complete">Received orders</option> >- <option value="cancelled">Cancelled orders</option> >+ [% IF orderstatus == "" %] >+ <option value=""></option> >+ [% ELSE %] >+ <option value="" selected="selected"></option> >+ [% END %] >+ [% IF orderstatus == "new" %] >+ <option value="new" selected="selected">New</option> >+ [% ELSE %] >+ <option value="new">New</option> >+ [% END %] >+ [% IF orderstatus == "ordered" %] >+ <option value="ordered" selected="selected">Pending</option> >+ [% ELSE %] >+ <option value="ordered">Pending</option> >+ [% END %] >+ [% IF orderstatus == "partial" %] >+ <option value="partial" selected="selected">Partially received</option> >+ [% ELSE %] >+ <option value="partial">Partially received</option> >+ [% END %] >+ [% IF orderstatus == "complete" %] >+ <option value="complete" selected="selected">Received</option> >+ [% ELSE %] >+ <option value="complete">Received</option> >+ [% END %] >+ [% IF orderstatus == "cancelled" %] >+ <option value="cancelled" selected="selected">Cancelled</option> >+ [% ELSE %] >+ <option value="cancelled">Cancelled</option> >+ [% END %] > </select> > </li> > <li> >@@ -66,7 +90,11 @@ > [% FOREACH bp_loo IN bp_loop %] > <optgroup label="[% bp_loo.budget_period_description %]"> > [% FOREACH h_loo IN bp_loo.hierarchy %] >+ [% IF h_loo.budget_id == budget_id %] >+ <option type="text" value="[% h_loo.budget_id %]" branchcode="[% h_loo.budget_branchcode %]" selected="selected"> >+ [% ELSE %] > <option type="text" value="[% h_loo.budget_id %]" branchcode="[% h_loo.budget_branchcode %]"> >+ [% END %] > [% h_loo.budget_display_name %] > </option> > [% END %] >@@ -94,6 +122,7 @@ > <thead> > <tr> > <th>Order line (parent)</th> >+ <th>Status</th> > <th>Basket</th> > <th>Basket group</th> > <th>Invoice number</th> >@@ -101,7 +130,6 @@ > <th>Vendor</th> > <th>Placed on</th> > <th>Received on</th> >- <th>Status</th> > <th>Quantity received</th> > <th>Pending order</th> > <th>Unit cost</th> >@@ -115,6 +143,15 @@ > [% order.ordernumber %] > [% IF order.ordernumber != order.parent_ordernumber %]([% order.parent_ordernumber %])[% END %] > </td> >+ <td> >+ [% SWITCH order.orderstatus %] >+ [% CASE 'new' %]New >+ [% CASE 'ordered' %]Pending >+ [% CASE 'partial' %]Partially received >+ [% CASE 'complete' %]Received >+ [% CASE 'cancelled' %]Cancelled >+ [% END %] >+ </td> > <td>[% order.basketname %] (<a href="basket.pl?basketno=[% order.basketno %]">[% order.basketno %]</a>)</td> > <td> > [% IF ( order.basketgroupid ) %] >@@ -140,15 +177,6 @@ > [% order.datereceived | $KohaDates %] > [% END %] > </td> >- <td> >- [% SWITCH order.orderstatus %] >- [% CASE 'new' %]New orders >- [% CASE 'ordered' %]Pending orders >- [% CASE 'partial' %]Partially received orders >- [% CASE 'complete' %]Received orders >- [% CASE 'cancelled' %]Cancelled orders >- [% END %] >- </td> > <td>[% order.quantityreceived %]</td> > <td>[% order.quantity %]</td> > <td>[% order.ecost %]</td> >@@ -156,8 +184,11 @@ > </tr> > [% END %] > </tbody> >- </table> >- </div>[% ELSE %][% END %] >+ </table> >+ </div> >+ [% ELSIF search_done %] >+ There is no order for this search. >+ [% END %] > > </div> > </div> >-- >1.7.10.4
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 5336
:
11603
|
11604
|
11792
|
14067
|
14188
|
14213
|
16013
|
17612
|
17613
|
17615
|
17617
|
17648
|
17649
|
17650
|
17883
|
18187
|
18411
|
18412
|
18413
|
18414
|
18496
|
18497
|
18498
|
18499
|
18500
|
19317
|
19318
|
19319
|
19320
|
19321
|
19550
|
20894
|
20895
|
20896
|
20897
|
20898
|
20899
|
21689
|
21690
|
21691
|
21692
|
21693
|
21694
|
21695
|
21696
|
21697
|
21707
|
21708
|
21709
|
21751
|
21752
|
21753
|
21754
|
21755
|
21756
|
21757
|
21758