Merge lp:~openerp-dev/openobject-addons/7.0-opw-586615-pna into lp:openobject-addons/7.0

[Pinakin Nayi (OpenERP)]

Hello ,

I Fixed issue in procurement module for reordering rules and add constraint to restrict qty.

Steps to reproduce :
Create Reordering rules for product, now enter minimum qty more than maximum qty it will save it without warning.

Expected Result :
Not allowing minimum qty more than maximum qty.

Thanks,
pna

[Naresh(OpenERP) (nch-openerp)]

Hi,

Please remove the changes concerning the *.pot. The terms should not be added manually in these files, they are meant to be automatically created on each re-synchronization of the translation templates.

Thanks,
Naresh Soni
OpenERP Enterprise Services

review: Needs Fixing

[lp:~openerp-dev/openobject-addons/7.0-opw-586615-pna]

8774. By Pinakin Nayi (OpenERP) on 2013-06-17

[REV]procurement: remove unwanted pot changes

8775. By Pinakin Nayi (OpenERP) on 2013-06-17

[REV]procurement: remove unwanted pot changes

[Pinakin Nayi (OpenERP) (pna-openerp)]

Hello Sir,

I just removed unwanted *.pot changes as per suggestion.

Thanks,
pna

Unmerged revisions

8775. By Pinakin Nayi (OpenERP) on 2013-06-17

[REV]procurement: remove unwanted pot changes

8774. By Pinakin Nayi (OpenERP) on 2013-06-17

[REV]procurement: remove unwanted pot changes

8773. By Pinakin Nayi (OpenERP) on 2013-02-27

[IMP]procurement:whem set reordering rules it takes min qty more than max qty so add constraint

=== modified file 'procurement/procurement.py'
--- procurement/procurement.py	2013-06-07 11:38:29 +0000
+++ procurement/procurement.py	2013-06-17 12:52:43 +0000
@@ -510,6 +510,14 @@
             
         return True
 
+    def _check_product_qty(self, cr, uid, ids, context=None):
+        if not context:
+            context = {}
+        for rule in self.browse(cr, uid, ids, context=context):
+            if rule.product_min_qty > rule.product_max_qty :
+                return False
+        return True
+
     _columns = {
         'name': fields.char('Name', size=32, required=True),
         'active': fields.boolean('Active', help="If the active field is set to False, it will allow you to hide the orderpoint without removing it."),
@@ -544,6 +552,7 @@
     ]
     _constraints = [
         (_check_product_uom, 'You have to select a product unit of measure in the same category than the default unit of measure of the product', ['product_id', 'product_uom']),
+        (_check_product_qty, 'Please verify the minimum quantity should not be more than maximum quantity', ['product_min_qty', 'product_max_qty']),
     ]
 
     def default_get(self, cr, uid, fields, context=None):