Initialize the field widget manager in the update-step, such that the…

… widget update step is only responsible for actually updating the widgets. The change is required to support the situation where you want to change the common widget prefix (which defaults to 'widgets.').

CHANGES.txt

@@ -2,6 +2,16 @@
 CHANGES
 =======
 
+In next release ...
+
+- Initialize widgets in ``update`` step. The ``updateWidgets`` method
+  is now responsible only for actually updating the widgets.
+
+  This allows updating the common widgets prefix before the individual
+  widgets are updated, useful for situations where neither a form, nor
+  a widgets prefix is desired.
+
+
 2.8.3 (unreleased)
 ------------------
 

src/z3c/form/form.py

@@ -126,8 +126,6 @@ def getContent(self):
 
     def updateWidgets(self):
         '''See interfaces.IForm'''
-        self.widgets = zope.component.getMultiAdapter(
-            (self, self.request, self.getContent()), interfaces.IWidgets)
         self.widgets.mode = self.mode
         self.widgets.ignoreContext = self.ignoreContext
         self.widgets.ignoreRequest = self.ignoreRequest
@@ -148,6 +146,8 @@ def extractData(self, setErrors=True):
 
     def update(self):
         '''See interfaces.IForm'''
+        self.widgets = zope.component.getMultiAdapter(
+            (self, self.request, self.getContent()), interfaces.IWidgets)
         self.updateWidgets()
 
     def render(self):

src/z3c/form/group.py

@@ -34,8 +34,6 @@ def __init__(self, context, request, parentForm):
 
     def updateWidgets(self):
         '''See interfaces.IForm'''
-        self.widgets = zope.component.getMultiAdapter(
-            (self, self.request, self.getContent()), interfaces.IWidgets)
         for attrName in ('mode', 'ignoreRequest', 'ignoreContext',
                          'ignoreReadonly'):
             value = getattr(self.parentForm.widgets, attrName)
@@ -44,7 +42,7 @@ def updateWidgets(self):
 
     def update(self):
         '''See interfaces.IForm'''
-        self.updateWidgets()
+        super(Group, self).update()
         groups = []
         for groupClass in self.groups:
             # only instantiate the groupClass if it hasn't already
@@ -121,9 +119,10 @@ def applyChanges(self, data):
 
         return changed
 
-    def update(self):
+    def updateWidgets(self):
         '''See interfaces.IForm'''
-        self.updateWidgets()
+        super(GroupForm, self).updateWidgets()
+
         groups = []
         for groupClass in self.groups:
             # only instantiate the groupClass if it hasn't already
@@ -135,7 +134,3 @@ def update(self):
             group.update()
             groups.append(group)
         self.groups = tuple(groups)
-        self.updateActions()
-        self.actions.execute()
-        if self.refreshActions:
-            self.updateActions()