[IMP] merge add company_registry, but use a button instead of onchange to generate the footer

bzr revid: al@openerp.com-20110824234829-10lvn7cs0uuxaty4

2 files changed, 31 insertions, 28 deletions

diff --git a/openerp/addons/base/base_update.xml b/openerp/addons/base/base_update.xml
index 6f4dbf6a1f9..cef36dda8e4 100644
--- a/openerp/addons/base/base_update.xml
+++ b/openerp/addons/base/base_update.xml
@@ -224,16 +224,18 @@
                             <field name="city"/>
                             <field name="country_id"/>
                             <field name="state_id"/>
-                            <field name="phone" on_change="on_change_header(phone, email, fax, website, vat)"/>
-                            <field name="email" on_change="on_change_header(phone, email, fax, website, vat)"/>
-                            <field name="fax" on_change="on_change_header(phone, email, fax, website, vat)"/>
-                            <field name="website" on_change="on_change_header(phone, email, fax, website, vat)"/>
-                            <field name="vat" on_change="on_change_header(phone, email, fax, website, vat)"/>
+                            <field name="phone"/>
+                            <field name="email"/>
+                            <field name="fax"/>
+                            <field name="website"/>
+                            <field name="vat"/>
+                            <field name="company_registry"/>
                             <separator string="Header/Footer of Reports" colspan="4"/>
                             <group colspan="4" col="3">
                                 <field name="rml_header1" colspan="3"/>
                                 <newline/>
-                                <field name="rml_footer1" colspan="3"/>
+                                <field name="rml_footer1" colspan="2"/>
+                                <button name="generate_header" string="Generate" type="object" icon="gtk-go-forward"/>
                                 <newline/>
                                 <field name="rml_footer2" colspan="2"/>
                                 <button name="%(bank_account_update)d" string="Set Bank Accounts" type="action" icon="gtk-go-forward"/>
@@ -242,10 +244,10 @@
                                 <button name="%(preview_report)d" string="Preview Header" type="action" icon="gtk-print"/>
                             </group>
                         </page>
-                        <page string="Header/Footer" groups="base.group_extended">
+                        <page string="External Template" groups="base.group_extended">
                             <field colspan="4" name="rml_header" nolabel="1"/>
                         </page>
-                        <page string="Internal Header/Footer" groups="base.group_extended">
+                        <page string="Internal Template" groups="base.group_extended">
                             <separator string="Portrait" colspan="2"/>
                             <separator string="Landscape" colspan="2"/>
                             <field colspan="2" name="rml_header2" nolabel="1"/>
diff --git a/openerp/addons/base/res/res_company.py b/openerp/addons/base/res/res_company.py
index fc63ac42dd9..e1aa5e6b5e9 100644
--- a/openerp/addons/base/res/res_company.py
+++ b/openerp/addons/base/res/res_company.py
@@ -88,18 +88,6 @@ class res_company(osv.osv):
                         result[company.id][field] = address[field] or False
         return result
 
-
-    def _get_bank_data(self, cr, uid, ids, field_names, arg, context=None):
-        """ Read the 'address' functional fields. """
-        result = {}
-        for company in self.browse(cr, uid, ids, context=context):
-            r = []
-            for bank in company.bank_ids:
-                if bank.footer:
-                    r.append(bank.name_get(context=context)[0][1])
-            result[company.id] = ' | '.join(r)
-        return result
-
     def _set_address_data(self, cr, uid, company_id, name, value, arg, context=None):
         """ Write the 'address' functional fields. """
         company = self.browse(cr, uid, company_id, context=context)
@@ -114,6 +102,16 @@ class res_company(osv.osv):
                 address_obj.create(cr, uid, {name: value or False, 'partner_id': company.partner_id.id}, context=context)
         return True
 
+    def _get_bank_data(self, cr, uid, ids, field_names, arg, context=None):
+        """ Read the 'address' functional fields. """
+        result = {}
+        for company in self.browse(cr, uid, ids, context=context):
+            r = []
+            for bank in company.bank_ids:
+                if bank.footer:
+                    r.append(bank.name_get(context=context)[0][1])
+            result[company.id] = ' | '.join(r)
+        return result
 
     _columns = {
         'name': fields.char('Company Name', size=64, required=True),
@@ -143,15 +141,8 @@ class res_company(osv.osv):
         'fax': fields.function(_get_address_data, fnct_inv=_set_address_data, size=64, type='char', string="Fax", multi='address'), 
         'website': fields.related('partner_id', 'website', string="Website", type="char", size=64), 
         'vat': fields.related('partner_id', 'vat', string="Tax ID", type="char", size=32), 
+        'company_registry': fields.char('Company Registry', size=64),
     }
-    def on_change_header(self, cr, uid, ids, phone, email, fax, website, vat, context={}):
-        val = []
-        if phone: val.append(_('Phone: ')+phone)
-        if fax: val.append(_('Fax: ')+fax)
-        if website: val.append(_('Website: ')+website)
-        if vat: val.append(_('VAT: ')+vat)
-        return {'value': {'rml_footer1':' | '.join(val)}}
-
 
     def _search(self, cr, uid, args, offset=0, limit=None, order=None,
             context=None, count=False, access_rights_uid=None):
@@ -234,6 +225,16 @@ class res_company(osv.osv):
         self.cache_restart(cr)
         return super(res_company, self).write(cr, *args, **argv)
 
+    def generate_header(self, cr, uid, ids, context=None):
+        for c in self.browse(cr, uid, ids, context=context):
+            val = []
+            if c.phone: val.append(_('Phone: ')+c.phone)
+            if c.fax: val.append(_('Fax: ')+c.fax)
+            if c.website: val.append(_('Website: ')+c.website)
+            if c.vat: val.append(_('VAT: ')+c.vat)
+            if c.company_registry: val.append(_('Reg: ')+c.company_registry)
+            self.write(cr,uid, [c.id], {'rml_footer1':' | '.join(val)}, context)
+
     def _get_euro(self, cr, uid, context={}):
         try:
             return self.pool.get('res.currency').search(cr, uid, [])[0]