Bug 6068 - New feature: capability to blanking subfields in BatchMod
Summary: New feature: capability to blanking subfields in BatchMod
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Tools (show other bugs)
Version: 3.6
Hardware: All All
: PATCH-Sent (DO NOT USE) enhancement
Assignee: Alex Arnaud
QA Contact: Bugs List
URL:
Keywords:
: 4172 (view as bug list)
Depends on: 6018 6688
Blocks:
  Show dependency treegraph
 
Reported: 2011-04-05 13:48 UTC by Alex Arnaud
Modified: 2012-10-25 23:09 UTC (History)
5 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments
capability to blanking subfields in BatchMod (6.51 KB, patch)
2011-04-06 09:21 UTC, Alex Arnaud
Details | Diff | Splinter Review
Revised for Template::Toolkit (6.43 KB, patch)
2011-05-24 19:55 UTC, Owen Leonard
Details | Diff | Splinter Review
proposed patch (passed QA) (6.39 KB, patch)
2011-08-09 16:12 UTC, Paul Poulain
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Alex Arnaud 2011-04-05 13:48:50 UTC

    
Comment 1 Alex Arnaud 2011-04-06 09:21:28 UTC Comment hidden (obsolete)
Comment 2 Alex Arnaud 2011-04-06 09:26:51 UTC
This feature was in biblibre/master. I adapted it for 3.4.
Comment 3 Owen Leonard 2011-05-24 19:55:33 UTC Comment hidden (obsolete)
Comment 4 Nicole C. Engard 2011-07-06 16:04:09 UTC
This patch works as described.  I would recommend an improvement that does what Owen suggests and make it so that you can't select if the check box is checked - but it works so I'm signing off.

Nicole
Comment 5 Paul Poulain 2011-08-09 16:12:17 UTC
Created attachment 4911 [details] [review]
proposed patch (passed QA)

QA comment 
I agree with Owen, and will create a new bugzilla entry about this small ENH (disable the field when you check the "empty" button)
It's not worth rejecting QA, as it works as announced, and the suggestion is a small glitch/improvement
Comment 6 Robin Sheat 2011-08-15 03:58:42 UTC
*** Bug 4172 has been marked as a duplicate of this bug. ***
Comment 7 Chris Cormack 2011-08-15 21:12:56 UTC
Pushed, please test
Comment 8 Nicole C. Engard 2011-08-17 11:29:12 UTC
Appears to work as described.
Comment 9 Robin Sheat 2011-08-17 12:10:26 UTC
Also tested and works for me.