Bugzilla – Attachment 27648 Details for
Bug 11433
Remove attachbasket op value in basket.pl
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[PATCH] Bug 11433 - Remove attachbasket op value in basket.pl and rename it in basketgroup.pl
0001-patch (text/plain), 1.83 KB, created by
Mathieu Saby
on 2014-04-26 12:27:27 UTC
(
hide
)
Description:
[PATCH] Bug 11433 - Remove attachbasket op value in basket.pl and rename it in basketgroup.pl
Filename:
MIME Type:
Creator:
Mathieu Saby
Created:
2014-04-26 12:27:27 UTC
Size:
1.83 KB
patch
obsolete
>From e2a14ac0be80e523d05b52fc919608194ee60024 Mon Sep 17 00:00:00 2001 >From: Mathieu Saby <mathieu.saby@univ-rennes2.fr> >Date: Fri, 20 Dec 2013 23:21:32 +0100 >Subject: [PATCH] Bug 11433 - Remove attachbasket op value in basket.pl >Content-Type: text/plain; charset="utf-8" > >This patch cleans code in basket.pl > >In basket.pl, some code is supposed to be executed if $op eq 'attachbasket'. But it is never the case (grep attachbasket * -r), so this condition must be removed. > >This patch >- removes code related to 'attachbasket' in basket.pl > >No functional change expected. >Regression test only : >Make a complete acquisition process, from the creation of a basket to the closure of a basketgroup, and check everything is OK >On a basket page, try to change the basketgroup it belongs to, and check everything is OK >On a basketgroup page, try to edit the content of a basketgroup (put a new basket in it, change the deliverybranch...), and check everything is OK >On a basketgroup page, try to reopen a closed basketgroup, and close an open basketgroup, and check everything is OK > >--- > acqui/basket.pl | 3 --- > 1 file changed, 3 deletions(-) > >diff --git a/acqui/basket.pl b/acqui/basket.pl >index 047ae84..4f2a37e 100755 >--- a/acqui/basket.pl >+++ b/acqui/basket.pl >@@ -189,9 +189,6 @@ if ( $op eq 'delete_confirm' ) { > address3 => $bookseller->{'address3'}, > address4 => $bookseller->{'address4'}, > ); >-} elsif ($op eq 'attachbasket' && $template->{'VARS'}->{'CAN_user_acquisition_group_manage'} == 1) { >- print $query->redirect('/cgi-bin/koha/acqui/basketgroup.pl?basketno=' . $basket->{'basketno'} . '&op=attachbasket&booksellerid=' . $booksellerid); >- # check if we have to "close" a basket before building page > } elsif ($op eq 'export') { > print $query->header( > -type => 'text/csv', >-- >1.7.9.5 >
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 11433
:
23740
|
25367
|
27037
|
27648
|
27990
|
28024