[html-formfu] model-config "read_only" for Checkboxgroup

[Wolfgang Kinkeldei]

Hi Carl,

Thanks for your quick reply. After my first thoughts we figured out that the best way would be to add an additional check for model_config->{read_only} during a save operation from checkboxgroup fields.

I prepared the patch and expanded a test. A pull request is on the way. I hope you find the patch OK.

Am 02.12.2011 um 11:44 schrieb Carl Franks:

> Hi,
> If you could submit a test file that uses "no_follow", it'll ensure it
> does't get broken or removed.
> A github pull request is preferred.
> Otherwise, submit a bug report stating that it isn't documented or
> tested - that'll at least keep it on the radar.
> Carl
> 
> 
> On 2 December 2011 10:35, Wolfgang Kinkeldei <wolfgang at kinkeldei.de> wrote:
>> Hello,
>> 
>> during the last few days we had some trouble getting certain things working. The main problem was that
>>    model_config => { read_only => 1 }
>> 
>> is not honored during saving relations, only for "simple" fields. Is there something we missed?
>> 
>> Also, inside the private method '_save_relationships' inside HTML::FormFu::Model::DBIC we discovered
>>    return if $attrs->{no_follow};
>> 
>> which is not documented but does what we wanted but as long as it is not documented, the danger of breaking our code due to updates is high.
>> 
>> Does anybody have a hint for me?
>> 
>> 
>> Grüße,
>> 
>> Wolfgang Kinkeldei
>> 
>> --
>> 
>> ' /\_/\ ' .print[split??,"".(($/=q|Cms)+-03467:;<=|)=~tr!C-z -B! -z!)x
>> '( o.o )' .$/]->[hex]foreach split qr<>,qq+1ecd039ad65b025b8063475b+||
>> ' > ^ < ' .q<!-- Wolfgang Kinkeldei - wolfgang at kinkeldei dot de -->
>> 
>> 
>> _______________________________________________
>> HTML-FormFu mailing list
>> HTML-FormFu at lists.scsys.co.uk
>> http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/html-formfu
> 
> _______________________________________________
> HTML-FormFu mailing list
> HTML-FormFu at lists.scsys.co.uk
> http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/html-formfu



Grüße,

Wolfgang Kinkeldei

-- 

' /\_/\ ' .print[split//,"".(($/=q|Cms)+-03467:;<=|)=~tr!C-z -B! -z!)x
'( o.o )' .$/]->[hex]foreach split qr<>,qq+1ecd039ad65b025b8063475b+||
' > ^ < ' .q<!-- Wolfgang Kinkeldei - wolfgang at kinkeldei dot de -->