Attachment 22441 Details for Bug 11108 – [PASSED QA] Bug 11108: fix duplicate display of basket whose receipt has just been cancelled

[patch] [PASSED QA] Bug 11108: fix duplicate display of basket whose receipt has just been cancelled

PASSED-QA-Bug-11108-fix-duplicate-display-of-baske.patch (text/plain), 3.16 KB, created by Katrin Fischer on 2013-10-27 04:12:27 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2013-10-27 04:12:27 UTC

Size: 3.16 KB

patch

obsolete

>From 2ef7be3d62f791e78705ca03cd1bc5f6d2b91dd6 Mon Sep 17 00:00:00 2001
>From: Galen Charlton <gmc@esilibrary.com>
>Date: Tue, 22 Oct 2013 00:14:47 +0000
>Subject: [PATCH] [PASSED QA] Bug 11108: fix duplicate display of basket whose
> receipt has just been cancelled
>
>After receiving an order basket, before finishing receiving the shipment,
>one has an option to cancel the receipt.  This patch fixes a bug where
>a basket whose receipt was just cancelled displays on both the pending orders
>and already received tabs.
>
>This patch also fixes a minor unitialized variable warning.
>
>To test:
>
>[1] Create a basket with at least one order and close it.
>[2] Receive the order, then on the row in the 'Already received'
>    table, click the cancel receipt link.
>[3] In the page that displays, the basket just cancelled displays
>    on both tables.  Clicking the cancel receipt link again results
>    in an error message.
>[4] Apply the patch.
>[5] Repeat steps 1 and 2.  This time, the cancelled basket displays
>    only in the pending orders table, as expected.
>[6] Verify that after applying the patch, the following no longer
>    is logged in the Apache error log:
>
>parcel.pl: Use of uninitialized value in string eq at acqui/parcel.pl line...
>
>Signed-off-by: Galen Charlton <gmc@esilibrary.com>
>Signed-off-by: Ed Veal <ed.veal@bywatersolutions.com>
>Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
>Passes test plan, all tests and QA script.
>Works as described.
>---
> acqui/parcel.pl |   24 +++++++++++++-----------
> 1 file changed, 13 insertions(+), 11 deletions(-)
>
>diff --git a/acqui/parcel.pl b/acqui/parcel.pl
>index 028e957..42f2960 100755
>--- a/acqui/parcel.pl
>+++ b/acqui/parcel.pl
>@@ -117,6 +117,18 @@ my ($template, $loggedinuser, $cookie)
>                  debug => 1,
> });
> 
>+my $op = $input->param('op') // '';
>+
>+# process cancellation first so that list of
>+# orders to display is calculated after
>+if ($op eq 'cancelreceipt') {
>+    my $ordernumber = $input->param('ordernumber');
>+    my $parent_ordernumber = CancelReceipt($ordernumber);
>+    unless($parent_ordernumber) {
>+        $template->param(error_cancelling_receipt => 1);
>+    }
>+}
>+
> my $invoiceid = $input->param('invoiceid');
> my $invoice;
> $invoice = GetInvoiceDetails($invoiceid) if $invoiceid;
>@@ -130,16 +142,6 @@ unless( $invoiceid and $invoice->{invoiceid} ) {
>     exit;
> }
> 
>-my $op = $input->param('op') // '';
>-
>-if ($op eq 'cancelreceipt') {
>-    my $ordernumber = $input->param('ordernumber');
>-    my $parent_ordernumber = CancelReceipt($ordernumber);
>-    unless($parent_ordernumber) {
>-        $template->param(error_cancelling_receipt => 1);
>-    }
>-}
>-
> my $booksellerid = $invoice->{booksellerid};
> my $bookseller = GetBookSellerFromId($booksellerid);
> my $gst = $bookseller->{gstrate} // C4::Context->preference("gist") // 0;
>@@ -203,7 +205,7 @@ push @book_foot_loop, map { $_ } values %foot;
> my @loop_orders = ();
> if(!defined $invoice->{closedate}) {
>     my $pendingorders;
>-    if($input->param('op') eq "search"){
>+    if($op eq "search"){
>         my $search   = $input->param('summaryfilter') || '';
>         my $ean      = $input->param('eanfilter') || '';
>         my $basketname = $input->param('basketfilter') || '';
>-- 
>1.7.9.5

Actions: View | Diff | Splinter Review

Attachments on bug 11108: 22249 | 22260 | 22441