Changeset 6273 for main/waeup.sirp/trunk/src/waeup/sirp/utils/converters.py

Timestamp:

4 Jun 2011, 02:29:21 (13 years ago)

Author:

uli

Message:

Finally make the new converter work. API-wise it is as good as the old one (can import everyting, the old one could),
but design-wise it might be much more powerfull. Basically it can handle/convert all content-types for which one can
create an Add- or EditForm? successfully. In other words: if you manage to write an edit form for some content type,
then you can also create an importer for that content-type. Still finetuning needed (for dates, bool data, etc.) but
the main things work.

File:

• main/waeup.sirp/trunk/src/waeup/sirp/utils/converters.py (modified) (3 diffs)

main/waeup.sirp/trunk/src/waeup/sirp/utils/converters.py

@@ -245 +245 @@
 from zope.formlib import form
 from zope.formlib.form import (
-    _widgetKey, WidgetInputError, ValidationError, InputErrors, expandPrefix)
+    _widgetKey, WidgetInputError, ValidationError, InputErrors, expandPrefix,
+    setUpInputWidgets)
 from zope.formlib.interfaces import IInputWidget
 from zope.publisher.browser import TestRequest
+from waeup.sirp.interfaces import IObjectConverter
 
 def getWidgetsData(widgets, form_prefix, data):
+    """Get data and validation errors from `widgets` for `data`.
+
+    Updates the dict in `data` with values from the widgets in
+    `widgets`.
+
+    Returns a list of tuples ``(<WIDGET_NAME>, <ERROR>)`` where
+    ``<WIDGET_NAME>`` is a widget name (normally the same as the
+    associated field name) and ``<ERROR>`` is the exception that
+    happened for that widget/field.
+
+    This is merely a copy from the same-named function in
+    :mod:`zope.formlib.form`. The only difference is that we also
+    store the fieldname for which a validation error happened in the
+    returned error list (what the original does not do).
+
+    """
     errors = []
     form_prefix = expandPrefix(form_prefix)
@@ -271 +289 @@
     return errors
 
-
-class IObjectConverter(Interface):
-    def __init__(iface):
-        """Create an converter.
-
-        `iface` denotes the interface to which we want to turn any
-        passed object.
-
-        """
-
-    def applyRowData(data_dict, context, form_fields=None):
-        """Apply data in `data_dict` to `context`.
-
-        `data_dict` is a dict containing field names as keys and an
-        object or string as `context`.
-
-        If `context` is a string, this is understood as a factory name
-        and we will try to create a proper object calling
-        ``createObject()``.
-
-        `form_fields` are by default (``None``) buildt from the given
-        `iface` but can also be passed in to override the
-        default. This might be handy if you want to omit or select
-        certains fields from the interface.
-
-        Returns a tuple ``(<ERROR_LIST, INV_ERR_LIST, OBJ>)`` where
-        ``ERROR_DICT`` is a dict of errors for single fields (if
-        happened), ``INV_ERR_LIST`` is a list of invariant errors
-        happened (errors that apply to several fields), and ``OBJ`` is
-        the created/updated object.
-        """
-
 class DefaultObjectConverter(grok.Adapter):
-    """An object converter can apply CSV data to objects.
-
-    Thus, in a way, it can turn CSV data into real objects.
+    """Turn string values into real values.
+
+    A converter can convert string values for objects that implement a
+    certain interface into real values based on the given interface.
     """
 
@@ -314 +301 @@
     def __init__(self, iface):
         self.iface = iface
-        self.form_fields = form.Fields(iface)
+        self.default_form_fields = form.Fields(iface)
         return
 
-    def applyRowData(self, data_dict, context, form_fields=None):
+    def fromStringDict(self, data_dict, context, form_fields=None):
+        """Convert values in `data_dict`.
+
+        Converts data in `data_dict` into real values based on
+        `context` and `form_fields`.
+
+        `data_dict` is a mapping (dict) from field names to values
+        represented as strings.
+
+        The fields (keys) to convert can be given in optional
+        `form_fields`. If given, form_fields should be an instance of
+        :class:`zope.formlib.form.Fields`. Suitable instances are for
+        example created by :class:`grok.AutoFields`.
+
+        If no `form_fields` are given, a default is computed from the
+        associated interface.
+
+        The `context` can be an existing object (implementing the
+        associated interface) or a factory name. If it is a string, we
+        try to create an object using
+        :func:`zope.component.createObject`.
+
+        Returns a tuple ``(<FIELD_ERRORS>, <INVARIANT_ERRORS>,
+        <DATA_DICT>)`` where
+
+        ``<FIELD_ERRORS>``
+           is a list of tuples ``(<FIELD_NAME>, <ERROR>)`` for each
+           error that happened when validating the input data in
+           `data_dict`
+
+        ``<INVARIANT_ERRORS>``
+           is a list of invariant errors concerning several fields
+
+        ``<DATA_DICT>``
+           is a dict with the values from input dict converted.
+
+        If errors happen, i.e. the error lists are not empty, always
+        an empty ``<DATA_DICT>`` is returned.
+
+        If ``<DATA_DICT>` is non-empty, there were no errors.
+        """
         if form_fields is None:
-            form_fields = self.form_fields
+            form_fields = self.default_form_fields
+
+        request = TestRequest(form={})
+        for key, val in data_dict.items():
+            request.form['form.%s' % key] = val
 
         obj = context
         if isinstance(context, basestring):
             obj = createObject(context)
-        request = TestRequest(form={})
-        for key, val in data_dict.items():
-            request.form['form.%s' % key] = val
-        widgets = form.setUpWidgets(
+
+        widgets = form.setUpInputWidgets(
             form_fields, 'form', obj, request)
-        errors = getWidgetsData(widgets, 'form', data_dict)
-        err_messages = []
-        if errors:
-            for key, error in errors:
-                message = error.args[0]
-                err_messages.append((key, message))
-        invariant_errors = form.checkInvariants(form_fields, data_dict)
-        invariant_errors = [err.message for err in invariant_errors]
-        if not errors and not invariant_errors:
-            changed = form.applyChanges(
-                obj, form_fields, data_dict)
-        return err_messages, invariant_errors, obj
+
+        new_data = dict()
+        errors = getWidgetsData(widgets, 'form', new_data)
+
+        invariant_errors = form.checkInvariants(form_fields, new_data)
+        if errors or invariant_errors:
+            err_messages = [(key, err.args[0]) for key, err in errors]
+            invariant_errors = [err.message for err in invariant_errors]
+            return err_messages, invariant_errors, {}
+
+        return errors, invariant_errors, new_data