[Checkins] SVN: z3c.form/trunk/ Initialize the field widget manager

David Glick (Plone) david.glick at plone.org
Tue Oct 30 17:19:02 UTC 2012


On 10/30/12 7:12 AM, Malthe Borch wrote:
> On 30 October 2012 14:46, Roger <dev at projekt01.ch> wrote:
>> I do not understand what you are trying to solve. Can you describe
>> your Problem with the previous implementation?
> I'm trying to solve the problem of changing the widgets prefix in a
> reasonable way.
>
> Previously, the widgets component lookup and the update step was
> together, so you had no chance to adjust or set the prefix.
>
> With my change, ``updateWidgets`` does exactly that: update the
> existing widgets.
>
>> The widget setup in update is very bad because this now get
>> called in the super call order now.
> Why don't you just move the fields setup to your ``update`` method?
>
> I understand that it's an API change and that some implementations
> will need a migration, but it's relatively minor. You just need to
> shift your fields setup code to the ``update`` method.
>
>
I'm also -1 on the API change, which will break a lot of forms.



More information about the checkins mailing list