[Bug 24850] New: Koha::DateUtils ignored offsets in RFC3339 datetimes
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Bug ID: 24850 Summary: Koha::DateUtils ignored offsets in RFC3339 datetimes Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: koha-bugs@lists.koha-community.org Reporter: martin.renvoize@ptfs-europe.com QA Contact: testopia@bugs.koha-community.org Depends on: 24643 The final test added in bug 24643 incorrectly tests RFC3339 timezone handling. If an offset is passed it should be applied to the given time to return an epoch relative to UTC and then the koha timezone config value should be applied against that to give us a koha instance relative time. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24643 [Bug 24643] Koha::DateUtils::dt_from_string rfc3339 cannot handle high precision seconds -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@bugs.koha-c | |ommunity.org -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #1 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 100585 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=100585&action=edit Bug 24850: Correct tests for offset handling in RFC3339 dates The tests were incorrectly passing for RFC3339 dates passed with an offset. This patch corrects the test. Test plan: 1/ Read the change 2/ Agree the change adheres to the RFC 3/ Run the test and varify it now fails 3/ Signoff -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #2 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 100586 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=100586&action=edit Bug 24850: Correct offset handling in dt_from_string This patch adds correct handling for when an offset is passed within an RFC3339 formatted datetime. Test plan 1/ Run the DateUtils test and varify it now passes. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #3 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 100587 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=100587&action=edit Bug 24850: Add tests for invalid RFC3339 dates We should die on invalidly formatted dates being passed. This patch adds such a test case. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |martin.renvoize@ptfs-europe |ity.org |.com -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=24608 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #100585|0 |1 is obsolete| | --- Comment #4 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 100592 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=100592&action=edit Bug 24850: Correct tests for offset handling in RFC3339 dates The tests were incorrectly passing for RFC3339 dates passed with an offset. This patch corrects the test. Test plan: 1/ Read the change 2/ Agree the change adheres to the RFC 3/ Run the test and varify it now fails 3/ Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #100586|0 |1 is obsolete| | --- Comment #5 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 100593 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=100593&action=edit Bug 24850: Correct offset handling in dt_from_string This patch adds correct handling for when an offset is passed within an RFC3339 formatted datetime. Test plan 1/ Run the DateUtils test and varify it now passes. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #100587|0 |1 is obsolete| | --- Comment #6 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 100594 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=100594&action=edit Bug 24850: Add tests for invalid RFC3339 dates We should die on invalidly formatted dates being passed. This patch adds such a test case. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #7 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 100595 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=100595&action=edit Bug 24850: Don't fallback when rfc3339 format is passed -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |agustinmoyano@theke.io, | |andrew.isherwood@ptfs-europ | |e.com, | |colin.campbell@ptfs-europe. | |com, tomascohen@gmail.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #8 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- I'm not sure about the final followup.. it scares me that we're always falling back to a relatively loose regex at the moment; even if a date_format is passed in. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|enhancement |major -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=24455 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #9 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Finally.. I think we also need to add some handling into the DateTime::Format::MySQL datetime_parser that we are using via DBIx::Class.. it does not take into account offsets/timezones in the format_datetime method. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #100592|0 |1 is obsolete| | --- Comment #10 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 100678 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=100678&action=edit Bug 24850: Correct tests for offset handling in RFC3339 dates The tests were incorrectly passing for RFC3339 dates passed with an offset. This patch corrects the test. Test plan: 1/ Read the change 2/ Agree the change adheres to the RFC 3/ Run the test and varify it now fails 3/ Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #100593|0 |1 is obsolete| | --- Comment #11 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 100679 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=100679&action=edit Bug 24850: Correct offset handling in dt_from_string This patch adds correct handling for when an offset is passed within an RFC3339 formatted datetime. Test plan 1/ Run the DateUtils test and varify it now passes. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #100594|0 |1 is obsolete| | --- Comment #12 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 100680 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=100680&action=edit Bug 24850: Add tests for invalid RFC3339 dates We should die on invalidly formatted dates being passed. This patch adds such a test case. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #13 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 100681 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=100681&action=edit Bug 24850: Don't fallback when date_format is passed -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #100595|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #14 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- (In reply to Martin Renvoize from comment #8)
I'm not sure about the final followup.. it scares me that we're always falling back to a relatively loose regex at the moment; even if a date_format is passed in.
Corrected in final followup. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #15 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 100881 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=100881&action=edit Bug 24850: Always return 'instance' local time This patch adds handling to ensure we always convert a passed in time to the instance configured timezone.. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #100678|0 |1 is obsolete| | --- Comment #16 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 100901 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=100901&action=edit Bug 24850: Correct tests for offset handling in RFC3339 dates The tests were incorrectly passing for RFC3339 dates passed with an offset. This patch corrects the test. Test plan: 1/ Read the change 2/ Agree the change adheres to the RFC 3/ Run the test and varify it now fails 3/ Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #100679|0 |1 is obsolete| | --- Comment #17 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 100902 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=100902&action=edit Bug 24850: Correct offset handling in dt_from_string This patch adds correct handling for when an offset is passed within an RFC3339 formatted datetime. Test plan 1/ Run the DateUtils test and varify it now passes. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #100680|0 |1 is obsolete| | --- Comment #18 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 100903 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=100903&action=edit Bug 24850: Add tests for invalid RFC3339 dates We should die on invalidly formatted dates being passed. This patch adds such a test case. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #100681|0 |1 is obsolete| | --- Comment #19 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 100904 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=100904&action=edit Bug 24850: Don't fallback when date_format is passed -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #100881|0 |1 is obsolete| | --- Comment #20 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 100905 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=100905&action=edit Bug 24850: Always return 'instance' local time This patch adds handling to ensure we always convert a passed in time to the instance configured timezone.. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |24608 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24608 [Bug 24608] Allow modification of checkout due date -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |bgkriegel@gmail.com --- Comment #21 from Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> --- Martin, t/DateUtils.t fails always. An special setup is needed? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Koha::DateUtils ignored |Koha::DateUtils ignores |offsets in RFC3339 |offsets in RFC3339 |datetimes |datetimes -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #22 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- (In reply to Bernardo Gonzalez Kriegel from comment #21)
Martin, t/DateUtils.t fails always. An special setup is needed?
Nope.. seems I forgot to check the tests again after than final patch.. looking at it now. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #100901|0 |1 is obsolete| | --- Comment #23 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 101803 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=101803&action=edit Bug 24850: Correct tests for offset handling in RFC3339 dates The tests were incorrectly passing for RFC3339 dates passed with an offset. This patch corrects the test. Test plan: 1/ Read the change 2/ Agree the change adheres to the RFC 3/ Run the test and varify it now fails 3/ Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #100902|0 |1 is obsolete| | --- Comment #24 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 101804 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=101804&action=edit Bug 24850: Correct offset handling in dt_from_string This patch adds correct handling for when an offset is passed within an RFC3339 formatted datetime. Test plan 1/ Run the DateUtils test and varify it now passes. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #100903|0 |1 is obsolete| | --- Comment #25 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 101805 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=101805&action=edit Bug 24850: Add tests for invalid RFC3339 dates We should die on invalidly formatted dates being passed. This patch adds such a test case. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #100904|0 |1 is obsolete| | --- Comment #26 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 101806 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=101806&action=edit Bug 24850: Don't fallback when date_format is passed -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #100905|0 |1 is obsolete| | --- Comment #27 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 101807 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=101807&action=edit Bug 24850: Always return 'instance' local time This patch adds handling to ensure we always convert a passed in time to the instance configured timezone.. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #28 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 101808 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=101808&action=edit Bug 24850: Continue to handle default time and daylight saving This patch prevents the call to set_time_zone if we are handling a dateonly datetime string or a DST time -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kyle@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Agustín Moyano <agustinmoyano@theke.io> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #101803|0 |1 is obsolete| | --- Comment #29 from Agustín Moyano <agustinmoyano@theke.io> --- Created attachment 101837 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=101837&action=edit Bug 24850: Correct tests for offset handling in RFC3339 dates The tests were incorrectly passing for RFC3339 dates passed with an offset. This patch corrects the test. Test plan: 1/ Read the change 2/ Agree the change adheres to the RFC 3/ Run the test and varify it now fails 3/ Signoff Signed-off-by: Agustin Moyano <agustinmoyano@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Agustín Moyano <agustinmoyano@theke.io> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #101837|0 |1 is obsolete| | --- Comment #30 from Agustín Moyano <agustinmoyano@theke.io> --- Created attachment 101838 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=101838&action=edit Bug 24850: Correct tests for offset handling in RFC3339 dates The tests were incorrectly passing for RFC3339 dates passed with an offset. This patch corrects the test. Test plan: 1/ Read the change 2/ Agree the change adheres to the RFC 3/ Run the test and varify it now fails 3/ Signoff Signed-off-by: Agustin Moyano <agustinmoyano@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Agustín Moyano <agustinmoyano@theke.io> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Agustín Moyano <agustinmoyano@theke.io> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #101804|0 |1 is obsolete| | Attachment #101805|0 |1 is obsolete| | Attachment #101806|0 |1 is obsolete| | Attachment #101807|0 |1 is obsolete| | Attachment #101808|0 |1 is obsolete| | Attachment #101838|0 |1 is obsolete| | --- Comment #31 from Agustín Moyano <agustinmoyano@theke.io> --- Created attachment 101839 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=101839&action=edit Bug 24850: Correct tests for offset handling in RFC3339 dates The tests were incorrectly passing for RFC3339 dates passed with an offset. This patch corrects the test. Test plan: 1/ Read the change 2/ Agree the change adheres to the RFC 3/ Run the test and varify it now fails 3/ Signoff Signed-off-by: Agustin Moyano <agustinmoyano@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #32 from Agustín Moyano <agustinmoyano@theke.io> --- Created attachment 101840 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=101840&action=edit Bug 24850: Correct offset handling in dt_from_string This patch adds correct handling for when an offset is passed within an RFC3339 formatted datetime. Test plan 1/ Run the DateUtils test and varify it now passes. Signed-off-by: Agustin Moyano <agustinmoyano@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #33 from Agustín Moyano <agustinmoyano@theke.io> --- Created attachment 101841 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=101841&action=edit Bug 24850: Add tests for invalid RFC3339 dates We should die on invalidly formatted dates being passed. This patch adds such a test case. Signed-off-by: Agustin Moyano <agustinmoyano@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #34 from Agustín Moyano <agustinmoyano@theke.io> --- Created attachment 101842 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=101842&action=edit Bug 24850: Don't fallback when date_format is passed Signed-off-by: Agustin Moyano <agustinmoyano@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #35 from Agustín Moyano <agustinmoyano@theke.io> --- Created attachment 101843 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=101843&action=edit Bug 24850: Always return 'instance' local time This patch adds handling to ensure we always convert a passed in time to the instance configured timezone.. Signed-off-by: Agustin Moyano <agustinmoyano@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #36 from Agustín Moyano <agustinmoyano@theke.io> --- Created attachment 101844 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=101844&action=edit Bug 24850: Continue to handle default time and daylight saving This patch prevents the call to set_time_zone if we are handling a dateonly datetime string or a DST time Signed-off-by: Agustin Moyano <agustinmoyano@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #37 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- + # Default to UTC (when 'Z' is passed) for inbound timezone. Do you have an example where this "Z" is passed? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #38 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- (In reply to Jonathan Druart from comment #37)
+ # Default to UTC (when 'Z' is passed) for inbound timezone.
Do you have an example where this "Z" is passed?
Z is part of the RFC ;). I don't yet have an example of where we pass it in Koha, but it's valid via the API and so should be catered for. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #39 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- https://github.com/mojolicious/json-validator/blob/master/lib/JSON/Validator... shows we support passing Z in our validation routines. https://tools.ietf.org/html/rfc3339#section-2 for reference in the RFC -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |BLOCKED CC| |m.de.rooy@rijksmuseum.nl -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|BLOCKED |Failed QA --- Comment #40 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Part of the rfc3339 regex: ([Tt\s]) I am not sure if lowercase t is part of the syntax. I think it should be capital T or space. Note that \s is more than space. I think we should test for space only here. # Default to UTC (when 'Z' is passed) for inbound timezone. $tz = DateTime::TimeZone->new( name => 'UTC' ); Yes, Z stands for UTC. But this line comes out of the blue. We are just building the regex here. Further down, we are testing for the offset and at that code location we should check for the Z too and set UTC timezone. + # Convert to configured timezone (unless we started with a dateonly string or had to drop to floating time) + $dt->set_time_zone($server_tz) unless ( $date_only || $floating ); This line is confusing too. Although I am not sure if we have calls in Koha where a timezone is passed already, so it may be theoretical. But aren't we effectively ignoring/overriding the $tz parameter here? Should this routine have a tz parameter anyhow? Note also that we did not interfere earlier in the code: return DateTime->now( time_zone => $tz ) In line with what we do at the end, we should not allow this return? So, actually the big question is here: Shouldnt we always return in server timezone ? Changing status. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #41 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- The idea of the input tz is that you're passing in the TZ of the string being passed in.. the idea of the tz conversion at the bottom (and in the call to now) is that we always pass back a DateTime object with a consistent config based timezone. doing this call at the bottom means we will convert internally to the correct instance timezone from the passed timezone. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |rel_20_11_target -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #42 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Did my reply make sense Marcel.. or are you still unhappy with this work. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #43 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- (In reply to Martin Renvoize from comment #42)
Did my reply make sense Marcel.. or are you still unhappy with this work.
I am afraid that it got lost between lots of other patch mails.. Keeping this one for Friday.. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #44 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- No worries... I forgot about it too.. might take a bit to get the brain back up to speed. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Patch doesn't apply --- Comment #45 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Please rebase -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #46 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- (In reply to Martin Renvoize from comment #41)
The idea of the input tz is that you're passing in the TZ of the string being passed in.. the idea of the tz conversion at the bottom (and in the call to now) is that we always pass back a DateTime object with a consistent config based timezone. doing this call at the bottom means we will convert internally to the correct instance timezone from the passed timezone.
Ok. But I have the impression that many questions raised in comment40 are not answered yet.. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords|rel_20_11_target |rel_21_05_candidate -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC|colin.campbell@ptfs-europe. | |com | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #101839|0 |1 is obsolete| | --- Comment #47 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 115851 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115851&action=edit Bug 24850: Correct tests for offset handling in RFC3339 dates The tests were incorrectly passing for RFC3339 dates passed with an offset. This patch corrects the test. Test plan: 1/ Read the change 2/ Agree the change adheres to the RFC 3/ Run the test and varify it now fails 3/ Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #101840|0 |1 is obsolete| | --- Comment #48 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 115852 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115852&action=edit Bug 24850: Correct offset handling in dt_from_string This patch adds correct handling for when an offset is passed within an RFC3339 formatted datetime. Test plan 1/ Run the DateUtils test and varify it now passes. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #101841|0 |1 is obsolete| | --- Comment #49 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 115853 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115853&action=edit Bug 24850: Add tests for invalid RFC3339 dates We should die on invalidly formatted dates being passed. This patch adds such a test case. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #101842|0 |1 is obsolete| | --- Comment #50 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 115854 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115854&action=edit Bug 24850: Don't fallback when date_format is passed -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #101843|0 |1 is obsolete| | --- Comment #51 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 115855 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115855&action=edit Bug 24850: Always return 'instance' local time This patch adds handling to ensure we always convert a passed in time to the instance configured timezone.. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #101844|0 |1 is obsolete| | --- Comment #52 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 115856 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115856&action=edit Bug 24850: Continue to handle default time and daylight saving This patch prevents the call to set_time_zone if we are handling a dateonly datetime string or a DST time -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Signed Off --- Comment #53 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- This is blocking bug 24608 and I don't know how to move it on.. I've answered the questions and asked for feedback if the answer isn't understood.. so much time has passed that without going back to the begging of all these issues I can't really give more detail... But 24608 highlights the issues we have by not properly handling offsets and this bug resolves all those issues whilst maintaining existing compatibility. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #54 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 115857 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115857&action=edit Bug 24850: (QA follow-up) Add more comments To try and clarify the logic here I've added an addition comment to the code. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |24609 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24609 [Bug 24609] Add routes for getting and setting checkout due date -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks|24608 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24608 [Bug 24608] Allow modification of checkout due date -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dcook@prosentient.com.au --- Comment #55 from David Cook <dcook@prosentient.com.au> --- I'm still curious why we don't use DateTime::Format::Strptime (from the author of DateTime). That said, it doesn't allow for optional parts of patterns, so you have to do a few tests from more specific to less specific. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #56 from David Cook <dcook@prosentient.com.au> --- Comment on attachment 115852 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115852 Bug 24850: Correct offset handling in dt_from_string Review of attachment 115852: --> (https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=24850&attachment=115852) ----------------------------------------------------------------- ::: Koha/DateUtils.pm @@ +117,4 @@
(?<minute>\d{2}) : (?<second>\d{2}) + (\.\d{1,3})?(([Zz])|((?<offset>[\+|\-])(?<hours>[01][0-9]|2[0-3]):(?<minutes>[0-5][0-9])))
I don't think lowercase "z" is valid here? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #57 from Martin Renvoize <martin.renvoize@ptfs-europe.com> ---
From memory, I think you are right that the Z should always be capitalised.. but I think I read somewhere that most libraries are lenient with the case so I adopted that approach.. Happy to switch it out of course.
-- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #58 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- (In reply to David Cook from comment #55)
I'm still curious why we don't use DateTime::Format::Strptime (from the author of DateTime).
That said, it doesn't allow for optional parts of patterns, so you have to do a few tests from more specific to less specific.
Mmmm, I've often wondered that... history.. Though I note that lib seems to have had a bunch of fixes recently.. so it wasn't perfect.. might be better now. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #59 from David Cook <dcook@prosentient.com.au> --- (In reply to Martin Renvoize from comment #58)
Mmmm, I've often wondered that... history.. Though I note that lib seems to have had a bunch of fixes recently.. so it wasn't perfect.. might be better now.
I think in the past we were using a different library. It was DateTime::Format::DateParse which is super old and unmaintained. DateTime::Format::Strptime, on the other hand, is well maintained. I have an application that deals a lot with time and I love DateTime::Format::Strptime for it heh. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #60 from David Cook <dcook@prosentient.com.au> --- Comment on attachment 115855 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115855 Bug 24850: Always return 'instance' local time Review of attachment 115855: --> (https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=24850&attachment=115855) ----------------------------------------------------------------- ::: Koha/DateUtils.pm @@ +54,5 @@
return if $date_string and $date_string =~ m|^0000-0|;
my $do_fallback = defined($date_format) ? 0 : 1; + my $server_tz = C4::Context->tz; + $tz = C4::Context->tz unless $tz;
This might read easier if it said the following: $tz = $server_tz unless $tz; -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #61 from David Cook <dcook@prosentient.com.au> --- Comment on attachment 115852 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115852 Bug 24850: Correct offset handling in dt_from_string Review of attachment 115852: --> (https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=24850&attachment=115852) ----------------------------------------------------------------- ::: Koha/DateUtils.pm @@ +160,5 @@
second => $+{second}, ); $ampm = $+{ampm}; + if ( $+{offset} ) { + $tz = DateTime::TimeZone->new( name => $+{offset} . $+{hours} . $+{minutes} );
You might want to call them offset_hours and offset_minutes, because they're so close to "hour" and "minute" that I had to do a double-take to make sure they were right. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #62 from David Cook <dcook@prosentient.com.au> --- I actually don't have the brainpower to QA this one today, but since it's marked as "Needs Signoff", I can at least do that. My test plan: 0) Use koha-testing-docker 1) prove t/DateUtils.t 2) Noted "All tests successful." -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #63 from David Cook <dcook@prosentient.com.au> --- Honestly I really dislike dt_from_string. At some point, it should probably have its internals refactored into a number of more specific private sub-functions. But that change is outside the scope of this change. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #115851|0 |1 is obsolete| | --- Comment #64 from David Cook <dcook@prosentient.com.au> --- Created attachment 116299 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116299&action=edit Bug 24850: Correct tests for offset handling in RFC3339 dates The tests were incorrectly passing for RFC3339 dates passed with an offset. This patch corrects the test. Test plan: 1/ Read the change 2/ Agree the change adheres to the RFC 3/ Run the test and varify it now fails 3/ Signoff Signed-off-by: David Cook <dcook@prosentient.com.au> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #115852|0 |1 is obsolete| | --- Comment #65 from David Cook <dcook@prosentient.com.au> --- Created attachment 116300 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116300&action=edit Bug 24850: Correct offset handling in dt_from_string This patch adds correct handling for when an offset is passed within an RFC3339 formatted datetime. Test plan 1/ Run the DateUtils test and varify it now passes. Signed-off-by: David Cook <dcook@prosentient.com.au> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #115853|0 |1 is obsolete| | --- Comment #66 from David Cook <dcook@prosentient.com.au> --- Created attachment 116301 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116301&action=edit Bug 24850: Add tests for invalid RFC3339 dates We should die on invalidly formatted dates being passed. This patch adds such a test case. Signed-off-by: David Cook <dcook@prosentient.com.au> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #115854|0 |1 is obsolete| | --- Comment #67 from David Cook <dcook@prosentient.com.au> --- Created attachment 116302 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116302&action=edit Bug 24850: Don't fallback when date_format is passed Signed-off-by: David Cook <dcook@prosentient.com.au> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #115855|0 |1 is obsolete| | --- Comment #68 from David Cook <dcook@prosentient.com.au> --- Created attachment 116303 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116303&action=edit Bug 24850: Always return 'instance' local time This patch adds handling to ensure we always convert a passed in time to the instance configured timezone.. Signed-off-by: David Cook <dcook@prosentient.com.au> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #115856|0 |1 is obsolete| | --- Comment #69 from David Cook <dcook@prosentient.com.au> --- Created attachment 116304 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116304&action=edit Bug 24850: Continue to handle default time and daylight saving This patch prevents the call to set_time_zone if we are handling a dateonly datetime string or a DST time Signed-off-by: David Cook <dcook@prosentient.com.au> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #115857|0 |1 is obsolete| | --- Comment #70 from David Cook <dcook@prosentient.com.au> --- Created attachment 116305 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116305&action=edit Bug 24850: (QA follow-up) Add more comments To try and clarify the logic here I've added an addition comment to the code. Signed-off-by: David Cook <dcook@prosentient.com.au> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #71 from David Cook <dcook@prosentient.com.au> --- Comment on attachment 115854 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115854 Bug 24850: Don't fallback when date_format is passed Review of attachment 115854: --> (https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=24850&attachment=115854) ----------------------------------------------------------------- ::: Koha/DateUtils.pm @@ +146,4 @@
)? )? |xms; + $regex .= $time_re unless ( $date_format eq 'rfc3339' );
Should this be when $date_format eq 'rfc3339' or any time $date_format is passed? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off --- Comment #72 from David Cook <dcook@prosentient.com.au> --- I think something I find challenging about this patch set is that it makes a number of changes beyond including support for the timezone offset. I think it might be easier to get this through if it were broken up into different bugzilla issues. For instance: 1. Correct tests for offset handling in RFC3339 dates 2. Don't fallback when date_format is passed 3. Correct offset handling in dt_from_string 4. Always return 'instance' local time Honestly, the number of different changes just makes me less confident that I'm thoroughly testing everything and that makes me uncomfortable QAing this one. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |BLOCKED --- Comment #73 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- QA: Looking here -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #74 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> ---
From the RFC: time-second = 2DIGIT ; 00-58, 00-59, 00-60 based on leap second ; rules
And the code: + (\.\d{1,3})?(([Zz])|((?<offset>[\+|\-])(?<hours>[01][0-9]|2[0-3]):(?<minutes>[0-5][0-9]))) No blocker, just noting -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #75 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- (In reply to Marcel de Rooy from comment #74)
From the RFC: time-second = 2DIGIT ; 00-58, 00-59, 00-60 based on leap second ; rules
And the code: + (\.\d{1,3})?(([Zz])|((?<offset>[\+|\-])(?<hours>[01][0-9]|2[0-3]): (?<minutes>[0-5][0-9])))
No blocker, just noting
No, thats fine. Looking at seconds.. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Small patch Status|BLOCKED |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #116299|0 |1 is obsolete| | --- Comment #76 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 116355 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116355&action=edit Bug 24850: Correct tests for offset handling in RFC3339 dates The tests were incorrectly passing for RFC3339 dates passed with an offset. This patch corrects the test. Test plan: 1/ Read the change 2/ Agree the change adheres to the RFC 3/ Run the test and varify it now fails 3/ Signoff Signed-off-by: David Cook <dcook@prosentient.com.au> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #116300|0 |1 is obsolete| | --- Comment #77 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 116356 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116356&action=edit Bug 24850: Correct offset handling in dt_from_string This patch adds correct handling for when an offset is passed within an RFC3339 formatted datetime. Test plan 1/ Run the DateUtils test and varify it now passes. Signed-off-by: David Cook <dcook@prosentient.com.au> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #116301|0 |1 is obsolete| | --- Comment #78 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 116357 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116357&action=edit Bug 24850: Add tests for invalid RFC3339 dates We should die on invalidly formatted dates being passed. This patch adds such a test case. Signed-off-by: David Cook <dcook@prosentient.com.au> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #116302|0 |1 is obsolete| | --- Comment #79 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 116358 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116358&action=edit Bug 24850: Don't fallback when date_format is passed Signed-off-by: David Cook <dcook@prosentient.com.au> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #116303|0 |1 is obsolete| | --- Comment #80 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 116359 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116359&action=edit Bug 24850: Always return 'instance' local time This patch adds handling to ensure we always convert a passed in time to the instance configured timezone.. Signed-off-by: David Cook <dcook@prosentient.com.au> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #116304|0 |1 is obsolete| | --- Comment #81 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 116360 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116360&action=edit Bug 24850: Continue to handle default time and daylight saving This patch prevents the call to set_time_zone if we are handling a dateonly datetime string or a DST time Signed-off-by: David Cook <dcook@prosentient.com.au> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #116305|0 |1 is obsolete| | --- Comment #82 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 116361 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116361&action=edit Bug 24850: (QA follow-up) Add more comments To try and clarify the logic here I've added an addition comment to the code. Signed-off-by: David Cook <dcook@prosentient.com.au> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |m.de.rooy@rijksmuseum.nl |y.org | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #83 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- (In reply to Marcel de Rooy from comment #79)
Created attachment 116358 [details] [review] Bug 24850: Don't fallback when date_format is passed
I was trying to understand where this $do_fallback was tested. I removed it to the following line: + } elsif ( $do_fallback && $date_string =~ $fallback_re ) { but the test is not failing explicitly: t/DateUtils.t .. 1/80 # Looks like you planned 80 tests but ran 79. Can you highlight this use case in the tests please? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #84 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 116511 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116511&action=edit Bug 24850: (follow-up) Add tests for invalid RFC3339 dates Make this failure explicit, otherwise we get a "bad plan" failure. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #85 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- I think this line is not tested: 229 $dt->set_time_zone($server_tz) unless ( $date_only || $floating ); And cannot we remove $floating and compare something like $dt->time_zone eq 'floating'? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |ASSIGNED --- Comment #86 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Sorry chaps, I'll swing back around to this tomorrow.. been a bit bogged down the past few days -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #87 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- (In reply to David Cook from comment #72)
I think something I find challenging about this patch set is that it makes a number of changes beyond including support for the timezone offset.
I think it might be easier to get this through if it were broken up into different bugzilla issues. For instance:
1. Correct tests for offset handling in RFC3339 dates 2. Don't fallback when date_format is passed 3. Correct offset handling in dt_from_string 4. Always return 'instance' local time
Honestly, the number of different changes just makes me less confident that I'm thoroughly testing everything and that makes me uncomfortable QAing this one.
Whilst I agree.. I think that's how I started but found that half the tests failed if I didn't follow through in one step. So much time has passed since I first wrote these patches that I must admit I'm struggling to unpick them myself :(. I'm going to take a look at refactoring the tests to get my head around what each test is actually testing. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Jonathan Druart <jonathan.druart+koha@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords|rel_21_05_candidate |rel_21_11_candidate -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #116355|0 |1 is obsolete| | --- Comment #88 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 126182 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=126182&action=edit Bug 24850: Correct tests for offset handling in RFC3339 dates The tests were incorrectly passing for RFC3339 dates passed with an offset. This patch corrects the test. Test plan: 1/ Read the change 2/ Agree the change adheres to the RFC 3/ Run the test and varify it now fails 3/ Signoff Signed-off-by: David Cook <dcook@prosentient.com.au> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #116356|0 |1 is obsolete| | --- Comment #89 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 126183 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=126183&action=edit Bug 24850: Correct offset handling in dt_from_string This patch adds correct handling for when an offset is passed within an RFC3339 formatted datetime. Test plan 1/ Run the DateUtils test and varify it now passes. Signed-off-by: David Cook <dcook@prosentient.com.au> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #116357|0 |1 is obsolete| | --- Comment #90 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 126184 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=126184&action=edit Bug 24850: Add tests for invalid RFC3339 dates We should die on invalidly formatted dates being passed. This patch adds such a test case. Signed-off-by: David Cook <dcook@prosentient.com.au> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #116358|0 |1 is obsolete| | --- Comment #91 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 126185 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=126185&action=edit Bug 24850: Don't fallback when date_format is passed Signed-off-by: David Cook <dcook@prosentient.com.au> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #116359|0 |1 is obsolete| | --- Comment #92 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 126186 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=126186&action=edit Bug 24850: Always return 'instance' local time This patch adds handling to ensure we always convert a passed in time to the instance configured timezone.. Signed-off-by: David Cook <dcook@prosentient.com.au> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #116360|0 |1 is obsolete| | --- Comment #93 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 126187 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=126187&action=edit Bug 24850: Continue to handle default time and daylight saving This patch prevents the call to set_time_zone if we are handling a dateonly datetime string or a DST time Signed-off-by: David Cook <dcook@prosentient.com.au> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #116361|0 |1 is obsolete| | --- Comment #94 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 126188 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=126188&action=edit Bug 24850: (QA follow-up) Add more comments To try and clarify the logic here I've added an addition comment to the code. Signed-off-by: David Cook <dcook@prosentient.com.au> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #116511|0 |1 is obsolete| | --- Comment #95 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 126189 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=126189&action=edit Bug 24850: (follow-up) Add tests for invalid RFC3339 dates Make this failure explicit, otherwise we get a "bad plan" failure. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #96 from Tomás Cohen Arazi <tomascohen@gmail.com> --- What is the status of this bug? I find the change pretty simple and the tests are clear as well. I see sign offs from a bunch of people as well. I tested it too, but not sure what is the real status. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #97 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- (In reply to Tomás Cohen Arazi from comment #96)
What is the status of this bug? I find the change pretty simple and the tests are clear as well. I see sign offs from a bunch of people as well. I tested it too, but not sure what is the real status.
See comments 85-87 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |28377 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28377 [Bug 28377] Use the API to suspend/resume holds -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Passed QA --- Comment #98 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- I have nothing to add here.. This bug is a massive blocker for any API bugs that deal with dates.. to me the tests are as clear as I can make them and I don't trust myself to re-write them all.. that's kinda the point of tests.. to prevent regressions.. re-writing them all at this point would be asking for regressions. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|martin.renvoize@ptfs-europe |koha-bugs@lists.koha-commun |.com |ity.org -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |BLOCKED -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #99 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Ok, so we need some more eyes/typing hands/Perl brains on this one. Joubu, could you clarify maybe the points that need attention before inclusion? -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|BLOCKED |In Discussion --- Comment #100 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Deciding on "in Discussion" as we concluded that those brains/hands/eyes need to meet up to discuss the path forward with timezone handling first. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #101 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- API's that deal with dates in any way are fundamentally broken without the patches here.. so I will be failing anything that touches dates in API's until this has been looked at. I personally don't have the mental capacity to go down this road for another time so am stepping away. Someone else will have to take this one. Burnt out... -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Jonathan Druart <jonathan.druart+koha@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|In Discussion |Passed QA --- Comment #102 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- Tested on top of bug 24609. This script: """ use Koha::DateUtils qw( dt_from_string output_pref ); use Koha::Patrons; use C4::Context; use C4::Circulation qw(AddIssue ); use t::lib::Mocks; t::lib::Mocks::mock_config('America/Cordoba'); my $dbh = C4::Context->dbh; my $patron = Koha::Patron->new({userid => 'xxx', categorycode => 'S', branchcode => 'CPL'})->store; t::lib::Mocks::mock_userenv({patron => $patron}); my $item = Koha::Items->find(1); my $today = dt_from_string; say "NOW: " . $today. " : " . $today->time_zone_short_name; my $issue = AddIssue($patron->unblessed, $item->barcode); my $date_due = $issue->date_due; say "OBJECT :" . $date_due; say "UI :" . output_pref({str => $date_due}); say "ID: " . $issue->id; """ Creates an checkout and outputs: NOW: 2021-10-22T10:46:15 : -03 OBJECT :2021-10-27 23:59:00 UI :10/27/2021 23:59 ID: 4 GET http://kohadev-intra.mydnsname.org:8081/api/v1/checkouts/4 => "due_date": "2021-10-27T23:59:00-03:00", PUT http://kohadev-intra.mydnsname.org:8081/api/v1/checkouts/4 { "due_date": "2021-10-27T23:59:00-03:00", "issuer_id": null, "note_seen": false, "patron_id": 179, "unseen_renewals": 0 } GET http://kohadev-intra.mydnsname.org:8081/api/v1/checkouts/4 => "due_date": "2021-10-27T23:59:00-03:00", PUT http://kohadev-intra.mydnsname.org:8081/api/v1/checkouts/4 { "due_date": "2021-10-27T20:59:00-06:00", "issuer_id": null, "note_seen": false, "patron_id": 179, "unseen_renewals": 0 } GET http://kohadev-intra.mydnsname.org:8081/api/v1/checkouts/4 => "due_date": "2021-10-27T23:59:00-03:00", PUT http://kohadev-intra.mydnsname.org:8081/api/v1/checkouts/4 { "due_date": "2021-10-27T23:59:00-06:00", "issuer_id": null, "note_seen": false, "patron_id": 179, "unseen_renewals": 0 } GET http://kohadev-intra.mydnsname.org:8081/api/v1/checkouts/4 => "due_date": "2021-10-28T02:59:00-03:00", -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Jonathan Druart <jonathan.druart+koha@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |release-notes-needed -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #103 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- I haven't tried what's happening if no TZ is passed. Do we reject it, assuming it's instance tz or UTC? -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #104 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- (In reply to Jonathan Druart from comment #103)
I haven't tried what's happening if no TZ is passed. Do we reject it, assuming it's instance tz or UTC?
It's mandatory. However this is weird: "due_date": "2021-10-27 23:59", => "Does not match date-time" "due_date": "2021-10-27 23:59:00", => "Missing property." -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=29322 -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #105 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- (In reply to Jonathan Druart from comment #104)
(In reply to Jonathan Druart from comment #103)
I haven't tried what's happening if no TZ is passed. Do we reject it, assuming it's instance tz or UTC?
It's mandatory. However this is weird:
"due_date": "2021-10-27 23:59", => "Does not match date-time"
"due_date": "2021-10-27 23:59:00", => "Missing property."
Added as bug 29322.. I think it's tied up inside JSON::Validator.. I'll ask Tomas to take a look. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Text to go in the| |Prior to this patch our release notes| |date handling library | |ignored offset data passed | |with rfc3339 dates. This | |could lead to problems if | |an API client converted to | |UTC or was in a different | |timezone to the Koha | |instance time setting. | | | |This patch adds proper | |handling to dt_from_string | |such that if an REF3339 | |date is input, we parse out | |the offset and then adjust | |the time to match the | |instance timezone for | |storage. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords|release-notes-needed, | |rel_21_11_candidate | -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |rel_21_11_candidate -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Jonathan Druart <jonathan.druart+koha@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |martin.renvoize@ptfs-europe |ity.org |.com -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Jonathan Druart <jonathan.druart+koha@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to master Version(s)| |21.11.00 released in| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #106 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- Pushed to master for 21.11, thanks to everybody involved! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #107 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- There is a test failing because of this: t/db_dependent/Koha/Object.t .. 9/21 # No tests run! # Failed test 'No tests run for subtest "date and date-time handling tests"' # at t/db_dependent/Koha/Object.t line 649. # Looks like you planned 2 tests but ran 1. # Looks like you failed 1 test of 1 run. # Failed test 'attributes_from_api() tests' # at t/db_dependent/Koha/Object.t line 669. [A bad parameter was given]# Looks like your test exited with 255 just after 14. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Jonathan Druart <jonathan.druart+koha@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |additional_work_needed -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #108 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- And many more actually, see the last builds -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #109 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- (In reply to Jonathan Druart from comment #108)
And many more actually, see the last builds
So for Object.t it's failing on date_or_birth because of: The given date (2019-12-27) does not match the date format (rfc3339) at /kohadevbox/koha/Koha/DateUtils.pm line 193. We lost the fallback, certainly caused by 'Bug 24850: Don't fallback when date_format is passed' -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #110 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- We could assume that 'date' won't be rfc3339, but I am not sure it's correct. The following patches seem to fix the tests. diff --git a/Koha/Object.pm b/Koha/Object.pm index 5d0f3f720ff..ec1316c45d7 100644 --- a/Koha/Object.pm +++ b/Koha/Object.pm @@ -751,7 +751,7 @@ sub attributes_from_api { elsif ( _date_or_datetime_column_type( $columns_info->{$koha_field_name}->{data_type} ) ) { try { if ( $columns_info->{$koha_field_name}->{data_type} eq 'date' ) { - $value = $dtf->format_date(dt_from_string($value, 'rfc3339')) + $value = $dtf->format_date(dt_from_string($value)) if defined $value; } else { -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #111 from Tomás Cohen Arazi <tomascohen@gmail.com> --- This commit broke all the things: 398f7b45f074b023b03e5075821f0abb08cda66c -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #112 from Tomás Cohen Arazi <tomascohen@gmail.com> --- That's what we're getting. The given date (2021-11-02) does not match the date format (rfc3339) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #113 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- Created attachment 127231 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=127231&action=edit Bug 24850: Date only are not RFC3339 We removed the fallback on 'Bug 24850: Don't fallback when date_format is passed', and the dates passed for date only (YYYY-MM-DD) are not RFC3339 formatted but ISO8601. We must assume that all dates will be YYYY-MM-DD, without fallback. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Jonathan Druart <jonathan.druart+koha@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #127231|0 |1 is obsolete| | --- Comment #114 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- Created attachment 127232 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=127232&action=edit Bug 24850: Date only are not RFC3339 We removed the fallback on 'Bug 24850: Don't fallback when date_format is passed', and the dates passed for date only (YYYY-MM-DD) are not RFC3339 formatted but ISO8601. We must assume that all dates will be YYYY-MM-DD, without fallback. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #115 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Date-only is part of RFC 3339. It is the full-date spec which matches ISO 8601 [1] Maybe we need a dateonly param so things are clear? [1] https://xml2rfc.tools.ietf.org/public/rfc/html/rfc3339.html#anchor14 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #116 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- I actually read it the other way around.. RFC3339 is full date-time specific and is a subspec of ISO8609 which is looser and allows for date only. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #117 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- https://stackoverflow.com/questions/522251/whats-the-difference-between-iso-... Highlighting the bit that raised my thoughts: `Most notably RFC 3339 specifies a complete representation of date and time (only fractional seconds are optional)` -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #118 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- But yes.. I did wonder if we should actually have a 'date only' parameter we can pass so our iso fallback can explicitly ban time parts for this case. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #119 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- But, for here.. I think passing 'iso' as the format explicitly for date only where Jonathans follow-up does seems like the most pragmatic solution in the time.. Adding a 'date-only' parameter should be in a desperate bug in my opinion. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #120 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- There is still one test failing: # Failed test '404 Not Found' # at t/db_dependent/api/v1/holds.t line 220. # got: '400' # expected: '404' # Failed test 'has value for JSON Pointer "/error"' # at t/db_dependent/api/v1/holds.t line 220. I don't understand it and it's a mess to debug (We definitely need to improve our error handling when testing the REST API). -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #121 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- Created attachment 127235 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=127235&action=edit Bug 24850: Fix api/v1/holds.t -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #122 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- Read those 2 comments the other way around, there is still one test failing :) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #123 from Tomás Cohen Arazi <tomascohen@gmail.com> --- (In reply to Martin Renvoize from comment #119)
But, for here.. I think passing 'iso' as the format explicitly for date only where Jonathans follow-up does seems like the most pragmatic solution in the time.. Adding a 'date-only' parameter should be in a desperate bug in my opinion.
I agree, what I meant to say is that passing an ISO full-date formatted date is correct in the Swagger 2.0 spec, and we should better have explicit params. My attempt yesterday night was to add something explicit: $value = $dtf->format_datetime(dt_from_string($value, 'iso')) but either works, as long as we document this conversation -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #124 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Created attachment 127236 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=127236&action=edit Bug 24850: suspended_until is a date-time Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #125 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- 77 "suspended_until": { 78 "type": ["string", "null"], 79 "format": "date-time", 80 "description": "Date until which the hold has been suspended" 81 }, But the code is converting everything to date-only... What do we want here? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #126 from Tomás Cohen Arazi <tomascohen@gmail.com> --- (In reply to Jonathan Druart from comment #125)
77 "suspended_until": { 78 "type": ["string", "null"], 79 "format": "date-time", 80 "description": "Date until which the hold has been suspended" 81 },
But the code is converting everything to date-only...
What do we want here?
The spec was written taking the DB structure into account. If you think the spec is wrong and this field should be date-only, let's do it on a separate bug? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #127 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- /suspension is expecting a date other routes are expecting a datetime That seems inconsistent and wrong. IMO we should open a separate bug report and base the test fix on top of it. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Jonathan Druart <jonathan.druart+koha@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #127232|0 |1 is obsolete| | --- Comment #128 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- Created attachment 127239 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=127239&action=edit Bug 24850: Date only are not RFC3339 We removed the fallback on 'Bug 24850: Don't fallback when date_format is passed', and the dates passed for date only (YYYY-MM-DD) are not RFC3339 formatted but ISO8601. We must assume that all dates will be YYYY-MM-DD, without fallback. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |29401 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29401 [Bug 29401] Hold suspensions should match the resolution of other holds date fields -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #129 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- Created attachment 127240 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=127240&action=edit Bug 24850: Correctly handle suspended_until for PUT if we are passing suspended_until to the body, it should be a rfc3339 (what the spec is expecting) but $hold->suspend_until will be iso -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #127235|0 |1 is obsolete| | --- Comment #130 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 127241 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=127241&action=edit Bug 24850: Fix api/v1/holds.t Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #127236|0 |1 is obsolete| | --- Comment #131 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 127242 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=127242&action=edit Bug 24850: suspended_until is a date-time Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #127239|0 |1 is obsolete| | --- Comment #132 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 127243 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=127243&action=edit Bug 24850: Date only are not RFC3339 We removed the fallback on 'Bug 24850: Don't fallback when date_format is passed', and the dates passed for date only (YYYY-MM-DD) are not RFC3339 formatted but ISO8601. We must assume that all dates will be YYYY-MM-DD, without fallback. Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #127240|0 |1 is obsolete| | --- Comment #133 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 127244 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=127244&action=edit Bug 24850: Correctly handle suspended_until for PUT if we are passing suspended_until to the body, it should be a rfc3339 (what the spec is expecting) but $hold->suspend_until will be iso Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |29402 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29402 [Bug 29402] We should add a 'dateonly' switch option to dt_from_string -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 --- Comment #134 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- OK.. I've tested the follow-ups and agree that, although not perfect in the wider scheme, they are what we should push for now.. as such I've signed off. I have added two new bugs depending on this one however, to continue the work to clarify and improve our date handling situation. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |29403 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29403 [Bug 29403] dt_from_string should fail if passed and rfc3339 formatted string when 'iso' is selected as dateformat -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Jonathan Druart <jonathan.druart+koha@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords|additional_work_needed, | |rel_21_11_candidate | --- Comment #135 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- Last patches pushed to master. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to master |RESOLVED Resolution|--- |FIXED --- Comment #136 from Kyle M Hall <kyle@bywatersolutions.com> --- Unit tests fail in 21.05.x, please rebase and ping me if this is needed for 21.05.x -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Andrew Nugged <nugged@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |29906 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29906 [Bug 29906] When changing hold parameters over API (PUT) it forcibly gets to "suspended" state -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 Andrew Isherwood <bugzilla@warmlight.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- CC|bugzilla@warmlight.co.uk | -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org