Attachment 90827 Details for Bug 23166 – Bug 23166: Remove baskets_deletedorders code

[patch] Bug 23166: Remove baskets_deletedorders code

Bug-23166-Remove-basketsdeletedorders-code.patch (text/plain), 5.86 KB, created by Jonathan Druart on 2019-06-20 02:37:45 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2019-06-20 02:37:45 UTC

Size: 5.86 KB

patch

obsolete

>From c0dda683e952994f9c5f672fb87ed0fba50c2f3e Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Wed, 19 Jun 2019 21:11:20 -0500
>Subject: [PATCH] Bug 23166: Remove baskets_deletedorders code
>
>This is never used.
>---
> catalogue/ISBDdetail.pl        | 4 ----
> catalogue/MARCdetail.pl        | 4 ----
> catalogue/detail.pl            | 6 ------
> catalogue/imageviewer.pl       | 4 ----
> catalogue/labeledMARCdetail.pl | 4 ----
> catalogue/moredetail.pl        | 4 ----
> 6 files changed, 26 deletions(-)
>
>diff --git a/catalogue/ISBDdetail.pl b/catalogue/ISBDdetail.pl
>index d63b833ee4..889c71eda7 100755
>--- a/catalogue/ISBDdetail.pl
>+++ b/catalogue/ISBDdetail.pl
>@@ -145,15 +145,11 @@ $template->param (
> my @allorders_using_biblio = GetOrdersByBiblionumber ($biblionumber);
> my @deletedorders_using_biblio;
> my @orders_using_biblio;
>-my @baskets_deletedorders;
> 
> foreach my $myorder (@allorders_using_biblio) {
>     my $basket = $myorder->{'basketno'};
>     if ((defined $myorder->{'datecancellationprinted'}) and  ($myorder->{'datecancellationprinted'} ne '0000-00-00') ){
>         push @deletedorders_using_biblio, $myorder;
>-        unless (grep{ $_ eq $basket } @baskets_deletedorders){
>-            push @baskets_deletedorders,$myorder->{'basketno'};
>-        }
>     }
>     else {
>         push @orders_using_biblio, $myorder;
>diff --git a/catalogue/MARCdetail.pl b/catalogue/MARCdetail.pl
>index 4c175b5268..d7e2854c56 100755
>--- a/catalogue/MARCdetail.pl
>+++ b/catalogue/MARCdetail.pl
>@@ -321,15 +321,11 @@ $template->param (
> my @allorders_using_biblio = GetOrdersByBiblionumber ($biblionumber);
> my @deletedorders_using_biblio;
> my @orders_using_biblio;
>-my @baskets_deletedorders;
> 
> foreach my $myorder (@allorders_using_biblio) {
>     my $basket = $myorder->{'basketno'};
>     if ((defined $myorder->{'datecancellationprinted'}) and  ($myorder->{'datecancellationprinted'} ne '0000-00-00') ){
>         push @deletedorders_using_biblio, $myorder;
>-        unless (grep{ $_ eq $basket } @baskets_deletedorders){
>-            push @baskets_deletedorders,$myorder->{'basketno'};
>-        }
>     }
>     else {
>         push @orders_using_biblio, $myorder;
>diff --git a/catalogue/detail.pl b/catalogue/detail.pl
>index cadedab62e..3c2b2dc434 100755
>--- a/catalogue/detail.pl
>+++ b/catalogue/detail.pl
>@@ -515,15 +515,11 @@ if ($StaffDetailItemSelection) {
> my @allorders_using_biblio = GetOrdersByBiblionumber ($biblionumber);
> my @deletedorders_using_biblio;
> my @orders_using_biblio;
>-my @baskets_deletedorders;
> 
> foreach my $myorder (@allorders_using_biblio) {
>     my $basket = $myorder->{'basketno'};
>     if ((defined $myorder->{'datecancellationprinted'}) and  ($myorder->{'datecancellationprinted'} ne '0000-00-00') ){
>         push @deletedorders_using_biblio, $myorder;
>-        unless (grep{ $_ eq $basket } @baskets_deletedorders){
>-            push @baskets_deletedorders,$myorder->{'basketno'};
>-        }
>     }
>     else {
>         push @orders_using_biblio, $myorder;
>@@ -536,6 +532,4 @@ $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 (basketsdeletedorders => \@baskets_deletedorders);
>-
> output_html_with_http_headers $query, $cookie, $template->output;
>diff --git a/catalogue/imageviewer.pl b/catalogue/imageviewer.pl
>index b1d62dd929..b7079fe9f9 100755
>--- a/catalogue/imageviewer.pl
>+++ b/catalogue/imageviewer.pl
>@@ -82,15 +82,11 @@ $template->{VARS}->{'biblio'} = $biblio;
> my @allorders_using_biblio = GetOrdersByBiblionumber ($biblionumber);
> my @deletedorders_using_biblio;
> my @orders_using_biblio;
>-my @baskets_deletedorders;
> 
> foreach my $myorder (@allorders_using_biblio) {
>     my $basket = $myorder->{'basketno'};
>     if ((defined $myorder->{'datecancellationprinted'}) and  ($myorder->{'datecancellationprinted'} ne '0000-00-00') ){
>         push @deletedorders_using_biblio, $myorder;
>-        unless (grep{ $_ eq $basket } @baskets_deletedorders){
>-            push @baskets_deletedorders,$myorder->{'basketno'};
>-        }
>     }
>     else {
>         push @orders_using_biblio, $myorder;
>diff --git a/catalogue/labeledMARCdetail.pl b/catalogue/labeledMARCdetail.pl
>index e6bb96c6ca..ce5dc01203 100755
>--- a/catalogue/labeledMARCdetail.pl
>+++ b/catalogue/labeledMARCdetail.pl
>@@ -128,15 +128,11 @@ $template->param (
> my @allorders_using_biblio = GetOrdersByBiblionumber ($biblionumber);
> my @deletedorders_using_biblio;
> my @orders_using_biblio;
>-my @baskets_deletedorders;
> 
> foreach my $myorder (@allorders_using_biblio) {
>     my $basket = $myorder->{'basketno'};
>     if ((defined $myorder->{'datecancellationprinted'}) and  ($myorder->{'datecancellationprinted'} ne '0000-00-00') ){
>         push @deletedorders_using_biblio, $myorder;
>-        unless (grep { $_ eq $basket } @baskets_deletedorders){
>-            push @baskets_deletedorders,$myorder->{'basketno'};
>-        }
>     }
>     else {
>         push @orders_using_biblio, $myorder;
>diff --git a/catalogue/moredetail.pl b/catalogue/moredetail.pl
>index 875b3a9a78..f553512cba 100755
>--- a/catalogue/moredetail.pl
>+++ b/catalogue/moredetail.pl
>@@ -228,15 +228,11 @@ $template->{'VARS'}->{'searchid'} = $query->param('searchid');
> my @allorders_using_biblio = GetOrdersByBiblionumber ($biblionumber);
> my @deletedorders_using_biblio;
> my @orders_using_biblio;
>-my @baskets_deletedorders;
> 
> foreach my $myorder (@allorders_using_biblio) {
>     my $basket = $myorder->{'basketno'};
>     if ((defined $myorder->{'datecancellationprinted'}) and  ($myorder->{'datecancellationprinted'} ne '0000-00-00') ){
>         push @deletedorders_using_biblio, $myorder;
>-        unless (grep{ $_ eq $basket } @baskets_deletedorders){
>-            push @baskets_deletedorders,$myorder->{'basketno'};
>-        }
>     }
>     else {
>         push @orders_using_biblio, $myorder;
>-- 
>2.11.0

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