Attachment 34545 Details for Bug 10848 – [PASSED QA] Bug 10848 - Allow configuration of mandatory/required fields on the suggestion form in OPAC #2/2

[patch] [PASSED QA] Bug 10848 - Allow configuration of mandatory/required fields on the suggestion form in OPAC #2/2

PASSED-QA-Bug-10848---Allow-configuration-of-manda.patch (text/plain), 3.73 KB, created by Kyle M Hall (khall) on 2014-12-19 14:15:42 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2014-12-19 14:15:42 UTC

Size: 3.73 KB

patch

obsolete

>From 3b298a615de3a4e28b21e109717c0045ecb3f16f Mon Sep 17 00:00:00 2001
>From: Jacek Ablewicz <abl@biblos.pk.edu.pl>
>Date: Sun, 8 Jun 2014 12:52:08 +0200
>Subject: [PATCH] [PASSED QA] Bug 10848 - Allow configuration of mandatory/required fields on the suggestion form in OPAC #2/2
>
>(part #2: new syspref 'OPACSuggestionMandatoryFields' + DB updates)
>
>Signed-off-by: Nick Clemens <nick@debian>
>
>Signed-off-by: Valerie Bertrand <valerie.bertrand@univ-lyon3.fr>
>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>---
> installer/data/mysql/sysprefs.sql                  |    1 +
> installer/data/mysql/updatedatabase.pl             |    7 +++++++
> .../prog/en/modules/admin/preferences/opac.pref    |    7 +++++++
> 3 files changed, 15 insertions(+), 0 deletions(-)
>
>diff --git a/installer/data/mysql/sysprefs.sql b/installer/data/mysql/sysprefs.sql
>index c3e308f..15b95f1 100644
>--- a/installer/data/mysql/sysprefs.sql
>+++ b/installer/data/mysql/sysprefs.sql
>@@ -305,6 +305,7 @@ INSERT INTO systempreferences ( `variable`, `value`, `options`, `explanation`, `
> ('OPACShowUnusedAuthorities','1','','Show authorities that are not being used in the OPAC.','YesNo'),
> ('OpacStarRatings','all','disable|all|details',NULL,'Choice'),
> ('OpacSuggestionManagedBy',1,'','Show the name of the staff member who managed a suggestion in OPAC','YesNo'),
>+('OPACSuggestionMandatoryFields','',NULL,'Define the mandatory fields for a patron purchase suggestions made via OPAC.','free'),
> ('OpacSuppression','0','','Turn ON the OPAC Suppression feature, requires further setup, ask your system administrator for details','YesNo'),
> ('OpacSuppressionByIPRange','','','Restrict the suppression to IP adresses outside of the IP range','free'),
> ('OpacSuppressionMessage','','Display this message on the redirect page for suppressed biblios','70|10','Textarea'),
>diff --git a/installer/data/mysql/updatedatabase.pl b/installer/data/mysql/updatedatabase.pl
>index a0698ef..85ae446 100755
>--- a/installer/data/mysql/updatedatabase.pl
>+++ b/installer/data/mysql/updatedatabase.pl
>@@ -9585,6 +9585,13 @@ if ( CheckVersion($DBversion) ) {
>     SetVersion ($DBversion);
> }
> 
>+$DBversion = "3.19.00.XXX";
>+if (CheckVersion($DBversion)) {
>+    $dbh->do("INSERT INTO systempreferences (variable,value,options,explanation,type) VALUES('OPACSuggestionMandatoryFields', '', 'NULL', 'Define the mandatory fields for a patron purchase suggestions made via OPAC.', 'Free');");
>+    print "Upgrade to $DBversion done (Bug 10848 - Allow configuration of mandatory/required fields on the suggestion form in OPAC)\n";
>+    SetVersion($DBversion);
>+}
>+
> =head1 FUNCTIONS
> 
> =head2 TableExists($table)
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/opac.pref b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/opac.pref
>index 45df9f8..458b3d1 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/opac.pref
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/opac.pref
>@@ -530,6 +530,13 @@ OPAC:
>                   yes: Allow
>             - patrons to select library when making a purchase suggestion
>         -
>+            - "Fields (separated with |) that should be mandatory for patron purchase suggestions:"
>+            - pref: OPACSuggestionMandatoryFields
>+              class: multi
>+            - "<br />Field names you can set here as mandatory for OPAC suggestions include: <code>author</code>, <code>copyrightdate</code>,"
>+            - "<code>isbn</code>, <code>publishercode</code>, <code>place</code>, <code>itemtype</code> and <code>patronreason</code>."
>+            - "Note: <code>title</code> field would be allways implicitly required, no need to enter it here."
>+        -
>             - pref: OpacHiddenItems
>               type: textarea
>               class: code
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 10848: 28721 | 28722 | 28792 | 29091 | 29092 | 34317 | 34318 | 34319 | 34324 | 34325 | 34326 | 34544 | 34545 | 34546 | 34547 | 35830 | 54016 | 54017 | 54018 | 54019 | 54020 | 54021 | 54083 | 54084 | 54356 | 54430 | 54431 | 54432