I published a different fix that does not use resumable exceptions,
but instead asks the parent if there are any errors to report
Magritte-Seaside-lr.286.
Lukas
On Mon, Mar 23, 2009 at 11:37 PM, Keith Hodges <keith_hodges(a)yahoo.co.uk> wrote:
Lukas Renggli wrote:
>> I found that MAInternalEditorComponent is doing this
>>
>> renderEditorOn: html
>> super renderEditorOn: html.
>> html hiddenInput callback: [ self component commit ]
>>
>> Am I correct in thinking that this means that the commit to the model is
>> occurring every time the form submits? I thought that the commit should
>> be to the memento, when validation etc has passed.
>>
> Yeah, looks suspicious. I committed a fix for this particular issue in
> Magritte-Seaside-lr.285.
>
> Note, that I rarely use the to-many relationship descriptions, so that
> there might be more bugs lingering around ...
>
One or two ;-)
I adopted your fix to see what difference it makes. Looks good so far.
I have posted fixes to two further issues.
Controls one level down i.e.
Container(1)>>Control=Container(2)>>Control
dont have access to the errors list of container (1) in order to display
class: 'error', in #forClass:
my solution uses a Notification.
Validation in the component heirarchy raises non-resumable errors,
whereas the model validation raises resumable errors resulting in wierd
behaviour.
Again I uploaded a fix for this
Keith
_______________________________________________
SmallWiki, Magritte, Pier and Related Tools ...
https://www.iam.unibe.ch/mailman/listinfo/smallwiki
--
Lukas Renggli
http://www.lukas-renggli.ch