Bug 10887

Summary: Batch item deletion -- doesn't need Use default values section
Product: Koha Reporter: HB-NEKLS <gwilliams>
Component: ToolsAssignee: Galen Charlton <gmcharlt>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: abbyrobertson.student, chris, jonathan.druart, katrin.fischer, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 9533    
Bug Blocks:    
Attachments: Bug 10887: Batch item deletion -- doesn't need Use default values section
Bug 10887: Batch item deletion -- doesn't need Use default values section
[SIGNED OFF] Bug 10887: Batch item deletion -- doesn't need Use default values section
[PASSED QA] Bug 10887: Batch item deletion -- doesn't need Use default values section

Description HB-NEKLS 2013-09-13 16:01:57 UTC
The batch item deletion tool doesn't need to have the option for "use default values section" to "Populate fields with default values from default framework." 

The option is applicable to the batch item modification tool, but not to batch item deletion -- because all the user is doing with the batch deletion is deleting items. Should this option be removed, as a result?
Comment 1 Katrin Fischer 2015-01-06 21:12:44 UTC
Hi Heather, good catch - it makes no sense there.
Comment 2 Abby 2015-01-15 21:37:17 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2015-01-19 06:09:44 UTC
Hi Abby, 

the same page (bachMod.tt) is used for 2 tools: batch item modification and batch item delete. The 'default values' option only makes sense if you want to batch modify, but not if you want to batch delete. Your patch removes from both tools, what we need is some kind of condition, so it shows only for one, but not the other.
Comment 4 Jonathan Druart 2015-02-16 14:47:02 UTC Comment hidden (obsolete)
Comment 5 Nick Clemens (kidclamp) 2015-02-19 19:28:38 UTC Comment hidden (obsolete)
Comment 6 Kyle M Hall 2015-02-20 12:01:30 UTC
Created attachment 36070 [details] [review]
[PASSED QA] Bug 10887: Batch item deletion -- doesn't need Use default values section

To test:
1. Go to Home > Tools > Batch item deletion
2. Notice that there is an unnecessary option for "use default values
section" to "Populate fields with default values from default work"
3. Apply the patch.
4. Check that this section has disappeared.

Signed-off-by: Nick <Nick@quechelibrary.org>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 7 Tomás Cohen Arazi 2015-04-08 14:37:45 UTC
Patch pushed to master.

Thanks Abby!
Comment 8 Chris Cormack 2015-04-11 05:29:29 UTC
Pushed to 3.18.x will be in 3.18.6 release