Bug 32902 - Add ability to choose if items overlay if any record matches or only if the item is on the matched record
Summary: Add ability to choose if items overlay if any record matches or only if the i...
Status: Failed QA
Alias: None
Product: Koha
Classification: Unclassified
Component: Tools (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Kyle M Hall (khall)
QA Contact: Testopia
URL:
Keywords:
Depends on: 32901
Blocks:
  Show dependency treegraph
 
Reported: 2023-02-07 17:44 UTC by Kyle M Hall (khall)
Modified: 2023-02-08 22:49 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments
Bug 32902: Update Schema File [DO NOT PUSH] (1.43 KB, patch)
2023-02-08 14:37 UTC, Kyle M Hall (khall)
Details | Diff | Splinter Review
Bug 32902: Update Schema File [DO NOT PUSH] (1.43 KB, patch)
2023-02-08 14:37 UTC, Kyle M Hall (khall)
Details | Diff | Splinter Review
Bug 32902: Add option to only overlay if matched item is on matched record (11.17 KB, patch)
2023-02-08 14:37 UTC, Kyle M Hall (khall)
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Kyle M Hall (khall) 2023-02-07 17:44:23 UTC
Bug 32804 resolves an bug where an overlaid item could have its' biblionumber changed. Fixing this has the effect of enabling items to be updated even if the item is on a different record. This can be wanted or unwanted behavior depending on the situation. It would be good if the cataloger could choose which behavior is preferred on a per-batch bases.
Comment 1 Kyle M Hall (khall) 2023-02-08 14:37:16 UTC
Created attachment 146392 [details] [review]
Bug 32902: Update Schema File [DO NOT PUSH]
Comment 2 Kyle M Hall (khall) 2023-02-08 14:37:48 UTC
Created attachment 146394 [details] [review]
Bug 32902: Update Schema File [DO NOT PUSH]
Comment 3 Kyle M Hall (khall) 2023-02-08 14:37:56 UTC
Created attachment 146395 [details] [review]
Bug 32902: Add option to only overlay if matched item is on matched record

Bug 32084 resolves an bug where an overlaid item could have its' biblionumber changed. Fixing this has the effect of enabling items to be updated even if the item is on a different record. This can be wanted or unwanted behavior depending on the situation. It would be good if the cataloger could choose which behavior is preferred on a per-batch bases.

Test Plan:
1) Grab the sample marc record file from bug 32804
2) Upload this file, chose to match on biblionumber and replace matching
   item if any matching record was found
3) Import the batch
4) Note the item was replaced
5) Revert the import
6) Change the rule to replace matching items if matching item is on
   matching record
7) Apply different matching rules
8) Import the batch into the catalog
9) Note the item was not replaced
Comment 4 Andrew Fuerste-Henry 2023-02-08 20:45:42 UTC
The file from bug 32804 contains 3 items, 2 that match the biblio record and 1 that does not. When I stage the file with the choice "replace matching items if any matching record was found," all 3 items update as expected on import. But when 
I stage the file with "Replace matching items if matching item is on matching record" then none of the 3 items update, including the 2 on the matching record.

In the latter case, should the two items on the matching biblio record not also update?
Comment 5 Andrew Fuerste-Henry 2023-02-08 20:51:10 UTC
Also, reverting the import at step 5 actually deleted the item with the mismatched biblionumber. I haven't had a chance to test what happens here without your patch, this may be a separate existing bug.
> 
> Test Plan:
> 1) Grab the sample marc record file from bug 32804
> 2) Upload this file, chose to match on biblionumber and replace matching
>    item if any matching record was found
> 3) Import the batch
> 4) Note the item was replaced
> 5) Revert the import
> 6) Change the rule to replace matching items if matching item is on
>    matching record
> 7) Apply different matching rules
> 8) Import the batch into the catalog
> 9) Note the item was not replaced
Comment 6 Andrew Fuerste-Henry 2023-02-08 22:49:42 UTC
This behavior is bug 14210, "Manage staged MARC records deletes replaced item when undoing import into catalog."

(In reply to Andrew Fuerste-Henry from comment #5)
> Also, reverting the import at step 5 actually deleted the item with the
> mismatched biblionumber. I haven't had a chance to test what happens here
> without your patch, this may be a separate existing bug.
> > 
> > Test Plan:
> > 1) Grab the sample marc record file from bug 32804
> > 2) Upload this file, chose to match on biblionumber and replace matching
> >    item if any matching record was found
> > 3) Import the batch
> > 4) Note the item was replaced
> > 5) Revert the import
> > 6) Change the rule to replace matching items if matching item is on
> >    matching record
> > 7) Apply different matching rules
> > 8) Import the batch into the catalog
> > 9) Note the item was not replaced