[Bug 30775] New: 952w should have datepicker plugin enabled for it by default
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30775 Bug ID: 30775 Summary: 952w should have datepicker plugin enabled for it by default Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Cataloging Assignee: koha-bugs@lists.koha-community.org Reporter: stalkernoid@gmail.com QA Contact: testopia@bugs.koha-community.org CC: m.de.rooy@rijksmuseum.nl -- 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=30775 Peter Vashchuk <stalkernoid@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |stalkernoid@gmail.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=30775 Peter Vashchuk <stalkernoid@gmail.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=30775 --- Comment #1 from Peter Vashchuk <stalkernoid@gmail.com> --- Created attachment 135018 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135018&action=edit Bug 30775: Make 952w to have datepicker plugin by default This field (Price effective from) is very similar to 952d, but it doesn't have dateaccessioned.pl plugin by default, Apart of worse usability of this it is also allows to enter wrong date which will be converted in 0000-00-00 in DB and even lead to crashes by code in other places. So, adding this plugin not only improves usability (user can have datepicker) but also adds date field validation. Test plan: 1. Head over to MARC frameworks from your administration page, check 952 subfield structure of your default framework structure. 2. dateaccessioned.pl is set as a plugin for 952d by default but is missing from 952w. 3. Apply the patch and reset your koha, drop db and use reset_all alias. 4. Check frameworks structure again and ensure that datepicker plugin is set by default for 952w. 5. Edit some item to ensure that datepicker works for that 952w. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30775 David Nind <david@davidnind.com> 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=30775 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #135018|0 |1 is obsolete| | --- Comment #2 from David Nind <david@davidnind.com> --- Created attachment 135461 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135461&action=edit Bug 30775: Make 952w to have datepicker plugin by default This field (Price effective from) is very similar to 952d, but it doesn't have dateaccessioned.pl plugin by default, Apart of worse usability of this it is also allows to enter wrong date which will be converted in 0000-00-00 in DB and even lead to crashes by code in other places. So, adding this plugin not only improves usability (user can have datepicker) but also adds date field validation. Test plan: 1. Head over to MARC frameworks from your administration page, check 952 subfield structure of your default framework structure. 2. dateaccessioned.pl is set as a plugin for 952d by default but is missing from 952w. 3. Apply the patch and reset your koha, drop db and use reset_all alias. 4. Check frameworks structure again and ensure that datepicker plugin is set by default for 952w. 5. Edit some item to ensure that datepicker works for that 952w. Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30775 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david@davidnind.com --- Comment #3 from David Nind <david@davidnind.com> --- Testing notes (using koha-testing-docker): - For step 3: apply the patch, then reset_all ==> 952$w should have dateaccessioned.pl -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30775 Victor Grousset/tuxayo <victor@tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30775 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Text to go in the| |This enhancement adds the release notes| |date picker to 952$w (price | |effective from). | | | |This | |improves usability (952$d | |(date acquired) has the | |date picker enabled) and | |also adds date field | |validation (the date is | |added to the database as | |YYYY-MM-DD and when entered | |incorrectly it can lead to | |crashes in other areas). -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30775 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #135461|0 |1 is obsolete| | --- Comment #4 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 136152 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=136152&action=edit Bug 30775: Make 952w to have datepicker plugin by default This field (Price effective from) is very similar to 952d, but it doesn't have dateaccessioned.pl plugin by default, Apart of worse usability of this it is also allows to enter wrong date which will be converted in 0000-00-00 in DB and even lead to crashes by code in other places. So, adding this plugin not only improves usability (user can have datepicker) but also adds date field validation. Test plan: 1. Head over to MARC frameworks from your administration page, check 952 subfield structure of your default framework structure. 2. dateaccessioned.pl is set as a plugin for 952d by default but is missing from 952w. 3. Apply the patch and reset your koha, drop db and use reset_all alias. 4. Check frameworks structure again and ensure that datepicker plugin is set by default for 952w. 5. Edit some item to ensure that datepicker works for that 952w. Signed-off-by: David Nind <david@davidnind.com> 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=30775 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA CC| |martin.renvoize@ptfs-europe | |.com --- Comment #5 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Good call, works as described and is a sensible default. PQA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30775 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |martin.renvoize@ptfs-europe |y.org |.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30775 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to master Version(s)| |22.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=30775 --- Comment #6 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Pushed to master for 22.11. Nice work everyone, thanks! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30775 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to master |Pushed to stable Version(s)|22.11.00 |22.11.00, 22.05.04 released in| | CC| |lucas@bywatersolutions.com --- Comment #7 from Lucas Gass <lucas@bywatersolutions.com> --- Backported to 22.05.x for 22.05.04 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30775 Arthur Suzuki <arthur.suzuki@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |arthur.suzuki@biblibre.com Version(s)|22.11.00, 22.05.04 |22.11.00, 22.05.04, released in| |21.11.11 Status|Pushed to stable |Pushed to oldstable --- Comment #8 from Arthur Suzuki <arthur.suzuki@biblibre.com> --- pushed to 21.11.x for 21.11.11 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30775 Victor Grousset/tuxayo <victor@tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to oldstable |RESOLVED CC| |victor@tuxayo.net Resolution|--- |FIXED --- Comment #9 from Victor Grousset/tuxayo <victor@tuxayo.net> --- Not backported to oldoldstable (21.05.x). Feel free to ask if it's needed. Nothing to document, marking resolved. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org