Bug 16358

Summary: Rancor - Deleting records when Rancor is enabled just opens them
Product: Koha Reporter: Jesse Weaver <jweaver>
Component: CatalogingAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: brendan, f.demians, jonathan.druart, julian.maurice, m.de.rooy, mirko, nick
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:
Attachments: Bug 16358: Do not redirect to the editor when deleting records
Bug 16358: Do not redirect to the editor when deleting records
Bug 16358: Do not redirect to the editor when deleting records

Description Jesse Weaver 2016-04-26 19:07:49 UTC
The "Delete record" button goes to addbiblio.pl?op=delete, which incorrectly redirects to Rancor when selected.
Comment 1 Jonathan Druart 2016-05-02 15:00:26 UTC
Jesse, do you plan to provide a patch (if so, please switch the status to assigned).
Comment 2 Jonathan Druart 2016-09-13 14:06:19 UTC
Created attachment 55533 [details] [review]
Bug 16358: Do not redirect to the editor when deleting records

The "Delete record" button goes to addbiblio.pl?op=delete, which
incorrectly redirects to Rancor when selected.

Test plan:
0/ Enable EnableAdvancedCatalogingEditor
1/ Edit a record
2/ Switch to the Advanced editor
3/ click on "View the record"
4/ Delete the record
=> Without this patch you are redirected to the editor
=> With this patch you are redirected to the advanced search
Comment 3 Mirko Tietgen 2016-10-13 07:54:51 UTC
Created attachment 56305 [details] [review]
Bug 16358: Do not redirect to the editor when deleting records

The "Delete record" button goes to addbiblio.pl?op=delete, which
incorrectly redirects to Rancor when selected.

Test plan:
0/ Enable EnableAdvancedCatalogingEditor
1/ Edit a record
2/ Switch to the Advanced editor
3/ click on "View the record"
4/ Delete the record
=> Without this patch you are redirected to the editor
=> With this patch you are redirected to the advanced search

Signed-off-by: Mirko Tietgen <mirko@abunchofthings.net>
Comment 4 Kyle M Hall 2016-10-13 15:11:20 UTC
Created attachment 56414 [details] [review]
Bug 16358: Do not redirect to the editor when deleting records

The "Delete record" button goes to addbiblio.pl?op=delete, which
incorrectly redirects to Rancor when selected.

Test plan:
0/ Enable EnableAdvancedCatalogingEditor
1/ Edit a record
2/ Switch to the Advanced editor
3/ click on "View the record"
4/ Delete the record
=> Without this patch you are redirected to the editor
=> With this patch you are redirected to the advanced search

Signed-off-by: Mirko Tietgen <mirko@abunchofthings.net>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 5 Brendan Gallagher 2016-10-17 23:04:37 UTC
Pushed to Master - Should be in the November 16.11 Release.  Thanks!
Comment 6 Frédéric Demians 2016-10-22 12:50:55 UTC
Pushed in 16.05. Will be in 16.05.05.
Comment 7 Julian Maurice 2016-10-24 10:32:21 UTC
Pushed in 3.22.x, will be in 3.22.12