Hi yaniv Taking your idea a little further, one could also just get the LDM injected, which actually archives what I wanted. However this has strayed a lot from the topic. So I think we should start a new thread on howto manage Wicket with IOC frameworks..
regards Nino 2010/6/8 yaniv kessler <[email protected]>: > Perhaps inject the model instead of manually creating it or maybe inject a > data service into the page and use that reference in the model. My point is, > passing the injector around (or in this case InjectorHolder), should be > avoided, since as it degrades the entire Guice DI into a simple service > locator pattern (which has its uses and shortcomings) > > Anyways, putting code habits aside, I was unable to understand from your > last mail if you succeeded in solving your problem, did you ? > > On Tue, Jun 8, 2010 at 2:26 AM, nino martinez wael < > [email protected]> wrote: > >> Well as I manually create the the loadabledetachmodel which extends >> the ClassWithDao, it has to call injectorholder. I am not aware of any >> other method that can solve this.. >> >> 2010/6/8 yaniv kessler <[email protected]>: >> > I agree with Jorge here and additionally, would like to ask what is the >> > motivation to use InjectorHolder directly? >> > >> > Are you simply doing new ClassWithDao() somewhere in your code ? >> > >> > On Mon, Jun 7, 2010 at 9:21 PM, Jorge Rodrigez <[email protected]> >> wrote: >> > >> >> 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] >> >> > >> >> > >> >> >> > >> > >> > >> > -- >> > 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] >> >> > > > -- > 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]
