Nom de l'équipement identique à l'ID du plugin

[Spine]

Bonjour,

Je découvre le plugin et n’étant pas très inspiré j’ai bêtement nommé mon premier équipement LogManager, j’ai ensuite mis un bon moment à comprendre pourquoi la configuration du niveau de log de l’équipement faisait sauter la configuration du niveau de log du plugin et inversement.
Après avoir renommé l’équipement tout est rentré dans l’ordre, je propose donc d’interdire cette possibilité.

---

Informations Jeedom

Core : 4.4.2 (V4-stable)
DNS Jeedom : non

Plugin : LogManager
Version : 2023-10-01 01:19:37 (stable)

[Mips]

Salut,

Bonne idée, ca aurait dû être le cas dès le départ bien sûr.
En même temps c’est le premier retour de la sorte

[Mips]

C’est dispo en beta, à valider

[Spine]

Bonjour,

Je peux toujours créer un équipement nommé LogManager :

Je ne vois pas ce que vous avez fait dans le code.
J’ai mis ça dans preSave() :

if (strtolower($this->getName()) == 'logmanager') {
	throw new Exception(__('Le nom de l\'équipement doit être différent de l\'ID du plugin', __FILE__));
}

Ca bloque bien la création et la sauvegarde :

[Mips]

version du plugin?
probablement pas encore synchro, j’aurais du écrire

C’est dispo dans la prochaine beta, à valider

pcq forcément si tu as une version du plugin < 2024-04-08… c’est pas dedans!

[Michel_F]

Salut Mips,

pour mon information et celle de tout le monde, pourquoi devrait-ce être le cas ? C’est spécifique à ce plugin ?
Ce n’est que le nom, soit un champs de la DB, je ne comprends pas pourquoi ça devrait faire planter quoi que ce soit.

NB: je laisse volontairement cette question dans ce fil

A+
Michel

[Spine]

J’ai rien dit !
Je reteste demain.

[Mips]

souvent j’attends un jour avant de répondre pour être sur, j’ai répondu trop vite:

@Michel_F

pourquoi devrait-ce être le cas ?

la réponse est dans la doc:

Un équipement correspond à un log, le nom de l’équipement sera utilisé comme nom du fichier log.

[Spine]

Bonjour,

Testé et approuvé, merci bien.

[Mips]

ca sera en stable demain