[Bug 7784] New: Improve clarity of batch modification operations
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7784 Priority: P5 - low Change sponsored?: --- Bug ID: 7784 Assignee: oleonard@myacpl.org Summary: Improve clarity of batch modification operations QA Contact: koha.sekjal@gmail.com Severity: enhancement Classification: Unclassified OS: All Reporter: oleonard@myacpl.org URL: /cgi-bin/koha/tools/batchMod.pl Hardware: All Status: NEW Version: master Component: Tools Product: Koha In the batch item modification interfaces: "Checking the box right next the subfield label will disable the entry and delete the values of that subfield on all selected items" I think it's more accurate to say "disable the entry and delete the contents of the subfield." Some enhancements: - indicate this by emptying/resetting the field and adding a disabled attribute. - append a note which says "The contents of this field will be deleted" It would also be helpful to add a note saying that leaving fields empty will make no changes to any records. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7784 --- Comment #1 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 11709 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11709&action=edit Bug 7784 - Improve clarity of batch modification operations - Improve clarity of hint about checkboxes - Inicate that leaving fields blank will make no change - Add a hint when checking checkboxes stating that the field will be deleted. - Block deletion of required subfields - Add "required" style to required subfields (matching additem.pl) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7784 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7784 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #11709|0 |1 is obsolete| | --- Comment #2 from Nicole C. Engard <nengard@gmail.com> --- Created attachment 11712 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11712&action=edit [SIGNED-OFF] Bug 7784 - Improve clarity of batch modification operations - Improve clarity of hint about checkboxes - Inicate that leaving fields blank will make no change - Add a hint when checking checkboxes stating that the field will be deleted. - Block deletion of required subfields - Add "required" style to required subfields (matching additem.pl) Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> Works as described -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7784 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |nengard@gmail.com -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7784 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA CC| |jonathan.druart@biblibre.co | |m --- Comment #3 from Jonathan Druart <jonathan.druart@biblibre.com> --- QA Comment: Just one question: what is the utility of the last line of your patch: + <span class="hint" id="hint[% ite.tag %][% ite.subfield %][% ite.random %]"></span> Else nothing to say, works as expected, no regression. Marked as Passed QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7784 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|koha.sekjal@gmail.com |jonathan.druart@biblibre.co | |m -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7784 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master CC| |paul.poulain@biblibre.com Version|master |rel_3_10 --- Comment #4 from Paul Poulain <paul.poulain@biblibre.com> --- Patch pushed to master -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7784 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |RESOLVED Resolution|--- |FIXED --- Comment #5 from Owen Leonard <oleonard@myacpl.org> --- (In reply to comment #3)
Just one question: what is the utility of the last line of your patch: + <span class="hint" id="hint[% ite.tag %][% ite.subfield %][% ite.random %]"></span>
The hint is shown if you check the box to delete a subfield: "This subfield will be deleted." The ID is attached so that if you uncheck the box the hint can be removed again. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org