Merge lp:~camptocamp/sale-wkfl/product_procurement_type into lp:~sale-core-editors/sale-wkfl/7.0

[Alexandre Fayolle - camptocamp]

Hello,

Here are two new addons who brings a simple field to manage several combinations
for two procurements fields : procure_method & supply_method :
First addon adds:
- Normal: MTS + buy
- Bill of Materials: MTO + produce
Second addon depends of sale_dropshipping addon and adds:
- Dropshipping: MTO + buy

I'm not sure if it's the right community branch to propose it, if you think about a better
place, let me know.

Regards,

Romain

[Yannick Vaucher @ Camptocamp (yvaucher-c2c)]

LGTM for the code

However, I don't think we have a proper branch to extend procurement.

Shall we create one?

review: Approve (code review, no tests)

[Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c)]

> LGTM for the code
>
> However, I don't think we have a proper branch to extend procurement.
>
> Shall we create one?

Since this is mainly affecting sales, I think this branch is correct.

[Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c)]

need to set the value of the attribute from current product configuration when the module is installed.

setting status to "Work in progress" until this is resolved

review: Needs Fixing

[Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c)]

> > LGTM for the code
> >
> > However, I don't think we have a proper branch to extend procurement.
> >
> > Shall we create one?
>
> Since this is mainly affecting sales, I think this branch is correct.

However I'm not sure this is not too customer specific to get in OCA. What do other think?

review: Needs Information

[Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c)]

> need to set the value of the attribute from current product configuration when
> the module is installed.
>
> setting status to "Work in progress" until this is resolved

dons

[Joël Grand-Guillaume @ camptocamp (jgrandguillaume-c2c)]

LGTM, and I agree about adding this module here. Thanks

review: Approve (code review, no tests)

[Leonardo Pistone (lepistone)]

I approve the code.

I share Alexandre's question if there is interest to have that in OCA. If not, being it approved technically, we can quickly merge it to a specific branch.

review: Approve (code review)

[Stefan Rijnhart (Opener) (stefan-opener)]

I don't see a problem having these modules here. About the code: there is a migration script, and I think its purpose is to update existing products upon installation of the module. However, I don't think the script is run at module installation time. Are you sure that this is the case?

Otherwise, you will need to put it in the model's _auto_init() method. It will run every time the module is upgraded (or installed) but that should not hurt. What you can do is check if column 'procurement_type' exists, call super() and run the migration queries only if the column did not exist when you checked. This indicates the installation of the module.

review: Needs Information

[Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c)]

> I don't see a problem having these modules here. About the code: there is a
> migration script, and I think its purpose is to update existing products upon
> installation of the module. However, I don't think the script is run at module
> installation time. Are you sure that this is the case?

Yes it is run at installation time, I checked for this.

> Otherwise, you will need to put it in the model's _auto_init() method. It will
> run every time the module is upgraded (or installed) but that should not hurt.
> What you can do is check if column 'procurement_type' exists, call super() and
> run the migration queries only if the column did not exist when you checked.
> This indicates the installation of the module.

Not sure what the cleanest way is. Do you have an opinion about the cleanness of either options, and what we should aim for in OCA modules?

[Stefan Rijnhart (Opener) (stefan-opener)]

Great, that means that we have the pre- and post-installation scripts that I always wanted after all! Much better solution than the _auto_init one.

review: Approve

[Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c)]

I found an issue (possibly a framework bug) which prevents the new attribute from being imported from a CSV file.

review: Needs Resubmitting (test)

[Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c)]

I updated the MP with a workaround described in http://bazaar.launchpad.net/~camptocamp/sale-wkfl/product_procurement_type/revision/43

review: Approve (code review, test)

[Lorenzo Battistini (elbati)]

Hello Alexandre,

line 223: if vals['type'] == 'service', should we run _compute_procurement_vals anyway?

lines 585 and 597: as 'product_procurement_type_dropshipping' depends on 'product_procurement_type', why do the write and create methods call the '_compute_procurement_vals' method, already called by 'product_procurement_type'?

Thanks!

review: Needs Information

[Joël Grand-Guillaume @ camptocamp (jgrandguillaume-c2c)]

Hi,

Regarding Lorenzo's comments:

 * L223 : IMO think that's not necessary
 * L585 : I agree with Lorenzo's opinion here, you should remove the write and create override in the product_procurement_type_dropshipping module. Already made in the product_procurement_type one.

Thanks !

review: Needs Fixing (code review + no tests)

[lp:~camptocamp/sale-wkfl/product_procurement_type]

44. By Alexandre Fayolle - camptocamp on 2014-06-04

[IMP] remove unnecessary code pointed out by lpistone

45. By Alexandre Fayolle - camptocamp on 2014-06-04

the migration scripts are not longer called on installation -> move to init method

46. By Romain Deheele - Camptocamp on 2014-06-10

[UPD] update manifest according to selection values

47. By Romain Deheele - Camptocamp on 2014-06-10

replace ValueError exception with False value that will be refused because a value is required : avoid ugly traceback

48. By Romain Deheele - Camptocamp on 2014-06-10

[UPD] update manifest description

Unmerged revisions

48. By Romain Deheele - Camptocamp on 2014-06-10

[UPD] update manifest description

47. By Romain Deheele - Camptocamp on 2014-06-10

replace ValueError exception with False value that will be refused because a value is required : avoid ugly traceback

46. By Romain Deheele - Camptocamp on 2014-06-10

[UPD] update manifest according to selection values

45. By Alexandre Fayolle - camptocamp on 2014-06-04

the migration scripts are not longer called on installation -> move to init method

44. By Alexandre Fayolle - camptocamp on 2014-06-04

[IMP] remove unnecessary code pointed out by lpistone

43. By Alexandre Fayolle - camptocamp on 2014-03-03

[FIX] issue when importing products from CSV with procurement_type attribute

There is an issue in the framework (I think) which will break the previous
version of the code in CSV imports: the get_product_procurement_type method (in
product_template) is called with an instance of product_product for self, and
the call to super() crashed. This redesign works around the issue by extracting
get_product_procurement_type to a function and thus avoiding self altogether.

42. By Alexandre Fayolle - camptocamp on 2014-02-26

[FIX] tests

41. By Alexandre Fayolle - camptocamp on 2014-02-26

[ADD] migration script to ensure that the new attribute is populated

40. By Alexandre Fayolle - camptocamp on 2014-02-26

[FIX] values of procurement_type in dispatch method + help string + module version number

39. By Alexandre Fayolle - camptocamp on 2014-02-25

[IMP] follow changes in product_procurement_type_dropshipping

=== added directory 'product_procurement_type'
=== added file 'product_procurement_type/__init__.py'
--- product_procurement_type/__init__.py	1970-01-01 00:00:00 +0000
+++ product_procurement_type/__init__.py	2014-06-04 22:48:39 +0000
@@ -0,0 +1,22 @@
+# -*- coding: utf-8 -*-
+##############################################################################
+#
+#    Author: Romain Deheele
+#    Copyright 2014 Camptocamp SA
+#
+#    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 . import product

=== added file 'product_procurement_type/__openerp__.py'
--- product_procurement_type/__openerp__.py	1970-01-01 00:00:00 +0000
+++ product_procurement_type/__openerp__.py	2014-06-04 22:48:39 +0000
@@ -0,0 +1,50 @@
+# -*- coding: utf-8 -*-
+##############################################################################
+#
+#    Author: Romain Deheele
+#    Copyright 2014 Camptocamp SA
+#
+#    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 Procurement Types",
+    "version": "0.1",
+    "author": "Camptocamp",
+    "license": "AGPL-3",
+    "category": "Sales",
+    "description":
+    """
+    Add a procurement type selection on products.
+    Let create products as :
+     - Standard
+     - Bill of Materials
+
+    Choose "Standard" changes automatically
+    procurement method as "make to stock" and supply method as "buy".
+    Choose "Bill of Materials" changes automatically
+    procurement method as "make to order" and supply method as "produce".
+
+    """,
+    "website": "http://camptocamp.com",
+    "depends": ['sale',
+                'procurement',
+                'mrp'],
+    "data": ['product_view.xml'],
+    "demo": [],
+    "test": ['test/test_onchange_procurement_type.yml'],
+    "active": False,
+    "installable": True,
+}

=== added file 'product_procurement_type/product.py'
--- product_procurement_type/product.py	1970-01-01 00:00:00 +0000
+++ product_procurement_type/product.py	2014-06-04 22:48:39 +0000
@@ -0,0 +1,139 @@
+# -*- coding: utf-8 -*-
+##############################################################################
+#
+#    Author: Romain Deheele
+#    Copyright 2014 Camptocamp SA
+#
+#    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 logging
+
+from openerp.osv import orm, fields
+
+_logger = logging.getLogger('openerp.upgrade')
+
+def get_procurement_type_selection():
+    return [('buy_stock', 'On stock, buy'),
+            ('buy_demand', 'On demand, buy'),
+            ('produce_demand', 'On demand, produce'),
+            ('produce_stock', 'On stock, produce'),
+            ]
+
+class product_template(orm.Model):
+
+    _inherit = 'product.template'
+
+    def init(self, cr):
+        _logger.info('Migrating product_procurement_type')
+        query = ("UPDATE product_template pt "
+                 "SET procurement_type=%(procurement_type)s "
+                 "WHERE procure_method=%(procure_method)s and supply_method=%(supply_method)s")
+        fixes = [{'procurement_type': 'buy_stock',
+                  'procure_method': 'make_to_stock',
+                  'supply_method': 'buy'},
+                 {'procurement_type': 'buy_demand',
+                  'procure_method': 'make_to_order',
+                  'supply_method': 'buy'},
+                 {'procurement_type': 'produce_stock',
+                  'procure_method': 'make_to_stock',
+                  'supply_method': 'produce'},
+                 {'procurement_type': 'produce_demand',
+                  'procure_method': 'make_to_order',
+                  'supply_method': 'produce'},
+                 ]
+        for fix in fixes:
+            cr.execute(query, fix)
+
+    def get_procurement_type_selection(self, cr, uid, context=None):
+        """
+        Has to be inherited to add procurement type
+        """
+        return get_procurement_type_selection()
+
+    def _compute_procurement_vals(self, vals):
+        if vals['procurement_type'] == 'buy_stock':
+            vals.update({'procure_method': 'make_to_stock',
+                         'supply_method': 'buy',
+                         })
+        elif vals['procurement_type'] == 'produce_demand':
+            vals.update({'procure_method': 'make_to_order',
+                         'supply_method': 'produce',
+                         })
+        elif vals['procurement_type'] == 'buy_demand':
+            vals.update({'procure_method': 'make_to_order',
+                         'supply_method': 'buy',
+                         })
+        elif vals['procurement_type'] == 'produce_stock':
+            vals.update({'procure_method': 'make_to_stock',
+                         'supply_method': 'produce',
+                         })
+        else:
+            raise ValueError(vals['procurement_type'])
+        return vals
+
+    _columns = {
+        'procurement_type': fields.selection(
+            get_procurement_type_selection,
+            'Procurement Type',
+            required=True,
+            help='On stock, buy: Procurement Method: Make to Stock, '
+                 'Supply Method: Buy.\n'
+                 'On stock, produce: Procurement Method: Make to Stock, '
+                 'Supply Method: Produce.\n'
+                 'On order, buy: Procurement Method: Make to Order, '
+                 'Supply Method: Buy.\n'
+                 'On order, produce: Procurement Method: Make to Order, '
+                 'Supply Method: Produce.\n'
+            ),
+    }
+
+    _defaults = {
+        'procurement_type': 'buy_stock',
+    }
+
+    def onchange_procurement_type(self, cr, uid, ids, type, procurement_type,
+                                  context=None):
+        vals = {'procurement_type': procurement_type}
+        if type != 'service':
+            vals = self._compute_procurement_vals(vals)
+        return {'value': vals}
+
+    def write(self, cr, uid, ids, vals, context=None):
+        for product in self.read(cr, uid, ids, ['type'], context=context):
+            if product['type'] != 'service' and \
+                    vals.get('procurement_type'):
+                vals = self._compute_procurement_vals(vals)
+        res = super(product_template, self).write(cr, uid, ids, vals,
+                                                  context=context)
+        return res
+
+    def create(self, cr, uid, vals, context=None):
+        if vals['type'] != 'service' and vals.get('procurement_type'):
+            vals = self._compute_procurement_vals(vals)
+        return super(product_template, self).create(cr, uid, vals,
+                                                    context=context)
+
+
+class product_product(orm.Model):
+
+    _inherit = 'product.product'
+
+    def onchange_procurement_type(self, cr, uid, ids, type, procurement_type,
+                                  context=None):
+        tmpl_obj = self.pool['product.template']
+        res = tmpl_obj.onchange_procurement_type(cr, uid, ids, type,
+                                                 procurement_type,
+                                                 context=context)
+        return {'value': res['value']}

=== added file 'product_procurement_type/product_view.xml'
--- product_procurement_type/product_view.xml	1970-01-01 00:00:00 +0000
+++ product_procurement_type/product_view.xml	2014-06-04 22:48:39 +0000
@@ -0,0 +1,42 @@
+<?xml version="1.0" encoding="utf-8"?>
+<openerp>
+    <data>
+        <record id="product_normal_form_view_procurement_type" model="ir.ui.view">
+            <field name="name">product.product.procurement_type</field>
+            <field name="model">product.product</field>
+            <field name="inherit_id" ref="procurement.product_form_view_procurement_button"/>
+            <field name="arch" type="xml">
+                <data>
+                    <field name="procure_method" position="before">
+                        <field name="procurement_type" on_change="onchange_procurement_type(type, procurement_type)" attrs="{'invisible':[('type','=','service')]}"/>
+                    </field>
+                    <field name="supply_method" position="attributes">
+                     <attribute name="attrs">{'invisible':[('type','!=','service')]}</attribute>
+                    </field>
+                    <field name="procure_method" position="attributes">
+                     <attribute name="attrs">{'invisible':[('type','!=','service')]}</attribute>
+                    </field>
+                </data>
+            </field>
+        </record>
+        
+        <record model="ir.ui.view" id="product_template_form_view_procurement_type">
+            <field name="name">product.template.procurement_type</field>
+            <field name="model">product.template</field>
+            <field name="inherit_id" ref="procurement.product_template_form_view_procurement"/>
+            <field name="arch" type="xml">
+                <data>
+                    <field name="procure_method" position="before">
+                        <field name="procurement_type" on_change="onchange_procurement_type(type, procurement_type)" attrs="{'invisible':[('type','=','service')]}"/>
+                    </field>
+                    <field name="procure_method" position="attributes">
+                     <attribute name="invisible">{'invisible':[('type','!=','service')]}</attribute>
+                    </field>
+                    <field name="supply_method" position="attributes">
+                     <attribute name="invisible">{'invisible':[('type','!=','service')]}</attribute>
+                    </field>
+                </data>
+            </field>
+        </record>        
+    </data>
+</openerp>

=== added directory 'product_procurement_type/test'
=== added file 'product_procurement_type/test/test_onchange_procurement_type.yml'
--- product_procurement_type/test/test_onchange_procurement_type.yml	1970-01-01 00:00:00 +0000
+++ product_procurement_type/test/test_onchange_procurement_type.yml	2014-06-04 22:48:39 +0000
@@ -0,0 +1,56 @@
+-
+  Testing change of procurement type on product
+-
+  I create a product "Store" with type buy_stock
+-
+  !record {model: product.product, id: product1}:
+    name: Store
+    type: product
+    procurement_type: buy_stock
+-
+  I check if the product is MTS + buy
+-
+  !assert {model: product.product, id: product1}:
+    - procure_method == 'make_to_stock' 
+    - supply_method == 'buy' 
+-
+  I create a product "Curtain" with type produce_demand
+-
+  !record {model: product.product, id: product2}:
+    name: Curtain
+    type: product
+    procurement_type: produce_demand
+-
+  I check if the product is MTO + produce
+-
+  !assert {model: product.product, id: product2}:
+    - procure_method == 'make_to_order'
+    - supply_method == 'produce'
+
+-
+  I create a product "Curtain2" with type produce_stock
+-
+  !record {model: product.product, id: product3}:
+    name: Curtain2
+    type: product
+    procurement_type: produce_stock
+-
+  I check if the product is MTS + produce
+-
+  !assert {model: product.product, id: product3}:
+    - procure_method == 'make_to_stock' 
+    - supply_method == 'produce'
+
+-
+  I create a product "Store2" with type buy_demand
+-
+  !record {model: product.product, id: product4}:
+    name: Store2
+    type: product
+    procurement_type: buy_demand
+-
+  I check if the product is MTO + buy
+-
+  !assert {model: product.product, id: product4}:
+    - procure_method == 'make_to_order' 
+    - supply_method == 'buy' 

=== added directory 'product_procurement_type_dropshipping'
=== added file 'product_procurement_type_dropshipping/__init__.py'
--- product_procurement_type_dropshipping/__init__.py	1970-01-01 00:00:00 +0000
+++ product_procurement_type_dropshipping/__init__.py	2014-06-04 22:48:39 +0000
@@ -0,0 +1,22 @@
+# -*- coding: utf-8 -*-
+##############################################################################
+#
+#    Author: Romain Deheele
+#    Copyright 2014 Camptocamp SA
+#
+#    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 . import product

=== added file 'product_procurement_type_dropshipping/__openerp__.py'
--- product_procurement_type_dropshipping/__openerp__.py	1970-01-01 00:00:00 +0000
+++ product_procurement_type_dropshipping/__openerp__.py	2014-06-04 22:48:39 +0000
@@ -0,0 +1,46 @@
+# -*- coding: utf-8 -*-
+##############################################################################
+#
+#    Author: Romain Deheele
+#    Copyright 2014 Camptocamp SA
+#
+#    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 Procurement Type Dropshipping",
+    "version": "0.1",
+    "author": "Camptocamp",
+    "license": "AGPL-3",
+    "category": "Sales",
+    "description":
+    """
+    Add a choice named 'dropshipping' in procurement_type selection
+    introduced by product_procurement_type addon.
+    Choose "Dropshipping" changes automatically
+    procurement method as "make to order" and supply method as "buy".
+    It checks if at least one supplier is associated on supplier list
+    in product form.
+
+    """,
+    "website": "http://camptocamp.com",
+    "depends": ['product_procurement_type',
+                'sale_dropshipping'],
+    "data": ['product_view.xml'],
+    "demo": [],
+    "test": ['test/test_onchange_procurement_type.yml'],
+    "active": False,
+    "installable": True,
+}

=== added file 'product_procurement_type_dropshipping/product.py'
--- product_procurement_type_dropshipping/product.py	1970-01-01 00:00:00 +0000
+++ product_procurement_type_dropshipping/product.py	2014-06-04 22:48:39 +0000
@@ -0,0 +1,115 @@
+# -*- coding: utf-8 -*-
+##############################################################################
+#
+#    Author: Romain Deheele
+#    Copyright 2014 Camptocamp SA
+#
+#    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 logging
+
+from openerp.osv import orm, fields
+from openerp.tools.translate import _
+from openerp.addons.product_procurement_type.product import get_procurement_type_selection
+
+_logger = logging.getLogger(__name__)
+def get_procurement_type_selection_with_dropshipping():
+    selections = get_procurement_type_selection()
+    selections.append(('direct_delivery', 'Drop Shipping'))
+    return selections
+
+class product_template(orm.Model):
+
+    _inherit = 'product.template'
+
+    def init(self, cr):
+        _logger.info('Migrating product_procurement_type_dropshipping')
+        query = ("UPDATE product_template AS pt "
+                 "SET procurement_type=%(procurement_type)s "
+                 "FROM product_supplierinfo AS psi "
+                 "WHERE procure_method=%(procure_method)s "
+                 "  AND supply_method=%(supply_method)s "
+                 "  AND psi.product_id = pt.id "
+                 "  AND psi.direct_delivery_flag = true")
+        fixes = [{'procurement_type': 'direct_delivery', 'procure_method': 'make_to_order', 'supply_method': 'buy'},
+                 ]
+        for fix in fixes:
+            cr.execute(query, fix)
+
+
+    def get_procurement_type_selection(self, cr, uid, context=None):
+        """
+        Adds type dropshipping in procurement_type selection
+        """
+        return get_procurement_type_selection_with_dropshipping()
+
+    def _compute_procurement_vals(self, vals):
+        if vals['procurement_type'] == 'direct_delivery':
+            vals.update({'procure_method': 'make_to_order',
+                         'supply_method': 'buy',
+                         })
+        else:
+            vals = super(product_template, self)._compute_procurement_vals(vals)
+        return vals
+
+    def _check_sellers(self, cr, uid, ids, procurement_type, context=None):
+        dd_flag = procurement_type == 'direct_delivery'
+        seller_pool = self.pool['product.supplierinfo']
+        seller_ids = seller_pool.search(cr, uid, [('product_id', 'in', ids)],
+                                        context=context)
+        if seller_ids:
+            seller_pool.write(cr, uid, seller_ids,
+                              {'direct_delivery_flag': dd_flag},
+                              context=context)
+        else:
+            if dd_flag:
+                raise orm.except_orm(
+                    _('Warning'),
+                    _('No suppliers defined'))
+
+    _columns = {
+        'procurement_type': fields.selection(
+            get_procurement_type_selection,
+            'Procurement Type',
+            required=True,
+            help='On stock, buy: Procurement Method: Make to Stock, '
+                 'Supply Method: Buy.\n'
+                 'On stock, produce: Procurement Method: Make to Stock, '
+                 'Supply Method: Produce.\n'
+                 'On order, buy: Procurement Method: Make to Order, '
+                 'Supply Method: Buy.\n'
+                 'On order, produce: Procurement Method: Make to Order, '
+                 'Supply Method: Produce.\n'
+                 'DropShipping: Procurement Method: Make to Order, '
+                 'Supply Method: Buy + supplier does drop shipping'),
+    }
+
+    def onchange_procurement_type(self, cr, uid, ids, type,
+                                  procurement_type, context=None):
+        res = super(product_template, self).\
+            onchange_procurement_type(cr,
+                                      uid,
+                                      ids,
+                                      type,
+                                      procurement_type)
+        if type != 'service':
+            res['value'].update(
+                self._compute_procurement_vals(
+                    {'procurement_type': procurement_type})
+                )
+            self._check_sellers(cr, uid, ids, procurement_type,
+                                context=context)
+        return res
+

=== added file 'product_procurement_type_dropshipping/product_view.xml'
--- product_procurement_type_dropshipping/product_view.xml	1970-01-01 00:00:00 +0000
+++ product_procurement_type_dropshipping/product_view.xml	2014-06-04 22:48:39 +0000
@@ -0,0 +1,17 @@
+<?xml version="1.0" encoding="utf-8"?>
+<openerp>
+    <data>
+        <record id="view_product_supplier_context_dropshipping" model="ir.ui.view">
+            <field name="name">product.supplier.context.dropshipping</field>
+            <field name="model">product.product</field>
+            <field name="inherit_id" ref="purchase.view_product_supplier_inherit"/>
+            <field name="arch" type="xml">
+                <data>
+                    <field name="seller_ids" position="attributes">
+                     <attribute name="context">{'uom_id': uom_id, 'default_direct_delivery_flag': procurement_type == 'direct_delivery'}</attribute>
+                    </field>
+                </data>
+            </field>
+        </record>
+    </data>
+</openerp>

=== added directory 'product_procurement_type_dropshipping/test'
=== added file 'product_procurement_type_dropshipping/test/test_onchange_procurement_type.yml'
--- product_procurement_type_dropshipping/test/test_onchange_procurement_type.yml	1970-01-01 00:00:00 +0000
+++ product_procurement_type_dropshipping/test/test_onchange_procurement_type.yml	2014-06-04 22:48:39 +0000
@@ -0,0 +1,52 @@
+-
+  Testing change of procurement type on product
+-
+  I create a product "Jalousie"
+-
+  !record {model: product.product, id: product3}:
+    name: Jalousie
+    type: product
+-
+  I define the product with dropshipping
+  but it fails because the product has no suppliers defined.
+-
+  !python {model: product.product}: |
+    from openerp.osv import orm
+    product3_id = ref("product3")
+    try:
+        self.write(cr, uid, [product3_id], {'procurement_type': 'direct_delivery'})
+    except orm.except_orm as err:
+        assert 'Error Expected'
+-
+  I create a supplier "Dupont"
+-
+  !record {model: res.partner, id: partner1}:
+    name: Dupont
+    supplier: True
+-
+  I set this supplier on the product "Jalousie"
+-
+  !record {model: product.supplierinfo, id: psupinfo1}:
+    name: partner1
+    product_id: product3
+    min_qty: 1.0
+-
+  I define dropshipping on the product.
+-
+  !python {model: product.product}: |
+    product3_id = ref("product3")
+    try:
+        values = self.onchange_procurement_type(cr, uid, [product3_id], 'product', 'direct_delivery')
+        self.write(cr, uid, [product3_id], values['value'])
+    except:
+        raise AssertionError('Expected no error')
+-
+  I check if the product is now made to order
+-
+  !assert {model: product.product, id: product3}:
+    - procure_method == 'make_to_order'
+-
+  I check if supplier is checked as doing dropshipping
+-
+  !assert {model: product.supplierinfo, id: psupinfo1}:
+    - direct_delivery_flag == 1
\ No newline at end of file