bank statement - default value for balance_start

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

Bug Description

In addons/account/account_bank_statement.py, class account.bank.statement, the default value for balance_start is calculated in the _default_balance_start method. However, in that method, the select is executed always against journal_id = 1. The default value for journal_id is calculated in the _default_journal_id method, where the journal selected is filtered by a domain clause based on the context. I think that the _default_balance_start method should also select the journal as the _default_journal_id method is done instead of using journal_id = 1.

Tags: 6.0.2 low

Related branches

Changed in openobject-addons:
assignee: nobody → OpenERP R&D Addons Team 3 (openerp-dev-addons3)
importance: Undecided → Low
status: New → Confirmed
Revision history for this message
Mustufa Rangwala (Open ERP) (mra-tinyerp) wrote :

We can remove _default_balance_start method as we have already defined onchange_journal_id which sets the starting balance on bank statement.

Thanks Manu,
mra

Changed in openobject-addons:
milestone: none → 6.1
Changed in openobject-addons:
status: Confirmed → Fix Committed
Revision history for this message
Bharat Devnani (Open ERP) (bde-openerp) wrote :

Hello Manu,

The Revision Number and Revision ID of this bug are as follows:

Revision Number : 4836
Revision ID : bde@tiny2-desktop-20110630134250-1qu7vdtvtnrb3266

Thanks for reporting this bug. This bug is been solved and will be merged in addons soon.

Thanks & Regards,
Devnani Bharat R.

qdp (OpenERP) (qdp)
Changed in openobject-addons:
status: Fix Committed → 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.