Bugzilla – Attachment 109037 Details for
Bug 23166
Simplify code related to orders in catalogue/*detail.pl
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 23166: Remove baskets_orders code
Bug-23166-Remove-basketsorders-code.patch (text/plain), 5.22 KB, created by
Nick Clemens (kidclamp)
on 2020-08-24 20:32:04 UTC
(
hide
)
Description:
Bug 23166: Remove baskets_orders code
Filename:
MIME Type:
Creator:
Nick Clemens (kidclamp)
Created:
2020-08-24 20:32:04 UTC
Size:
5.22 KB
patch
obsolete
>From 2951e85d46c2818011b646bf21e4775c1cb9f4df Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >Date: Wed, 19 Jun 2019 21:10:15 -0500 >Subject: [PATCH] Bug 23166: Remove baskets_orders code > >This is never used. > >Signed-off-by: Nick Clemens <nick@bywatersolutions.com> >--- > catalogue/ISBDdetail.pl | 4 ---- > catalogue/MARCdetail.pl | 4 ---- > catalogue/detail.pl | 5 ----- > catalogue/imageviewer.pl | 4 ---- > catalogue/labeledMARCdetail.pl | 4 ---- > catalogue/moredetail.pl | 4 ---- > 6 files changed, 25 deletions(-) > >diff --git a/catalogue/ISBDdetail.pl b/catalogue/ISBDdetail.pl >index 9f333e1b33..6999c0455f 100755 >--- a/catalogue/ISBDdetail.pl >+++ b/catalogue/ISBDdetail.pl >@@ -154,7 +154,6 @@ $template->param ( > my @allorders_using_biblio = GetOrdersByBiblionumber ($biblionumber); > my @deletedorders_using_biblio; > my @orders_using_biblio; >-my @baskets_orders; > my @baskets_deletedorders; > > foreach my $myorder (@allorders_using_biblio) { >@@ -167,9 +166,6 @@ foreach my $myorder (@allorders_using_biblio) { > } > else { > push @orders_using_biblio, $myorder; >- unless (grep{ $_ eq $basket } @baskets_orders){ >- push @baskets_orders,$myorder->{'basketno'}; >- } > } > } > >diff --git a/catalogue/MARCdetail.pl b/catalogue/MARCdetail.pl >index 44bc8d0801..bd6341b590 100755 >--- a/catalogue/MARCdetail.pl >+++ b/catalogue/MARCdetail.pl >@@ -347,7 +347,6 @@ $template->param ( > my @allorders_using_biblio = GetOrdersByBiblionumber ($biblionumber); > my @deletedorders_using_biblio; > my @orders_using_biblio; >-my @baskets_orders; > my @baskets_deletedorders; > > foreach my $myorder (@allorders_using_biblio) { >@@ -360,9 +359,6 @@ foreach my $myorder (@allorders_using_biblio) { > } > else { > push @orders_using_biblio, $myorder; >- unless (grep { $_ eq $basket } @baskets_orders){ >- push @baskets_orders,$myorder->{'basketno'}; >- } > } > } > >diff --git a/catalogue/detail.pl b/catalogue/detail.pl >index 276f33c270..cda10b7c60 100755 >--- a/catalogue/detail.pl >+++ b/catalogue/detail.pl >@@ -555,7 +555,6 @@ if ($StaffDetailItemSelection) { > my @allorders_using_biblio = GetOrdersByBiblionumber ($biblionumber); > my @deletedorders_using_biblio; > my @orders_using_biblio; >-my @baskets_orders; > my @baskets_deletedorders; > > foreach my $myorder (@allorders_using_biblio) { >@@ -568,9 +567,6 @@ foreach my $myorder (@allorders_using_biblio) { > } > else { > push @orders_using_biblio, $myorder; >- unless (grep{ $_ eq $basket } @baskets_orders){ >- push @baskets_orders,$myorder->{'basketno'}; >- } > } > } > >@@ -582,7 +578,6 @@ $template->param (countorders => $count_orders_using_biblio); > my $count_deletedorders_using_biblio = scalar @deletedorders_using_biblio ; > $template->param (countdeletedorders => $count_deletedorders_using_biblio); > >-$template->param (basketsorders => \@baskets_orders); > $template->param (basketsdeletedorders => \@baskets_deletedorders); > > output_html_with_http_headers $query, $cookie, $template->output; >diff --git a/catalogue/imageviewer.pl b/catalogue/imageviewer.pl >index 67ded03c46..21151d6263 100755 >--- a/catalogue/imageviewer.pl >+++ b/catalogue/imageviewer.pl >@@ -90,7 +90,6 @@ $template->{VARS}->{'biblio'} = $biblio; > my @allorders_using_biblio = GetOrdersByBiblionumber ($biblionumber); > my @deletedorders_using_biblio; > my @orders_using_biblio; >-my @baskets_orders; > my @baskets_deletedorders; > > foreach my $myorder (@allorders_using_biblio) { >@@ -103,9 +102,6 @@ foreach my $myorder (@allorders_using_biblio) { > } > else { > push @orders_using_biblio, $myorder; >- unless (grep{ $_ eq $basket } @baskets_orders){ >- push @baskets_orders,$myorder->{'basketno'}; >- } > } > } > >diff --git a/catalogue/labeledMARCdetail.pl b/catalogue/labeledMARCdetail.pl >index 446a679fba..ead7b9aeed 100755 >--- a/catalogue/labeledMARCdetail.pl >+++ b/catalogue/labeledMARCdetail.pl >@@ -136,7 +136,6 @@ $template->param ( > my @allorders_using_biblio = GetOrdersByBiblionumber ($biblionumber); > my @deletedorders_using_biblio; > my @orders_using_biblio; >-my @baskets_orders; > my @baskets_deletedorders; > > foreach my $myorder (@allorders_using_biblio) { >@@ -149,9 +148,6 @@ foreach my $myorder (@allorders_using_biblio) { > } > else { > push @orders_using_biblio, $myorder; >- unless (grep{ $_ eq $basket } @baskets_orders){ >- push @baskets_orders,$myorder->{'basketno'}; >- } > } > } > >diff --git a/catalogue/moredetail.pl b/catalogue/moredetail.pl >index aca129468a..13176493fe 100755 >--- a/catalogue/moredetail.pl >+++ b/catalogue/moredetail.pl >@@ -270,7 +270,6 @@ $template->{'VARS'}->{'searchid'} = $query->param('searchid'); > my @allorders_using_biblio = GetOrdersByBiblionumber ($biblionumber); > my @deletedorders_using_biblio; > my @orders_using_biblio; >-my @baskets_orders; > my @baskets_deletedorders; > > foreach my $myorder (@allorders_using_biblio) { >@@ -283,9 +282,6 @@ foreach my $myorder (@allorders_using_biblio) { > } > else { > push @orders_using_biblio, $myorder; >- unless (grep { $_ eq $basket } @baskets_orders){ >- push @baskets_orders,$myorder->{'basketno'}; >- } > } > } > >-- >2.11.0
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 23166
:
90824
|
90825
|
90826
|
90827
|
90828
|
104365
|
104366
|
104367
|
104368
|
104369
|
104370
|
108991
|
108992
|
108993
|
108994
|
108995
|
108996
|
108997
|
108998
|
108999
|
109000
|
109001
|
109002
|
109019
|
109020
|
109021
|
109022
|
109023
|
109024
|
109025
|
109026
|
109037
|
109038
|
109039
|
109040
|
109041
|
109042
|
109043
|
109044
|
109045
|
109046
|
109047
|
109048
|
109049
|
109050
|
109058
|
109059