[Bug 8585] New: Add System Preference to specify Holds to Pull List Start Date
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 Priority: P5 - low Change sponsored?: --- Bug ID: 8585 CC: gmcharlt@gmail.com Assignee: koha-bugs@lists.koha-community.org Summary: Add System Preference to specify Holds to Pull List Start Date Severity: enhancement Classification: Unclassified OS: All Reporter: dcook@prosentient.com.au Hardware: All Status: NEW Version: master Component: Hold requests Product: Koha When you are on the "Holds to Pull" page in the Circulation module, the default "Start Date" in the "Refine Results" section is 2 days in the past. I plan to introduce a system preference that changes the number of days to however many the librarian chooses. If the system preference isn't set, the default of 2 days will continue to be used. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff Assignee|koha-bugs@lists.koha-commun |dcook@prosentient.com.au |ity.org | --- Comment #1 from David Cook <dcook@prosentient.com.au> --- Created attachment 11516 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11516&action=edit Bug 8585 : Add System Preference to specify Holds to Pull List Start Date -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 --- Comment #2 from Owen Leonard <oleonard@myacpl.org> --- A system preference seems like overkill for this. Could we set a cookie that stored the offset so that the page would show the same offset later? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 --- Comment #3 from David Cook <dcook@prosentient.com.au> --- Good point. I don't know much about setting cookies, but I like the idea. That said, it would mean that different staff members would have different results in different browsers rather than one consistent result across the board. In my experience, staff don't tend to think about the date ranges on the side. They either just press the submit button or accept the results that they're given (if the report is automatically generated as in Bug 8454). In terms of policy, it's easier to have staff check a set range of dates that the manager or systems person has set up with the system preference, rather than relying on staff's knowledge of how far back to check for holds. Admittedly, a cookie would solve the problem initially, but it could create a problem where someone narrows the date range and then forgets to put it back...and then staff think that there are no holds to pull, because they don't realize that the range was changed. It seems too variable :/. But that's just my opinion. (In reply to comment #2)
A system preference seems like overkill for this. Could we set a cookie that stored the offset so that the page would show the same offset later?
-- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #11516|0 |1 is obsolete| | --- Comment #4 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 12052 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12052&action=edit Bug 8585 : Add System Preference to specify Holds to Pull List Start Date Currently, Koha creates a default value of 2 days ago (-2) for the start date of the Holds to pull List. This system preference allows users to specify their own default start date for this list, since users might not want to have to manually change the date all the time when they already know the set date period they want to view. The system preference value is specified as a positive integer, which is then passed as a negative integer in the handler script. This saves users from having to include a qualifer to the sys pref value. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kyle@bywatersolutions.com --- Comment #5 from Kyle M Hall <kyle@bywatersolutions.com> --- Works as advertised. I also am for the system preference. A cookie means the librarian will have to set it every session. We could possibly have the combination of both, but I think the syspref alone is better than a cookie alone. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 --- Comment #6 from Paul Poulain <paul.poulain@biblibre.com> --- Created attachment 12659 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12659&action=edit Bug 8585 follow-up Passing syspref to template Passing the syspref HoldsToPullStartDate to get the number of days displayed on the screen (instead of "two" that was hardcoded) Also s/tab/4 spaces/g -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 --- Comment #7 from Paul Poulain <paul.poulain@biblibre.com> --- QA comment: * passes koha-qa.pl * the feature work * after applying the patch, the koha/circ/pendingreserves.pl still says: "(Inclusive, default is two days ago to today, set other date ranges as needed. )". I've written a tiny follow-up that display the syspref instead of "two" passed QA (with the follow-up) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA CC| |paul.poulain@biblibre.com Version|master |rel_3_10 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master --- Comment #8 from Paul Poulain <paul.poulain@biblibre.com> --- Patch pushed to master -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 --- Comment #9 from Nicole C. Engard <nengard@gmail.com> --- Created attachment 12796 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12796&action=edit Follow up to Bug 8585: add missing word to preference The preference added in this bug is missing the word 'day(s)'. This patch adds it in. http://bugs.koha-community.org/show_bug.cgi?id=4118 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #12052|0 |1 is obsolete| | --- Comment #10 from Nicole C. Engard <nengard@gmail.com> --- Comment on attachment 12052 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12052 Bug 8585 : Add System Preference to specify Holds to Pull List Start Date pushed to maaster -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #12659|0 |1 is obsolete| | --- Comment #11 from Nicole C. Engard <nengard@gmail.com> --- Comment on attachment 12659 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12659 Bug 8585 follow-up Passing syspref to template pushed to master -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |ASSIGNED CC| |nengard@gmail.com -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff Severity|enhancement |normal --- Comment #12 from Nicole C. Engard <nengard@gmail.com> --- string follow up needs sign off. changed from enhancement to get in to 3.10. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 --- Comment #13 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 12825 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12825&action=edit [SIGNED-OFF] Follow up to Bug 8585: add missing word to preference The preference added in this bug is missing the word 'day(s)'. This patch adds it in. http://bugs.koha-community.org/show_bug.cgi?id=4118 Signed-off-by: Owen Leonard <oleonard@myacpl.org> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #12796|0 |1 is obsolete| | --- Comment #14 from Owen Leonard <oleonard@myacpl.org> --- Comment on attachment 12796 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12796 Follow up to Bug 8585: add missing word to preference
From 5d09752547e6cde4f9840c01ec54ad3a381c166f Mon Sep 17 00:00:00 2001 From: Nicole C. Engard <nengard@bywatersolutions.com> Date: Thu, 11 Oct 2012 21:23:48 -0400 Subject: [PATCH] Follow up to Bug 8585: add missing word to preference
The preference added in this bug is missing the word 'day(s)'. This patch adds it in.
http://bugs.koha-community.org/show_bug.cgi?id=4118 --- .../en/modules/admin/preferences/circulation.pref | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref index 62938b4..6d58217 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref @@ -55,7 +55,7 @@ Circulation: - Set the default start date for the Holds to pull list to - pref: HoldsToPullStartDate class: integer - - ago. + - day(s) ago. - - pref: AllowAllMessageDeletion choices: -- 1.7.2.3
-- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA --- Comment #15 from Paul Poulain <paul.poulain@biblibre.com> --- follow-up passes QA of course -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master Severity|normal |enhancement --- Comment #16 from Paul Poulain <paul.poulain@biblibre.com> --- back to pushed to master & ENH, follow-up pushed -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 --- Comment #17 from David Cook <dcook@prosentient.com.au> --- I might be missing something, but where did the original patch and other follow-ups go? It looks like they've been mistakenly marked as obsolete... While circulation.pref looks good in master, it seems to me that no one will be able to apply this patch using git bz. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 Jane Wagner <jwagner@ptfs.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jwagner@ptfs.com --- Comment #18 from Jane Wagner <jwagner@ptfs.com> --- For background, see Bug 2368 -- the original default date range was 10 years. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 --- Comment #19 from Nicole C. Engard <nengard@gmail.com> --- David, After a patch is pushed it's marked obsolete to avoid clutter and make it clear which patches were pushed. Nicole -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|enhancement |trivial -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|trivial |enhancement -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8585 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |m.de.rooy@rijksmuseum.nl See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=9761 --- Comment #20 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Bug 9761 adds a similar preference for the end date. This does not only apply to Holds to pull but does extend to confirming future pending holds. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org