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.

Sign up for GitHub

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

Jump to bottom

Fix flow issue #6349 (#20255) (CP: 2.11) #20279

Merged

mshabarov merged 1 commit into 2.11 from cherry-pick-20255-to-2.11-1729238243460

Oct 18, 2024

Collaborator

vaadin-bot commented Oct 18, 2024

No description provided.


          fix: clear CurrentInstance before invoking new session tasks (#6349) (#…

8f2c8d6

…20255)

Commands enqueued by VaadinSession.access() in general have nothing to do with each other. The only thing they have in common is they share the same VaadinSession (and, by implication, VaadinService).

Therefore, if command №1 invoked UI.setCurrent() and command №2 invokes UI.getCurrent(), command №2 should read null, not the random UI from command №1 that it has nothing to do with.

Fixes #6349

sonarcloud bot commented Oct 18, 2024

Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

vaadin-bot added the +0.0.1 label

mshabarov approved these changes

View reviewed changes

mshabarov merged commit 32add3c into 2.11

9 checks passed

mshabarov deleted the cherry-pick-20255-to-2.11-1729238243460 branch

October 18, 2024 09:48

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels