Comments (10)
Thanks, started working on this.
Original comment by [email protected]
on 3 Oct 2012 at 8:45
- Changed state: Accepted
from dellroad-stuff.
Just as an update: Basic stuff on
[https://github.com/aanno/dellroad-stuff-aanno] is now working, including
@VaadinConfigurable. Biggest hurdles for general consumption are now (a) I'm
extending VaadinServlet (and I think this is a very bad idea) and (b) I'm not
fully understanding the new concepts (and lifecycles) of UI (and their ids),
VaadinServiceSession, and VaadinServlet at present.
Original comment by [email protected]
on 4 Oct 2012 at 6:37
from dellroad-stuff.
Well, I probably should mention that my changes are on the 'tp' branch while
the 'master' branch just tracks the original upstream svn repository.
Original comment by [email protected]
on 4 Oct 2012 at 7:25
from dellroad-stuff.
Thanks. I am trying to implement this (from scratch) using the approach
suggested by the Vaadin folks. See http://dev.vaadin.com/ticket/9733 for a
related discussion.
Once I get it committed (hopefully soon), I'd very much appreciate your review.
BTW, extending VaadinServlet is indeed required.
Original comment by [email protected]
on 4 Oct 2012 at 1:04
from dellroad-stuff.
Just as an update: After reading your link, and reasoning on myself
[https://vaadin.com/forum/-/message_boards/view_message/1875525], I've
concluded that the 'normal' spring context scope for vaadin7 should be bound to
VaadinServiceSession. That's what the code does now. I've also added a Spring
UI-bound scope that can be used within (and outside!) this context. On top of
that I'm experimenting with an UI-scoped EventBus (as a building block for some
Spring-based MVP support inspired by
[https://vaadin.com/directory#addon/mvp-lite]).
The is another (competing vaadin7 Spring integration)
[https://vaadin.com/forum/-/message_boards/view_message/1811100]. I will have a
look on this soon to decide if there are ideas there that are missing here.
However, I'm really not interested at forking your stuff. Best thing that could
happen to my that your implementation is working again soon. And I would be
glad if you incorporate the more useful ideas (well, not the hacked code,
probably).
Original comment by [email protected]
on 11 Oct 2012 at 12:24
from dellroad-stuff.
I should have something ready to commit in the next day or two. You are correct
that VaadinServiceSession is the new place to hang the "Vaadin application" and
Spring context.
Original comment by [email protected]
on 11 Oct 2012 at 1:35
from dellroad-stuff.
r528 seems to work more or less. The dellroad-stuff example on github (branch
'vaadin7') has been updated to work with it.
Check it out and let me know what you think.
Original comment by [email protected]
on 11 Oct 2012 at 10:42
- Changed state: Fixed
- Added labels: Type-Enhancement
- Removed labels: Type-Defect
from dellroad-stuff.
Dear Archie,
thank you for the regular updates to the vaadin 7 stuff. I basically use your
code out-of-the-box. However there are some minor glitches (like the broken
'mvn deploy' compatibility (especially then it comes to SNAPSHOTs), basic
eclipse support, make AbstractUpdatingDataSource compatible with java7, ...)
that makes me still 'rebase'ing my tp2 branch on top of your code.
There is only one issue that perhaps need more profound consideration: It's the
scope of an vaadin application. You decided to bound the vaadin application
lifecycle to the lifetime of the UI. This is a valid choice, as it makes
possible to log-in to the same web application with *two* different users in
two different tabs. As the UI is not shared, all works as expected.
However, this is not the only possible choice. If you want *one* web
application that supports several browser native tabs, this is possible with
vaadin as well - but will be difficult with your spring integration. In this
case the lifecyle of the vaadin application should be bound to the HttpSession
(as a new UI is just another tab in your browser).
The bad news is that I need the later. (I need to support mobile (phone+tablet)
clients, and because of this I would favourite native browser tabs above the
tab widget of vaadin.)
A possible solution to with is to bound the spring context to the HttpSession
and in addition provide a Spring scope that is UI-bound. With way, both
scenarios would be supported...
Original comment by [email protected]
on 5 Dec 2012 at 2:10
from dellroad-stuff.
Not sure I understand the scope question. The latest vaadin7 package associates
the Vaadin application context to the VaadinSession, not any particular UI
instance. See the latest Javadoc API:
http://dellroad-stuff.googlecode.com/svn/trunk/publish/reports/javadoc/org/dellr
oad/stuff/vaadin7/SpringVaadinServlet.html
Or am I misunderstanding something?
Original comment by [email protected]
on 5 Dec 2012 at 7:43
from dellroad-stuff.
I've folded in some of your changes into SVN. Thanks!
Original comment by [email protected]
on 5 Dec 2012 at 7:56
from dellroad-stuff.
Related Issues (20)
- Include dependencies in maven meta-data HOT 1
- SpringContextApplication does not implement Root HOT 8
- PostCompletionAspect can not be registered - needs to be removed from aop.xml HOT 6
- Fix deprecation warnings HOT 1
- Problem with Spring 3.1.3 HOT 14
- Patch for /trunk/src/addon/pom.xml HOT 1
- Patch for /trunk/src/addon/pom.xml HOT 2
- Add support for Vaadin Touchkit and Spring in Tomcat 6 HOT 3
- Actions queued using VaadinUtil.invokeLater(Runnable action) are not triggered for run HOT 7
- bad URL on front page HOT 1
- @ThreadConfigurable and @ThreadTransactional HOT 1
- dellroad-stuff-maven-plugin: Single quotes in `exec-set-property` result HOT 2
- Maven deployment in support of dependent projects HOT 4
- support for Portlet API HOT 11
- Remove dependecy from Servlet 3.0 HOT 4
- servlet-api must have scope provided HOT 11
- Circular autowiring of SpringContextApplication fails HOT 1
- Extract "transactional file" support from PersistentObject into a separate class HOT 1
- NotSerializableException thrown when serializing subclass of ContextApplication HOT 3
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from dellroad-stuff.