[IMP] connector_carepoint: Centralize model attrs

* Add default backend * Move created_at and updated_at to connector * Remove duplicated attribute definitions * Remove duplicated sql constraints
LasLabs · Aug 30, 2016 · 68a7602 · 68a7602
1 parent 80c92bc
commit 68a7602
Show file tree

Hide file tree

Showing 20 changed files with 17 additions and 264 deletions.
diff --git a/connector_carepoint/connector.py b/connector_carepoint/connector.py
@@ -37,16 +37,29 @@ class CarepointBinding(models.AbstractModel):
         comodel_name='carepoint.backend',
         string='Carepoint Backend',
         required=True,
+        readonly=True,
         ondelete='restrict',
+        default=lambda s: s._default_backend_id(),
     )
     # fields.Char because 0 is a valid Carepoint ID
     carepoint_id = fields.Char(string='ID on Carepoint')
+    created_at = fields.Date('Created At (on Carepoint)')
+    updated_at = fields.Date('Updated At (on Carepoint)')
 
     _sql_constraints = [
         ('carepoint_uniq', 'unique(backend_id, carepoint_id)',
          'A binding already exists with the same Carepoint ID.'),
     ]
 
+    @api.model
+    def _default_backend_id(self):
+        return self.env['easypost.backend'].search([
+            ('is_default', '=', True),
+            ('active', '=', True),
+        ],
+            limit=1,
+        )
+
 
 def add_checkpoint(session, model_name, record_id, backend_id):
     """ Add a row in the model ``connector.checkpoint`` for a record,

diff --git a/connector_carepoint/models/account.py b/connector_carepoint/models/account.py
@@ -29,14 +29,6 @@ class CarepointAccount(models.Model):
         required=True,
         ondelete='restrict',
     )
-    backend_id = fields.Many2one(
-        comodel_name='carepoint.backend',
-        string='Carepoint Backend',
-        store=True,
-        readonly=True,
-        # override 'carepoint.binding', can't be INSERTed if True:
-        required=False,
-    )
 
 
 class Account(models.Model):

diff --git a/connector_carepoint/models/account_invoice_line.py b/connector_carepoint/models/account_invoice_line.py
@@ -37,21 +37,6 @@ class CarepointAccountInvoiceLine(models.Model):
         required=True,
         ondelete='cascade'
     )
-    backend_id = fields.Many2one(
-        comodel_name='carepoint.backend',
-        string='Carepoint Backend',
-        store=True,
-        readonly=True,
-        # override 'carepoint.binding', can't be INSERTed if True:
-        required=False,
-    )
-    created_at = fields.Date('Created At (on Carepoint)')
-    updated_at = fields.Date('Updated At (on Carepoint)')
-
-    _sql_constraints = [
-        ('odoo_uniq', 'unique(backend_id, odoo_id)',
-         'A Carepoint binding for this order already exists.'),
-    ]
 
 
 class AccountInvoiceLine(models.Model):

diff --git a/connector_carepoint/models/address.py b/connector_carepoint/models/address.py
@@ -36,21 +36,6 @@ class CarepointCarepointAddress(models.Model):
         required=True,
         ondelete='cascade'
     )
-    backend_id = fields.Many2one(
-        comodel_name='carepoint.backend',
-        string='Carepoint Backend',
-        store=True,
-        readonly=True,
-        # override 'carepoint.binding', can't be INSERTed if True:
-        required=False,
-    )
-    created_at = fields.Date('Created At (on Carepoint)')
-    updated_at = fields.Date('Updated At (on Carepoint)')
-
-    _sql_constraints = [
-        ('odoo_uniq', 'unique(backend_id, odoo_id)',
-         'A Carepoint binding for this address already exists.'),
-    ]
 
 
 class CarepointAddress(models.Model):

diff --git a/connector_carepoint/models/address_patient.py b/connector_carepoint/models/address_patient.py
@@ -33,21 +33,6 @@ class CarepointCarepointAddressPatient(models.Model):
         required=True,
         ondelete='cascade'
     )
-    backend_id = fields.Many2one(
-        comodel_name='carepoint.backend',
-        string='Carepoint Backend',
-        store=True,
-        readonly=True,
-        # override 'carepoint.binding', can't be INSERTed if True:
-        required=False,
-    )
-    created_at = fields.Date('Created At (on Carepoint)')
-    updated_at = fields.Date('Updated At (on Carepoint)')
-
-    _sql_constraints = [
-        ('odoo_uniq', 'unique(backend_id, odoo_id)',
-         'A Carepoint binding for this address already exists.'),
-    ]
 
 
 class CarepointAddressPatient(models.Model):

diff --git a/connector_carepoint/models/address_pharmacy.py b/connector_carepoint/models/address_pharmacy.py
@@ -32,21 +32,6 @@ class CarepointCarepointAddressPharmacy(models.Model):
         required=True,
         ondelete='cascade'
     )
-    backend_id = fields.Many2one(
-        comodel_name='carepoint.backend',
-        string='Carepoint Backend',
-        store=True,
-        readonly=True,
-        # override 'carepoint.binding', can't be INSERTed if True:
-        required=False,
-    )
-    created_at = fields.Date('Created At (on Carepoint)')
-    updated_at = fields.Date('Updated At (on Carepoint)')
-
-    _sql_constraints = [
-        ('odoo_uniq', 'unique(backend_id, odoo_id)',
-         'A Carepoint binding for this address already exists.'),
-    ]
 
 
 class CarepointAddressPharmacy(models.Model):

diff --git a/connector_carepoint/models/address_physician.py b/connector_carepoint/models/address_physician.py
@@ -32,21 +32,6 @@ class CarepointCarepointAddressPhysician(models.Model):
         required=True,
         ondelete='cascade'
     )
-    backend_id = fields.Many2one(
-        comodel_name='carepoint.backend',
-        string='Carepoint Backend',
-        store=True,
-        readonly=True,
-        # override 'carepoint.binding', can't be INSERTed if True:
-        required=False,
-    )
-    created_at = fields.Date('Created At (on Carepoint)')
-    updated_at = fields.Date('Updated At (on Carepoint)')
-
-    _sql_constraints = [
-        ('odoo_uniq', 'unique(backend_id, odoo_id)',
-         'A Carepoint binding for this address already exists.'),
-    ]
 
 
 class CarepointAddressPhysician(models.Model):

diff --git a/connector_carepoint/models/carepoint_person.py b/connector_carepoint/models/carepoint_person.py
@@ -31,21 +31,6 @@ class CarepointResUsers(models.Model):
         required=True,
         ondelete='cascade'
     )
-    backend_id = fields.Many2one(
-        comodel_name='carepoint.backend',
-        string='Carepoint Backend',
-        store=True,
-        readonly=True,
-        # override 'carepoint.binding', can't be INSERTed if True:
-        required=False,
-    )
-    created_at = fields.Date('Created At (on Carepoint)')
-    updated_at = fields.Date('Updated At (on Carepoint)')
-
-    _sql_constraints = [
-        ('odoo_uniq', 'unique(backend_id, odoo_id)',
-         'A Carepoint binding for this user already exists.'),
-    ]
 
 
 class MedicalUser(models.Model):

diff --git a/connector_carepoint/models/medical_patient.py b/connector_carepoint/models/medical_patient.py
@@ -37,21 +37,6 @@ class CarepointMedicalPatient(models.Model):
         required=True,
         ondelete='cascade'
     )
-    backend_id = fields.Many2one(
-        comodel_name='carepoint.backend',
-        string='Carepoint Backend',
-        store=True,
-        readonly=True,
-        # override 'carepoint.binding', can't be INSERTed if True:
-        required=False,
-    )
-    created_at = fields.Date('Created At (on Carepoint)')
-    updated_at = fields.Date('Updated At (on Carepoint)')
-
-    _sql_constraints = [
-        ('odoo_uniq', 'unique(backend_id, odoo_id)',
-         'A Carepoint binding for this patient already exists.'),
-    ]
 
 
 class MedicalPatient(models.Model):

diff --git a/connector_carepoint/models/medical_pharmacy.py b/connector_carepoint/models/medical_pharmacy.py
@@ -31,21 +31,6 @@ class CarepointMedicalPharmacy(models.Model):
         required=True,
         ondelete='cascade'
     )
-    backend_id = fields.Many2one(
-        comodel_name='carepoint.backend',
-        string='Carepoint Backend',
-        store=True,
-        readonly=True,
-        # override 'carepoint.binding', can't be INSERTed if True:
-        required=False,
-    )
-    created_at = fields.Date('Created At (on Carepoint)')
-    updated_at = fields.Date('Updated At (on Carepoint)')
-
-    _sql_constraints = [
-        ('odoo_uniq', 'unique(backend_id, odoo_id)',
-         'A Carepoint binding for this partner already exists.'),
-    ]
 
 
 class MedicalPharmacy(models.Model):

diff --git a/connector_carepoint/models/medical_physician.py b/connector_carepoint/models/medical_physician.py
@@ -33,21 +33,6 @@ class CarepointMedicalPhysician(models.Model):
         required=True,
         ondelete='cascade'
     )
-    backend_id = fields.Many2one(
-        comodel_name='carepoint.backend',
-        string='Carepoint Backend',
-        store=True,
-        readonly=True,
-        # override 'carepoint.binding', can't be INSERTed if True:
-        required=False,
-    )
-    created_at = fields.Date('Created At (on Carepoint)')
-    updated_at = fields.Date('Updated At (on Carepoint)')
-
-    _sql_constraints = [
-        ('odoo_uniq', 'unique(backend_id, odoo_id)',
-         'A Carepoint binding for this physician already exists.'),
-    ]
 
 
 class MedicalPhysician(models.Model):

diff --git a/connector_carepoint/models/medical_prescription_order.py b/connector_carepoint/models/medical_prescription_order.py
@@ -33,21 +33,6 @@ class CarepointMedicalPrescriptionOrder(models.Model):
         required=True,
         ondelete='cascade'
     )
-    backend_id = fields.Many2one(
-        comodel_name='carepoint.backend',
-        string='Carepoint Backend',
-        store=True,
-        readonly=True,
-        # override 'carepoint.binding', can't be INSERTed if True:
-        required=False,
-    )
-    created_at = fields.Date('Created At (on Carepoint)')
-    updated_at = fields.Date('Updated At (on Carepoint)')
-
-    _sql_constraints = [
-        ('odoo_uniq', 'unique(backend_id, odoo_id)',
-         'A Carepoint binding for this prescription already exists.'),
-    ]
 
 
 class MedicalPrescriptionOrder(models.Model):

diff --git a/connector_carepoint/models/medical_prescription_order_line.py b/connector_carepoint/models/medical_prescription_order_line.py
@@ -32,21 +32,6 @@ class CarepointMedicalPrescriptionOrderLine(models.Model):
         required=True,
         ondelete='cascade'
     )
-    backend_id = fields.Many2one(
-        comodel_name='carepoint.backend',
-        string='Carepoint Backend',
-        store=True,
-        readonly=True,
-        # override 'carepoint.binding', can't be INSERTed if True:
-        required=False,
-    )
-    created_at = fields.Date('Created At (on Carepoint)')
-    updated_at = fields.Date('Updated At (on Carepoint)')
-
-    _sql_constraints = [
-        ('odoo_uniq', 'unique(backend_id, odoo_id)',
-         'A Carepoint binding for this prescription already exists.'),
-    ]
 
 
 class MedicalPrescriptionOrderLine(models.Model):

diff --git a/connector_carepoint/models/procurement_order.py b/connector_carepoint/models/procurement_order.py
@@ -33,21 +33,6 @@ class CarepointProcurementOrder(models.Model):
         required=True,
         ondelete='cascade',
     )
-    backend_id = fields.Many2one(
-        comodel_name='carepoint.backend',
-        string='Carepoint Backend',
-        store=True,
-        readonly=True,
-        # override 'carepoint.binding', can't be INSERTed if True:
-        required=False,
-    )
-    created_at = fields.Date('Created At (on Carepoint)')
-    updated_at = fields.Date('Updated At (on Carepoint)')
-
-    _sql_constraints = [
-        ('odoo_uniq', 'unique(backend_id, odoo_id)',
-         'A Carepoint binding for this patient already exists.'),
-    ]
 
 
 class ProcurementOrder(models.Model):

diff --git a/connector_carepoint/models/res_users.py b/connector_carepoint/models/res_users.py
@@ -31,21 +31,6 @@ class CarepointResUsers(models.Model):
         required=True,
         ondelete='cascade'
     )
-    backend_id = fields.Many2one(
-        comodel_name='carepoint.backend',
-        string='Carepoint Backend',
-        store=True,
-        readonly=True,
-        # override 'carepoint.binding', can't be INSERTed if True:
-        required=False,
-    )
-    created_at = fields.Date('Created At (on Carepoint)')
-    updated_at = fields.Date('Updated At (on Carepoint)')
-
-    _sql_constraints = [
-        ('odoo_uniq', 'unique(backend_id, odoo_id)',
-         'A Carepoint binding for this user already exists.'),
-    ]
 
 
 class MedicalUser(models.Model):

diff --git a/connector_carepoint/models/sale_order.py b/connector_carepoint/models/sale_order.py
@@ -30,21 +30,6 @@ class CarepointSaleOrder(models.Model):
         required=True,
         ondelete='cascade'
     )
-    backend_id = fields.Many2one(
-        comodel_name='carepoint.backend',
-        string='Carepoint Backend',
-        store=True,
-        readonly=True,
-        # override 'carepoint.binding', can't be INSERTed if True:
-        required=False,
-    )
-    created_at = fields.Date('Created At (on Carepoint)')
-    updated_at = fields.Date('Updated At (on Carepoint)')
-
-    _sql_constraints = [
-        ('odoo_uniq', 'unique(backend_id, odoo_id)',
-         'A Carepoint binding for this order already exists.'),
-    ]
 
 
 class SaleOrder(models.Model):

diff --git a/connector_carepoint/models/sale_order_line.py b/connector_carepoint/models/sale_order_line.py
@@ -34,21 +34,6 @@ class CarepointSaleOrderLine(models.Model):
         required=True,
         ondelete='cascade'
     )
-    backend_id = fields.Many2one(
-        comodel_name='carepoint.backend',
-        string='Carepoint Backend',
-        store=True,
-        readonly=True,
-        # override 'carepoint.binding', can't be INSERTed if True:
-        required=False,
-    )
-    created_at = fields.Date('Created At (on Carepoint)')
-    updated_at = fields.Date('Updated At (on Carepoint)')
-
-    _sql_constraints = [
-        ('odoo_uniq', 'unique(backend_id, odoo_id)',
-         'A Carepoint binding for this order already exists.'),
-    ]
 
 
 class SaleOrderLine(models.Model):

diff --git a/connector_carepoint/models/sale_order_line_non_rx.py b/connector_carepoint/models/sale_order_line_non_rx.py
@@ -33,21 +33,6 @@ class CarepointSaleOrderLineNonRx(models.Model):
         required=True,
         ondelete='cascade'
     )
-    backend_id = fields.Many2one(
-        comodel_name='carepoint.backend',
-        string='Carepoint Backend',
-        store=True,
-        readonly=True,
-        # override 'carepoint.binding', can't be INSERTed if True:
-        required=False,
-    )
-    created_at = fields.Date('Created At (on Carepoint)')
-    updated_at = fields.Date('Updated At (on Carepoint)')
-
-    _sql_constraints = [
-        ('odoo_uniq', 'unique(backend_id, odoo_id)',
-         'A Carepoint binding for this order already exists.'),
-    ]
 
 
 class SaleOrderLineNonRx(models.Model):