[Bug 8498] New: can't specify an hour when specifying due date
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8498 Priority: P5 - low Change sponsored?: --- Bug ID: 8498 CC: gmcharlt@gmail.com, kyle.m.hall@gmail.com Assignee: koha-bugs@lists.koha-community.org Summary: can't specify an hour when specifying due date Severity: normal Classification: Unclassified OS: All Reporter: nengard@gmail.com Hardware: All Status: NEW Version: master Component: Circulation Product: Koha With hourly loans Koha can make items due specific hours, but if you have the specify due date box on the checkout screen you can't enter an hour in it ... -- 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=8498 --- Comment #1 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 11115 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11115&action=edit Bug 8498 - can't specify an hour when specifying due date With hourly loans Koha can make items due specific hours, but if you have the specify due date box on the checkout screen you can't enter an hour/minute. It will always default to 23:59. * Add jquery timepicker plugin * Update jquery ui to include timepicker prerequisites * Set timepicker's default time to 23:59, remove the hard coded 23:59 setting from circulation.pl -- 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=8498 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff CC| |kyle@bywatersolutions.com -- 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=8498 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA Assignee|koha-bugs@lists.koha-commun |kyle@bywatersolutions.com |ity.org | --- Comment #2 from Owen Leonard <oleonard@myacpl.org> --- This patch adds *all* jQueryUI interactions, widgets, and effects to Koha, whereas I was trying to add them only as needed. I would prefer it if you could use the jQueryUI download configuration tool to add just the additional widgets necessary for the timepicker. The current baseline is: - Core - Autocomplete - Tabs - Datepicker -- 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=8498 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |paul.poulain@biblibre.com --- Comment #3 from Paul Poulain <paul.poulain@biblibre.com> --- (In reply to comment #2)
This patch adds *all* jQueryUI interactions, widgets, and effects to Koha, whereas I was trying to add them only as needed. I would prefer it if you could use the jQueryUI download configuration tool to add just the additional widgets necessary for the timepicker.
The current baseline is:
- Core - Autocomplete - Tabs - Datepicker
Sounds OK/fair to me. What about adding a wiki page (linked to coding guidelines -maybe not coding guidelines page itself) about those rules ? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8498 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #11115|0 |1 is obsolete| | --- Comment #4 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 11123 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11123&action=edit Bug 8498 - can't specify an hour when specifying due date With hourly loans Koha can make items due specific hours, but if you have the specify due date box on the checkout screen you can't enter an hour/minute. It will always default to 23:59. * Add jquery timepicker plugin * Update jquery ui to include timepicker prerequisites * Set timepicker's default time to 23:59, remove the hard coded 23:59 setting from circulation.pl This variation only adds the jquery ui slider to the existing jquery ui file. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8498 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #11123|0 |1 is obsolete| | --- Comment #5 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 11124 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11124&action=edit Bug 8498 - can't specify an hour when specifying due date With hourly loans Koha can make items due specific hours, but if you have the specify due date box on the checkout screen you can't enter an hour/minute. It will always default to 23:59. * Add jquery timepicker plugin * Update jquery ui to include timepicker prerequisite slider * Set timepicker's default time to 23:59, remove the hard coded 23:59 setting from circulation.pl -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8498 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8498 --- Comment #6 from Marc Véron <veron@veron.ch> --- Created attachment 11196 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11196&action=edit In input field last minute is not displayed Patch works as expected. - Minor issue with CSS: Input field does not fully display the time (last minute not visible). See screenshot. - Question: If an item is issued, should "Renewal due date" display time as well? Marc -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8498 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |veron@veron.ch -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8498 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #11124|0 |1 is obsolete| | --- Comment #7 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 11201 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11201&action=edit Bug 8498 - can't specify an hour when specifying due date With hourly loans Koha can make items due specific hours, but if you have the specify due date box on the checkout screen you can't enter an hour/minute. It will always default to 23:59. * Add jquery timepicker plugin * Update jquery ui to include timepicker prerequisite slider * Set timepicker's default time to 23:59, remove the hard coded 23:59 setting from circulation.pl * Resize input fields such that the entire date and time are visible. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8498 --- Comment #8 from Kyle M Hall <kyle@bywatersolutions.com> --- (In reply to comment #6)
Created attachment 11196 [details] In input field last minute is not displayed
Patch works as expected.
- Minor issue with CSS: Input field does not fully display the time (last minute not visible). See screenshot.
Fixed in new patch.
- Question: If an item is issued, should "Renewal due date" display time as well?
It already does. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8498 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #11201|0 |1 is obsolete| | --- Comment #9 from Marc Véron <veron@veron.ch> --- Created attachment 11202 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11202&action=edit [SIGNED-OFF] Bug 8498 - can't specify an hour when specifying due date With hourly loans Koha can make items due specific hours, but if you have the specify due date box on the checkout screen you can't enter an hour/minute. It will always default to 23:59. * Add jquery timepicker plugin * Update jquery ui to include timepicker prerequisite slider * Set timepicker's default time to 23:59, remove the hard coded 23:59 setting from circulation.pl * Resize input fields such that the entire date and time are visible. Signed-off-by: Marc Veron <veron@veron.ch> Works as expected, input field displays entire date and time. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8498 Marc Véron <veron@veron.ch> 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=8498 --- Comment #10 from Paul Poulain <paul.poulain@biblibre.com> --- QA Comment: * Owen/Kyle, I need a confirmation: is the upgrade - * jQuery UI CSS Framework 1.8.20 + * jQuery UI CSS Framework 1.8.22 needed, why ? are you sure it's harmless ? * If you don't use hourly loans, is it possible not to have the hour/minute picker ? (not sure) Other than this, nothing against a passed QA status -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8498 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de --- Comment #11 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Paul, I think at the moment we don't have a system prefernce for hourly loans, so to not see the minutes the system would have to check the circulation rules? I would like it a bit better too, if it possible to easily differentiate between loans in days and hourly loans. For example I think 'normal' loans shouldn't display the minute part, as it will probably be confusing to some libraries. But probably way out of scope for this bug :) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8498 --- Comment #12 from Kyle M Hall <kyle@bywatersolutions.com> --- (In reply to comment #10)
QA Comment: * Owen/Kyle, I need a confirmation: is the upgrade - * jQuery UI CSS Framework 1.8.20 + * jQuery UI CSS Framework 1.8.22 needed, why ? are you sure it's harmless ?
I needed to add components that were not in our current version. The jQuery UI builder does not give the option of selecting a previous version.
* If you don't use hourly loans, is it possible not to have the hour/minute picker ? (not sure)
Not in this iteration, but I don't feel that it gets in the way, even if it is not hidden. We aren't hiding the hours portion of due dates anywhere else ( at least not on purpose ). -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8498 Mason James <mtj@kohaaloha.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA CC| |mtj@kohaaloha.com --- Comment #13 from Mason James <mtj@kohaaloha.com> --- (In reply to comment #10)
Other than this, nothing against a passed QA status
i think Kyle has answered Cait's and Paul's questions to satisfaction, so... patch looks good, passing QA :) mason@xen1:~/git/head$ koha-qa.pl * e99beda Bug 8498 - can't specify an hour when specifying due date circ/circulation.pl koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css koha-tmpl/intranet-tmpl/prog/en/lib/jquery/jquery-ui.css koha-tmpl/intranet-tmpl/prog/en/lib/jquery/jquery-ui.js koha-tmpl/intranet-tmpl/prog/en/lib/jquery/plugins/jquery-ui-timepicker-addon.js koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt - perlcritic-progressive tests... OK - perl -c syntax tests... OK - xt/tt_valid.t tests... OK - xt/author/valid-template.t tests... OK -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8498 --- Comment #14 from Owen Leonard <oleonard@myacpl.org> --- FYI I have added (In reply to comment #3)
Sounds OK/fair to me. What about adding a wiki page (linked to coding guidelines -maybe not coding guidelines page itself) about those rules ?
I updated this section with the information: http://wiki.koha-community.org/wiki/Coding_Guidelines#Using_jQueryUI_widgets -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8498 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master Version|master |rel_3_8 --- Comment #15 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=8498 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |chris@bigballofwax.co.nz --- Comment #16 from Chris Cormack <chris@bigballofwax.co.nz> --- Does not apply to 3.8.x please submit a patch for 3.8.x if one is needed -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8498 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |RESOLVED Resolution|--- |FIXED --- Comment #17 from Owen Leonard <oleonard@myacpl.org> --- (In reply to comment #16)
Does not apply to 3.8.x please submit a patch for 3.8.x if one is needed
Not applicable to 3.8.x. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org