Bug 11747 - Default to logged in library for Overdue notice/status triggers
Summary: Default to logged in library for Overdue notice/status triggers
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Architecture, internals, and plumbing (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Kyle M Hall
QA Contact: Jonathan Druart
URL:
Keywords:
Depends on: 11625
Blocks:
  Show dependency treegraph
 
Reported: 2014-02-12 13:02 UTC by Kyle M Hall
Modified: 2017-06-14 22:10 UTC (History)
8 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 11747 - Default to logged in library for Overdue notice/status triggers (2.04 KB, patch)
2014-02-12 13:07 UTC, Kyle M Hall
Details | Diff | Splinter Review
Bug 11747 - Default to logged in library for Overdue notice/status triggers (2.10 KB, patch)
2014-02-14 21:23 UTC, Biblibre Sandboxes
Details | Diff | Splinter Review
Bug 11747 - Default to logged in library for Overdue notice/status triggers (2.19 KB, patch)
2014-02-17 11:47 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 11747 [QA Followup] - Don't default to branch specific rules if only one branch exists (1018 bytes, patch)
2014-03-05 12:56 UTC, Kyle M Hall
Details | Diff | Splinter Review
Bug 11747 - Default to logged in library for Overdue notice/status triggers (2.19 KB, patch)
2015-01-22 14:07 UTC, Kyle M Hall
Details | Diff | Splinter Review
Bug 11747 [QA Followup] - Don't default to branch specific rules if only one branch exists (989 bytes, patch)
2015-01-22 14:07 UTC, Kyle M Hall
Details | Diff | Splinter Review
Bug 11747 [QA Followup 2] - Use system preference to control behavior (4.80 KB, patch)
2015-01-22 14:07 UTC, Kyle M Hall
Details | Diff | Splinter Review
Bug 11747 [QA Followup 2] - Use system preference to control behavior (1.71 KB, patch)
2015-04-18 03:24 UTC, Mark Tompsett
Details | Diff | Splinter Review
Bug 11747 - Add system preference DefaultToLoggedInLibraryOverdueTriggers (3.76 KB, patch)
2015-04-21 15:20 UTC, Kyle M Hall
Details | Diff | Splinter Review
Bug 11747 - Implement DefaultToLoggedInLibraryOverdueTriggers (2.23 KB, patch)
2015-04-21 15:20 UTC, Kyle M Hall
Details | Diff | Splinter Review
Bug 11747 - Implement DefaultToLoggedInLibraryOverdueTriggers (2.23 KB, patch)
2015-04-21 15:22 UTC, Kyle M Hall
Details | Diff | Splinter Review
Bug 11747 [QA Followup] - Use system preference (1.31 KB, patch)
2015-04-22 15:22 UTC, Kyle M Hall
Details | Diff | Splinter Review
Bug 11747 - Add system preference DefaultToLoggedInLibraryOverdueTriggers (3.89 KB, patch)
2015-10-16 11:53 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 11747 - Implement DefaultToLoggedInLibraryOverdueTriggers (2.22 KB, patch)
2015-10-16 11:53 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 11747 [QA Followup] - Use system preference (1.38 KB, patch)
2015-10-16 11:53 UTC, Jonathan Druart
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Kyle M Hall 2014-02-12 13:02:09 UTC
As with bug 11625, we should make the default selection for overdue notice/status triggers the logged in library.
Comment 1 Kyle M Hall 2014-02-12 13:07:05 UTC Comment hidden (obsolete)
Comment 2 Biblibre Sandboxes 2014-02-14 21:23:13 UTC
Patch tested with a sandbox, by Nick Clemens <nick@quecheelibrary.org>
Comment 3 Biblibre Sandboxes 2014-02-14 21:23:31 UTC Comment hidden (obsolete)
Comment 4 Jonathan Druart 2014-02-17 11:47:55 UTC Comment hidden (obsolete)
Comment 5 Galen Charlton 2014-02-19 17:38:37 UTC
Setting to in discussion for the same reason I set bug 11625 to in discussion.  I suggest that we use bug 11625 for all discussion, as whatever resolution occurs for that bug will, for the sake of consistency, should become the resolution for this one.
Comment 6 Kyle M Hall 2014-03-05 12:56:28 UTC Comment hidden (obsolete)
Comment 7 Katrin Fischer 2014-04-27 16:15:43 UTC
This depends on 11625 as we are still trying to agree how this should be working.
Comment 8 Kyle M Hall 2015-01-22 14:07:05 UTC Comment hidden (obsolete)
Comment 9 Kyle M Hall 2015-01-22 14:07:19 UTC Comment hidden (obsolete)
Comment 10 Kyle M Hall 2015-01-22 14:07:22 UTC Comment hidden (obsolete)
Comment 11 Kyle M Hall 2015-01-22 14:07:38 UTC
I don't think we'll ever get consensus on this, so I think the best way to move forward is to make this behavior controlled via system preferences.
Comment 12 Jonathan Druart 2015-02-11 11:35:22 UTC
This is blocked by the discussion on bug 11625.
Comment 13 Jonathan Druart 2015-04-17 08:35:00 UTC
It needs QA with bug 11625.
Comment 14 Jonathan Druart 2015-04-17 09:20:22 UTC
Bug 11625 does not apply, please also rebase this one.
Comment 15 Mark Tompsett 2015-04-18 03:24:45 UTC Comment hidden (obsolete)
Comment 16 Mark Tompsett 2015-04-18 03:26:51 UTC
Set to signed off from patch doesn't apply, since I believe that's where it should be.
Comment 17 Jonathan Druart 2015-04-20 10:57:40 UTC
Kyle,
I am almost ready to mark bug 11625 and this one as pass qa, but the updatedb entry is not clear.
Could you please add a clean patch for each pref (1 updatedb entry + sysprefs.sql + pref template)? This means 1 patch here for DefaultToLoggedInLibraryOverdueTriggers and 1 patch on bug 11625 for DefaultToLoggedInLibraryCircRules.
Comment 18 Kyle M Hall 2015-04-21 15:20:43 UTC Comment hidden (obsolete)
Comment 19 Kyle M Hall 2015-04-21 15:20:52 UTC Comment hidden (obsolete)
Comment 20 Kyle M Hall 2015-04-21 15:22:14 UTC Comment hidden (obsolete)
Comment 21 Jonathan Druart 2015-04-22 14:56:38 UTC
DefaultToLoggedInLibraryOverdueTriggers is never used in the code?
Comment 22 Kyle M Hall 2015-04-22 15:22:51 UTC Comment hidden (obsolete)
Comment 23 Marcel de Rooy 2015-08-20 08:02:21 UTC
Applying: Bug 11747 - Add system preference DefaultToLoggedInLibraryOverdueTriggers
/home/koha/testclone/.git/rebase-apply/patch:50: trailing whitespace.
            - When editing overdue notice/status triggers show the
fatal: sha1 information is lacking or useless (installer/data/mysql/sysprefs.sql).
Repository lacks necessary blobs to fall back on 3-way merge.
Cannot fall back to three-way merge.
Comment 24 Marcel de Rooy 2015-08-20 08:03:30 UTC
(In reply to Marcel de Rooy from comment #23)
> Applying: Bug 11747 - Add system preference
> DefaultToLoggedInLibraryOverdueTriggers
> /home/koha/testclone/.git/rebase-apply/patch:50: trailing whitespace.
>             - When editing overdue notice/status triggers show the
> fatal: sha1 information is lacking or useless
> (installer/data/mysql/sysprefs.sql).
> Repository lacks necessary blobs to fall back on 3-way merge.
> Cannot fall back to three-way merge.

oops, dependency
Comment 25 Jonathan Druart 2015-10-16 11:53:10 UTC
Created attachment 43499 [details] [review]
Bug 11747 - Add system preference DefaultToLoggedInLibraryOverdueTriggers

Signed-off-by: Nick Clemens <nick@quecheelibrary.org>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Amended patch: INSERT IGNORE INTO + msg
Comment 26 Jonathan Druart 2015-10-16 11:53:13 UTC
Created attachment 43500 [details] [review]
Bug 11747 - Implement DefaultToLoggedInLibraryOverdueTriggers

This patch makes the default circ rules and notices/slips to edit
the logged in branch, if one is set.

Test Plan:
1) Apply this patch
2) Log into staff side, and set your library
3) Browse to Tools/Overdue notice/status triggers
4) Note the actions displayed are for your library
5) Select "All libraries"
6) Note the actions displayed are for "All libraries"

Signed-off-by: Nick Clemens <nick@quecheelibrary.org>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 27 Jonathan Druart 2015-10-16 11:53:23 UTC
Created attachment 43501 [details] [review]
Bug 11747 [QA Followup] - Use system preference

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 28 Kyle M Hall 2015-12-30 11:49:26 UTC
Pushed to master!