tapestry-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Davor Hrg" <hrgda...@gmail.com>
Subject Re: [jira] Created: (TAPESTRY-2116) @Retain and @Persist should generate a setter and/or getter for the decorated field
Date Mon, 04 Feb 2008 09:33:12 GMT
The separate @Property annotation still has issues in combination with @Persist

@Persist replaces all write access with a write_method...
and the new setter uses the field so persist misses it...

adding claimField seemed to fix thing, but did not, because
the values was simply retained...


Davor Hrg

On Feb 4, 2008 8:54 AM, Davor Hrg <hrgdavor@gmail.com> wrote:
> :):):):)
>
> haven't noticed it :D :D
>
> since it is an JIRA issue,
> I'll just create a patch since I belive also that this should alerady
> be part of the core
>
> Davor Hrg
>
>
>
> On Feb 3, 2008 9:46 PM, Kevin Menard <kmenard@servprise.com> wrote:
> > I don't have anything important to add here.  I just thought it funny that
> > Davor pointed Sven to his own article.
> >
> > Davor, maybe you could update the article with your contributions?
> >
> > --
> > Kevin
> >
> >
> > On 2/3/08 12:26 PM, in article
> > 84d1dff00802030926i94bc5f3l54ad159f004289b2@mail.gmail.com, "Davor Hrg"
> >
> > <hrgdavor@gmail.com> wrote:
> >
> > > it is trivial to make a third @Property worker
> > > that adds getter and setter ...
> > > so you have
> > >
> > > @Persist @Property private String name;
> > >
> > >
> > > I ve made such worker for my self easily following this wiki article:
> > > http://wiki.apache.org/tapestry/Tapestry5ManipulateClassesAtRuntime
> > >
> > > with difference ... that I used
> > >
> > > configuration.add("GridRowBean", new GridRowBeanWorker(),
> > > "after:Inject*","before:Persist");
> > > instead of:
> > > configuration.add("GridRowBean", new GridRowBeanWorker(), "after:Inject*")
> > >
> > >
> > > Davor Hrg
> > >
> > >
> > > On Feb 3, 2008 10:48 AM, Sven Homburg (JIRA) <dev@tapestry.apache.org>
wrote:
> > >> @Retain and @Persist should generate a setter and/or getter for the decorated
> > >> field
> > >> -----------------------------------------------------------------------------
> > >> ------
> > >>
> > >>                  Key: TAPESTRY-2116
> > >>                  URL: https://issues.apache.org/jira/browse/TAPESTRY-2116
> > >>              Project: Tapestry
> > >>           Issue Type: New Feature
> > >>           Components: tapestry-core
> > >>             Reporter: Sven Homburg
> > >>              Fix For: 5.0.10
> > >>
> > >>
> > >> sometime code looks incredible unclear, because its peppered with getter
and
> > >> setter methods.
> > >> i think it should be ease to extend the Persist- and RetainWorker for doing
> > >> that job.
> > >>
> > >> @Retain(getter=true, setter=true)
> > >>
> > >> as an exmaple
> > >>
> > >>
> > >> --
> > >> This message is automatically generated by JIRA.
> > >> -
> > >> You can reply to this email to add a comment to the issue online.
> > >>
> > >>
> > >> ---------------------------------------------------------------------
> > >> To unsubscribe, e-mail: dev-unsubscribe@tapestry.apache.org
> > >> For additional commands, e-mail: dev-help@tapestry.apache.org
> > >>
> > >>
> >
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@tapestry.apache.org
> > For additional commands, e-mail: dev-help@tapestry.apache.org
> >
> >
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tapestry.apache.org
For additional commands, e-mail: dev-help@tapestry.apache.org


Mime
View raw message