From 614c612ec9f764b8ea18404f823b4dfdad4265cb Mon Sep 17 00:00:00 2001 From: Juerd Waalboer Date: Sat, 4 Jan 2025 22:04:29 +0100 Subject: [PATCH] openepaperlink: The touch/mtime solution is not right. The hook gets fired for each product individually, so a single mtime update is not good enough. The timestamp needs to be recorded per product or tag. Meh. --- plugins/openepaperlink | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/plugins/openepaperlink b/plugins/openepaperlink index be2cf57..78eeb04 100644 --- a/plugins/openepaperlink +++ b/plugins/openepaperlink @@ -84,7 +84,7 @@ sub hook_product_deleted($class, $product, $mtime, @) { _run @macs; sleep 1 if $mtime == time; - _touch; + _touch; # XXX this is wrong; causes subsequent product changes to be ignored. }; } @@ -100,6 +100,6 @@ sub hook_product_changed($class, $old, $new, $mtime, @) { _run $new->{id}; sleep 1 if $mtime == time; - _touch; + _touch; # XXX this is wrong; causes subsequent product changes to be ignored. }; }