Merge lp:~openerp-dev/openobject-addons/6.1-opw-580818-nep into lp:openobject-addons/6.1

Proposed by Nehal Panchal (OpenERP)
Status: Approved
Approved by: Naresh(OpenERP)
Approved revision: 7048
Proposed branch: lp:~openerp-dev/openobject-addons/6.1-opw-580818-nep
Merge into: lp:openobject-addons/6.1
Diff against target: 13 lines (+0/-3)
1 file modified
sale/sale.py (+0/-3)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/6.1-opw-580818-nep
Reviewer Review Type Date Requested Status
Naresh(OpenERP) (community) Approve
Review via email: mp+132046@code.launchpad.net

Description of the change

Hello,

In Sales Order, when advance invoice is created and the invoice is validated the progress bar shows correct indication. But when the invoice is paid, the progress bar is 100% which is wrong.

For example:
1. Create Sales Order with price:100 of any Product.
2. Confirm Sales Order then create an advance invoice with price:50.
3. Validate the invoice, progress bar shows 50% which is correct.
4. Paid the invoice, progress bar shows 100% which is incorrect.

This fixes the issue.

Thanks

To post a comment you must log in.
Revision history for this message
Naresh(OpenERP) (nch-openerp) :
review: Approve
Revision history for this message
Naresh(OpenERP) (nch-openerp) wrote :

Hello,

This bug was qualified as Not Relevant on Trunk as there is no progress bar anymore in treeview of Sale order and so this bug has no meaning anymore. If this Merge Proposal could not be merged in v6.1 at the release of v7.0, it will be closed.

Thanks,
Naresh Soni

Unmerged revisions

7048. By Nehal Panchal (OpenERP)

[FIX] sale: Fixed the issue of wrong indication of progress bar for partially created invoice of SO

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'sale/sale.py'
--- sale/sale.py 2012-10-09 12:16:14 +0000
+++ sale/sale.py 2012-10-30 10:35:29 +0000
@@ -130,9 +130,6 @@
130 def _invoiced_rate(self, cursor, user, ids, name, arg, context=None):130 def _invoiced_rate(self, cursor, user, ids, name, arg, context=None):
131 res = {}131 res = {}
132 for sale in self.browse(cursor, user, ids, context=context):132 for sale in self.browse(cursor, user, ids, context=context):
133 if sale.invoiced:
134 res[sale.id] = 100.0
135 continue
136 tot = 0.0133 tot = 0.0
137 for invoice in sale.invoice_ids:134 for invoice in sale.invoice_ids:
138 if invoice.state not in ('draft', 'cancel'):135 if invoice.state not in ('draft', 'cancel'):