> -----Original Message----- > From: Bruno Dumon [mailto:[EMAIL PROTECTED] > Sent: vrijdag 21 november 2003 21:45 > To: [EMAIL PROTECTED] > Subject: RE: Woody: selection list problem > > > On Fri, 2003-11-21 at 20:37, Danny Bols wrote: > > > -----Original Message----- > > > From: Bruno Dumon [mailto:[EMAIL PROTECTED] > > > Sent: vrijdag 21 november 2003 17:35 > > > To: [EMAIL PROTECTED] > > > Subject: Re: Woody: selection list problem > > > > > > > > > On Fri, 2003-11-21 at 10:31, Danny Bols wrote: > > > > Hello, > > > > > > > > when trying to populate a selection list from within flow > > > script I ran into > > > > a problem. The following piece of code did not work: > > > > > > > > function myform(form) { > > > > var widget = form.form.getWidget("myfield"); > > > > var selectionList = new > > > > > > > Packages.org.apache.cocoon.woody.datatype.StaticSelectionList(widg > > > et.getData > > > > type()); > > > > selectionList.addItem("AL", "Alabama"); > > > > selectionList.addItem("AK", "Alaska"); > > > > widget.setSelectionList(selectionList); > > > > form.showForm("flow/myform.form"); > > > > } > > > > > > Well, if you would have read the javadoc of that class and method ... > > > > mmhh...I thought I read almost everything.....almost :-) > > > > > > > Anyway, this code changed recently, if you update to current CVS you > > > should be able to make it work with something like this: > > > > > > selectionList.addItem("AL", new > > > Packages.org.apache.cocoon.woody.util.StringMessage("Alabama")); > > > > Would it be a good idea to create a convenience method which accepts 2 > > string objects? > > A convenience method which takes a string for the second argument (the > label) would be a good idea. The first argument, the value itself, can > really be any object type so that doesn't matter.
Can you explain to me why the first parameter has to be an object? > What is needed though > is checking that the type of the value corresponds to the the type of > the datatype. Can be done using the isAssignableFrom method on the class > objects. > > It's just a couple of lines of code, but if you could provide a (tested) > patch that would be great. OK, I look what I can do. -- Danny
