anglo saxon - analytic accounts should only be set for P&L accounts

Bug #921877 reported by Ferdinand
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Fix Released
Medium
OpenERP R&D Addons Team 3

Bug Description

openobject-addons/account_anglo_saxon/invoice.py

IMHO for the moves gerenated along with customer invoices
analyitc account should only exists for moves on P&L accounts
and not on stock input/output accounts

this way the analytic account will show
* the net sales amount
* the cost of good sold
= contribution

Related branches

Revision history for this message
Ferdinand (office-chricar) wrote :
Revision history for this message
Amit Parik (amit-parik) wrote :

Hello Dr Ferdinand,

I have checked your issue but I am not able to understand you properly.

I don't understand why you describe that we don't allow analytic account there?
Also I don't agreed with your patch.

So would you please elaborate more on this issue and If you provide a proper use case of this then that should be batter.

Thanks and waiting for your reply!

Changed in openobject-addons:
status: New → Incomplete
Revision history for this message
Ferdinand (office-chricar) wrote :

the module creates 2 moves on the analytic account one with positive, one with negative amount, and hence balance 0,

querying the analytic account must only use P&L accounts or all but then other than P&L must not have analytic account set.

Revision history for this message
Amit Parik (amit-parik) wrote :

Hello Dr Ferdinand,

Thanks a lot for your reply!

I have faced the same problem as you described and I have tried your patch it's working fine as well as nice catch..;-).

Thanks for your contribution.

Changed in openobject-addons:
assignee: nobody → OpenERP R&D Addons Team 3 (openerp-dev-addons3)
importance: Undecided → Medium
status: Incomplete → Confirmed
Revision history for this message
qdp (OpenERP) (qdp) wrote :

Hello Ferdinand,

thanks for the bug report. It has been fixed and released in
revision: 6595
revision-id: <email address hidden>

Quentin

Changed in openobject-addons:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.