bitlair/revbank
6
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

openepaperlink: The touch/mtime solution is not right.

Browse source 
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.
This commit is contained in:
Juerd Waalboer 2025-01-04 22:04:29 +01:00
parent 5d910510b8
commit 614c612ec9
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
plugins/openepaperlink
View file

@ -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.
};
}