When debugging Dashboard UI tests locally I've seen the below warning several times
WARNING TagManager[2024-05-30 08:31:43.581651 UTC] [4c877] /Users/michal/Sites/matomo/plugins/TagManager/Context/WebContext.php(98): Warning - Undefined array key "id" - Matomo 5.2.0-alpha - Please report this message in the Matomo forums: https://forum.matomo.org (please do a search first as it might have been reported already)
so this is adding a safeguard for cases when the ID is not defined. Seems like in UI tests the environments may not be always accessible or not have the ID set, not sure. This change should not affect anything else as far as I can tell.
[ ] Potential edge cases thought about (behavior of the code with strange input, with strange internal state or possible interactions with other Matomo subsystems)
[ ] Usability review done (is anything maybe unclear or think about anything that would cause people to reach out to support)
Description:
When debugging Dashboard UI tests locally I've seen the below warning several times
WARNING TagManager[2024-05-30 08:31:43.581651 UTC] [4c877] /Users/michal/Sites/matomo/plugins/TagManager/Context/WebContext.php(98): Warning - Undefined array key "id" - Matomo 5.2.0-alpha - Please report this message in the Matomo forums: https://forum.matomo.org (please do a search first as it might have been reported already)so this is adding a safeguard for cases when the ID is not defined. Seems like in UI tests the environments may not be always accessible or not have the ID set, not sure. This change should not affect anything else as far as I can tell.
Review