openerp-dev-web team mailing list archive

["Atul Patel\(OpenERP\)" <atp@xxxxxxxxxxx>]

Atul Patel(OpenERP) has proposed merging lp:~openerp-dev/openobject-addons/trunk-bug-partner_segmentation_ind-atp into lp:~openerp-dev/openobject-addons/trunk-bugfixes-Ind.

Requested reviews:
  Husen Daudi (husendaudi)

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-partner_segmentation_ind-atp/+merge/60453

Hello,

Sales/Configuration/Automated Actions/Partners Segmentation:
------------------------------------------------------------
 - Fix Duplicate partner key error of compute segmentation button in partner segmentation form.
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-partner_segmentation_ind-atp/+merge/60453
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bugfixes-Ind.

=== modified file 'crm/crm_segmentation.py'
--- crm/crm_segmentation.py	2011-01-14 00:11:01 +0000
+++ crm/crm_segmentation.py	2011-05-10 05:09:24 +0000
@@ -98,8 +98,13 @@
                     partners.remove(pid)
 
             for partner_id in partners:
-                cr.execute('insert into res_partner_category_rel (category_id,partner_id) \
-                        values (%s,%s)', (categ['categ_id'][0], partner_id))
+                cr.execute('select partner_id from res_partner_category_rel where category_id = %s', (categ['categ_id'][0],))
+                partner_ids = [x[0] for x in cr.fetchall()]
+                if partner_id in partner_ids:
+                    continue
+                else:    
+                    cr.execute('insert into res_partner_category_rel (category_id,partner_id) \
+                      values (%s,%s)', (categ['categ_id'][0], partner_id))
 
             self.write(cr, uid, [id], {'state':'not running', 'partner_id':0})
         return True

=== modified file 'product/product.py'
--- product/product.py	2011-05-02 18:46:43 +0000
+++ product/product.py	2011-05-10 05:09:24 +0000
@@ -218,7 +218,7 @@
         'type' : lambda *a : 'normal',
     }
 
-    _order = "sequence"
+    _order = "sequence, name"
     def _check_recursion(self, cr, uid, ids, context=None):
         level = 100
         while len(ids):