Hello, Kriti and Rucha,
i seems quite better now. But eventually, let me criticize a bit the signature of the new _prepare_order_line_invoice(self, cr, uid, orders, line_ids, *args) method you introduced here http://bazaar.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-724131-ksa/revision/5491 Two things don't feel right for me: 1) why pass the orders param. Eventually pass just the order (although you can re(browse it from the order line) but is there any reason to pass the others orders? I mean may be but I'm just curious. 2) the other argument is line_ids. Too me it seems wrong as it has the _id suffix while it's not a id integer but a browse object and also it's in plural while it's actually a single order line. This seems rather confusing to me when you then have in the code things like if line_ids.product_id... Wouldn't it be better to just call it order_line instead? Thanks. ** Changed in: openobject-addons Status: Fix Committed => New -- You received this bug notification because you are a member of C2C OERPScenario, which is subscribed to OpenERP Project Group. https://bugs.launchpad.net/bugs/724131 Title: Purchase lines without Invoice Lines Relationship Status in OpenERP Addons (modules): New Bug description: Hello, Using branch addons-6.0 revision number: 4422 I created a purchase order, but the relationship between purchase order line and invoice line does not exist, the field purchase.order.line.invoice_lines is always empty. Renato Lima - http://www.akretion.com To manage notifications about this bug go to: https://bugs.launchpad.net/openobject-addons/+bug/724131/+subscriptions _______________________________________________ Mailing list: https://launchpad.net/~c2c-oerpscenario Post to : [email protected] Unsubscribe : https://launchpad.net/~c2c-oerpscenario More help : https://help.launchpad.net/ListHelp
