Bug 18069

Summary: koha-rebuild-zebra still calls rebuild_zebra with -x
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Architecture, internals, and plumbingAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: normal    
Priority: P5 - low CC: brendan, katrin.fischer, m.de.rooy, mtj
Version: unspecified   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 17731    
Bug Blocks:    
Attachments: Bug 18069: Remove residue of rebuild_zebra -x
Bug 18069: Remove residue of rebuild_zebra -x
Bug 18069: Remove residue of rebuild_zebra -x

Description Jonathan Druart 2017-02-07 08:10:03 UTC

    
Comment 1 Jonathan Druart 2017-02-07 08:18:24 UTC
Created attachment 59970 [details] [review]
Bug 18069: Remove residue of rebuild_zebra -x

Bug 17731 removed the -x option of rebuild_zebra but koha-rebuild-zebra
still cals the script with this option.

"Warning: You passed -x which is already the default and is now deprecated"

Test plan:
sudo koha-rebuild-zebra -f
should no longer raise the warning
Comment 2 Mason James 2017-02-22 14:21:55 UTC
Created attachment 60556 [details] [review]
Bug 18069: Remove residue of rebuild_zebra -x

Bug 17731 removed the -x option of rebuild_zebra but koha-rebuild-zebra
still cals the script with this option.

"Warning: You passed -x which is already the default and is now deprecated"

Test plan:
sudo koha-rebuild-zebra -f
should no longer raise the warning

Signed-off-by: Mason James <mtj@kohaaloha.com>
Comment 3 Marcel de Rooy 2017-03-07 13:59:31 UTC
Created attachment 60874 [details] [review]
Bug 18069: Remove residue of rebuild_zebra -x

Bug 17731 removed the -x option of rebuild_zebra but koha-rebuild-zebra
still cals the script with this option.

"Warning: You passed -x which is already the default and is now deprecated"

Test plan:
sudo koha-rebuild-zebra -f
should no longer raise the warning

Signed-off-by: Mason James <mtj@kohaaloha.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 4 Brendan Gallagher 2017-03-23 12:17:19 UTC
Pushed to Master - Should be in the 17.05 Release.  Thanks
Comment 5 Katrin Fischer 2017-03-26 20:55:12 UTC
This patch has been pushed to 16.11.x and will be in 16.11.06.
Comment 6 Mason James 2017-05-03 03:43:41 UTC
Pushed to 16.05.x, for 16.05.12 release