c2c-oerpscenario team mailing list archive
-
c2c-oerpscenario team
-
Mailing list archive
-
Message #36763
[Bug 724131] Re: Purchase lines without Invoice Lines Relationship
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
References