improve adherance to coding conventions

base_partner_sequence/__init__.py

@@ -18,6 +18,5 @@
 #    along with this program.  If not, see <http://www.gnu.org/licenses/>.
 #
 ##############################################################################
-import partner_sequence
+import partner
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
-

base_partner_sequence/__openerp__.py

@@ -21,25 +21,24 @@
 #
 ##############################################################################
 {
-    "name" : "Add a sequence on customers' code",
-    "version" : "1.1",
-    "author" : "initOS GmbH & Co. KG",
-    "category" : "Generic Modules/Base",
+    "name": "Add a sequence on customers' code",
+    "version": "1.1",
+    "author": "initOS GmbH & Co. KG",
+    "category": "Generic Modules/Base",
     "website": "http://www.initos.com",
-    "depends" : ["base"],
-    "summary" : "Sets customer's code from a sequence",
+    "depends": ["base"],
+    "summary": "Sets customer's code from a sequence",
     "description": """
         This module adds the possibility to define a sequence for
         the partner code. This code is then set as default when you
-        create a new customer, using the defined sequence.
+        create a new partner (not a contact), using the defined sequence.
         This module is a migration of the original base_partner_sequence
         addon to OpenERP version 7.0.
     """,
-    "demo_xml" : [],
-    "init_xml" : ['partner_sequence.xml'],
-    "update_xml" : [],
+    "demo_xml": [],
+    "init_xml": ['partner_sequence.xml'],
+    "update_xml": [],
     "active": False,
     "installable": True
 }
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
-

base_partner_sequence/partner_sequence.py → base_partner_sequence/partner.py

@@ -21,22 +21,23 @@
 #
 ##############################################################################
 
-from osv import osv, fields
-import logging
-_logger = logging.getLogger(__name__)
+from openerp.osv import orm, fields
+
+
+class ResPartner(orm.Model):
+    """Assigns 'ref' from a sequence on creation"""
 
-class partner_sequence(osv.osv):
     _inherit = 'res.partner'
+
     def create(self, cr, uid, vals, context={}):
         # only assign a 'ref' if it is not a child object
         #  (such as a shipping/invoice address)
-        if vals.get('parent_id', 0) > 0:
+        if not vals.get('parent_id'):
             vals['ref'] = self.pool.get('ir.sequence').get(cr, uid, 'res.partner')
-        res = super(partner_sequence, self).create(cr, uid, vals, context)
-        return res
+        return super(ResPartner, self).create(cr, uid, vals, context)
+
     _columns = {
         'ref': fields.char('Code', size=64, readonly=True),
     }
-partner_sequence()
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4: