search

alump / LazyLayouts

Vaadin layouts for your lazy loading needs
Apache License 2.0
4 stars 3 forks source link

Please make LazyComponentProvider extend Serializable #9

Closed mvysny closed 7 years ago

mvysny commented 7 years ago

If one writes code as follows:

vl.enableLazyLoading(new LazyComponentProvider() {
...
});

Then that provider is not Serializable, and e.g. Tomcat will complain later on that

Apr 03, 2017 4:17:50 PM org.apache.catalina.session.StandardSession doWriteObject
WARNING: Cannot serialize session attribute com.vaadin.server.VaadinSession.MyUIServlet for session D69B79DCA3E3FBEE840CC9E7D1C185DC
java.io.NotSerializableException: org.test.MyUI$1

Thanks! ;)

alump commented 7 years ago

What version do you use (read: do you have Vaadin 7 or Vaadin 8 project)

alump commented 7 years ago

0.2.1 has this fixed, if required I can also add it to Vaadin 7 versions (0.1.x)

alump commented 7 years ago

0.1.4 also has this fix now (for Vaadin 7 users)

mvysny commented 7 years ago

Thanks so much! I'm using Vaadin 8. Thanks for fixing :)