Mmmmh @Pierre-Gilles … je vois que cette issue « critique » a été fermée par le stale bot mais n’a jamais été traitée… Je l’ai réouverte :
ouvert 04:30AM - 20 May 22 UTC
fermé 01:26PM - 28 Sep 23 UTC
bug
effort:medium
follows #1530
![Bug save mqtt same external_id](https://user-images.githubuse… rcontent.com/35010958/169449866-410c6b90-7613-4a94-ad57-27b694351ca0.gif)
Should not overwrite the existing device but rather return an explicit error.
The bug does not occur if 2 devices are created in 2 different integrations.
![image](https://user-images.githubusercontent.com/35010958/169450605-542a3783-24ff-4e78-a777-499f353232f7.png)
![image](https://user-images.githubusercontent.com/35010958/169450660-5d27209c-3164-478e-ae6c-0da394123edc.png)
The bug also occurs with 2 features created with the same name in the same device.
But if 2 features with the same name are created in 2 different devices, saving the second feature is not possible and returns an error...
Issue found here: https://community.gladysassistant.com/t/debutant-gladys-tests-avant-de-pourquoi-pas-migrer-depuis-domoticz/7281/19
Je pense même que tu n’y avais jamais prêté attention… je viens de me faire avoir de nouveau … heureusement avec des nouveaux devices. Je te renvoi vers le poste d’origine :
Salut @pierre-gilles ,
Je profite de cela pour relevé un bug dans le même thème mais beaucoup plus inquiétant … Je voulais créer la PR et du coup j’ai fais un test sur l’intégration mqtt pour voir le message qui apparaissait … et là c’est le drame !!!
MQTT ne renvoi aucune erreur mais écrase tout simplement le device directement !! en supprimant les fonctionnalités de l’autre device au passage …
[Bug save mqtt same external_id]
J’ai également fais le test pour 2 features dan…
Il suffit de repondre nimporte quoi pour que le stale soit retiré dans le github…
Oui, @spenceur pour quelqu’un qui n’aurait pas créé l’issue, mais je l’ai réouverte aussitôt car j’étais l’initiateur !
Le sujet forum est surtout pour y repenser et pouvoir le traiter ^^
Effectivement! Merci d’avoir re-créé l’issue
Je viens de traiter l’issue dans cette PR : MQTT: Add conflict error when creating a device with same selector by Pierre-Gilles · Pull Request #1902 · GladysAssistant/Gladys · GitHub
Cette PR est mergée sur master et partira dans la prochaine version de Gladys.
Merci d’avoir reporté ce bug
2 « J'aime »
Correctif live dans Gladys Assistant 4.29 :
Salut à tous !
Nouvelle version de Gladys aujourd’hui avec Gladys Assistant 4.29, une version qui apporte pas mal de nouveautés
Je vous en parle sur le blog :
N’hésitez pas à féliciter @Lokkye pour toute ces chouettes contributions récentes
1 « J'aime »