27 Jan
2010
27 Jan
'10
4:10 p.m.
Hi
Yesterday I rebuilt my image using the latest
Metacello ConfigurationOfSeaside30 (loadLatestVersion) and the latest code
in Pier2 and Magritte2 repositories. I've noticed a new problem when I
attempt to add a component using the Pier web ui which results in a form
validation error: 'Component Class: Invalid input given'
To reproduce:
* download latest build from:
http://hudson.lukas-renggli.ch/job/Seaside%203.0/
<http://hudson.lukas-renggli.ch/job/Seaside%203.0/>* open and start web
server
* open a workspace and execute:
PRPierFrame registerAsApplication: 'pier' kernel: (PRKernel named:
'defaultKernel').
* browse to http://localhost:8080/pier and select the edit link
* type: '+acomponent+' into the text in the box and press 'save'
* click on the 'acomponent' link and in the type select list choose
'Component' and press 'Add'
* select any class from the 'Component Class' select list and press
'save'
* note the form validation error: "Component Class: Invalid input given"
Investigation:
* MAKindError is being generated as the component selected is always
WAComponent regardless of the user selection.
* WAComponent is the default value ; the selected value is not being set as
the callback in MASelectListComponent>>renderOption:on: is never called.
This appears to be a Seaside bug; it appears that the callbacks for a
grouped select list are not called on form submission.
Further investigation:
* using the latest build browse to http://localhost:8080/magritte/editor
* select a type other than 'Boolean' from the grouped select box eg choose
'File' and press Add
* notice that whatever type you choose 'Boolean' is always selected.
* place a 'self halt' in the callback in
MASelectListComponent>>renderOption:on:
and notice that it is never called.
I noticed that there have been bug fixes on the select tag:
http://lists.squeakfoundation.org/pipermail/seaside/2010-January/022358.html
I made sure that the build I was using had the latest Seaside-Canvas
(Seaside-Canvas-lr.88.mcz). My previous image with working grouped selection
list was based on Seaside-Canvas-pmm.74.mcz
Hope I'm not reporting something that has already been fixed or noticed
elsewhere
Nick