[REF] account :In balance sheet report wizard add reserve_account_id as per xrg patch

bzr revid: ara@tinyerp.com-20101227054032-8ic2n16ahkwnzo2u
2010-12-27 11:10:32 +05:30
parent 81381fb098
commit 944a8f501e
2 changed files with 35 additions and 18 deletions
@@ -32,36 +32,47 @@ class account_bs_report(osv.osv_memory):
    _inherit = "account.common.account.report"
    _description = 'Account Balance Sheet Report'

+    def _get_def_reserve_account(self, cr, uid, context=None):
+        chart_id = self._get_account(cr, uid, context=context)
+        # Reuse the onchange function, for symmetry
+        res = self.onchange_chart_id(cr, uid, chart_id, context=context)
+        if not res:
+            return False
+        return res['value']['reserve_account_id']
+
    _columns = {
        'display_type': fields.boolean("Landscape Mode"),
-        'reserve_account_id': fields.many2one('account.account', 'Reserve & Profit/Loss Account',required = True,
-                                      help='This Account is used for transferring Profit/Loss(If It is Profit: Amount will be added, Loss : Amount will be duducted.), Which is calculated from Profilt & Loss Report', domain = [('type','=','payable')]),
+        'reserve_account_id': fields.many2one('account.account', 'Reserve & Profit/Loss Account',
+                                      required=True,
+                                      help='This Account is used for transfering Profit/Loss ' \
+                                           '(Profit: Amount will be added, Loss: Amount will be duducted), ' \
+                                           'which is calculated from Profilt & Loss Report',
+                                      domain = [('type','=','payable')]),
    }

    _defaults={
        'display_type': True,
        'journal_ids': [],
+        'reserve_account_id': _get_def_reserve_account,
    }

-    def fields_view_get(self, cr, uid, view_id=None, view_type='form', context=None, toolbar=False, submenu=False):
-        res = super(account_bs_report, self).fields_view_get(cr, uid, view_id=view_id, view_type=view_type, context=context, toolbar=toolbar, submenu=False)
-        doc = etree.XML(res['arch'])
-        nodes = doc.xpath("//field[@name='journal_ids']")
-        for node in nodes:
-            node.set('readonly', '1')
-            node.set('required', '0')
-        res['arch'] = etree.tostring(doc)
-        return res
+    def onchange_chart_id(self, cr, uid, chart_id, context=None):
+        if not chart_id:
+            return False
+        account = self.pool.get('account.account').browse(cr, uid, chart_id , context=context)
+        if not account.company_id.property_reserve_and_surplus_account:
+            return False # We cannot raise an exception, because that's before the wizard
+        return { 'value': {'reserve_account_id': account.company_id.property_reserve_and_surplus_account.id}}
+

    def _print_report(self, cr, uid, ids, data, context=None):
        if context is None:
            context = {}
-        data = self.pre_print_report(cr, uid, ids, data, context=context)
-        account = self.pool.get('account.account').browse(cr, uid, data['form']['chart_account_id'], context=context)
-        if not account.company_id.property_reserve_and_surplus_account:
+        data['form'].update(self.read(cr, uid, ids, ['display_type','reserve_account_id'])[0])
+        if not data['form']['reserve_account_id']:
+            # only in < v6.1, where orm_memory does not honor required fields
            raise osv.except_osv(_('Warning'),_('Please define the Reserve and Profit/Loss account for current user company !'))
-        data['form']['reserve_account_id'] = account.company_id.property_reserve_and_surplus_account.id
-        data['form'].update(self.read(cr, uid, ids, ['display_type'])[0])
+        data = self.pre_print_report(cr, uid, ids, data, context=context)
        if data['form']['display_type']:
            return {
                'type': 'ir.actions.report.xml',
@@ -77,4 +88,4 @@ class account_bs_report(osv.osv_memory):

 account_bs_report()

-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
@@ -9,6 +9,12 @@
            <field name="inherit_id" ref="account.account_common_report_view" />
            <field name="arch" type="xml">
            <data>
+            <xpath expr="//field[@name='chart_account_id']" position="replace">
+                <field name="chart_account_id" widget='selection' onchange="onchange_chart_id"/>
+            </xpath>
+            <xpath expr="//field[@name='journal_ids']" position="replace">
+                <field name="journal_ids" colspan="4" nolabel="1" required="0" readonly="1"/>
+            </xpath>
            <xpath expr="/form/label[@string='']" position="replace">
                <separator string="Balance Sheet" colspan="4"/>
                <label nolabel="1" colspan="4" string="This report allows you to print or generate a pdf of your trial balance allowing you to quickly check the balance of each of your accounts in a single report"/>
@@ -16,7 +22,7 @@
            <xpath expr="//field[@name='target_move']" position="after">
                <field name="display_account"/>
                <field name="display_type"/>
-                <field name="reserve_account_id" required="0" invisible="1"/>
+                <field name="reserve_account_id" required="1"/>
                <newline/>
            </xpath>
            </data>