You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by er...@apache.org on 2010/06/14 18:07:17 UTC

svn commit: r954534 - /ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelForm.java

Author: erwan
Date: Mon Jun 14 16:07:16 2010
New Revision: 954534

URL: http://svn.apache.org/viewvc?rev=954534&view=rev
Log:
As spotted by Scott on the dev mailing-list, this is a better writing

Modified:
    ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelForm.java

Modified: ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelForm.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelForm.java?rev=954534&r1=954533&r2=954534&view=diff
==============================================================================
--- ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelForm.java (original)
+++ ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelForm.java Mon Jun 14 16:07:16 2010
@@ -1989,8 +1989,7 @@ public class ModelForm extends ModelWidg
         Locale locale = UtilMisc.ensureLocale(context.get("locale"));
         MapStack<String> localContext = MapStack.create(context);
 
-        FlexibleStringExpander containerIdExpander = FlexibleStringExpander.getInstance(this.getContainerId());
-        String retVal = containerIdExpander.expandString(localContext, locale);
+        String retVal = FlexibleStringExpander.expandString(this.getContainerId(), localContext, locale); 
 
         Integer itemIndex = (Integer) context.get("itemIndex");
         if (itemIndex != null && "list".equals(this.getType())) {



Re: svn commit: r954534 - /ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelForm.java

Posted by Erwan de FERRIERES <er...@nereide.fr>.
Le 14/06/2010 18:31, Jacques Le Roux a écrit :
> It was Adrian ;o)
>
> Jacques
Thanks Jacques, and Sorry for that Adrian and Scott...
Commit log is now corrected !

Cheers,

-- 
Erwan de FERRIERES
www.nereide.biz

Re: svn commit: r954534 - /ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelForm.java

Posted by Jacques Le Roux <ja...@les7arts.com>.
It was Adrian ;o)

Jacques

From: <er...@apache.org>
> Author: erwan
> Date: Mon Jun 14 16:07:16 2010
> New Revision: 954534
>
> URL: http://svn.apache.org/viewvc?rev=954534&view=rev
> Log:
> As spotted by Scott on the dev mailing-list, this is a better writing
>
> Modified:
>    ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelForm.java
>
> Modified: ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelForm.java
> URL: 
> http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelForm.java?rev=954534&r1=954533&r2=954534&view=diff
> ==============================================================================
> --- ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelForm.java (original)
> +++ ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelForm.java Mon Jun 14 16:07:16 2010
> @@ -1989,8 +1989,7 @@ public class ModelForm extends ModelWidg
>         Locale locale = UtilMisc.ensureLocale(context.get("locale"));
>         MapStack<String> localContext = MapStack.create(context);
>
> -        FlexibleStringExpander containerIdExpander = FlexibleStringExpander.getInstance(this.getContainerId());
> -        String retVal = containerIdExpander.expandString(localContext, locale);
> +        String retVal = FlexibleStringExpander.expandString(this.getContainerId(), localContext, locale);
>
>         Integer itemIndex = (Integer) context.get("itemIndex");
>         if (itemIndex != null && "list".equals(this.getType())) {
>
>