Quantcast

[Bug 9988] Leave larger authority merges to merge_authorities cronjob ( pref AuthorityMergeLimit)

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 9988] Leave larger authority merges to merge_authorities cronjob ( pref AuthorityMergeLimit)

bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9988

--- Comment #40 from Marcel de Rooy <[hidden email]> ---
(In reply to Jacek Ablewicz from comment #39)

> For "real" merge usage case (merging two different authority records), if
> the merge gets postponed, when merge() is called later by the cron job,
> authority record with $mergefrom ID is no longer there (it got deleted from
> the database by authorities/merge.pl script ?). This search in postponed
> merge() call
>
>    my $authfrom = Koha::Authorities->find($mergefrom);
>
> will fail, and $authtypefrom | $auth_tag_to_report_from | @record_from would
> be empty. I.e.: old authority record heading, stored in the merge requests
> table will never be used.
>
> (but there is a strong possibility I'm very much mistaken about that,
> authorities-related code got changed quite a lot lately).

The code is designed to overcome this situation. There are tests simulating it
too.
Take a look :)

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Loading...