Bugzilla – Attachment 6068 Details for
Bug 6880
Reason for suggestions do not save
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[SIGNED-OFF] Bug 6880- Reason for suggestions do not save
SIGNED-OFF-Bug-6880--Reason-for-suggestions-do-not.patch (text/plain), 2.58 KB, created by
Katrin Fischer
on 2011-10-26 22:33:12 UTC
(
hide
)
Description:
[SIGNED-OFF] Bug 6880- Reason for suggestions do not save
Filename:
MIME Type:
Creator:
Katrin Fischer
Created:
2011-10-26 22:33:12 UTC
Size:
2.58 KB
patch
obsolete
>From a934b0aa8cb3e4a6225f91c3ee087e144a7dbe14 Mon Sep 17 00:00:00 2001 >From: Owen Leonard <oleonard@myacpl.org> >Date: Wed, 26 Oct 2011 14:17:52 -0400 >Subject: [PATCH] [SIGNED-OFF] Bug 6880- Reason for suggestions do not save > >The fix for Bug 1883 seems to have lacked the necessary >update to the database to add the "patronreason" column >to the suggestions table. This patch does so. > >Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> >Patch adds database update and change to kohastructure. >After this patch is applied, the patron's reason for >making a suggestion (form in opac) is saved to the database >correctly. The reason is also shown in the detail view >of the suggestion. > >This needs to be backported to 3.6 >--- > installer/data/mysql/kohastructure.sql | 3 ++- > installer/data/mysql/updatedatabase.pl | 7 +++++++ > 2 files changed, 9 insertions(+), 1 deletions(-) > >diff --git a/installer/data/mysql/kohastructure.sql b/installer/data/mysql/kohastructure.sql >index e4388a4..35967ac 100644 >--- a/installer/data/mysql/kohastructure.sql >+++ b/installer/data/mysql/kohastructure.sql >@@ -1861,7 +1861,8 @@ CREATE TABLE `suggestions` ( -- purchase suggestions > `isbn` varchar(30) default NULL, -- isbn of the suggested item > `mailoverseeing` smallint(1) default 0, > `biblionumber` int(11) default NULL, -- foreign key linking the suggestion to the biblio table after the suggestion has been ordered >- `reason` text, -- reason for making the suggestion >+ `reason` text, -- reason for accepting or rejecting the suggestion >+ `patronreason` text, -- reason for making the suggestion > budgetid INT(11), -- foreign key linking the suggested budget to the aqbudgets table > branchcode VARCHAR(10) default NULL, -- foreign key linking the suggested branch to the branches table > collectiontitle text default NULL, -- collection name for the suggested item >diff --git a/installer/data/mysql/updatedatabase.pl b/installer/data/mysql/updatedatabase.pl >index a8cd166..7d6090c 100755 >--- a/installer/data/mysql/updatedatabase.pl >+++ b/installer/data/mysql/updatedatabase.pl >@@ -4550,6 +4550,13 @@ if (C4::Context->preference("Version") < TransformToNum($DBversion)) { > SetVersion ($DBversion); > } > >+$DBversion = "3.06.00.XXX"; >+if (C4::Context->preference("Version") < TransformToNum($DBversion)) { >+ $dbh->do("ALTER TABLE `suggestions` ADD `patronreason` TEXT NULL AFTER `reason`"); >+ print "Upgrade to $DBversion done (Add column to suggestions table to store patrons' reasons for submitting a suggestion. )\n"; >+ SetVersion($DBversion); >+} >+ > =head1 FUNCTIONS > > =head2 DropAllForeignKeys($table) >-- >1.7.5.4
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 6880
:
6066
| 6068