Attachment 31737 Details for Bug 12841 – [PASSED QA] Bug 12841: aqorders fields should not be mapped

[patch] [PASSED QA] Bug 12841: aqorders fields should not be mapped

PASSED-QA-Bug-12841-aqorders-fields-should-not-be-.patch (text/plain), 2.05 KB, created by Kyle M Hall (khall) on 2014-09-19 12:36:01 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2014-09-19 12:36:01 UTC

Size: 2.05 KB

patch

obsolete

>From 3572c45b7d5a6f17ed68c8619c49473936198883 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@biblibre.com>
>Date: Tue, 26 Aug 2014 10:50:22 +0200
>Subject: [PATCH] [PASSED QA] Bug 12841: aqorders fields should not be mapped
>
>Bug 5336 introduced some code which should have been introduced by bug
>7294.
>Since the idea behind bug 7294 has been abandoned (map the aqorders
>fields), the code can be removed.
>
>Test plan:
>Verify that Koha does not allow you to map the aqorders fields with a
>MARC subfield.
>Verify there is no regression on adding/updating an order.
>
>Signed-off-by: Zeno Tajoli <z.tajoli@cineca.it>
>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>---
> acqui/addorder.pl |    7 -------
> 1 files changed, 0 insertions(+), 7 deletions(-)
>
>diff --git a/acqui/addorder.pl b/acqui/addorder.pl
>index da1beb0..264b247 100755
>--- a/acqui/addorder.pl
>+++ b/acqui/addorder.pl
>@@ -250,13 +250,6 @@ if ( $orderinfo->{quantity} ne '0' ) {
>                 "biblio.copyrightdate"        => $$orderinfo{publicationyear} ? $$orderinfo{publicationyear}: "",
>                 "biblioitems.itemtype"        => $$orderinfo{itemtype} ? $$orderinfo{itemtype} : "",
>                 "biblioitems.editionstatement"=> $$orderinfo{editionstatement} ? $$orderinfo{editionstatement} : "",
>-                "aqorders.branchcode"         => $$orderinfo{branchcode} ? $$orderinfo{branchcode} : "",
>-                "aqorders.quantity"           => $$orderinfo{quantity} ? $$orderinfo{quantity} : "",
>-                "aqorders.listprice"          => $$orderinfo{listprice} ? $$orderinfo{listprice} : "",
>-                "aqorders.uncertainprice"     => $$orderinfo{uncertainprice} ? $$orderinfo{uncertainprice} : "",
>-                "aqorders.rrp"                => $$orderinfo{rrp} ? $$orderinfo{rrp} : "",
>-                "aqorders.ecost"              => $$orderinfo{ecost} ? $$orderinfo{ecost} : "",
>-                "aqorders.discount"           => $$orderinfo{discount} ? $$orderinfo{discount} : "",
>             });
> 
>         # create the record in catalogue, with framework ''
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 12841: 31245 | 31573 | 31576 | 31737