Sorry, It's working. I remember getting an error of nullField sometime ago and therefore stopped using this version of findList and started passing a list from screen using <entity-condition>.
On Fri, May 31, 2013 at 6:50 PM, Jacques Le Roux < [email protected]> wrote: > I just tried by adding > <#assign orderHeaders = delegator.findList("OrderHeader", null, null, > null, null, false)/> > orderHeaders=========${orderHeaders} > > to OrderSalesReps.ftl in trunk locally, works perfectly > > Which release.revision are you using? > > Jacques > > From: "Deepak Agarwal" <[email protected]> > >I might have missed something but findList("EntityName", null, null, null, > > null, false) doesn't work for me from ftl because of 'null' . > > > > On Fri, May 31, 2013 at 1:13 AM, Jacques Le Roux < > > [email protected]> wrote: > > > >> Last pragmatic cons note for findAll("EntityName"), > >> instead of findList("EntityName", null, null, null, null, false) > >> > >> It's much easier to remember when you have to use it in envs where there > >> is no auto-completer, like Groovy scripts in Eclipse. > >> > >> This would mean to revert r938947 and I guess some others commits, not > >> sure all will agree... > >> > >> If ever we agree on this. Are there other high level methods (like > >> findAll("EntityName"), ie w/o parameters just EntityName) we would > >> reintroduce? > >> > >> Jacques > >> > >> Jacques Le Roux wrote: > >> > About findAll(), yes it's a moot point > >> > > >> > David got into this direction 5 years ago and nobody was really > against > >> it > >> > http://markmail.org/message/thcsl7khketkqagh > >> > > >> > One of the ideas was to get rif of all the variations and use rather > >> parameters, take for instance > >> > http://svn.apache.org/viewvc?view=revision&revision=938948 > >> > > >> > IMO, Jonathan had a good point about owen buttons in the thread above > >> > One thing we would consider is reintroducing most simplifed ones, like > >> findAll(), yes! > >> > > >> > But maybe Scott would be agains it > >> > http://svn.apache.org/viewvc?view=revision&revision=938947 > >> > > >> > Opinions? > >> > > >> > Jacques > >> > > >> > From: "Skip" <[email protected]> > >> >> Jacques > >> >> > >> >> That works fine. findAll() works for me because I added it back in. > >> >> findAll() is a better function to use because in my opinion, it sez > >> exactly > >> >> what it does and that makes the code more readable. I have hundreds > of > >> java > >> >> files written over the years and it was way easier to add back in the > >> >> functionality you guys deprecated/removed than to replace all my > entity > >> >> engine calls. > >> >> > >> >> This time however, I put them all together so it would be easier next > >> time > >> >> you guys do a major release. > >> >> > >> >> BTW, I have had a few weeks now to go pretty thoroughly through lots > of > >> this > >> >> code and it is very nice. I am especially pleased with the re-write > of > >> the > >> >> widget code and the use of jquery. > >> >> > >> >> Skip > >> >> > >> >> -----Original Message----- > >> >> From: Jacques Le Roux [mailto:[email protected]] > >> >> Sent: Thursday, May 30, 2013 6:38 AM > >> >> To: [email protected] > >> >> Subject: Re: But in CategoryTree.groovy > >> >> > >> >> > >> >> Thanks for report Skip, > >> >> > >> >> BTW you must have missed something, findAll does not exist anymore > for 3 > >> >> years ;) > >> >> http://svn.apache.org/viewvc?view=revision&revision=938947 > >> >> > >> >> Using > >> >> prodCatalogs = delegator.findList("ProdCatalog", null, null, null, > null, > >> >> false); > >> >> Should be the way. Could you please confirm before I commit? > >> >> > >> >> Thanks > >> >> > >> >> Jacques > >> >> > >> >> From: "Skip" <[email protected]> > >> >>> There is a line at 50 that sez: > >> >>> > >> >>> prodCatalogs = delegator.findByAnd("ProdCatalog"); > >> >>> > >> >>> This causes a groovy ambiguious error when you go to the catalog > >> screen in > >> >>> 12.04. > >> >>> > >> >>> Changing this to: > >> >>> prodCatalogs = delegator.findAll("ProdCatalog"); > >> >>> > >> >>> Fixes the problem. > >> >>> > >> >>> I expect you guys already did this, there here it is for any others > >> like > >> >> me > >> >>> that just download the zip file. > >> >>> > >> >>> Skip > >> > > > > > > > > -- > > Thanks, > > Deepak Agarwal, > > > > Mobile: +91 9501190044 > > > -- Thanks, Deepak Agarwal, Mobile: +91 9501190044
