Skip to content

Compléter la correction des groupes multi-hubs : callbacks et migration - #56

Merged
zoic21 merged 1 commit into
betafrom
fix/multi-hub-groups
Oct 20, 2026
Merged

zoic21 merged 1 commit into
betafrom
fix/multi-hub-groups

Conversation

@zoic21

@zoic21 zoic21 commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Description

La PR #44 a déjà corrigé les trois points décrits dans #43 sur beta, mais deux chemins utilisent encore incorrectement les identifiants de groupe :

  • Le callback reçoit l’identifiant Ajax brut (00000001) et le recherche directement, alors que l’équipement Jeedom s’appelle désormais hubId_00000001. Les mises à jour d’état des groupes synchronisés sont donc ignorées.
  • Lors de la migration, un groupe ancien est repris sans vérifier son hub_id. Si le groupe existe pour le hub B et que le hub A est synchronisé en premier, l’équipement et ses commandes sont réaffectés au mauvais hub.

Le même lookup hub + groupe sert maintenant à la synchronisation et aux callbacks. Le fallback vers un ancien identifiant n’est accepté que pour un équipement de type groupe appartenant au même hub. Les identifiants des équipements existants et de leurs commandes sont préservés pour le bon hub. Le callback reconnaît GROUP et Group, tout en conservant le routage habituel des autres équipements.

Fixes #43

Suggested changelog entry

Correction des mises à jour en temps réel des groupes et de leur migration sur les installations multi-hubs.

Related issues/external references

Types of changes

  • Bug fix

Validation

Cinq tests de non-régression sans compte Ajax ni installation Jeedom :

  • Groupes de deux hubs ayant le même identifiant et synchronisation répétée.
  • Migration d’un ancien groupe du second hub sans récupération par le premier.
  • Rafraîchissement des groupes et URLs des commandes d’armement.
  • Réponse /groups invalide : warning et conservation des équipements.
  • Exécution du vrai callback avec une entrée JSON de test : groupes préfixés, ancien groupe, isolation entre hubs et conservation des mises à jour d’un équipement ordinaire.

Validation réussie sur PHP 7.4 et 8.3 : les 16 tests de session existants et les 5 tests de groupes passent.

Syntaxe YAML et git diff --check vérifiés. Validation sur matériel réel non effectuée.

@zoic21
zoic21 merged commit 4d43290 into beta Oct 20, 2026
7 checks passed
@zoic21
zoic21 deleted the fix/multi-hub-groups branch October 20, 2026 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: Gestion des groupes (zones) cassée pour les configurations multi-hubs + retour false silencieux

1 participant