Bug 20133

Summary: "Hide patron information" feature should not affect all library groups
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: System AdministrationAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: gmcharlt, josef.moravec, nick
Version: unspecified   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: ---
Patch complexity: --- Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
Circulation function:
Bug Depends on: 18403    
Bug Blocks: 20157    
Attachments: Bug 20133: Add new column library_groups.ft_hide_patron_info
Bug 20133: DBIC Schema changes
Bug 20133: UI Changes
Bug 20133: Enable/disable the "Hide patron information" feature per group of libraries
Bug 20133 [QA Followup] - Tweak new 'group features' section
Bug 20133: Add new column library_groups.ft_hide_patron_info
Bug 20133: Add new column library_groups.ft_hide_patron_info
Bug 20133: DBIC Schema changes
Bug 20133: UI Changes
Bug 20133: Enable/disable the "Hide patron information" feature per group of libraries
Bug 20133 [QA Followup] - Tweak new 'group features' section

Description Jonathan Druart 2018-02-02 17:20:33 UTC
The feature added by bug 18403 should not apply to all library groups. We should add the ability to enable/disable it only for specific hierarchies.
Comment 1 Jonathan Druart 2018-02-02 17:28:19 UTC
Created attachment 71156 [details] [review]
Bug 20133: Add new column library_groups.ft_hide_patron_info
Comment 2 Jonathan Druart 2018-02-02 17:28:22 UTC
Created attachment 71157 [details] [review]
Bug 20133: DBIC Schema changes
Comment 3 Jonathan Druart 2018-02-02 17:28:27 UTC
Created attachment 71158 [details] [review]
Bug 20133: UI Changes

Add a checkbox to enable/disable the feature for given group of
libraries.

Important note: Only the top level node can turn the feature, the idea
is to avoid confusion, ease testing (by limiting the number of cases)
and force libraries to define one tree per feature (Maybe this is wrong,
but maybe good to start)
Comment 4 Jonathan Druart 2018-02-02 17:28:31 UTC
Created attachment 71159 [details] [review]
Bug 20133: Enable/disable the "Hide patron information" feature per group of libraries

In order to control this feature correctly it needs to be enabled for
group of libraries and not for all groups defined in the system.
Groups will be used for different usages and so the feature must not be
enabled by default for all of them.

Test plan:
Retest bug 18403 with the feature turned on/off for a given tree
Comment 5 Kyle M Hall (khall) 2018-02-07 17:41:26 UTC
Created attachment 71296 [details] [review]
Bug 20133 [QA Followup] - Tweak new 'group features' section
Comment 6 Kyle M Hall (khall) 2018-02-09 11:43:36 UTC
Created attachment 71368 [details] [review]
Bug 20133: Add new column library_groups.ft_hide_patron_info

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 7 Kyle M Hall (khall) 2018-02-09 11:44:01 UTC
Created attachment 71369 [details] [review]
Bug 20133: Add new column library_groups.ft_hide_patron_info

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 8 Kyle M Hall (khall) 2018-02-09 11:44:07 UTC
Created attachment 71370 [details] [review]
Bug 20133: DBIC Schema changes

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 9 Kyle M Hall (khall) 2018-02-09 11:44:10 UTC
Created attachment 71371 [details] [review]
Bug 20133: UI Changes

Add a checkbox to enable/disable the feature for given group of
libraries.

Important note: Only the top level node can turn the feature, the idea
is to avoid confusion, ease testing (by limiting the number of cases)
and force libraries to define one tree per feature (Maybe this is wrong,
but maybe good to start)

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 10 Kyle M Hall (khall) 2018-02-09 11:44:13 UTC
Created attachment 71372 [details] [review]
Bug 20133: Enable/disable the "Hide patron information" feature per group of libraries

In order to control this feature correctly it needs to be enabled for
group of libraries and not for all groups defined in the system.
Groups will be used for different usages and so the feature must not be
enabled by default for all of them.

Test plan:
Retest bug 18403 with the feature turned on/off for a given tree

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 11 Kyle M Hall (khall) 2018-02-09 11:44:15 UTC
Created attachment 71373 [details] [review]
Bug 20133 [QA Followup] - Tweak new 'group features' section

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 12 Jonathan Druart 2018-02-09 21:46:21 UTC
I am going to push this change along with bug 18403.
Comment 13 Jonathan Druart 2018-02-12 18:47:41 UTC
Pushed to master for 18.05, thanks to everybody involved!
Comment 14 Nick Clemens (kidclamp) 2018-02-16 01:40:36 UTC
Awesome work all! Enhancement, not backported to stable.