Why phoneDao is transient ? Wicket-Guice injects a Serializable proxy and thus after deserialization the field will be non-null.
On Mon, Jun 7, 2010 at 7:31 PM, nino martinez wael < [email protected]> wrote: > sure: > > public class ClassWithDao { > > public ClassWithDao() { > InjectorHolder.getInjector().inject(this); > } > > @Inject > private transient PhoneDao phoneDao; > > public PhoneDao getPhoneDao() { > return phoneDao; > } > > public void setPhoneDao(PhoneDao phoneDao) { > this.phoneDao = phoneDao; > } > > That's where I get the failure... And the working version you have seen.. > > 2010/6/7 yaniv kessler <[email protected]>: > > Nino, can you show the ClassWithDao code before the so called "ugliness" > was > > introduced? > > > > On Sun, Jun 6, 2010 at 1:44 PM, nino martinez wael < > > [email protected]> wrote: > > > >> im still working on the quickstart.. So far I've been unable to > >> replicate.. I'll continue tomorrow > >> > >> 2010/6/5 nino martinez wael <[email protected]>: > >> > I was too quick to say it worked, as it appears it does not. I'll try > >> > to make a quickstart based on legup, where should I put it? Just on > >> > jira? > >> > > >> > -Nino > >> > > >> > 2010/6/4 nino martinez wael <[email protected]>: > >> >> yeah me too, what Igor said worked so must be true or a bug.. I don't > >> >> think it's a bug.. > >> >> > >> >> 2010/6/3 James Carman <[email protected]>: > >> >>> I thought the filters were executed in the order of their > >> >>> filter-mappings, not their filter definitions. > >> >>> > >> >>> On Thu, Jun 3, 2010 at 5:23 PM, nino martinez wael > >> >>> <[email protected]> wrote: > >> >>>> Argh, how stupid.. Thanks a lot igor, how can I buy you a beer or > >> Coke? > >> >>>> > >> >>>> I spend a tremendous time trying to figure out what was wrong. I > even > >> >>>> considered going back to spring.. > >> >>>> > >> >>>> regards Nino > >> >>>> > >> >>>> 2010/6/3 Igor Vaynberg <[email protected]>: > >> >>>>> noep, the filters are processed in the order they are defined in > >> >>>>> web.xml, move the wicket filter decl below the warp persist stuff. > >> >>>>> > >> >>>>> -igor > >> >>>>> > >> >>>>> On Thu, Jun 3, 2010 at 1:05 PM, nino martinez wael > >> >>>>> <[email protected]> wrote: > >> >>>>>> yes, i think so: > >> >>>>>> > >> >>>>>> <?xml version="1.0" encoding="ISO-8859-1"?> > >> >>>>>> <web-app xmlns="http://java.sun.com/xml/ns/j2ee" > >> >>>>>> xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" > >> >>>>>> xsi:schemaLocation="http://java.sun.com/xml/ns/j2ee > >> >>>>>> http://java.sun.com/xml/ns/j2ee/web-app_2_4.xsd" > >> >>>>>> version="2.4"> > >> >>>>>> > >> >>>>>> <display-name>IVR Web frontend</display-name> > >> >>>>>> > >> >>>>>> <filter> > >> >>>>>> <filter-name>wicket.WicketWarp</filter-name> > >> >>>>>> > >> > <filter-class>org.apache.wicket.protocol.http.WicketFilter</filter-class> > >> >>>>>> <init-param> > >> >>>>>> > <param-name>applicationClassName</param-name> > >> >>>>>> > >> > <param-value>com.netdesign.codan.webadmin.WicketApplication</param-value> > >> >>>>>> </init-param> > >> >>>>>> > >> >>>>>> <init-param> > >> >>>>>> <param-name>configuration</param-name> > >> >>>>>> <param-value>deployment</param-value> > >> >>>>>> </init-param> > >> >>>>>> </filter> > >> >>>>>> > >> >>>>>> <filter> > >> >>>>>> <filter-name>warpPersistFilter</filter-name> > >> >>>>>> > >> > <filter-class>com.wideplay.warp.persist.PersistenceFilter</filter-class> > >> >>>>>> </filter> > >> >>>>>> > >> >>>>>> <filter-mapping> > >> >>>>>> <filter-name>warpPersistFilter</filter-name> > >> >>>>>> <url-pattern>/*</url-pattern> > >> >>>>>> </filter-mapping> > >> >>>>>> > >> >>>>>> > >> >>>>>> > >> >>>>>> > >> >>>>>> <filter-mapping> > >> >>>>>> <filter-name>wicket.WicketWarp</filter-name> > >> >>>>>> <url-pattern>/*</url-pattern> > >> >>>>>> </filter-mapping> > >> >>>>>> > >> >>>>>> > >> >>>>>> </web-app> > >> >>>>>> > >> >>>>>> > >> >>>>>> 2010/6/2 Igor Vaynberg <[email protected]>: > >> >>>>>>> did you install warp's open entity manager in view filter > *before* > >> >>>>>>> wicket's filter? > >> >>>>>>> > >> >>>>>>> -igor > >> >>>>>>> > >> >>>>>>> On Wed, Jun 2, 2010 at 5:46 AM, nino martinez wael > >> >>>>>>> <[email protected]> wrote: > >> >>>>>>>> Hi I somehow think theres something wrong with the Wicket Guice > >> proxy > >> >>>>>>>> (probably only if you are using guice 2 & possibly warp > persist) > >> >>>>>>>> > >> >>>>>>>> Anyhow I have had to change my LDM's to this code, notice the > >> injector > >> >>>>>>>> holder in the getter method really really bad. But if not I get > an > >> >>>>>>>> "entity manager is closed": > >> >>>>>>>> > >> >>>>>>>> public class ClassWithDao { > >> >>>>>>>> > >> >>>>>>>> public ClassWithDao() { > >> >>>>>>>> InjectorHolder.getInjector().inject(this); > >> >>>>>>>> } > >> >>>>>>>> > >> >>>>>>>> @Inject > >> >>>>>>>> private transient PhoneDao phoneDao; > >> >>>>>>>> > >> >>>>>>>> /** > >> >>>>>>>> * This method contains a fix, should ordinary be > avoided! > >> There > >> >>>>>>>> should be no need for calling the extra InjectorHolder method! > >> >>>>>>>> * @return > >> >>>>>>>> */ > >> >>>>>>>>>> public PhoneDao getPhoneDao() { > >> >>>>>>>>>> InjectorHolder.getInjector().inject(this); > >> >>>>>>>>>> return phoneDao; > >> >>>>>>>>>> } > >> >>>>>>>> > >> >>>>>>>> public void setPhoneDao(PhoneDao phoneDao) { > >> >>>>>>>> this.phoneDao = phoneDao; > >> >>>>>>>> } > >> >>>>>>>> > >> >>>>>>>> Am I doing anything wrong I wonder, however the first requests > are > >> >>>>>>>> always working it's after something has been trough the session > >> store > >> >>>>>>>> it goes wrong.. > >> >>>>>>>> > >> >>>>>>>> -regards Nino > >> >>>>>>>> > >> >>>>>>>> > >> --------------------------------------------------------------------- > >> >>>>>>>> To unsubscribe, e-mail: [email protected] > >> >>>>>>>> For additional commands, e-mail: [email protected] > >> >>>>>>>> > >> >>>>>>>> > >> >>>>>>> > >> >>>>>>> > >> --------------------------------------------------------------------- > >> >>>>>>> To unsubscribe, e-mail: [email protected] > >> >>>>>>> For additional commands, e-mail: [email protected] > >> >>>>>>> > >> >>>>>>> > >> >>>>>> > >> >>>>>> > >> --------------------------------------------------------------------- > >> >>>>>> To unsubscribe, e-mail: [email protected] > >> >>>>>> For additional commands, e-mail: [email protected] > >> >>>>>> > >> >>>>>> > >> >>>>> > >> >>>>> > --------------------------------------------------------------------- > >> >>>>> To unsubscribe, e-mail: [email protected] > >> >>>>> For additional commands, e-mail: [email protected] > >> >>>>> > >> >>>>> > >> >>>> > >> >>>> > --------------------------------------------------------------------- > >> >>>> To unsubscribe, e-mail: [email protected] > >> >>>> For additional commands, e-mail: [email protected] > >> >>>> > >> >>>> > >> >>> > >> >>> > --------------------------------------------------------------------- > >> >>> To unsubscribe, e-mail: [email protected] > >> >>> For additional commands, e-mail: [email protected] > >> >>> > >> >>> > >> >> > >> > > >> > >> --------------------------------------------------------------------- > >> To unsubscribe, e-mail: [email protected] > >> For additional commands, e-mail: [email protected] > >> > >> > > > > > > -- > > Tel: +972-77-4100977 > > Fax: +972-77-4100976 > > Mobile: +972-54-7755427 > > > > http://www.codeark.com > > http://blogs.microsoft.co.il/blogs/yanush/ > > http://headtoscreencollision.blogspot.com > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > >
