From cb17720f47b34d5677b1e4c3245c084359358df0 Mon Sep 17 00:00:00 2001 From: Tomas Cohen Arazi Date: Fri, 23 Dec 2016 15:54:02 -0300 Subject: [PATCH] Bug 17813: Add a primary key to borrower_attributes This patch adds 'borrower_attributes' a field (if) which will act as a primary key. This is needed for DBIC to be used to handle rows, and also will help when faced with the implementation of the REST api for this resource. To test: - Run all patron modification / attributes and verify nothing breaks - Sign off :-D Signed-off-by: Kyle M Hall Signed-off-by: Jonathan Druart Signed-off-by: Kyle M Hall --- .../atomicupdate/bug_17813_borrower_attributes_pk.sql | 11 +++++++++++ installer/data/mysql/kohastructure.sql | 1 + 2 files changed, 12 insertions(+) create mode 100644 installer/data/mysql/atomicupdate/bug_17813_borrower_attributes_pk.sql diff --git a/installer/data/mysql/atomicupdate/bug_17813_borrower_attributes_pk.sql b/installer/data/mysql/atomicupdate/bug_17813_borrower_attributes_pk.sql new file mode 100644 index 0000000000..04da55daf9 --- /dev/null +++ b/installer/data/mysql/atomicupdate/bug_17813_borrower_attributes_pk.sql @@ -0,0 +1,11 @@ +ALTER TABLE `borrower_attributes` ADD `id` int(11) NOT NULL AUTO_INCREMENT PRIMARY KEY FIRST; + +-- $DBversion = "16.12.00.XXX"; +-- if(CheckVersion($DBversion)) { +-- $dbh->do(q{ +-- ALTER TABLE `borrower_attributes` ADD `id` int(11) NOT NULL AUTO_INCREMENT PRIMARY KEY FIRST +-- }); +-- +-- print "Upgrade to $DBversion done (Bug 17813: Table borrower_attributes needs a primary key\n"; +-- SetVersion ($DBversion); +-- } diff --git a/installer/data/mysql/kohastructure.sql b/installer/data/mysql/kohastructure.sql index 2533e9d66e..cc57e3d9fa 100644 --- a/installer/data/mysql/kohastructure.sql +++ b/installer/data/mysql/kohastructure.sql @@ -1676,6 +1676,7 @@ CREATE TABLE `borrowers` ( -- this table includes information about your patrons DROP TABLE IF EXISTS `borrower_attributes`; CREATE TABLE `borrower_attributes` ( -- values of custom patron fields known as extended patron attributes linked to patrons/borrowers + `id` int(11) NOT NULL AUTO_INCREMENT PRIMARY KEY, -- Row id field `borrowernumber` int(11) NOT NULL, -- foreign key from the borrowers table, defines which patron/borrower has this attribute `code` varchar(10) NOT NULL, -- foreign key from the borrower_attribute_types table, defines which custom field this value was entered for `attribute` varchar(255) default NULL, -- custom patron field value -- 2.39.5