[Koha-bugs] [Bug 22437] Subsequent authority merges in cron may cause biblios to lose authority information

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22437

--- Comment #20 from Marcel de Rooy <m.de.rooy at rijksmuseum.nl> ---
(In reply to Victor Grousset/tuxayo from comment #17)

Thx Victor for testing in the first place !

> Note: They don't have the same framework, it that an issue (the 3 a a
> different framework)
AFAIC that should not be relevant.

> select * from need_merge_authorities \G
> shows the merge but I don't know how to check about 'BB merge' is that:
>     authid: 35 (id of B)
> authid_new: 35

I do not know the record numbers. But that looks to me as BB merge, yes.

> >     Verify that the linkcount to A is the former total of ABC.
> 6+9+1 != 15 record(s)
> Noooooo
> Wait, isn't that expected due to AuthorityMergeLimit = 2 ?
> Maybe I'm mixing things up.

6+9+1= 16 If you got 15, then it could be that one of the records formerly
linked to B or C already contained a link to A too. 
AuthorityMergeLimit=2 means that a merge XY where linkcount X>=2 is, will be
postponed to the cron job instead of being executed rightaway. Which is an
essential part of test plan and bug fix.

> - Given the above notes, was there a mistake in the execution of the test
> plan?
The only thing I am thinking of, is that you might have missed that one record
was already linked to A. Which would explain the difference between 15 and 16.

> - What does the last result mean?
The result of 15 means that all records were merged correctly. If you would
have doen the same without this patch, you would have lost links to A in a
number of records. See comment1.

> - given comment 7 should the status of this bug be Needs signoff ? (@Marcel)
Comment7 is obsolete now. It applied to the first patch on this report. If you
look in the history, you can see that I submitted an alternate patch and
changed status after that. So no worries :)

-- 
You are receiving this mail because:
You are watching all bug changes.