Flake 8

pledra · Mar 30, 2017 · abadc4d · abadc4d
1 parent 5c31f89
commit abadc4d
Show file tree

Hide file tree

Showing 6 changed files with 10 additions and 5 deletions.
diff --git a/product_configurator/__manifest__.py b/product_configurator/__manifest__.py
@@ -4,7 +4,7 @@
     'version': '10.0.1.0.0',
     'category': 'Generic Modules/Base',
     'summary': 'Base for product configuration interface modules',
-    'author': 'Pledra',  # pylint: disable=C8101
+    'author': 'Pledra',
     'license': 'AGPL-3',
     'website': 'http://www.pledra.com/',
     'depends': ['sale_stock'],

diff --git a/product_configurator_mrp/__manifest__.py b/product_configurator_mrp/__manifest__.py
@@ -4,7 +4,7 @@
     'version': '10.0.1.0.0',
     'category': 'Manufacturing',
     'summary': 'BOM Support for configurable products',
-    'author': 'Pledra',  # pylint: disable=C8101
+    'author': 'Pledra',
     'license': 'AGPL-3',
     'website': 'http://www.pledra.com/',
     'depends': [

diff --git a/product_configurator_wizard/__manifest__.py b/product_configurator_wizard/__manifest__.py
@@ -4,7 +4,7 @@
     'version': '10.0.1.0.0',
     'category': 'Generic Modules/Base',
     'summary': 'Back-end Product Configurator',
-    'author': 'Pledra',  # pylint: disable=C8101
+    'author': 'Pledra',
     'license': 'AGPL-3',
     'website': 'http://www.pledra.com/',
     'depends': ['sale', 'product_configurator'],

diff --git a/product_configurator_wizard/wizard/product_configurator.py b/product_configurator_wizard/wizard/product_configurator.py
@@ -439,6 +439,11 @@ def add_dynamic_fields(self, res, dynamic_fields, wiz):
             dependencies = config_lines.filtered(
                 lambda cl: cl.attribute_line_id == attr_line)
 
+            # If an attribute field depends on another field from the same
+            # configuration step then we must use attrs to enable/disable the
+            # required and readonly depending on the value entered in the
+            # dependee
+
             if attr_line.value_ids <= dependencies.mapped('value_ids'):
                 attr_depends = {}
                 domain_lines = dependencies.mapped('domain_id.domain_line_ids')

diff --git a/website_product_configurator/__manifest__.py b/website_product_configurator/__manifest__.py
@@ -3,7 +3,7 @@
     'name': "Website Product Configurator",
     'version': '10.0.1.0.0',
     'summary': """Configure products in e-shop""",
-    'author': "Pledra",  # pylint: disable=C8101
+    'author': "Pledra",
     'license': 'AGPL-3',
     'website': 'http://www.pledra.com/',
     'category': 'website',

diff --git a/website_product_configurator_mrp/__manifest__.py b/website_product_configurator_mrp/__manifest__.py
@@ -4,7 +4,7 @@
     'version': '10.0.1.0.0',
     'category': 'Website',
     'summary': 'Website integration of MRP',
-    'author': 'Pledra',  # pylint: disable=C8101
+    'author': 'Pledra',
     'license': 'AGPL-3',
     'website': 'http://www.pledra.com/',
     'depends': [