Repository navigation
Compléter la correction des groupes multi-hubs : callbacks et migration - #56
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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 :00000001) et le recherche directement, alors que l’équipement Jeedom s’appelle désormaishubId_00000001. Les mises à jour d’état des groupes synchronisés sont donc ignorées.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
GROUPetGroup, 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
id,hubId,type,updates) : https://support.ajax.systems/api/how-to-use-enterprise-API.pdfTypes of changes
Validation
Cinq tests de non-régression sans compte Ajax ni installation Jeedom :
/groupsinvalide : warning et conservation des équipements.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 --checkvérifiés. Validation sur matériel réel non effectuée.