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

SDL transfers AddSubMenu request with non-existent parentID parameter to HMI #3524

Closed
VjKlepikov opened this issue Oct 1, 2020 · 3 comments
Labels
7.0 Release issues introduced by 7.0 release candidate

Comments

@VjKlepikov
Copy link

VjKlepikov commented Oct 1, 2020

Bug Report

SDL transfers AddSubMenu request with non-existent parentID parameter to HMI

Preconditions:

  1. SDL and HMI are started
  2. Mobile device connected
  3. Mobile app is registered and activated
Reproduction Steps
  1. Add submenu item AddSubMenu RPC with non-existent parentID parameter
    AddSubMenu(parentID=1)
Expected Behavior

APP→SDL: AddSubMenu ( "menuName":"Submenu Name 1","parentID":1, "menuID":1 )
SDL→APP: AddSubMenu ( resultCode: INVALID_ID, success=false ).

Observed Behavior

APP→SDL: AddSubMenu ( "menuName":"Submenu Name 1","parentID":1, "menuID":1 )
SDL→HMI: UI.AddSubMenu ( params, "menuParams":{"menuName":"Submenu Name 1","parentID":1,"position":0}, "menuID":1} )

OS & Version Information

SDL: https://github.com/smartdevicelink/sdl_core/commits/develop (9f61819)
HMI: https://github.com/smartdevicelink/sdl_hmi/tree/develop (f712d4e)

Test Case, Sample Code, and / or Example App

Logs:SDL_logs.zip

@dboltovskyi
Copy link
Contributor

@theresalech Please notice issue is related to SDL-0148 Additional SubMenus proposal which is in scope of 7.0 Release.
Luxoft team is going to prepare a fix.

@theresalech
Copy link
Contributor

@dboltovskyi, thank you for letting us know. Please advise when a PR is ready for Livio review.

@iCollin
Copy link
Collaborator

iCollin commented Oct 20, 2020

Closed via #3534

@iCollin iCollin closed this as completed Oct 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
7.0 Release issues introduced by 7.0 release candidate
Projects
None yet
Development

No branches or pull requests

4 participants