Merge lp:~eoc/server-env-tools/6.1-mass_editing-fix_bug_1187937 into lp:~openerp-community/server-env-tools/6.1-mass_editing

[Mariano Ruiz]

This branch fix the error #1187937.

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

Thanks for the fix! You might want to use model_ids.append(found_model_ids[0]) though.

review: Needs Fixing

[lp:~eoc/server-env-tools/6.1-mass_editing-fix_bug_1187937]

42. By Mariano Ruiz on 2013-06-05

[FIX] Better operation syntax

[Mariano Ruiz (marianoruiz)]

Done! Rev 42.

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

You're quick! Thanks for the change.

review: Approve

[Holger Brunn (Therp) (hbrunn)]

review: Approve (code review)

[Guewen Baconnier @ Camptocamp (gbaconnier-c2c)]

LGTM

review: Approve (code review, no test)

[Guewen Baconnier @ Camptocamp (gbaconnier-c2c)]

Is it really intended to be merge in lp:~openerp-community/server-env-tools/6.1-mass_editing and not in lp:server-env-tools/6.1 ?

review: Needs Information

[Holger Brunn (Therp) (hbrunn)]

I ask myself the same thing

review: Needs Information

[debaetsr (rubendebaets)]

Hi friend!

Look at that! Wooow! It's so amazing and interesting, and even sometimes shocking! Just take a look here <http://suit.kitefamily.us/e4ccrzv>

Thanks for your consideration, ruben

=== modified file 'mass_editing/mass_editing.py'
--- mass_editing/mass_editing.py	2013-05-07 18:28:03 +0000
+++ mass_editing/mass_editing.py	2013-06-05 21:51:27 +0000
@@ -50,7 +50,7 @@
                 for key, val in active_model_obj._inherits.items():
                     found_model_ids = model_obj.search(cr, uid, [('model', '=', key)])
                     if found_model_ids:
-                        model_ids += found_model_ids[0]
+                        model_ids.append(found_model_ids[0])
         return {'value': {'model_ids': [(6, 0, model_ids)]}}
 
     def create_action(self, cr, uid, ids, context=None):