[Koha-patches] [PATCH] fir for #2338

paul.poulain at biblibre.com paul.poulain at biblibre.com
Sun Apr 19 01:37:32 CEST 2009


From: Paul Poulain <paul.poulain at biblibre.com>

When I go to serials/claims.pl, "All Suppliers (2)" is selected in the supplier
dropdown list, and no missing issues are listed. When I choose another supplier
from that list ("Supplier 1 (2)"), then two missing issues are listed. Should
"All Suppliers" instead read "Choose a Supplier" ?

chris: I think it might be safe to remove the all suppliers option all together as its
doing nothing

paul: agreed, this patch removes the All Supplier line, that is useless, confusing & untranslatable
---
 C4/Serials.pm |    3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)

diff --git a/C4/Serials.pm b/C4/Serials.pm
index 68fecdf..8fe5019 100644
--- a/C4/Serials.pm
+++ b/C4/Serials.pm
@@ -106,9 +106,6 @@ sub GetSuppliersWithLateIssues {
     while ( my ( $id, $name ) = $sth->fetchrow ) {
         $supplierlist{$id} = $name;
     }
-    if ( C4::Context->preference("RoutingSerials") ) {
-        $supplierlist{''} = "All Suppliers";
-    }
     return %supplierlist;
 }
 
-- 
1.5.6.3




More information about the Koha-patches mailing list