You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by kn...@apache.org on 2008/04/21 23:32:38 UTC
svn commit: r650277 -
/wicket/trunk/wicket/src/main/java/org/apache/wicket/model/Model.java
Author: knopp
Date: Mon Apr 21 14:32:36 2008
New Revision: 650277
URL: http://svn.apache.org/viewvc?rev=650277&view=rev
Log:
detach model object when model is detached
Modified:
wicket/trunk/wicket/src/main/java/org/apache/wicket/model/Model.java
Modified: wicket/trunk/wicket/src/main/java/org/apache/wicket/model/Model.java
URL: http://svn.apache.org/viewvc/wicket/trunk/wicket/src/main/java/org/apache/wicket/model/Model.java?rev=650277&r1=650276&r2=650277&view=diff
==============================================================================
--- wicket/trunk/wicket/src/main/java/org/apache/wicket/model/Model.java (original)
+++ wicket/trunk/wicket/src/main/java/org/apache/wicket/model/Model.java Mon Apr 21 14:32:36 2008
@@ -116,6 +116,10 @@
*/
public void detach()
{
+ if (object instanceof IDetachable)
+ {
+ ((IDetachable)object).detach();
+ }
}
/**
Re: svn commit: r650277 - /wicket/trunk/wicket/src/main/java/org/apache/wicket/model/Model.java
Posted by Johan Compagner <jc...@gmail.com>.
yes i think so, but i think matej ordered me to do it
Will look at it tonight
johan
On Sun, Apr 27, 2008 at 4:28 PM, Martijn Dashorst <
martijn.dashorst@gmail.com> wrote:
> Should this also go into wicket-1.3.x?
>
> Martijn
>
> On 4/21/08, knopp@apache.org <kn...@apache.org> wrote:
> > Author: knopp
> > Date: Mon Apr 21 14:32:36 2008
> > New Revision: 650277
> >
> > URL: http://svn.apache.org/viewvc?rev=650277&view=rev
> > Log:
> > detach model object when model is detached
> >
> > Modified:
> > wicket/trunk/wicket/src/main/java/org/apache/wicket/model/Model.java
> >
> > Modified:
> wicket/trunk/wicket/src/main/java/org/apache/wicket/model/Model.java
> > URL:
> http://svn.apache.org/viewvc/wicket/trunk/wicket/src/main/java/org/apache/wicket/model/Model.java?rev=650277&r1=650276&r2=650277&view=diff
> >
> ==============================================================================
> > ---
> wicket/trunk/wicket/src/main/java/org/apache/wicket/model/Model.java
> (original)
> > +++
> wicket/trunk/wicket/src/main/java/org/apache/wicket/model/Model.java Mon Apr
> 21 14:32:36 2008
> > @@ -116,6 +116,10 @@
> > */
> > public void detach()
> > {
> > + if (object instanceof IDetachable)
> > + {
> > + ((IDetachable)object).detach();
> > + }
> > }
> >
> > /**
> >
> >
> >
>
>
> --
> Buy Wicket in Action: http://manning.com/dashorst
> Apache Wicket 1.3.3 is released
> Get it now: http://www.apache.org/dyn/closer.cgi/wicket/1.3.3
>
Re: svn commit: r650277 - /wicket/trunk/wicket/src/main/java/org/apache/wicket/model/Model.java
Posted by Martijn Dashorst <ma...@gmail.com>.
Should this also go into wicket-1.3.x?
Martijn
On 4/21/08, knopp@apache.org <kn...@apache.org> wrote:
> Author: knopp
> Date: Mon Apr 21 14:32:36 2008
> New Revision: 650277
>
> URL: http://svn.apache.org/viewvc?rev=650277&view=rev
> Log:
> detach model object when model is detached
>
> Modified:
> wicket/trunk/wicket/src/main/java/org/apache/wicket/model/Model.java
>
> Modified: wicket/trunk/wicket/src/main/java/org/apache/wicket/model/Model.java
> URL: http://svn.apache.org/viewvc/wicket/trunk/wicket/src/main/java/org/apache/wicket/model/Model.java?rev=650277&r1=650276&r2=650277&view=diff
> ==============================================================================
> --- wicket/trunk/wicket/src/main/java/org/apache/wicket/model/Model.java (original)
> +++ wicket/trunk/wicket/src/main/java/org/apache/wicket/model/Model.java Mon Apr 21 14:32:36 2008
> @@ -116,6 +116,10 @@
> */
> public void detach()
> {
> + if (object instanceof IDetachable)
> + {
> + ((IDetachable)object).detach();
> + }
> }
>
> /**
>
>
>
--
Buy Wicket in Action: http://manning.com/dashorst
Apache Wicket 1.3.3 is released
Get it now: http://www.apache.org/dyn/closer.cgi/wicket/1.3.3