[REF] flake8 for 'mass_editing', 'auth_admin_passkey', 'disable_openerp_online' modules;

auth_admin_passkey/tests/test_auth_admin_passkey.py

@@ -91,7 +91,7 @@ class TestAuthAdminPasskey(TransactionCase):
         Test the correct behaviour of login with 'bad_login' / 'admin'"""
         exception_raised = False
         try:
-            res = self.ru_obj.authenticate(self.db, 'bad_login', 'admin', {})
+            self.ru_obj.authenticate(self.db, 'bad_login', 'admin', {})
         except:
             exception_raised = True
         self.assertEqual(

disable_openerp_online/__openerp__.py

@@ -30,7 +30,8 @@ come with the standard code:
 * update notifier code is deactivated and the function is overwritten
 * apps and updates menu items in settings are removed
 * help and account menu items in user menu are removed
-* prevent lookup of OPW for current database uuid and resulting 'unsupported' warning
+* prevent lookup of OPW for current database uuid and resulting"""
+    """ 'unsupported' warning
     """,
     "category": "",
     "depends": [

mass_editing/__openerp__.py

@@ -2,7 +2,8 @@
 ##############################################################################
 #
 #    This module uses OpenERP, Open Source Management Solution Framework.
-#    Copyright (C) 2012-Today Serpent Consulting Services (<http://www.serpentcs.com>)
+#    Copyright (C):
+#        2012-Today Serpent Consulting Services (<http://www.serpentcs.com>)
 #
 #    This program is free software: you can redistribute it and/or modify
 #    it under the terms of the GNU General Public License as published by
@@ -24,11 +25,16 @@
     "author": "Serpent Consulting Services",
     "category": "Tools",
     "website": "http://www.serpentcs.com",
-    "description": """This module provides the functionality to add, update or remove the values of more than one records on the fly at the same time.
+    "description": """
+This module provides the functionality to add, update or remove the values"""
+    """of more than one records on the fly at the same time.
     You can configure mass editing for any OpenERP model.
-        The video explaining the features and how-to for OpenERP Version 6 is here http://t.co/wukYMx1A
-        The video explaining the features and how-to for OpenERP Version 7 is here : http://www.youtube.com/watch?v=9BH0o74A748&feature=youtu.be 
-        For more details/customization/feedback contact us on contact@serpentcs.com. 
+    The video explaining the features and how-to for OpenERP Version 6"""
+    """ is here http://t.co/wukYMx1A
+    The video explaining the features and how-to for OpenERP Version 7 is"""
+    """ here : http://www.youtube.com/watch?v=9BH0o74A748&feature=youtu.be
+    For more details/customization/feedback contact us on"""
+    """ contact@serpentcs.com.
     """,
     'depends': ['base'],
     'data': [

mass_editing/mass_editing.py

@@ -2,7 +2,8 @@
 ##############################################################################
 #
 #    This module uses OpenERP, Open Source Management Solution Framework.
-#    Copyright (C) 2012-Today Serpent Consulting Services (<http://www.serpentcs.com>)
+#    Copyright (C):
+#        2012-Today Serpent Consulting Services (<http://www.serpentcs.com>)
 #
 #    This program is free software: you can redistribute it and/or modify
 #    it under the terms of the GNU General Public License as published by
@@ -19,36 +20,47 @@
 #
 ##############################################################################
 
-from openerp.osv import orm, fields
-import openerp.tools
+from openerp.osv import orm, fields, osv
 from openerp.tools.translate import _
-from lxml import etree
+
 
 class ir_model_fields(orm.Model):
     _inherit = 'ir.model.fields'
 
-    def search(self, cr, uid, args, offset=0, limit=0, order=None, context=None, count=False):
+    def search(
+            self, cr, uid, args, offset=0, limit=0, order=None, context=None,
+            count=False):
         model_domain = []
         for domain in args:
-            if domain[0] == 'model_id' and domain[2] and type(domain[2]) != list:
-                model_domain += [('model_id', 'in', map(int, domain[2][1:-1].split(',')))]
+            if domain[0] == 'model_id' and domain[2]\
+                    and type(domain[2]) != list:
+                model_domain += [(
+                    'model_id', 'in', map(int, domain[2][1:-1].split(',')))]
             else:
                 model_domain.append(domain)
-        return super(ir_model_fields, self).search(cr, uid, model_domain, offset=offset, limit=limit, order=order, context=context, count=count)
+        return super(ir_model_fields, self).search(
+            cr, uid, model_domain, offset=offset, limit=limit, order=order,
+            context=context, count=count)
 
 ir_model_fields()
 
+
 class mass_object(orm.Model):
     _name = "mass.object"
 
     _columns = {
         'name': fields.char("Name", size=64, required=True, select=1),
-        'model_id' : fields.many2one('ir.model', 'Model', required=True, select=1),
-        'field_ids' : fields.many2many('ir.model.fields', 'mass_field_rel', 'mass_id', 'field_id', 'Fields'),
-        'ref_ir_act_window':fields.many2one('ir.actions.act_window', 'Sidebar Action', readonly=True,
+        'model_id': fields.many2one(
+            'ir.model', 'Model', required=True, select=1),
+        'field_ids': fields.many2many(
+            'ir.model.fields', 'mass_field_rel', 'mass_id', 'field_id',
+            'Fields'),
+        'ref_ir_act_window': fields.many2one(
+            'ir.actions.act_window', 'Sidebar Action', readonly=True,
             help="Sidebar action to make this template available on records \
                  of the related document model"),
-        'ref_ir_value':fields.many2one('ir.values', 'Sidebar Button', readonly=True,
+        'ref_ir_value': fields.many2one(
+            'ir.values', 'Sidebar Button', readonly=True,
             help="Sidebar button to open the sidebar action"),
         'model_ids': fields.many2many('ir.model', string='Model List')
     }
@@ -58,22 +70,24 @@ class mass_object(orm.Model):
     ]
 
     def onchange_model(self, cr, uid, ids, model_id, context=None):
-        if context is None: context = {}
+        if context is None:
+            context = {}
         if not model_id:
             return {'value': {'model_ids': [(6, 0, [])]}}
         model_ids = [model_id]
         model_obj = self.pool.get('ir.model')
-        active_model_obj = self.pool.get(model_obj.browse(cr, uid, model_id).model)
+        active_model_obj = self.pool.get(model_obj.browse(
+            cr, uid, model_id).model)
         if active_model_obj._inherits:
             for key, val in active_model_obj._inherits.items():
-                found_model_ids = model_obj.search(cr, uid, [('model', '=', key)], context=context)
+                found_model_ids = model_obj.search(
+                    cr, uid, [('model', '=', key)], context=context)
                 model_ids += found_model_ids
         return {'value': {'model_ids': [(6, 0, model_ids)]}}
 
     def create_action(self, cr, uid, ids, context=None):
         vals = {}
         action_obj = self.pool.get('ir.actions.act_window')
-        data_obj = self.pool.get('ir.model.data')
         ir_values_obj = self.pool.get('ir.values')
         for data in self.browse(cr, uid, ids, context=context):
             src_obj = data.model_id.model
@@ -87,13 +101,15 @@ class mass_object(orm.Model):
                 'context': "{'mass_editing_object' : %d}" % (data.id),
                 'view_mode': 'form,tree',
                 'target': 'new',
-                 'auto_refresh':1
+                'auto_refresh': 1,
                 }, context)
             vals['ref_ir_value'] = ir_values_obj.create(cr, uid, {
                 'name': button_name,
                 'model': src_obj,
                 'key2': 'client_action_multi',
-                 'value': "ir.actions.act_window," + str(vals['ref_ir_act_window']),
+                'value': (
+                    "ir.actions.act_window,"
+                    + str(vals['ref_ir_act_window'])),
                 'object': True,
                 }, context)
         self.write(cr, uid, ids, {
@@ -106,12 +122,16 @@ class mass_object(orm.Model):
         for template in self.browse(cr, uid, ids, context=context):
             try:
                 if template.ref_ir_act_window:
-                    self.pool.get('ir.actions.act_window').unlink(cr, uid, template.ref_ir_act_window.id, context)
+                    self.pool.get('ir.actions.act_window').unlink(
+                        cr, uid, template.ref_ir_act_window.id, context)
                 if template.ref_ir_value:
                     ir_values_obj = self.pool.get('ir.values')
-                    ir_values_obj.unlink(cr, uid, template.ref_ir_value.id, context)
+                    ir_values_obj.unlink(
+                        cr, uid, template.ref_ir_value.id, context)
             except:
-                raise osv.except_osv(_("Warning"), _("Deletion of the action record failed."))
+                raise osv.except_osv(
+                    _("Warning"),
+                    _("Deletion of the action record failed."))
         return True
 
     def unlink(self, cr, uid, ids, context=None):
@@ -121,9 +141,9 @@ class mass_object(orm.Model):
     def copy(self, cr, uid, record_id, default=None, context=None):
         if default is None:
             default = {}
-
         default.update({'name': '', 'field_ids': []})
-        return super(mass_object, self).copy(cr, uid, record_id, default, context)
+        return super(mass_object, self).copy(
+            cr, uid, record_id, default, context)
 
 mass_object()
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

mass_editing/wizard/mass_editing_wizard.py

@@ -2,7 +2,8 @@
 ##############################################################################
 #
 #    This module uses OpenERP, Open Source Management Solution Framework.
-#    Copyright (C) 2012-Today Serpent Consulting Services (<http://www.serpentcs.com>)
+#    Copyright (C):
+#        2012-Today Serpent Consulting Services (<http://www.serpentcs.com>)
 #
 #    This program is free software: you can redistribute it and/or modify
 #    it under the terms of the GNU General Public License as published by
@@ -19,80 +20,165 @@
 #
 ##############################################################################
 
-from openerp.osv import orm, fields
+from openerp.osv import orm
 import openerp.tools as tools
 from lxml import etree
 
+
 class mass_editing_wizard(orm.TransientModel):
     _name = 'mass.editing.wizard'
 
     _columns = {
     }
 
-    def fields_view_get(self, cr, uid, view_id=None, view_type='form', context=None, toolbar=False, submenu=False):
-        result = super(mass_editing_wizard, self).fields_view_get(cr, uid, view_id, view_type, context, toolbar, submenu)
+    def fields_view_get(
+            self, cr, uid, view_id=None, view_type='form', context=None,
+            toolbar=False, submenu=False):
+        result = super(mass_editing_wizard, self).fields_view_get(
+            cr, uid, view_id, view_type, context, toolbar, submenu)
         if context.get('mass_editing_object'):
             mass_object = self.pool.get('mass.object')
-            editing_data = mass_object.browse(cr, uid, context.get('mass_editing_object'), context)
+            editing_data = mass_object.browse(
+                cr, uid, context.get('mass_editing_object'), context)
             all_fields = {}
-            xml_form = etree.Element('form', {'string': tools.ustr(editing_data.name), 'version':'7.0'})
+            xml_form = etree.Element('form', {
+                'string': tools.ustr(editing_data.name), 'version': '7.0'})
             xml_group = etree.SubElement(xml_form, 'group', {'colspan': '4'})
-            etree.SubElement(xml_group, 'label', {'string': '', 'colspan': '2'})
+            etree.SubElement(xml_group, 'label', {
+                'string': '', 'colspan': '2'})
             xml_group = etree.SubElement(xml_form, 'group', {'colspan': '4'})
             model_obj = self.pool.get(context.get('active_model'))
             field_info = model_obj.fields_get(cr, uid, [], context)
             for field in editing_data.field_ids:
                 if field.ttype == "many2many":
                     all_fields[field.name] = field_info[field.name]
-                    all_fields["selection__" + field.name] = {'type':'selection', 'string': field_info[field.name]['string'], 'selection':[('set', 'Set'), ('remove_m2m', 'Remove'), ('add', 'Add')]}
-                    xml_group = etree.SubElement(xml_group, 'group', {'colspan': '4'})
-                    etree.SubElement(xml_group, 'separator', {'string': field_info[field.name]['string'], 'colspan': '2'})
-                    etree.SubElement(xml_group, 'field', {'name': "selection__" + field.name, 'colspan': '2', 'nolabel':'1'})
-                    etree.SubElement(xml_group, 'field', {'name': field.name, 'colspan':'4', 'nolabel':'1', 'attrs':"{'invisible':[('selection__" + field.name + "','=','remove_m2m')]}"})
+                    all_fields["selection__" + field.name] = {
+                        'type': 'selection',
+                        'string': field_info[field.name]['string'],
+                        'selection': [
+                            ('set', 'Set'), ('remove_m2m', 'Remove'),
+                            ('add', 'Add')]}
+                    xml_group = etree.SubElement(xml_group, 'group', {
+                        'colspan': '4'})
+                    etree.SubElement(xml_group, 'separator', {
+                        'string': field_info[field.name]['string'],
+                        'colspan': '2'})
+                    etree.SubElement(xml_group, 'field', {
+                        'name': "selection__" + field.name,
+                        'colspan': '2', 'nolabel': '1'})
+                    etree.SubElement(xml_group, 'field', {
+                        'name': field.name, 'colspan': '4', 'nolabel': '1',
+                        'attrs': (
+                            "{'invisible':[('selection__"
+                            + field.name + "','=','remove_m2m')]}")})
                 elif field.ttype == "one2many":
-                    all_fields["selection__" + field.name] = {'type':'selection',
+                    all_fields["selection__" + field.name] = {
+                        'type': 'selection',
                         'string': field_info[field.name]['string'],
-						'selection':[('set', 'Set'),
-						('remove', 'Remove')]}
-                    all_fields[field.name] = {'type':field.ttype,
-						'string': field.field_description,
+                        'selection': [('set', 'Set'), ('remove', 'Remove')]}
+                    all_fields[field.name] = {
+                        'type': field.ttype, 'string': field.field_description,
                         'relation': field.relation}
-                    etree.SubElement(xml_group, 'field', 
-						{'name': "selection__" + field.name, 'colspan':'2'})
-                    etree.SubElement(xml_group, 'field', 
-						{'name': field.name, 'colspan':'4', 'nolabel':'1',
-							'attrs':"{'invisible':[('selection__" + field.name + "','=','remove_o2m')]}"})
+                    etree.SubElement(xml_group, 'field', {
+                        'name': "selection__" + field.name, 'colspan': '2'})
+                    etree.SubElement(xml_group, 'field', {
+                        'name': field.name, 'colspan': '4', 'nolabel': '1',
+                        'attrs': (
+                            "{'invisible':[('selection__"
+                            + field.name + "','=','remove_o2m')]}")})
                 elif field.ttype == "many2one":
-                    all_fields["selection__" + field.name] = {'type':'selection', 'string': field_info[field.name]['string'], 'selection':[('set', 'Set'), ('remove', 'Remove')]}
-                    all_fields[field.name] = {'type':field.ttype, 'string': field.field_description, 'relation': field.relation}
-                    etree.SubElement(xml_group, 'field', {'name': "selection__" + field.name, 'colspan':'2'})
-                    etree.SubElement(xml_group, 'field', {'name': field.name, 'nolabel':'1', 'colspan':'2', 'attrs':"{'invisible':[('selection__" + field.name + "','=','remove')]}"})
+                    all_fields["selection__" + field.name] = {
+                        'type': 'selection',
+                        'string': field_info[field.name]['string'],
+                        'selection': [('set', 'Set'), ('remove', 'Remove')]}
+                    all_fields[field.name] = {
+                        'type': field.ttype, 'string': field.field_description,
+                        'relation': field.relation}
+                    etree.SubElement(xml_group, 'field', {
+                        'name': "selection__" + field.name, 'colspan': '2'})
+                    etree.SubElement(xml_group, 'field', {
+                        'name': field.name, 'nolabel': '1', 'colspan': '2',
+                        'attrs': (
+                            "{'invisible':[('selection__"
+                            + field.name + "','=','remove')]}")})
                 elif field.ttype == "char":
-                    all_fields["selection__" + field.name] = {'type':'selection', 'string': field_info[field.name]['string'], 'selection':[('set', 'Set'), ('remove', 'Remove')]}
-                    all_fields[field.name] = {'type':field.ttype, 'string': field.field_description, 'size': field.size or 256}
-                    etree.SubElement(xml_group, 'field', {'name': "selection__" + field.name, 'colspan':'2', 'colspan':'2'})
-                    etree.SubElement(xml_group, 'field', {'name': field.name, 'nolabel':'1', 'attrs':"{'invisible':[('selection__" + field.name + "','=','remove')]}", 'colspan':'2'})
+                    all_fields["selection__" + field.name] = {
+                        'type': 'selection',
+                        'string': field_info[field.name]['string'],
+                        'selection': [('set', 'Set'), ('remove', 'Remove')]}
+                    all_fields[field.name] = {
+                        'type': field.ttype, 'string': field.field_description,
+                        'size': field.size or 256}
+                    etree.SubElement(xml_group, 'field', {
+                        'name': "selection__" + field.name,
+                        'colspan': '2',
+                        'colspan': '2'})
+                    etree.SubElement(xml_group, 'field', {
+                        'name': field.name, 'nolabel': '1',
+                        'attrs': (
+                            "{'invisible':[('selection__"
+                            + field.name + "','=','remove')]}"),
+                        'colspan': '2'})
                 elif field.ttype == 'selection':
-                    all_fields["selection__" + field.name] = {'type':'selection', 'string': field_info[field.name]['string'], 'selection':[('set', 'Set'), ('remove', 'Remove')]}
-                    etree.SubElement(xml_group, 'field', {'name': "selection__" + field.name, 'colspan':'2'})
-                    etree.SubElement(xml_group, 'field', {'name': field.name, 'nolabel':'1', 'colspan':'2', 'attrs':"{'invisible':[('selection__" + field.name + "','=','remove')]}"})
-                    all_fields[field.name] = {'type':field.ttype, 'string': field.field_description, 'selection': field_info[field.name]['selection']}
+                    all_fields["selection__" + field.name] = {
+                        'type': 'selection',
+                        'string': field_info[field.name]['string'],
+                        'selection': [('set', 'Set'), ('remove', 'Remove')]}
+                    etree.SubElement(xml_group, 'field', {
+                        'name': "selection__" + field.name, 'colspan': '2'})
+                    etree.SubElement(xml_group, 'field', {
+                        'name': field.name, 'nolabel': '1', 'colspan': '2',
+                        'attrs': (
+                            "{'invisible':[('selection__"
+                            + field.name + "','=','remove')]}")})
+                    all_fields[field.name] = {
+                        'type': field.ttype,
+                        'string': field.field_description,
+                        'selection': field_info[field.name]['selection']}
                 else:
-                    all_fields[field.name] = {'type':field.ttype, 'string': field.field_description}
-                    all_fields["selection__" + field.name] = {'type':'selection', 'string': field_info[field.name]['string'], 'selection':[('set', 'Set'), ('remove', 'Remove')]}
+                    all_fields[field.name] = {
+                        'type': field.ttype, 'string': field.field_description}
+                    all_fields["selection__" + field.name] = {
+                        'type': 'selection',
+                        'string': field_info[field.name]['string'],
+                        'selection': [('set', 'Set'), ('remove', 'Remove')]}
                     if field.ttype == 'text':
-                        xml_group = etree.SubElement(xml_group, 'group', {'colspan': '6'})
-                        etree.SubElement(xml_group, 'separator', {'string': all_fields[field.name]['string'], 'colspan': '2'})
-                        etree.SubElement(xml_group, 'field', {'name': "selection__" + field.name, 'colspan': '2', 'nolabel':'1'})
-                        etree.SubElement(xml_group, 'field', {'name': field.name, 'colspan':'4', 'nolabel':'1', 'attrs':"{'invisible':[('selection__" + field.name + "','=','remove')]}"})
+                        xml_group = etree.SubElement(xml_group, 'group', {
+                            'colspan': '6'})
+                        etree.SubElement(xml_group, 'separator', {
+                            'string': all_fields[field.name]['string'],
+                            'colspan': '2'})
+                        etree.SubElement(xml_group, 'field', {
+                            'name': "selection__" + field.name,
+                            'colspan': '2', 'nolabel': '1'})
+                        etree.SubElement(xml_group, 'field', {
+                            'name': field.name, 'colspan': '4', 'nolabel': '1',
+                            'attrs': (
+                                "{'invisible':[('selection__"
+                                + field.name + "','=','remove')]}")})
                     else:
-                        all_fields["selection__" + field.name] = {'type':'selection', 'string': field_info[field.name]['string'], 'selection':[('set', 'Set'), ('remove', 'Remove')]}
-                        etree.SubElement(xml_group, 'field', {'name': "selection__" + field.name, 'colspan': '2', })
-                        etree.SubElement(xml_group, 'field', {'name': field.name, 'nolabel':'1', 'attrs':"{'invisible':[('selection__" + field.name + "','=','remove')]}", 'colspan': '2', })
-            etree.SubElement(xml_form, 'separator', {'string' : '', 'colspan': '4'})
+                        all_fields["selection__" + field.name] = {
+                            'type': 'selection',
+                            'string': field_info[field.name]['string'],
+                            'selection': [(
+                                'set', 'Set'), ('remove', 'Remove')]}
+                        etree.SubElement(xml_group, 'field', {
+                            'name': "selection__" + field.name,
+                            'colspan': '2', })
+                        etree.SubElement(xml_group, 'field', {
+                            'name': field.name, 'nolabel': '1',
+                            'attrs': (
+                                "{'invisible':[('selection__"
+                                + field.name + "','=','remove')]}"),
+                            'colspan': '2', })
+            etree.SubElement(
+                xml_form, 'separator', {'string': '', 'colspan': '4'})
             xml_group3 = etree.SubElement(xml_form, 'footer', {})
-            etree.SubElement(xml_group3, 'button', {'string' :'Close', 'icon': "gtk-close", 'special' :'cancel'})
-            etree.SubElement(xml_group3, 'button', {'string' :'Apply', 'icon': "gtk-execute", 'type' :'object', 'name':"action_apply"})
+            etree.SubElement(xml_group3, 'button', {
+                'string': 'Close', 'icon': "gtk-close", 'special': 'cancel'})
+            etree.SubElement(xml_group3, 'button', {
+                'string': 'Apply', 'icon': "gtk-execute",
+                'type': 'object', 'name': "action_apply"})
             root = xml_form.getroottree()
             result['arch'] = etree.tostring(root)
             result['fields'] = all_fields
@@ -110,14 +196,17 @@ class mass_editing_wizard(orm.TransientModel):
                     elif val == 'remove':
                         dict.update({split_key: False})
                     elif val == 'remove_m2m':
-                        dict.update({split_key: [(3, id) for id in vals.get(split_key, False)[0][2]]})
+                        dict.update({split_key: [
+                            (3, id) for id in vals.get(
+                                split_key, False)[0][2]]})
                     elif val == 'add':
                         m2m_list = []
                         for m2m_id in vals.get(split_key, False)[0][2]:
                             m2m_list.append((4, m2m_id))
                         dict.update({split_key: m2m_list})
             if dict:
-                model_obj.write(cr, uid, context.get('active_ids'), dict, context)
+                model_obj.write(
+                    cr, uid, context.get('active_ids'), dict, context)
         result = super(mass_editing_wizard, self).create(cr, uid, {}, context)
         return result