Attachment 43724 Details for Bug 14247 – Bug 14247: Add system preference for news display

[patch] Bug 14247: Add system preference for news display

Bug-14247-Add-system-preference-for-news-display.patch (text/plain), 4.36 KB, created by Mark Tompsett on 2015-10-22 00:26:34 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Mark Tompsett

Created: 2015-10-22 00:26:34 UTC

Size: 4.36 KB

patch

obsolete

>From 86510f0f1292d96917b2f978b33e42cbdffc1cac Mon Sep 17 00:00:00 2001
>From: Martin Persson <xarragon@gmail.com>
>Date: Mon, 25 May 2015 14:33:13 +0200
>Subject: [PATCH] Bug 14247: Add system preference for news display
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>Test plan:
>
>* Apply dependant patch 14246 (if not done already).
>* Apply current patch.
>* Log into the Staff client, go to 'Administration' -> 'Global Preferences' -> 'Tools', a new section titled 'News' should appear with a single option named 'NewsAuthorDisplay'. Available values should be 'Not at all', 'OPAC only', 'Staff client only' and 'Both OPAC and staff client'. By default the value should be 'Not at all' (to mimic existing behaviour, important!).
>* Change the value to something different and save the changes.
>* Use the mysql command line client to connect to the Koha database:
>  $ mysql -u kohaadmin -p -D koha
>* Enter the following query:
>  SELECT * FROM systempreferences WHERE variable = 'NewsAuthorDisplay';
>* Check to make sure that the 'value' column has changed to reflect the choice you made in the above step.
>
>Separate SQL testing (clean installation of Koha):
>* Koha$ mysql -u root < create_koha_test_db.sql
>* Koha$ mysql -u koha_test -D koha_test < installer/data/mysql/kohastructure.sql
>* Koha$ mysql -u koha_test -D koha_test < installer/data/mysql/sysprefs.sql
>
>Testing the update script works the same way:
>* Koha$ echo "DELETE FROM systempreferences WHERE variable = 'NewsAuthorDisplay';" | mysql -u koha_test -D koha_test
>* Koha$ mysql -u koha_test -D koha_test < installer/data/mysql/atomicupdate/add_news_prefs.sql
>
>Signed-off-by: Marc VÃ©ron <veron@veron.ch>
>
>Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
>---
> installer/data/mysql/atomicupdate/add_news_prefs.sql           |  2 ++
> installer/data/mysql/sysprefs.sql                              |  1 +
> .../intranet-tmpl/prog/en/modules/admin/preferences/tools.pref | 10 ++++++++++
> 3 files changed, 13 insertions(+)
> create mode 100644 installer/data/mysql/atomicupdate/add_news_prefs.sql
>
>diff --git a/installer/data/mysql/atomicupdate/add_news_prefs.sql b/installer/data/mysql/atomicupdate/add_news_prefs.sql
>new file mode 100644
>index 0000000..caca0b5
>--- /dev/null
>+++ b/installer/data/mysql/atomicupdate/add_news_prefs.sql
>@@ -0,0 +1,2 @@
>+-- Add NewsAuthorDisplay system preference, bug 14247.
>+INSERT IGNORE INTO systempreferences (`variable`,`value`,`options`,`explanation`,`type`) VALUES('NewsAuthorDisplay','none','none|opac|staff|both','Display the author name for news items.','Choice');
>diff --git a/installer/data/mysql/sysprefs.sql b/installer/data/mysql/sysprefs.sql
>index 578ab79..83395e4 100644
>--- a/installer/data/mysql/sysprefs.sql
>+++ b/installer/data/mysql/sysprefs.sql
>@@ -217,6 +217,7 @@ INSERT INTO systempreferences ( `variable`, `value`, `options`, `explanation`, `
> ('MembershipExpiryDaysNotice',NULL,'Send an account expiration notice that a patron\'s card is about to expire after',NULL,'Integer'),
> ('minPasswordLength','3',NULL,'Specify the minimum length of a patron/staff password','free'),
> ('NewItemsDefaultLocation','','','If set, all new items will have a location of the given Location Code ( Authorized Value type LOC )',''),
>+('NewsAuthorDisplay','none','none|opac|staff|both','Display the author name for news items.','Choice'),
> ('noissuescharge','5','','Define maximum amount withstanding before check outs are blocked','Integer'),
> ('noItemTypeImages','0',NULL,'If ON, disables item-type images','YesNo'),
> ('NoLoginInstructions', '', '60|10', 'Instructions to display on the OPAC login form when a patron is not logged in', 'Textarea'),
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/tools.pref b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/tools.pref
>index 636a9f6..5a3ddce 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/tools.pref
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/tools.pref
>@@ -11,3 +11,13 @@ Tools:
>             - pref: ImageLimit
>               class: Integer
>             - images.
>+    News:
>+        -
>+            - "Show the author for news items:"
>+            - pref: NewsAuthorDisplay
>+              choices:
>+                  none: "Not at all"
>+                  opac: "OPAC only"
>+                  staff: "Staff client only"
>+                  both: "Both OPAC and staff client"
>+            -
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 14247: 39362 | 39414 | 39418 | 39440 | 39488 | 39527 | 43093 | 43538 | 43539 | 43540 | 43541 | 43542 | 43543 | 43724 | 43725 | 43914 | 43915