Skip to content
New issue

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

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

Already on GitHub? # to your account

The screen manager's soft button manager sends image buttons before images are uploaded #955

Closed
NicoleYarroch opened this issue May 8, 2018 · 0 comments
Assignees
Labels
bug A defect in the library
Milestone

Comments

@NicoleYarroch
Copy link
Contributor

NicoleYarroch commented May 8, 2018

Bug Report

The screen manager's soft button manager sends image buttons before said images are uploaded.

The code below:

...
} else if (([self sdl_currentStateHasImages] && ![self sdl_allCurrentStateImagesAreUploaded])
               && (self.softButtonCapabilities ? !self.softButtonCapabilities.imageSupported : YES)) {
...

should be:

...
} else if ((([self sdl_currentStateHasImages] && ![self sdl_allCurrentStateImagesAreUploaded])
               && (self.softButtonCapabilities ? self.softButtonCapabilities.imageSupported.boolValue : NO))
               || (self.softButtonCapabilities ? !self.softButtonCapabilities.imageSupported.boolValue : YES)) {
...
Reproduction Steps
  1. Connect app to Core and open SDL app. (App should have at least 1 soft button)
Expected Behavior

If the button has an image not yet uploaded to Core, the manager should wait to send the image button till the image has been successfully uploaded to Core.

Observed Behavior

If the button has an image not yet uploaded to Core, the manager still sends the image button. Core returns an INVALID_DATA error.

OS & Version Information
  • iOS Version: n/a
  • SDL iOS Version: 5.3
  • Testing Against: TDK (SYNC 3)
Test Case, Sample Code, and / or Example App

The SDL_iOS library example app.

@NicoleYarroch NicoleYarroch self-assigned this May 8, 2018
@NicoleYarroch NicoleYarroch added the bug A defect in the library label May 8, 2018
@NicoleYarroch NicoleYarroch added this to the 6.0.0 milestone May 8, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug A defect in the library
Projects
None yet
Development

No branches or pull requests

2 participants