Attachment 38402 Details for Bug 11006 – [PASSED QA] Bug 11006: Drop column aqorders.totalamount

[patch] [PASSED QA] Bug 11006: Drop column aqorders.totalamount

PASSED-QA-Bug-11006-Drop-column-aqorderstotalamoun.patch (text/plain), 2.56 KB, created by Kyle M Hall (khall) on 2015-04-23 11:27:16 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2015-04-23 11:27:16 UTC

Size: 2.56 KB

patch

obsolete

>From 0dbeb98880a773fe9f6a25a6388d8f71b632a0d2 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@biblibre.com>
>Date: Wed, 22 Apr 2015 13:03:53 +0200
>Subject: [PATCH] [PASSED QA] Bug 11006: Drop column aqorders.totalamount
>
>This column was not used in the Koha codebase, it can be removed.
>
>Test plan:
>Execute the updatedatabase.pl script and confirm that the column has
>been dropped.
>
>QA step:
>1/ git grep totalamount
>2/ prove t/db_dependent/Acquisition.t
>
>Signed-off-by: Marc Veron <veron@veron.ch>
>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>---
> .../bug_11006-DROP_COLUMN_totalamount.sql          |    1 +
> installer/data/mysql/kohastructure.sql             |    1 -
> t/db_dependent/Acquisition.t                       |    3 ---
> 3 files changed, 1 insertions(+), 4 deletions(-)
> create mode 100644 installer/data/mysql/atomicupdate/bug_11006-DROP_COLUMN_totalamount.sql
>
>diff --git a/installer/data/mysql/atomicupdate/bug_11006-DROP_COLUMN_totalamount.sql b/installer/data/mysql/atomicupdate/bug_11006-DROP_COLUMN_totalamount.sql
>new file mode 100644
>index 0000000..3cd0843
>--- /dev/null
>+++ b/installer/data/mysql/atomicupdate/bug_11006-DROP_COLUMN_totalamount.sql
>@@ -0,0 +1 @@
>+ALTER TABLE aqorders DROP COLUMN totalamount;
>diff --git a/installer/data/mysql/kohastructure.sql b/installer/data/mysql/kohastructure.sql
>index a74d81a..63a0b93 100644
>--- a/installer/data/mysql/kohastructure.sql
>+++ b/installer/data/mysql/kohastructure.sql
>@@ -3030,7 +3030,6 @@ CREATE TABLE `aqorders` ( -- information related to the basket line items
>   `quantity` smallint(6) default NULL, -- the quantity ordered
>   `currency` varchar(3) default NULL, -- the currency used for the purchase
>   `listprice` decimal(28,6) default NULL, -- the vendor price for this line item
>-  `totalamount` decimal(28,6) default NULL, -- not used? always NULL
>   `datereceived` date default NULL, -- the date this order was received
>   invoiceid int(11) default NULL, -- id of invoice
>   `freight` decimal(28,6) default NULL, -- shipping costs (not used)
>diff --git a/t/db_dependent/Acquisition.t b/t/db_dependent/Acquisition.t
>index f41a345..561b5af 100755
>--- a/t/db_dependent/Acquisition.t
>+++ b/t/db_dependent/Acquisition.t
>@@ -331,7 +331,6 @@ my @expectedfields = qw(
>   quantity
>   currency
>   listprice
>-  totalamount
>   datereceived
>   invoiceid
>   freight
>@@ -482,7 +481,6 @@ my @base_expectedfields = qw(
>   budget_expend
>   rrp
>   cn_sort
>-  totalamount
>   lccn
>   sort1
>   volume
>@@ -603,7 +601,6 @@ ok(
>   sort2_authcat
>   rrp
>   unitprice
>-  totalamount
>   sort1
>   ordernumber
>   datecreated
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 11006: 38320 | 38393 | 38402 | 39186