Skip to content

Fix shared cmd::update value mutation breaking live chart updates - #3452

Open
Salvialf wants to merge 3 commits into
developfrom
fix/cmd-value-mutation
Open

Fix shared cmd::update value mutation breaking live chart updates#3452
Salvialf wants to merge 3 commits into
developfrom
fix/cmd-value-mutation

Conversation

@Salvialf

@Salvialf Salvialf commented Aug 2, 2026

Copy link
Copy Markdown
Contributor
  • Two table state display callbacks (eqLogic.configure.php and plugin.template.js) mutated the value property of the shared cmd::update event payload object in place, instead of using a local copy for their own HTML-sanitized display.
  • This payload object is shared across every listener processing the same event (jeedom.cmd.refreshValue, jeedom.history.graphUpdate, and any other addUpdateFunction callback registered for the same command), so the mutation leaked into all code reading the value afterward, silently turning a numeric value into a string.
  • This caused live history chart updates (jeedom.history.graphUpdate) to receive a string instead of a number whenever a page or modal containing one of these table displays was open, breaking the chart's live rendering for that command.
  • While at it, aligned the tooltip date format across eqLogic.configure.php, plugin.template.js, cmd.action.slider.shutter.html and two mobile templates with the convention used across nearly all dashboard widget templates ({{Date de valeur}}: before {{Date de collecte}}: , no space before the colon), and added the missing {{}} translation wrapper around those strings in cmd.info.string.multiline.html (mobile), which had never been translatable.

@Salvialf Salvialf added the changelog-fix Use to generate release notes / changelog To be apply on PR label Aug 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-fix Use to generate release notes / changelog To be apply on PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant