From 9f6d69c75dff696afe1563bc99c60032a5089db8 Mon Sep 17 00:00:00 2001 From: Goffi Date: Thu, 14 Jul 2022 12:55:30 +0200 Subject: [PATCH] component AP gateway: use the new `priority` argument of `addManagedNode`: priority is needed because we'll need to inspect former items on retract to check the Acitivy to use (it may be an `Undo` activity is some cases), and this must be done before the cache is updated by pubsub cache plugin. rel 370 --- sat/plugins/plugin_comp_ap_gateway/__init__.py | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/sat/plugins/plugin_comp_ap_gateway/__init__.py b/sat/plugins/plugin_comp_ap_gateway/__init__.py index d6775c93..5f1477bf 100644 --- a/sat/plugins/plugin_comp_ap_gateway/__init__.py +++ b/sat/plugins/plugin_comp_ap_gateway/__init__.py @@ -132,7 +132,12 @@ class APGateway: self._t = host.plugins["TEXT_SYNTAXES"] self._i = host.plugins["IDENTITY"] self._p.addManagedNode( - "", items_cb=self._itemsReceived + "", + items_cb=self._itemsReceived, + # we want to be sure that the callbacks are launched before pubsub cache's + # one, as we need to inspect items before they are actually removed from cache + # or updated + priority=1000 ) self.pubsub_service = APPubsubService(self) host.trigger.add("messageReceived", self._messageReceivedTrigger, priority=-1000)