Attachment 168861 Details for Bug 36736 – Bug 36736: Load plugins at the start of background job processing

[patch] Bug 36736: Load plugins at the start of background job processing

Bug-36736-Load-plugins-at-the-start-of-background-.patch (text/plain), 1.71 KB, created by Nick Clemens (kidclamp) on 2024-07-11 18:14:06 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2024-07-11 18:14:06 UTC

Size: 1.71 KB

patch

obsolete

>From 5d98da5a767890914c3f1ecb20e3ad3372520428 Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Thu, 11 Jul 2024 18:08:08 +0000
>Subject: [PATCH] Bug 36736: Load plugins at the start of background job
> processing
>
>This patch adds a call to get_enabled_plugins before processing background jobs to ensure
>that all plugin hooks are loaded and cached
>
>To test:
>1 - Install a plugin that adds new objects e.g. the Contracts plugin
>    https://github.com/bywatersolutions/fs-koha-plugin-contracts
>    or the KOha Advent plugin:
>    https://gitlab.com/koha-community/koha-advent/koha-plugin-fancyplugin
>2 - Restart all
>3 - Tail all your logs
>4 - Stage and import a file containing items
>5 - Note in the logs
>    DBI Exception: DBD::mysql::st execute failed: Lock wait timeout exceeded; try restarting transaction
>6 - Apply this patch
>7 - Restart all
>8 - Stage and import again
>9 - Success!
>---
> Koha/BackgroundJob.pm | 7 +++++++
> 1 file changed, 7 insertions(+)
>
>diff --git a/Koha/BackgroundJob.pm b/Koha/BackgroundJob.pm
>index 4525122e8c3..33a6872ec03 100644
>--- a/Koha/BackgroundJob.pm
>+++ b/Koha/BackgroundJob.pm
>@@ -26,6 +26,7 @@ use C4::Context;
> use Koha::DateUtils qw( dt_from_string );
> use Koha::Exceptions;
> use Koha::Exceptions::BackgroundJob;
>+use Koha::Plugins;
> 
> use base qw( Koha::Object );
> 
>@@ -164,6 +165,12 @@ sub process {
> 
>     return {} if ref($self) ne 'Koha::BackgroundJob';
> 
>+    # Our background jobs are called in forked processes
>+    # to ensure we have all plugin hooks and data we call
>+    # get_enabled_plugins at the star of processing
>+    # to populate the cache
>+    Koha::Plugins->get_enabled_plugins();
>+
>     my $derived_class = $self->_derived_class;
> 
>     $args ||= {};
>-- 
>2.39.2

Actions: View | Diff | Splinter Review

Attachments on bug 36736: 168861 | 169389 | 169410 | 169440 | 169508 | 169555