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

Core stops generating vrHelp and helpPrompt after ResetGlobalProperties #3931

Closed
jacobkeeler opened this issue Jun 20, 2022 · 2 comments
Closed
Labels
Projects

Comments

@jacobkeeler
Copy link
Contributor

jacobkeeler commented Jun 20, 2022

Bug Report

If ResetGlobalProperties is sent for VRHELP, VRHELPTITLE, and HELPPROMPT, an auto-generated value for each of these fields is sent initially, but these auto-generated values no longer update after this.

Reproduction Steps
  1. Register App
  2. Send AddCommand, verify UI.SetGlobalProperties + TTS.SetGlobalProperties are sent with info for this command
  3. Send SetGlobalProperties with vrHelp + vrHelpTitle + helpPrompt
  4. Send ResetGlobalProperties with VR_HELP, VR_HELP_TITLE, verify UI.SetGlobalProperties + TTS.SetGlobalProperties matches step 1
  5. Send another AddCommand
Expected Behavior

UI.SetGlobalProperties + TTS.SetGlobalProperties are sent with data for both commands

Observed Behavior

UI.SetGlobalProperties + TTS.SetGlobalProperties are not sent

OS & Version Information
  • OS/Version: Ubuntu 20.04
  • SDL Core Version: master (Release 8.1)
  • Testing Against: RPC Builder JS
@jacobkeeler
Copy link
Contributor Author

Seems this is because the sending_type_ is never updated when ResetGlobalProperties is sent, see https://github.com/smartdevicelink/sdl_core/blob/master/src/components/application_manager/src/help_prompt_manager_impl.cc#L340

@ShobhitAd
Copy link
Contributor

Closed via #3937

@theresalech theresalech moved this from Development In Progress/PR Ready for Review to Completed Bug Fixes/ Enhancements in 8.2.0 Aug 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
8.2.0
Completed Bug Fixes / Enhancements
Development

No branches or pull requests

3 participants