Merge lp:~openerp-community/openerp-product-variant/fix-pep8-and-relative-import into lp:~anybox/openerp-product-variant/7.0

[Niels Huylebroeck]

PEP8 + relative import

[Pedro Manuel Baeza (pedro.baeza)]

LGTM

review: Approve (code review, no test)

[Quentin THEURET @Amaris (qtheuret)]

Code review ok with no tests.

But, if I run the flake8 in the branch (as described in http://pad.openerp.com/p/community-review), there are some errors : : https://cloud.theuret.net/public.php?service=files&t=325551f781f972b1eb5d088c259c07f5

[Niels Huylebroeck (red15)]

Quentin: cannot access that file. I will run flake8 on the module entirely and commit this, I had only done this on __openerp__.py as asked by Nicolas Bessi.

[Niels Huylebroeck (red15)]

A quick check with flake8 produces only one easily fixable style suggestion:
http://pastie.org/8336327

The space after the comma.

For all the other "problems" it's nagging about the 79 character limit.
Since no lines are longer than 100 characters this is a minimal discomfort imho and restructuring the code manually takes a long time.

Yes the long lines indicate a non-optimal writing. This is code that has been ported, not written from scratch.

Feel free to branch and commit (it's pushed to ~openerp-community team) and start cleaning up. Keep in mind that the larger the commit message the more time it takes to approve it.

Consider also that changing the indentation/line length will not necessarily produce cleaner code but can introduce errors in logic unless you are very careful or use automated tools.

Formatting tools (autopep8) turn the file from somewhat hard to read to impossible to read with the tons of auto-indentation it adds, I think to get a clean result you can only do it manually.

[Quentin THEURET @Amaris (qtheuret)]

Niels,

I expected this message :)

It was just a remark (that's why I didn't reject the MP) because the title of the branch contains fix-pep8 so I said me maybe you don't run flake8 on your code.

So, for me, the code is good.

[Christophe Combelles (ccomb)]

I just pushed another flake8 cleanup in our branch (most of us use a vim flake8 plugin, but we limited the line length to 100 instead of 80)

[Pedro Manuel Baeza (pedro.baeza)]

LGTM

review: Approve (code review, no test)

[Guewen Baconnier @ Camptocamp (gbaconnier-c2c)]

The MP's target is an Anybox's branch. Is it normal?

review: Needs Information

Unmerged revisions

200. By Niels Huylebroeck on 2013-09-18

[FIX] product_variant_multi: improve styling of __openerp__ and __init__

=== modified file 'product_variant_multi/__init__.py'
--- product_variant_multi/__init__.py	2013-04-22 22:54:34 +0000
+++ product_variant_multi/__init__.py	2013-09-18 14:52:26 +0000
@@ -20,4 +20,4 @@
 #
 ##############################################################################
 
-import product_variant
+from . import product_variant

=== modified file 'product_variant_multi/__openerp__.py'
--- product_variant_multi/__openerp__.py	2013-04-04 09:29:53 +0000
+++ product_variant_multi/__openerp__.py	2013-09-18 14:52:26 +0000
@@ -20,13 +20,13 @@
 #
 ##############################################################################
 {
-    "name" : "Product Variant Multi",
-    "version" : "1.0",
-    "author" : "OpenERP SA, Akretion",
-    "category" : "Sales Management",
+    "name": "Product Variant Multi",
+    "version": "1.0",
+    "author": "OpenERP SA, Akretion",
+    "category": "Sales Management",
     "license": "AGPL-3",
     "summary": "Products with multi-dimension variants",
-    "description":"""
+    "description": """
 Multi-axial varianted product support for OpenERP
 =================================================
 
@@ -60,9 +60,9 @@
 and only from product.template if not found on product.product. But at least you
 will have been warned.
     """,
-    "depends" : ["product"],
-    "demo" : ["demo_data.xml"],
-    "data" : [
+    "depends": ["product"],
+    "demo": ["demo_data.xml"],
+    "data": [
         "security/ir.model.access.csv",
         "product_view.xml",
     ],
@@ -70,4 +70,3 @@
     "active": False,
     "installable": True,
 }
-