Created attachment 132992 [details] [review] Bug 30467: Don't send the delete request for each item We are sending the request when all the items have been deleted.
Created attachment 134398 [details] [review] Bug 30467: Don't send the delete request for each item We are sending the request when all the items have been deleted.
Created attachment 134640 [details] [review] Bug 30467: Don't send the delete request for each item We are sending the request when all the items have been deleted. Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Yup, confirmed... All works and this prevents a double indexing process. Passing QA
Sorry does not work for me. In Koha::BackgroundJob::BatchDeleteItem : I see there is indexing for deleted biblio records : if (@deleted_biblionumbers) { my $indexer = Koha::SearchEngine::Indexer->new( { index => $Koha::SearchEngine::BIBLIOS_INDEX } ); $indexer->index_records( \@deleted_biblionumbers, 'recordDelete', "biblioserver", undef ); } But I see not indexing for deleted items. I see batch items update uses Koha::Items->batch_update() with : if (@modified_itemnumbers) { my @biblionumbers = uniq( Koha::Items->search( { itemnumber => \@modified_itemnumbers } ) ->get_column('biblionumber')); if ( @biblionumbers ) { my $indexer = Koha::SearchEngine::Indexer->new( { index => $Koha::SearchEngine::BIBLIOS_INDEX } ); $indexer->update_index( \@biblionumbers ); } } Should we also create Koha::Items->batch_delete() ? Maybe in another bug report.
Item updates result in biblio re-indexes and these item deletes are all attached to the biblio's we're deleting.. so the re-index is caught by the biblio re-index call you spotted. Or am I missing something more involved?
Created attachment 134672 [details] [review] Bug 30467: (QA follow-up) Restore indexing for item only deletion This patch restores indexing for item only bulk deletions.
I was totally missing something, good catch. Follow-up attached, but I'd love to get a QA stamp from someone to check my work on it
Created attachment 134675 [details] [review] Bug 30467: (QA follow-up) Restore indexing for item only deletion This patch restores indexing for item only bulk deletions. JD amended patch: * remove trailing whitespaces * Move if inside first condition Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Created attachment 134741 [details] [review] Bug 30467: (QA follow-up) Fix copy paste error to update updated_biblionumbers
Created attachment 134744 [details] [review] Bug 30467: (QA follow-up) Fix copy paste error to update updated_biblionumbers Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Works perfect, thanks for your fast and great work
Pushed to master for 22.05, thanks to everybody involved [U+1F984]