Hi yaniv
replying inline.. And thanks for the ideas.
2010/6/8 yaniv kessler <yan...@gmail.com>:
> 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)
Yeah I know and it annoys me too. So what you are saying is that the
the ioc serialise proxy will be discovered in my ldm and be replaced
on deserialise, also if I do not use injectorholder (that makes sense
to me). The idea behind using injectorholder directly was because I
wanted the ldm to be selfcontained.
>
> Anyways, putting code habits aside, I was unable to understand from your
> last mail if you succeeded in solving your problem, did you ?
Well I've started a quickstart project (based on the legup archetype,
but "upgraded" to guice 2) to trace what I am doing wrong, and have
been unable to replicate the problem so far. I'll continue searching
for what I've done wrong in the non working project.
>
> On Tue, Jun 8, 2010 at 2:26 AM, nino martinez wael <
> nino.martinez.w...@gmail.com> 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 <yan...@gmail.com>:
>> > 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 <mg.mli...@gmail.com>
>> 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 <
>> >> nino.martinez.w...@gmail.com> 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 <yan...@gmail.com>:
>> >> > > 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 <
>> >> > > nino.martinez.w...@gmail.com> 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 <nino.martinez.w...@gmail.com>:
>> >> > >> > 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 <nino.martinez.w...@gmail.com>:
>> >> > >> >> 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 <ja...@carmanconsulting.com>:
>> >> > >> >>> 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
>> >> > >> >>> <nino.martinez.w...@gmail.com> 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 <igor.vaynb...@gmail.com>:
>> >> > >> >>>>> 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
>> >> > >> >>>>> <nino.martinez.w...@gmail.com> 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 <igor.vaynb...@gmail.com>:
>> >> > >> >>>>>>> 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
>> >> > >> >>>>>>> <nino.martinez.w...@gmail.com> 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:
>> users-unsubscr...@wicket.apache.org
>> >> > >> >>>>>>>> For additional commands, e-mail:
>> >> users-h...@wicket.apache.org
>> >> > >> >>>>>>>>
>> >> > >> >>>>>>>>
>> >> > >> >>>>>>>
>> >> > >> >>>>>>>
>> >> > >>
>> ---------------------------------------------------------------------
>> >> > >> >>>>>>> To unsubscribe, e-mail:
>> users-unsubscr...@wicket.apache.org
>> >> > >> >>>>>>> For additional commands, e-mail:
>> users-h...@wicket.apache.org
>> >> > >> >>>>>>>
>> >> > >> >>>>>>>
>> >> > >> >>>>>>
>> >> > >> >>>>>>
>> >> > >>
>> ---------------------------------------------------------------------
>> >> > >> >>>>>> To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
>> >> > >> >>>>>> For additional commands, e-mail:
>> users-h...@wicket.apache.org
>> >> > >> >>>>>>
>> >> > >> >>>>>>
>> >> > >> >>>>>
>> >> > >> >>>>>
>> >> > ---------------------------------------------------------------------
>> >> > >> >>>>> To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
>> >> > >> >>>>> For additional commands, e-mail:
>> users-h...@wicket.apache.org
>> >> > >> >>>>>
>> >> > >> >>>>>
>> >> > >> >>>>
>> >> > >> >>>>
>> >> > ---------------------------------------------------------------------
>> >> > >> >>>> To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
>> >> > >> >>>> For additional commands, e-mail: users-h...@wicket.apache.org
>> >> > >> >>>>
>> >> > >> >>>>
>> >> > >> >>>
>> >> > >> >>>
>> >> > ---------------------------------------------------------------------
>> >> > >> >>> To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
>> >> > >> >>> For additional commands, e-mail: users-h...@wicket.apache.org
>> >> > >> >>>
>> >> > >> >>>
>> >> > >> >>
>> >> > >> >
>> >> > >>
>> >> > >>
>> ---------------------------------------------------------------------
>> >> > >> To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
>> >> > >> For additional commands, e-mail: users-h...@wicket.apache.org
>> >> > >>
>> >> > >>
>> >> > >
>> >> > >
>> >> > > --
>> >> > > 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: users-unsubscr...@wicket.apache.org
>> >> > For additional commands, e-mail: users-h...@wicket.apache.org
>> >> >
>> >> >
>> >>
>> >
>> >
>> >
>> > --
>> > 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: users-unsubscr...@wicket.apache.org
>> For additional commands, e-mail: users-h...@wicket.apache.org
>>
>>
>
>
> --
> 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: users-unsubscr...@wicket.apache.org
For additional commands, e-mail: users-h...@wicket.apache.org
