Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OnLanguageChange and OnSystemInfoChanged are sent twice #256

Closed
GetmanetsIrina opened this issue Nov 5, 2019 · 2 comments
Closed

OnLanguageChange and OnSystemInfoChanged are sent twice #256

GetmanetsIrina opened this issue Nov 5, 2019 · 2 comments
Assignees

Comments

@GetmanetsIrina
Copy link

Bug Report

OnLanguageChange and OnSystemInfoChanged are sent twice

Reproduction Steps
  1. SDL and HMI are started
  2. App is registered
  3. Change language on HMI
Expected Behavior

HMI sends OnLanguageChange and OnSystemInfoChanged once to SDL

Observed Behavior
FFW.UI.OnLanguageChange
RPCClient.js:317 HMI -> SDL [15:17:6:13]: {"jsonrpc":"2.0","method":"UI.OnLanguageChange","params":{"language":"DE-DE"}}
BasicCommunicationRPC.js:1036 FFW.BasicCommunication.OnSystemInfoChanged
RPCClient.js:317 HMI -> SDL [15:17:6:17]: {"jsonrpc":"2.0","method":"BasicCommunication.OnSystemInfoChanged","params":{"language":"DE-DE"}}
UIRPC.js:1589 FFW.UI.OnLanguageChange
RPCClient.js:317 HMI -> SDL [15:17:6:20]: {"jsonrpc":"2.0","method":"UI.OnLanguageChange","params":{"language":"DE-DE"}}
BasicCommunicationRPC.js:1036 FFW.BasicCommunication.OnSystemInfoChanged
RPCClient.js:317 HMI -> SDL [15:17:6:22]: {"jsonrpc":"2.0","method":"BasicCommunication.OnSystemInfoChanged","params":{"language":"DE-DE"}}
RPCClient.js:137 SDL -> HMI [15:17:6:126]: {"id":262,"jsonrpc":"2.0","method":"BasicCommunication.UpdateAppList","params":{"applications":[]}}
OS & Version Information
@YarikMamykin
Copy link
Contributor

@theresalech Please triage this issue and notice linked fix is ready for review.

@jacobkeeler
Copy link
Contributor

Closing with the merge of #263

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

No branches or pull requests

3 participants