Re: svn commit: r667448 - /ofbiz/trunk/framework/example/webapp/example/WEB-INF/actions/includes/findExampleFeatures.groovy

2008-06-13 Thread Ashish Vijaywargiya
Jacques, Everything looks perfect to me. +1. On Fri, Jun 13, 2008 at 5:55 AM, [EMAIL PROTECTED] wrote: Author: jleroux Date: Fri Jun 13 02:55:27 2008 New Revision: 667448 URL: http://svn.apache.org/viewvc?rev=667448view=rev Log: My 1st bsh-groovy migration, please check. I will change

Re: svn commit: r667448 - /ofbiz/trunk/framework/example/webapp/example/WEB-INF/actions/includes/findExampleFeatures.groovy

2008-06-13 Thread Scott Gray
Hi Jacques Looks good except for one thing: +autocompleteOptions = delegator.findList(ExampleFeature, entityConditionList, +new TreeSet(['exampleFeatureId', 'description'], ['-exampleFeatureId'], null, false); Introduces a bug, there is no closing bracket for the TreeSet. Also

Re: svn commit: r667448 - /ofbiz/trunk/framework/example/webapp/example/WEB-INF/actions/includes/findExampleFeatures.groovy

2008-06-13 Thread Ashish Vijaywargiya
Oops I missed closing braces :( Thanks Scott for catching that. -- Ashish On Fri, Jun 13, 2008 at 6:04 AM, Scott Gray [EMAIL PROTECTED] wrote: Hi Jacques Looks good except for one thing: +autocompleteOptions = delegator.findList(ExampleFeature, entityConditionList, +new

Re: svn commit: r667448 - /ofbiz/trunk/framework/example/webapp/example/WEB-INF/actions/includes/findExampleFeatures.groovy

2008-06-13 Thread Jacques Le Roux
Thanks for the Set tip Scoot, Jacques From: Scott Gray [EMAIL PROTECTED] Hi Jacques Looks good except for one thing: +autocompleteOptions = delegator.findList(ExampleFeature, entityConditionList, +new TreeSet(['exampleFeatureId', 'description'], ['-exampleFeatureId'], null,

Re: svn commit: r667448 - /ofbiz/trunk/framework/example/webapp/example/WEB-INF/actions/includes/findExampleFeatures.groovy

2008-06-13 Thread Vikas Mayur
Jacques, Looks like Scott name is in french :) Vikas On Fri, Jun 13, 2008 at 6:54 PM, Jacques Le Roux [EMAIL PROTECTED] wrote: Thanks for the Set tip Scoot, Jacques From: Scott Gray [EMAIL PROTECTED] Hi Jacques Looks good except for one thing: +autocompleteOptions =

Re: svn commit: r667448 - /ofbiz/trunk/framework/example/webapp/example/WEB-INF/actions/includes/findExampleFeatures.groovy

2008-06-13 Thread Ashish Vijaywargiya
I think it looks like Windows Side effect :-) -- Ashish On Fri, Jun 13, 2008 at 9:52 AM, Vikas Mayur [EMAIL PROTECTED] wrote: Jacques, Looks like Scott name is in french :) Vikas On Fri, Jun 13, 2008 at 6:54 PM, Jacques Le Roux [EMAIL PROTECTED] wrote: Thanks for the Set tip Scoot,

Re: svn commit: r667448 - /ofbiz/trunk/framework/example/webapp/example/WEB-INF/actions/includes/findExampleFeatures.groovy

2008-06-13 Thread Jacques Le Roux
Yes sorry Scott, This time it's more my fat fingers than Windows ;o) Jacques PS : Actually I just bought a Dell 530 with Ubuntu (really affordable) , but I can't find the time to migrate all to it. I guess at 1st I will simply plug my disks. It's a pity because I have completly enough of