[6.1] [mrp] Translate string uses concatenation instead of modifier resulting in incorrect stock move name

[Graeme Gellatly]

@@ -926,7 +926,7 @@
         stock_move = self.pool.get('stock.move')
         source_location_id = production.product_id.product_tmpl_id.property_stock_production.id
         destination_location_id = production.location_dest_id.id
- move_name = _('PROD: %s') + production.name
+ move_name = _('PROD: %s') % production.name
         data = {
             'name': move_name,
             'date': production.date_planned,

File is mrp.py - sorry didn't post patch as I've had to make quite a lot of changes already to that file but is simple enough.

[Ravish(OpenERP) (rmu-openerp)]

Hello,

I have checked with using % . It is printing PROD: MO/00016(just for example) instead of PROD: %sMO/00015.
So I am assign it to responsible team.

Thanks!!

[Mayur Maheshwari(OpenERP) (mma-openerp)]

Hello Graeme Gellatly,

Thanks for your contribution .
It has been fixed in lp:~openerp-dev/openobject-addons/trunk-bug-938251-mma With
Revision ID: <email address hidden>
Revision num: 6668

It will be available in trunk soon.

[Kirti Savalia(OpenERP) (ksa-openerp)]

Hello,

It has been fixed in http://bazaar.launchpad.net/~openerp-dev/openobject-addons/6.1-opw-579756-ksa With
Revision-id: <email address hidden>
Revno: 7007

It will be available in stable soon.

Thanks

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

Hello,

I see the bug was fixed on trunk (great) but the merge proposal was rejected on stable ? Why ? It is 0-risk, and while the bug is not harmful, it gives a bad impression to end users.