Review: Disapprove Hello, Sorry, this merge proposal is not correct and in fact not necessary. As invitu explains on the bug, it is trying to workaround a regression that was introduced in [1] due to a misunderstanding of what the "invoiced_rate" field means.
This field is meant to show how much of the so/po has been *invoiced*, not how much has been paid (hence the name). How much has been paid is a concern of the accounting department usually, not of the sales/purchase management. If needed, this could be implemented a new feature with a "paid_rate" field, but even this is probably unnecessary, as it's visible on invoices, and that's where it should be. The original commit [1] simply needs to be reverted (see with JVO) Thanks! [1] http://bazaar.launchpad.net/~openerp/openobject-addons/6.0/revision/4593 -- https://code.launchpad.net/~openerp-dev/openobject-addons/6.0-bug-795026-rha/+merge/64362 Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-addons/6.0-bug-795026-rha. _______________________________________________ Mailing list: https://launchpad.net/~openerp-dev-gtk Post to : [email protected] Unsubscribe : https://launchpad.net/~openerp-dev-gtk More help : https://help.launchpad.net/ListHelp
