Bugzilla – Attachment 109020 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.17 KB, created by
Joonas Kylmälä
on 2020-08-24 15:54:45 UTC
(
hide
)
Description:
Bug 23166: Remove baskets_orders code
Filename:
MIME Type:
Creator:
Joonas Kylmälä
Created:
2020-08-24 15:54:45 UTC
Size:
5.17 KB
patch
obsolete
>From bb8bb2ba20623515e8d7f76f042ca3c089ee709e 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. >--- > 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 4a2aa3afa7..5c7a1a6dfb 100755 >--- a/catalogue/detail.pl >+++ b/catalogue/detail.pl >@@ -556,7 +556,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) { >@@ -569,9 +568,6 @@ foreach my $myorder (@allorders_using_biblio) { > } > else { > push @orders_using_biblio, $myorder; >- unless (grep{ $_ eq $basket } @baskets_orders){ >- push @baskets_orders,$myorder->{'basketno'}; >- } > } > } > >@@ -583,7 +579,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