Plugin abonné à un topic incorrect (mapping) 🤔

Jeandhom · Novembre 23, 2023, 10:01

Bonjour,

Il semblerait que le mapping ne se mette pas à jour.

Capture d’écran du 2023-11-23 00-31-19

Sur le plugin-mqttdomutils, je suis passé de mqtt2/domutils à domutils.

Capture d’écran du 2023-11-23 14-36-37

Le mapping indiqué par le plugin-mqtt2 a-t-il une importance ?
Peut on supprimer l’entrée caduque en BD ?

Mips · Novembre 24, 2023, 5:20

Salut,

Le plugin indique ce que les autres plugins ont demandé donc si ce n’est pas correcte le problème est dans le plugin-mqttdomutils .
C’est à lui que revient la responsabilité de supprimer si nécessaire et de réinscrire.

Jeandhom · Novembre 24, 2023, 9:36

Il semblerait que ce message n’ait pas été vu par certains développeurs.

Capture d’écran du 2023-11-24 10-26-57

Les plugin-mqttnetatmo et plugin-mqttdomutils de @wocha-fr.
Le plugin-daikinrccloud de @Thibaut_T.

Mips · Novembre 24, 2023, 9:42

A ma connaissance, les seuls cas où plugin-mqtt2 supprime de lui même un « abonnement » / inscription c’est lorsqu’il recoit un message sur les topics en question et que soit:

le plugin n’est plus présent,
n’a pas ou plus de méthode handleMqttMessage()
ou que celle-ci provoque une erreur 500.

Mips · Novembre 24, 2023, 9:43

et donc oui, tu peux supprimer, si tu es sur de ce que tu fais

Jeandhom · Novembre 24, 2023, 9:55

Merci pour tes réponses, je laisse le sujet ouvert jusqu’à la prise en compte de la bonne gestion du mapping.

Salvialf · Novembre 24, 2023, 9:56

Salut,

Une fonction existe dans le plugin mqtt2 pour supprimer un plugin abonné :

mqtt2::removePluginTopic($_topic);

ou

mqtt2::removePluginTopicByPlugin($_plugin);

github.com

jeedom/plugin-mqtt2/blob/beta/core/class/mqtt2.class.php#L432


      
             }
             return __CLASS__;
          }
          
          public static function addPluginTopic($_plugin, $_topic) {
             $mapping = config::byKey('mapping', __CLASS__, array());
             $mapping[$_topic] = $_plugin;
             config::save('mapping', $mapping, __CLASS__);
          }
          
          public static function removePluginTopic($_topic) {
             $mapping = config::byKey('mapping', __CLASS__, array());
             unset($mapping[$_topic]);
             config::save('mapping', $mapping, __CLASS__);
          }
          
           public static function removePluginTopicByPlugin($_plugin) {
             $mapping = config::byKey('mapping', __CLASS__, array());
             foreach($mapping as $topic => $plugin){
                if($plugin == $_plugin){
                   unset($mapping[$topic]);

wocha-fr · Novembre 24, 2023, 10:51

Hello,
Je vais regarder et corriger cela pour mes plugins.
Bonne journée.

Jeandhom · Décembre 4, 2023, 11:36

Merci pour la mise à jour de tes deux plugins. Cela semble OK pour moi.

Capture d’écran du 2023-12-04 12-31-31

wocha-fr · Décembre 4, 2023, 12:39

De rien.
C’est normal de fournir quelque chose de fonctionnel
Bonne journée.