You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cocoon.apache.org by Max Pfingsthorn <m....@hippo.nl> on 2006/01/26 22:49:26 UTC
RE: svn commit: r372535 - /cocoon/trunk/cocoon-forms/cocoon-forms-impl/src/main/java/org/apache/cocoon/forms/samples/bindings/CustomValueWrapBinding.java
Hi!
Thanks for fixing this so fast!
It should have gone into the AbstractCustomBinding though, in order not to break other custom bindings. Libraries are of no interest to custom bindings anyway. Fixing this (and the typo) now... (making extra sure I don't break stuff again).
Bye!
max
> -----Original Message-----
> From: cziegeler@apache.org [mailto:cziegeler@apache.org]
> Sent: Thursday, January 26, 2006 16:00
> To: cvs@cocoon.apache.org
> Subject: svn commit: r372535 -
> /cocoon/trunk/cocoon-forms/cocoon-forms-impl/src/main/java/org
> /apache/co
> coon/forms/samples/bindings/CustomValueWrapBinding.java
>
>
> Author: cziegeler
> Date: Thu Jan 26 06:59:30 2006
> New Revision: 372535
>
> URL: http://svn.apache.org/viewcvs?rev=372535&view=rev
> Log:
> Fix compilation problems
>
> Modified:
>
> cocoon/trunk/cocoon-forms/cocoon-forms-impl/src/main/java/org/
> apache/cocoon/forms/samples/bindings/CustomValueWrapBinding.java
>
> Modified:
> cocoon/trunk/cocoon-forms/cocoon-forms-impl/src/main/java/org/
> apache/cocoon/forms/samples/bindings/CustomValueWrapBinding.java
> URL:
> http://svn.apache.org/viewcvs/cocoon/trunk/cocoon-forms/cocoon
> -forms-impl/src/main/java/org/apache/cocoon/forms/samples/bind
> ings/CustomValueWrapBinding.java?rev=372535&r1=372534&r2=37253
> 5&view=diff
> ==============================================================
> ================
> ---
> cocoon/trunk/cocoon-forms/cocoon-forms-impl/src/main/java/org/
> apache/cocoon/forms/samples/bindings/CustomValueWrapBinding.ja
> va (original)
> +++
> cocoon/trunk/cocoon-forms/cocoon-forms-impl/src/main/java/org/
> apache/cocoon/forms/samples/bindings/CustomValueWrapBinding.ja
> va Thu Jan 26 06:59:30 2006
> @@ -18,6 +18,7 @@
> import org.apache.cocoon.forms.binding.AbstractCustomBinding;
> import org.apache.cocoon.forms.binding.Binding;
> import org.apache.cocoon.forms.binding.BindingException;
> +import org.apache.cocoon.forms.binding.library.Library;
> import org.apache.cocoon.forms.formmodel.Widget;
> import org.apache.cocoon.forms.util.DomHelper;
> import org.apache.commons.jxpath.JXPathContext;
> @@ -96,4 +97,15 @@
> throw new BindingException("Could not create
> instance of CustomValueWrapBinding." ,e);
> }
> }
> +
> + public Library getEnclosingLibary() {
> + // TODO Auto-generated method stub
> + return null;
> + }
> +
> + public void setEnclosingLibary(Library lib) {
> + // TODO Auto-generated method stub
> +
> + }
> +
> }
>
>
>
Re: svn commit: r372535 - /cocoon/trunk/cocoon-forms/cocoon-forms-impl/src/main/java/org/apache/cocoon/forms/samples/bindings/CustomValueWrapBinding.java
Posted by Carsten Ziegeler <cz...@apache.org>.
Max Pfingsthorn schrieb:
> Hi!
>
> Thanks for fixing this so fast!
> It should have gone into the AbstractCustomBinding though,
> in order not to break other custom bindings. Libraries are
> of no interest to custom bindings anyway. Fixing this (and
> the typo) now... (making extra sure I don't break stuff again).
>
No problem, this happens to everyone from time to time :)
Carsten
--
Carsten Ziegeler - Open Source Group, S&N AG
http://www.s-und-n.de
http://www.osoco.org/weblogs/rael/