Created attachment 148518 [details] report schedule start time input fix Hello community, I offer an enhancement for the task scheduler form: /cgi-bin/koha/tools/scheduler.pl via scheduler.tt The time input element currently has an 'text' value for type attribute. This requires a user to enter a valid time string for any scheduled report manually which is no fun. I have created patch in scheduler.tt that changes the type attribute value from 'text' to 'time' which makes the input of valid time easy. This change leveraged standard web form principles of html5 and would be compatible with any modern browser such as chrome, firefox and safari etc. Patch incoming. Thoughts and feedback invited. Thanks. 8-)
Created attachment 148519 [details] [review] Bug 33304: Report schedule form time input element attribute type changed to improves valid time value entry I offer an enhancement for the task scheduler form: /cgi-bin/koha/tools/scheduler.pl via scheduler.tt The time input element currently has an 'text' value for type attribute. This requires a user to enter a valid time string for any scheduled report manually which is no fun. I have created patch in scheduler.tt that changes the type attribute value from 'text' to 'time' which makes the input of valid time easy. This change leveraged standard web form principles of html5 and would be compatible with any modern browser such as chrome, firefox and safari. To test: 1) login to koha instance 2) navigate to report module from home page 3) create test report with basic action and save. (no actual run of report is requires, simply need to save with valid syntax) ... SELECT id FROM biblio_metadata LIMIT 1 ... 4) Press 'Schedule' icon from nav options 5) Enter relevant time value for schedule in 'time' input element which now has type 'time' attribute 3.00 pm, added easily. 6) Enter valid future date 7) Enter valid email address and press 'Save' 8) SIGN OFF
Created attachment 149252 [details] [review] Bug 33304: Report schedule form time input element attribute type changed to improves valid time value entry I offer an enhancement for the task scheduler form: /cgi-bin/koha/tools/scheduler.pl via scheduler.tt The time input element currently has an 'text' value for type attribute. This requires a user to enter a valid time string for any scheduled report manually which is no fun. I have created patch in scheduler.tt that changes the type attribute value from 'text' to 'time' which makes the input of valid time easy. This change leveraged standard web form principles of html5 and would be compatible with any modern browser such as chrome, firefox and safari. To test: 1) login to koha instance 2) navigate to report module from home page 3) create test report with basic action and save. (no actual run of report is requires, simply need to save with valid syntax) ... SELECT id FROM biblio_metadata LIMIT 1 ... 4) Press 'Schedule' icon from nav options 5) Enter relevant time value for schedule in 'time' input element which now has type 'time' attribute 3.00 pm, added easily. 6) Enter valid future date 7) Enter valid email address and press 'Save' 8) SIGN OFF Signed-off-by: David Nind <david@davidnind.com>
This is a great idea!
Hi David, Thanks for the sign off! I know I would use this in my daily drive Koha once implemented. 8-)
(In reply to Adam Styles from comment #4) > Hi David, > > Thanks for the sign off! > > I know I would use this in my daily drive Koha once implemented. > > 8-) No problem Adam! Good luck with the QA process - there is a long list of items waiting for QA, so it may take some time 8-( (see the dashboard under 'Bug statuses' https://dashboard.koha-community.org/ ). David
This is a bit of an odd one for me. It looks like the form is not really up to 'Koha standard' to begin with. Meaning that time and date should not really be separate input fields. It should all be handled by he datepicker. Also: with the change, it doesn't work according to the TimeFormat preference still. In my case I see PM/AM times now, while TimeFormat is set to 24 hours. This makes me unsure of if it's better than before or not. Adam, would you be interested in having a look in how to use Flatpickr to achieve the same goal here? An example for date and time can be seen in circulation when specifying a due date for example.
Hi Katrin, Thanks for this feed back. I did not take into account any system preferences that might be applied to date/time format for this solution. I am happy to investigate how I can use Flatpickr to rebuild this. Thanks for the 'circulation' example. Stay tuned for a new patch soon. Thanks.
Hi Katrin, opps I will get onto this one. Apologies for delay in new patch. Stay tuned. Thanks.