Changeset 69585ad in tailbone-corepos

Timestamp:

08/02/2020 09:28:53 PM (3 years ago)

Author:

Lance Edgar <lance@…>

Branches:

master

Children:

fdb63fc

Parents:

acf6764

Message:

Fix handling of corepos_account_id field when editing customers

File:

• tailbone_corepos/views/customers.py (modified) (1 diff)

tailbone_corepos/views/customers.py

@@ -54 +54 @@
         g.set_filter('corepos_account_id', model.CoreCustomer.corepos_account_id)
 
+    def configure_form(self, f):
+        super(CustomerView, self).configure_form(f)
+        f.set_required('corepos_account_id', False)
+
+    def objectify(self, form, data=None):
+        if data is None:
+            data = form.validated
+        customer = form.model_instance
+
+        # this field lives in an extension table, but the column does not allow
+        # null, which means we don't want to pass an empty value along unless
+        # there is already an extension record in place for this customer
+        if 'corepos_account_id' in data and data['corepos_account_id'] is None:
+            if self.creating:
+                data.pop('corepos_account_id')
+            elif self.editing and not customer._corepos:
+                data.pop('corepos_account_id')
+
+        return super(CustomerView, self).objectify(form, data)
+
     def get_version_child_classes(self):
         model = self.rattail_config.get_model()