Déclenchements multiples d'action de désactivation

[Apern]

Bonjour,

Afin d’éviter le déclenchement multiple des actions configurées lors de la désactivation d’une alarme, je suggère de modifier alarm.class.php de la manière suivante :

701 +	if ($cmd_armed->execCmd() == 0) {                                                                                                                              
702 +		log::add('alarm', 'debug',$eqLogic->getHumanName(). __(' Alarme déjà désactivée', __FILE__));                                                                
703 +		return;                                                                                                                                                      
704 +	}

Je me suis simplement inspiré de la logique similaire déjà utilisée lors de l’activation d’une alarme :

if ($cmd_armed->execCmd() == 1) {
	log::add('alarm', 'debug',$eqLogic->getHumanName(). __(' Alarme déjà active', __FILE__));
	return;
}

Ce changement a été effectué sur la dernière version beta disponible (le repo n’étant pas ouvert). Si vous ouvrez le repo je peux faire le PR sans problème.

Cordialement,
Alexis

[Apern]

@Loic désolé de te déranger mais penses-tu qu’il serait possible d’ouvrir le repo pour que je puisse faire le PR ? Merci <3

[Aurel]

Salut,

A ma connaissance c’est un plugin payant ou inclus dans un service pack.
Du coup le principe c’est que la repo n’est pas ouvert

[Loic]

Salut

Le repo est ouvert depuis des mois voir année GitHub - jeedom/plugin-ajaxSystem · GitHub

[Aurel]

Il a taggé plugin-alarme pas plugin-ajaxsystem

[Apern]

Dans ce cas @Loic (ou un autre dev) si tu peux faire la modif directement ça serait top

[Mips]

Bonjour,

Franchement je comprends pas le problème… j’utilise le plugin et je n’ai jamais été impacté par des exécutions multiples.

Donc avant de proposer une solution, faudrait expliquer pourquoi et comment reproduire

[Loic]

À pardon c’est bon il est ouvert mais je suis pas d’accord avec la correction c’est justement fait exprès que lors d’une désactivation même si l’alarme est déjà inactive que ça rejoue les action de désactivation (dans le cas où par exemple y’a eu un soucis et la 1er désactivation n’a pas coupé la sirène). Ça évite de devoir réactiver puis désactiver.

Il faudrait plus comprendre pourquoi chez toi tu as des multiples désactivation.

[Apern]

Bonsoir,
Merci pour l’ouverture du repo et pour ta réponse. Je propose de continuer la discussion sur la PR

fix: prevent double alarm deactivations