Merge lp://staging/~openerp-dev/openobject-addons/6.0-opw-5625-ach into lp://staging/openobject-addons/6.0

Proposed by Anup(SerpentCS)
Status: Merged
Approved by: Jay Vora (Serpent Consulting Services)
Approved revision: no longer in the source branch.
Merged at revision: 4689
Proposed branch: lp://staging/~openerp-dev/openobject-addons/6.0-opw-5625-ach
Merge into: lp://staging/openobject-addons/6.0
Diff against target: 45 lines (+7/-3)
2 files modified
account_voucher/wizard/account_statement_from_invoice.py (+6/-2)
account_voucher/wizard/account_statement_from_invoice_view.xml (+1/-1)
To merge this branch: bzr merge lp://staging/~openerp-dev/openobject-addons/6.0-opw-5625-ach
Reviewer Review Type Date Requested Status
Jay Vora (Serpent Consulting Services) (community) Approve
Rifakat Husen (OpenERP) (community) Approve
Review via email: mp+62470@code.staging.launchpad.net

Description of the change

Hello,

   The Partially Paid Invoices were not coming in the Bank Statement when importing invoices due to a domain.
  Even after fixing the domain the amount was not coming and was giving the wrongly calculated amount.
Fixed that too.

Thanks.

To post a comment you must log in.
Revision history for this message
Rifakat Husen (OpenERP) (rha-openerp) :
review: Approve
Revision history for this message
Jay Vora (Serpent Consulting Services) (jayvora) wrote :

Anup,

Thanks for the fix, its really a good catch it would be a good change to add it into next release.

I prefer adding a comment while you code something special.

Thanks.

review: Approve
Revision history for this message
Anup(SerpentCS) (anup-serpent) wrote :

Hello Jay,

   I totally agree with you. I have added a comment for the fix.

Thanks.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.