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.

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

Align @RouteScoped implementation with @RouteScope impl for Spring add-on #369

Closed
denis-anisimov opened this issue May 31, 2021 · 3 comments · Fixed by #370
Closed

Align @RouteScoped implementation with @RouteScope impl for Spring add-on #369

denis-anisimov opened this issue May 31, 2021 · 3 comments · Fixed by #370
Assignees
@denis-anisimov
Labels
enhancement prerelease version for platform 21.0.0 prerelease version for platform 22.0.0 Released with Platform 21.0.0.alpha5 Released with Platform 22.0.0.alpha1

Comments

@denis-anisimov
Copy link

Several things needs to be done:

  • Don't store beans inside UI instance but use window name (when available) and store things inside a VaadinSession.
  • Don't allow to use scope when scope context is not available (right now the scope will be used regardless of the route component presence : but if the context is not available everything will be destroyed. The injection should not be even done at all if the scope context is not available). To be able to implement this BeforeEnterListener should be used along with AfterNavigationListener.
  • RouteScoped without RouteScopeOwner should behave differently than now: it should use the current bottom-most navigation target component as a context.
@denis-anisimov denis-anisimov changed the title Alight @RouteScoped implementation with @RouteScop impl for Spring add-on Align @RouteScoped implementation with @RouteScop impl for Spring add-on May 31, 2021
@denis-anisimov
Copy link
Author

see #293

@denis-anisimov denis-anisimov mentioned this issue May 31, 2021
Closed
@denis-anisimov denis-anisimov changed the title Align @RouteScoped implementation with @RouteScop impl for Spring add-on Align @RouteScoped implementation with @RouteScope impl for Spring add-on May 31, 2021
@denis-anisimov denis-anisimov mentioned this issue Jun 1, 2021
Open
@denis-anisimov denis-anisimov self-assigned this Jun 1, 2021
denis-anisimov pushed a commit that referenced this issue Jun 4, 2021
feat: reimplement @RouteScoped to keep beans for PreserveOnRefresh
73ae2d2 
- presere beans for PreserveOnRefresh views
- reimplement absent RouteScopeOwner semantic
- do not allow to use (route) scope when it's not available

fixes #369
@denis-anisimov denis-anisimov mentioned this issue Jun 4, 2021
Merged
9 tasks
denis-anisimov pushed a commit that referenced this issue Jun 4, 2021
feat: reimplement @RouteScoped to keep beans for PreserveOnRefresh
d90a680 
- presere beans for PreserveOnRefresh views
- reimplement absent RouteScopeOwner semantic
- do not allow to use (route) scope when it's not available

fixes #369
denis-anisimov pushed a commit that referenced this issue Jun 7, 2021
feat: reimplement @RouteScoped to keep beans for PreserveOnRefresh (#370
c4c6d37 
)

* feat: reimplement @RouteScoped to keep beans for PreserveOnRefresh

- presere beans for PreserveOnRefresh views
- reimplement absent RouteScopeOwner semantic
- do not allow to use (route) scope when it's not available

fixes #369

* chore: make minor corrections and add ITs

* chore: correct test class
@vaadin-bot
Copy link

This ticket/PR has been released with platform 21.0.0.alpha5. For prerelease versions, it will be included in its final version.

@vaadin-bot
Copy link

This ticket/PR has been released with platform 22.0.0.alpha1 and is also targeting the upcoming stable 22.0.0 version.

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

@denis-anisimov denis-anisimov

Labels
enhancement prerelease version for platform 21.0.0 prerelease version for platform 22.0.0 Released with Platform 21.0.0.alpha5 Released with Platform 22.0.0.alpha1
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: reimplement @RouteScoped to keep beans for PreserveOnRefresh vaadin/cdi
3 participants
@pleku @denis-anisimov @vaadin-bot