Bug 18577 - Importing a batch using a framework not fully set up causes and endless loop
Summary: Importing a batch using a framework not fully set up causes and endless loop
Status: Signed Off
Alias: None
Product: Koha
Classification: Unclassified
Component: MARC Bibliographic record staging/import (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low major (vote)
Assignee: Nick Clemens
QA Contact: Testopia
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-05-11 03:28 UTC by Bob Ewart
Modified: 2017-06-25 14:39 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Bot Control: ---
When did the bot last check this:
Who signed the patch off:
Text to go in the release notes:


Attachments
Bug 18577 - Importing a batch using a framework not fully set up causes and endless loop (1.99 KB, patch)
2017-06-23 14:14 UTC, Nick Clemens
Details | Diff | Splinter Review
Bug 18577 - Importing a batch using a framework not fully set up causes and endless loop (2.08 KB, patch)
2017-06-25 14:39 UTC, Alex Buckley
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Bob Ewart 2017-05-11 03:28:59 UTC
Importing a batch using a framework code that is defined but not set up causes the import to go into an endless loop.

Create a new framework code but don't set what it is based on.  
Then import a batch using that framework.
Comment 1 Nick Clemens 2017-06-23 14:14:36 UTC
Created attachment 64597 [details] [review]
Bug 18577 - Importing a batch using a framework not fully set up causes and endless loop

To test:
1 - Create a new framework - don't set it up
2 - Stage some records for import
3 - Manage the import, note your new framwork is in the list
4 - Apply patch
5 - Reload the page
6 - Note the framework is no longer an option
Comment 2 Alex Buckley 2017-06-25 14:39:32 UTC
Created attachment 64615 [details] [review]
Bug 18577 - Importing a batch using a framework not fully set up causes and endless loop

To test:
1 - Create a new framework - don't set it up
2 - Stage some records for import
3 - Manage the import, note your new framwork is in the list
4 - Apply patch
5 - Reload the page
6 - Note the framework is no longer an option

Followed test plan,  worked as intended
Signed-off-by: Alex Buckley <alexbuckley@catalyst.net.nz>