9245 Fixes visual bux with fines tab not highlighting
authorViktor Sarge <viktor.sarge@regionhalland.se>
Mon, 16 Jun 2014 11:21:06 +0000 (12:21 +0100)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Mon, 7 Jul 2014 16:13:43 +0000 (13:13 -0300)
This patch makes shure that mancredit.pl, maninvoice.pl and pay.pl passes the value "finesview=1" that makes the tab active.

Test plan:
1) Go to a library user and the fines tab. Note that the "Fines" tab loses it's highlight when you click "Pay fines", "Create manual invoice" and "Create manual credit".
2) Install the patch.
3) Redo the first step and make shure that the left menu tab "Fines" is now highlighted all the time.

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
Works as advertised, no regression found on the rest of the tabs.

members/mancredit.pl
members/maninvoice.pl
members/pay.pl

index 0ba52f3..ccb739e 100755 (executable)
@@ -89,6 +89,7 @@ if (C4::Context->preference('ExtendedPatronAttributes')) {
 }
     
     $template->param(
+        finesview => 1,
         borrowernumber => $borrowernumber,
         firstname => $data->{'firstname'},
         surname  => $data->{'surname'},
index 4bf294a..cfd3878 100755 (executable)
@@ -116,6 +116,7 @@ if (C4::Context->preference('ExtendedPatronAttributes')) {
     );
 }
        $template->param(
+                finesview => 1,
                 borrowernumber => $borrowernumber,
                firstname => $data->{'firstname'},
                 surname  => $data->{'surname'},
index 3e36073..14607b0 100755 (executable)
@@ -104,6 +104,7 @@ for (@names) {
 }
 
 $template->param(
+    finesview => 1,
     activeBorrowerRelationship => (C4::Context->preference('borrowerRelationship') ne ''),
     RoutingSerials => C4::Context->preference('RoutingSerials'),
 );