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

Setting bad data in one T&G field then good data quickly in another can lead to the good data failing. #1828

Closed
ChloeMJM opened this issue Sep 8, 2022 · 1 comment · Fixed by #1831
Labels
bug A defect in the library

Comments

@ChloeMJM
Copy link
Contributor

ChloeMJM commented Sep 8, 2022

Bug Summary

This bug is related to iOS Issue #1781

Based on how the Text & Graphic manager currently works, it is possible to have good data fail. A possible solution is that when a Show fails and the currentScreenData gets updated to also update the updateState of the T&G operation as well. This will require knowing which fields the T&G operation is updating so that they don't also get reset.

With a similar method we could potentially also stop sending all Show data and only send updated Show data.

Reproduction Steps

  1. Set "bad" data (data that will get rejected) into TextField1 and start the operation.
  2. Set "good" data (data that will not get rejected) into TextField2 and start the operation before the first operation gets a Show response.

Expected Behavior: TextField1 should remain in its original state and TextField2 should show the new data.

Observed Behavior: TextField1 and TextField2 will show the original state data because TextField2's operation includes the TextField1 data as well.

Which projects have you seen this bug on?

No response

Android Version(s)

No response

Android Device(s)

n/a

sdl_java_suite Version

5.5.0

Testing Environment(s)

Manticore

Relevant log output

No response

@ChloeMJM ChloeMJM added the bug A defect in the library label Sep 8, 2022
@JulianKast
Copy link
Contributor

Fixed with #1833

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A defect in the library
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants