Hidden inputs

>> 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 ...
>  

> 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 wrote:
>  
>> 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
>>
>>  
>>    
> Cool thanks,
>
> I thought that children asking parents was considered bad form.
>
> btw: The MenuWidget is clueless about lightbox links.
>
> Keith
>
>  

>> I have submitted a naive fix for the lightbox issue.
>>    
>
> That code should be part of the package Pier-Lightbox, not Pier-Seaside.
>
>  
>> How does one normally get back from the lightbox? (As in back to the
>> structure that you came from).
>>    
>
> Pressing the close button of the lightbox?
>
>