Bug 15325

Summary: rebuild_zebra.pl table option (-t) doesn't work
Product: Koha Reporter: Julian Maurice <julian.maurice>
Component: Command-line UtilitiesAssignee: Julian Maurice <julian.maurice>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: jonathan.druart, kyle, robin
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:
Circulation function:
Bug Depends on: 12368    
Bug Blocks:    
Attachments: Bug 15325: Fix --table option of rebuild_zebra.pl
Bug 15325: Fix --table option of rebuild_zebra.pl
Bug 15325: Fix --table option of rebuild_zebra.pl

Description Julian Maurice 2015-12-07 14:24:05 UTC
$table is always set to 'biblioitems', no matter what is given on the command line.
Comment 1 Julian Maurice 2015-12-07 14:28:25 UTC Comment hidden (obsolete)
Comment 2 Julian Maurice 2015-12-07 14:34:16 UTC
Test plan:
1. git checkout master
2. perl misc/migration_tools/rebuild_zebra.pl -b -t items --where "price = 42"
3. You should see errors printed on screen about an unknown column
4. Apply patch
5. perl misc/migration_tools/rebuild_zebra.pl -b -t items --where "price = 42"
6. No errors \o/
Comment 3 Frédéric Demians 2015-12-07 16:07:31 UTC Comment hidden (obsolete)
Comment 4 Jonathan Druart 2015-12-08 15:21:57 UTC
Created attachment 45497 [details] [review]
Bug 15325: Fix --table option of rebuild_zebra.pl

Option's value given on command line was never used and 'biblioitems'
was used instead.

Test plan:
1. git checkout master
2. perl misc/migration_tools/rebuild_zebra.pl -b -t items --where "price = 42"
3. You should see errors printed on screen about an unknown column
4. Apply patch
5. perl misc/migration_tools/rebuild_zebra.pl -b -t items --where "price = 42"
6. No errors \o/

Signed-off-by: Frédéric Demians <f.demians@tamil.fr>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 5 Jonathan Druart 2015-12-08 15:22:36 UTC
Erk, sorry about that.
Comment 6 Kyle M Hall (khall) 2015-12-11 16:17:01 UTC
Pushed to master!

Thanks Julian!
Comment 7 Julian Maurice 2015-12-14 07:18:33 UTC
Patch pushed to 3.22.x, will be in 3.22.1