[Bug 13536] New: Date due without time on CHECKOUT alert
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13536 Bug ID: 13536 Summary: Date due without time on CHECKOUT alert Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: minor Priority: P5 - low Component: Notices Assignee: koha-bugs@lists.koha-community.org Reporter: m.de.rooy@rijksmuseum.nl QA Contact: testopia@bugs.koha-community.org In the issues table date_due is a datetime field. If you do not have hourly loans, the 23:59:00 on the CHECKOUT notice may be somewhat confusing. (When does the library close?) In the patch submitted I virtually add a date_due_without_time column to the issues table by adding such a field in the SELECT statement in Letters.pm, routine _parseletter_sth. If you want to remove the time from the notice, now you can just edit the notice and replace <<issues.date_due>> by <<issues.date_due_without_time>>. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13536 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13536 --- Comment #1 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 35033 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=35033&action=edit Bug 13536: Remove time from CHECKOUT notice In the issues table date_due is a datetime field. If you do not have hourly loans, the 23:59:00 on the CHECKOUT notice may be somewhat confusing. (When does the library close?) In the patch submitted I virtually add a date_due_without_time column to the issues table by adding such a field in the SELECT statement in Letters.pm, routine _parseletter_sth. If you want to remove the time from the notice, now you can just edit the notice and replace <<issues.date_due>> by <<issues.date_due_without_time>>. Test plan: [1] Edit CHECKOUT notice as described above. [2] Check out a book to a patron (notice enabled in his messaging prefs). [3] Check the notice: Do you only see the date, not the time? -- 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=13536 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Small patch -- 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=13536 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |m.de.rooy@rijksmuseum.nl |ity.org | -- 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=13536 Aleisha Amohia <aleishaamohia@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |aleishaamohia@hotmail.com Status|Needs Signoff |Failed QA --- Comment #2 from Aleisha Amohia <aleishaamohia@hotmail.com> --- Can't see <<issues.date_due_without_time>> to replace <<issues.date_due>> when editing the notice - setting to Failed QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13536 M. de Rooy <m.de.rooy@rijksmuseum.nl> 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=13536 --- Comment #3 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 35443 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=35443&action=edit Bug 13536: Follow-up for adding date_due_without_time in column list This patch adds the ("virtual") column date_due_without_time to the list of available columns when editing a notice (for the issues table). Test plan: Add the column to the CHECKOUT notice. Verify that the actual notice at checkout now contains the expected data. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> I also tested this by adding the field to DUE and running the overdues cron job. Note that in that case it must be included in an <item> section. (This is a restriction imposed by the code in the overdue_notices.pl script.) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13536 --- Comment #4 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- (In reply to Aleisha Amohia from comment #2)
Can't see <<issues.date_due_without_time>> to replace <<issues.date_due>> when editing the notice - setting to Failed QA
Thanks, Aleisha for testing. Hopefully, you still have the time to check 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=13536 Aleisha Amohia <aleishaamohia@hotmail.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=13536 Aleisha Amohia <aleishaamohia@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #35443|0 |1 is obsolete| | --- Comment #5 from Aleisha Amohia <aleishaamohia@hotmail.com> --- Created attachment 35453 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=35453&action=edit Bug 13536: Follow-up for adding date_due_without_time in column list This patch adds the ("virtual") column date_due_without_time to the list of available columns when editing a notice (for the issues table). Test plan: Add the column to the CHECKOUT notice. Verify that the actual notice at checkout now contains the expected data. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> I also tested this by adding the field to DUE and running the overdues cron job. Note that in that case it must be included in an <item> section. (This is a restriction imposed by the code in the overdue_notices.pl script.) Signed-off-by: Aleisha <aleishaamohia@hotmail.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13536 --- Comment #6 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 35507 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=35507&action=edit Bug 13536 [Proposed Followup] - Use multi-select -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13536 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #35507|0 |1 is obsolete| | CC| |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=13536 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@biblibre.co | |m --- Comment #7 from Jonathan Druart <jonathan.druart@biblibre.com> --- Marcel, don't you think it could be better to let the choice to the user? We could imagine a filter like <<my_table.my_date_field | date_without_time>>. Like that we could have a global solution, for all fields, not only for date_due. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13536 --- Comment #8 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- (In reply to Jonathan Druart from comment #7)
Marcel, don't you think it could be better to let the choice to the user? We could imagine a filter like <<my_table.my_date_field | date_without_time>>. Like that we could have a global solution, for all fields, not only for date_due.
The user cannot choose now: he gets the time inclusive. I offer him the choice of removing the time with the new field. I would welcome a global solution. This is indeed quite ad-hoc (like more things in Koha, so very consistent :) Building a filter option in the notices would take more time. Actually, we could consider building the notices through TT and add translation as well.. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13536 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=13622 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13536 --- Comment #9 from Jonathan Druart <jonathan.druart@biblibre.com> --- (In reply to M. de Rooy from comment #8)
I would welcome a global solution.
I had a try on bug 13622, feedback welcomed ;) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13536 --- Comment #10 from Kyle M Hall <kyle@bywatersolutions.com> --- I think the best solution would be to automatically strip times from items due at 23:59 Would this not solve the issue completely without the need for any special markup? (In reply to Jonathan Druart from comment #9)
(In reply to M. de Rooy from comment #8)
I would welcome a global solution.
I had a try on bug 13622, feedback welcomed ;)
-- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13536 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |In Discussion -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13536 --- Comment #11 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- (In reply to Kyle M Hall from comment #10)
I think the best solution would be to automatically strip times from items due at 23:59 Could be. But imo it is in the same ad-hoc category as the proposed solution.
-- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13536 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |DUPLICATE Status|In Discussion |RESOLVED --- Comment #12 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- *** This bug has been marked as a duplicate of bug 13622 *** -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org