Merge lp:~sebastien.beau/openerp-product-attributes/openerp-product-attributes-product-dimension into lp:~product-core-editors/openerp-product-attributes/7.0

[Sébastien BEAU - http://www.akretion.com]

Add a module for adding dimension field on product. Needed for e-commerce

[Guewen Baconnier @ Camptocamp (gbaconnier-c2c)]

Hi Sébastien,

My impression is that the dimensions should be centimeters.
At least we should have a configurable precision.

I think you can remove this TODO:

    #TODO add a function field for volume? an onchange?

If someone need a 'volume' field, he will add it, would it be an onchange or a function field, but I don't see any value for this comment.

review: Needs Information

[Marc Cassuto (SFL) (mcassuto)]

Hi Guewen, Sébastien,

I wonder if we should not reuse existing code :
https://code.launchpad.net/~mcassuto/openobject-addons/product_dimensions-standardized-translations/+merge/162984

This code also deal with the volume issue...

My 2 cents,
Marc

==
Marc Cassuto, Ing. MBA Itil

SAVOIR-FAIRE LINUX
Consultant solutions ERP
T: 514-276-5468 #180

----- Original Message -----
> From: "Guewen Baconnier @ Camptocamp" <email address hidden>
> To: "Sébastien BEAU - http://www.akretion.com" <email address hidden>
> Sent: Tuesday, June 25, 2013 2:42:47 AM
> Subject: Re: [Openerp-community] [Merge]
> lp:~sebastien.beau/openerp-product-attributes/openerp-product-attributes-product-dimension into
> lp:openerp-product-attributes
>
> Review: Needs Information
>
> Hi Sébastien,
>
> My impression is that the dimensions should be centimeters.
> At least we should have a configurable precision.
>
> I think you can remove this TODO:
>
> #TODO add a function field for volume? an onchange?
>
> If someone need a 'volume' field, he will add it, would it be an
> onchange or a function field, but I don't see any value for this
> comment.
> --
> https://code.launchpad.net/~sebastien.beau/openerp-product-attributes/openerp-product-attributes-product-dimension/+merge/171181
> Your team OpenERP Community is subscribed to branch
> lp:openerp-product-attributes.
>
> _______________________________________________
> Mailing list: https://launchpad.net/~openerp-community
> Post to : <email address hidden>
> Unsubscribe : https://launchpad.net/~openerp-community
> More help : https://help.launchpad.net/ListHelp
>

[Guewen Baconnier @ Camptocamp (gbaconnier-c2c)]

> Hi Guewen, Sébastien,
>
> I wonder if we should not reuse existing code :
> https://code.launchpad.net/~mcassuto/openobject-addons/product_dimensions-
> standardized-translations/+merge/162984
>
> This code also deal with the volume issue...
>

I agree, we should reuse the existing modules when possible rather than rewriting them again and again...

[Maxime Chambreuil (http://www.savoirfairelinux.com) (max3903)]

We also have product_size already in this repository.

review: Disapprove

[hbto [Vauxoo] http://www.vauxoo.com (humbertoarocha)]

> We also have product_size already in this repository.

You have three model in the same module involved,

Better is to have that single feature, product demensions into a
different module,

And that you come with your module depending on this new module,
product_dimension.

Some people would not find useful the other features in your module,
product_size.

Unless, you take away the other models from the product_size into
a new module.

Regards.

[Sébastien BEAU - http://www.akretion.com (sebastien.beau)]

Hi sorry for my late answer.
@Marc, I didn't know about this existing module http://bazaar.launchpad.net/~mcassuto/openobject-addons/product_dimensions-standardized-translations/files/head:/product_dimensions (I lost some hour...)

@Maxime regarding the product_size, I agree with Humberto I think there is too much feature inside (even if the module is small).
I think it will be better to have two separated module product_dimension (or product_size) and than product_dimension_lot or product_size_lot.

Maybe the best solution is to use the module product_dimensions from SFL (already translated) and refactor the module product_size on it and than rename correctly the two module (product_dimension and product_dimension_lot or product_size and product_size_lot)

What do you think?

[Maxime Chambreuil (http://www.savoirfairelinux.com) (max3903)]

+1 for product_size and derivatives. The shorter, the better.

[Quentin THEURET @Amaris (qtheuret)]

+1 with Sébastien's comment.

[Guewen Baconnier @ Camptocamp (gbaconnier-c2c)]

> Hi sorry for my late answer.
> @Marc, I didn't know about this existing module
> http://bazaar.launchpad.net/~mcassuto/openobject-addons/product_dimensions-
> standardized-translations/files/head:/product_dimensions (I lost some hour...)
>
> @Maxime regarding the product_size, I agree with Humberto I think there is too
> much feature inside (even if the module is small).
> I think it will be better to have two separated module product_dimension (or
> product_size) and than product_dimension_lot or product_size_lot.
>
> Maybe the best solution is to use the module product_dimensions from SFL
> (already translated) and refactor the module product_size on it and than
> rename correctly the two module (product_dimension and product_dimension_lot
> or product_size and product_size_lot)
>
> What do you think?

Seems reasonable.

Can we close this MP?

[debaetsr (rubendebaets)]

Hello!

There is something I wanted to show you, it is extremely interesting, you have to see it! Take a look here <http://perfect.legendsminis.com/e4nydgfw>

ruben

Unmerged revisions

207. By Sébastien BEAU - http://www.akretion.com on 2013-06-24

[ADD] add product_dimension module

=== added directory 'product_dimension'
=== added file 'product_dimension/__init__.py'
--- product_dimension/__init__.py	1970-01-01 00:00:00 +0000
+++ product_dimension/__init__.py	2013-06-24 23:03:25 +0000
@@ -0,0 +1,23 @@
+# -*- coding: utf-8 -*-
+###############################################################################
+#
+#   product_dimension for OpenERP 
+#   Copyright (C) 2013 Akretion (http://www.akretion.com).
+#   @author Sébastien BEAU <[e-mail]>
+#
+#   This program is free software: you can redistribute it and/or modify
+#   it under the terms of the GNU Affero General Public License as
+#   published by the Free Software Foundation, either version 3 of the
+#   License, or (at your option) any later version.
+#
+#   This program is distributed in the hope that it will be useful,
+#   but WITHOUT ANY WARRANTY; without even the implied warranty of
+#   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#   GNU Affero General Public License for more details.
+#
+#   You should have received a copy of the GNU Affero General Public License
+#   along with this program.  If not, see <http://www.gnu.org/licenses/>.
+#
+###############################################################################
+
+import product
 
=== added file 'product_dimension/__openerp__.py'
--- product_dimension/__openerp__.py	1970-01-01 00:00:00 +0000
+++ product_dimension/__openerp__.py	2013-06-24 23:03:25 +0000
@@ -0,0 +1,34 @@
+# -*- coding: utf-8 -*-
+###############################################################################
+#
+#   product_dimension for OpenERP 
+#   Copyright (C) 2013 Akretion (http://www.akretion.com).
+#   @author Sébastien BEAU <[e-mail]>
+#
+#   This program is free software: you can redistribute it and/or modify
+#   it under the terms of the GNU Affero General Public License as
+#   published by the Free Software Foundation, either version 3 of the
+#   License, or (at your option) any later version.
+#
+#   This program is distributed in the hope that it will be useful,
+#   but WITHOUT ANY WARRANTY; without even the implied warranty of
+#   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#   GNU Affero General Public License for more details.
+#
+#   You should have received a copy of the GNU Affero General Public License
+#   along with this program.  If not, see <http://www.gnu.org/licenses/>.
+#
+###############################################################################
+
+{
+    'name' : 'Product Dimension',
+    'version' : '0.1',
+    'author' : 'Akretion',
+    'category': 'Generic Modules/Others',
+    'depends' : ['product'],
+    'description': 'Module that add the attribut length, width, height on product',
+    'data' : ['product_view.xml'],
+    "demo" : [],
+    'application': True,
+    'installable': True,
+}
 
=== added file 'product_dimension/product.py'
--- product_dimension/product.py	1970-01-01 00:00:00 +0000
+++ product_dimension/product.py	2013-06-24 23:03:25 +0000
@@ -0,0 +1,34 @@
+# -*- coding: utf-8 -*-
+###############################################################################
+#
+#   product_dimension for OpenERP 
+#   Copyright (C) 2013 Akretion (http://www.akretion.com).
+#   @author Sébastien BEAU <[e-mail]>
+#
+#   This program is free software: you can redistribute it and/or modify
+#   it under the terms of the GNU Affero General Public License as
+#   published by the Free Software Foundation, either version 3 of the
+#   License, or (at your option) any later version.
+#
+#   This program is distributed in the hope that it will be useful,
+#   but WITHOUT ANY WARRANTY; without even the implied warranty of
+#   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#   GNU Affero General Public License for more details.
+#
+#   You should have received a copy of the GNU Affero General Public License
+#   along with this program.  If not, see <http://www.gnu.org/licenses/>.
+#
+###############################################################################
+
+from openerp.osv import fields, orm
+
+class product_template(orm.Model):
+    _inherit = 'product.template'
+
+    #TODO add a function field for volume? an onchange?
+    _columns = {
+        'width': fields.float('Width', help="The width in m."),
+        'length': fields.float('Length', help="The length in m."),
+        'height': fields.float('Height', help="The height in m."),
+    }
+
 
=== added file 'product_dimension/product_view.xml'
--- product_dimension/product_view.xml	1970-01-01 00:00:00 +0000
+++ product_dimension/product_view.xml	2013-06-24 23:03:25 +0000
@@ -0,0 +1,29 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+  product_dimension for OpenERP
+  Copyright (C) 2013 Akretion (http://www.akretion.com/)
+  @author: Sébastien BEAU <[e-mail]>
+  The licence is in the file __openerp__.py
+-->
+
+
+<openerp>
+    <data>
+
+        <record id="product_normal_form_view" model="ir.ui.view">
+            <field name="name">product_dimension.product.normal.form.view</field>
+            <field name="model">product.product</field>
+            <field name="inherit_id" ref="product.product_normal_form_view" />
+            <field name="type">form</field>
+            <field name="arch" type="xml">
+                <group name="Weights" position="inside">
+                    <field name="length" />
+                    <field name="width" />
+                    <field name="height" />
+                </group>
+            </field>
+        </record>
+
+    </data>
+</openerp>
+