From 2d88e67c66cf5ba09100adb08464c89cab463b48 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 --- .../mysql/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 1a2898da5e..10b8dd4d20 100644 --- a/installer/data/mysql/kohastructure.sql +++ b/installer/data/mysql/kohastructure.sql @@ -1677,6 +1677,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.11.0