From dd68cae5cb1e18b7b9bab0a57cc75327b72093a2 Mon Sep 17 00:00:00 2001
From: Nhomar Hernandez <nhomar@gmail.com>
Date: Mon, 17 Jun 2013 14:03:54 -0530
Subject: [PATCH 1/2] [FIX] A comma, dont allow install the modules, sorry for
 the direct commit and push.

---
 account_financial_report/__openerp__.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/account_financial_report/__openerp__.py b/account_financial_report/__openerp__.py
index ff62479f..ad047711 100644
--- a/account_financial_report/__openerp__.py
+++ b/account_financial_report/__openerp__.py
@@ -42,7 +42,7 @@ In the [ Account's Sign on Reports ] Section in the Company will be able to set
     "init_xml" : [],
     "demo_xml" : [],
     "update_xml" : [
-        "security/ir.model.access.csv"
+        "security/ir.model.access.csv",
         "security/security.xml",
         "view/report.xml", 
         "view/wizard.xml",

From 564ae0f0f07cd2ffafeb4b61972be81670740f4e Mon Sep 17 00:00:00 2001
From: Luis Escobar <vxkikevx@gmail.com>
Date: Tue, 18 Jun 2013 13:14:59 -0530
Subject: [PATCH 2/2] [IMP] Modified view to version 7.0

---
 .../view/account_financial_report_view.xml    | 108 ++++++------------
 account_financial_report/view/wizard.xml      |  93 +++++++--------
 2 files changed, 74 insertions(+), 127 deletions(-)

diff --git a/account_financial_report/view/account_financial_report_view.xml b/account_financial_report/view/account_financial_report_view.xml
index 6be04f89..40cd1d8f 100644
--- a/account_financial_report/view/account_financial_report_view.xml
+++ b/account_financial_report/view/account_financial_report_view.xml
@@ -1,55 +1,44 @@
 <?xml version="1.0" encoding="utf-8"?>
 <openerp>
     <data>
-
         <record id="account_financial_report_view" model="ir.ui.view">
             <field name="name">Account Financial Report (Template)</field>
             <field name="model">afr</field>
             <field name="type">form</field>
             <field name="arch" type="xml">
-            <form string="Account Financial Report (Templates)">
-                <group colspan="4">
-                    <field name="name" colspan="4"/>
-                </group>
-                <group colspan="4">
-                    <group colspan="2">
-                        <separator string="Company Info" colspan="4"/>
-                        <field name="company_id" on_change='onchange_company_id(company_id)' colspan="4"/>
+                <form string="Account Financial Report (Templates)" version="7.0">
+                    <h2><label for="name"/></h2> 
+                    <field name="name" nolabel="1"/>
+                <group>
+                    <group string="Company Info" >
+                        <field name="company_id" on_change='onchange_company_id(company_id)' />
                         <field name="currency_id" 
                         domain="[('company_id','=',company_id)]" 
-                        attrs="{'readonly':[('analytic_ledger','=',True)]}" 
-                        colspan="4"/>
+                        attrs="{'readonly':[('analytic_ledger','=',True)]}" />
                     </group>
-                    <group colspan="2">
-                        <separator string="Report Info" colspan="4"/>
+                    <group string="Report Info" >
                         <field 
                             name="inf_type" 
-                            colspan="4"
-                            on_change='onchange_inf_type(inf_type)'
-                            />
-                        <field name="columns" on_change='onchange_columns(columns,fiscalyear_id,period_ids)' colspan="4"/>
-                        <field name="analytic_ledger" colspan="4"
+                            on_change='onchange_inf_type(inf_type)' />
+                        <field name="columns" on_change='onchange_columns(columns,fiscalyear_id,period_ids)' />
+                        <field name="analytic_ledger" 
                         attrs="{'readonly':['|',('columns','!=','four'),('inf_type','!=','BS')],}" 
-                        on_change='onchange_analytic_ledger(company_id,analytic_ledger)'
-                        />
+                        on_change='onchange_analytic_ledger(company_id,analytic_ledger)' />
                     </group>
                 </group>
-                <group colspan='4'>
-                    <group colspan="2">
-                        <separator string="Accounts Info" colspan="4"/>
-                        <field name="display_account" required="True" colspan="4"/>
-                        <field name="display_account_level" required="True" colspan="4"/>
+                <group>
+                    <group string="Accounts Info" >
+                        <field name="display_account" required="True" />
+                        <field name="display_account_level" required="True" />
                     </group>
-                    <group colspan="2">
-                        <separator string="FiscalYear Info" colspan="4"/>
-                        <field name="fiscalyear_id" on_change='onchange_columns(columns,fiscalyear_id,period_ids)' domain="[('company_id','=',company_id)]"/>
-                        <newline/>
+                    <group string="FiscalYear Info" >
+                        <field name="fiscalyear_id" on_change='onchange_columns(columns,fiscalyear_id,period_ids)' 
+                               domain="[('company_id','=',company_id)]"/>
                     </group>
                 </group>
-                <group colspan='4'>
-                    <group colspan="1">
-                        <separator string="Accounts to Include" colspan="4"/>
-                        <field name="account_ids" nolabel="1" colspan="4" domain="[('company_id','=',company_id)]">
+                <group>
+                    <group string="Accounts to Include">
+                        <field name="account_ids" nolabel="1"  domain="[('company_id','=',company_id)]">
                             <tree>
                                 <field name='code'/>
                                 <field name='name'/>
@@ -58,14 +47,11 @@
                             </tree>
                         </field>
                     </group>
-                    <group colspan="1">
-                        <separator string="Periods to Include" colspan="4"/>
+                    <group string="Periods to Include">
                         <field name="period_ids" 
-                        colspan="4" nolabel="1" 
+                         nolabel="1" 
                         domain="[('fiscalyear_id','=',fiscalyear_id),('special','=',False),('company_id','=',company_id)]"
-                        attrs="{'readonly':[('columns','in',('qtr','thirteen'))],}" 
-                        >
-                        
+                        attrs="{'readonly':[('columns','in',('qtr','thirteen'))],}" >
                             <tree>
                                 <field name='code'/>
                                 <field name='name'/>
@@ -74,37 +60,18 @@
                         </field>
                     </group>
                 </group>
-                <group colspan='4'>
-                    <separator string="Account's Sign Conventions" colspan="4"/>
+                <group string="Account's Sign Conventions">
                     <label string="Please check the [ Account's Sign on Reports ] Section in the Company"/>
                 </group>
-
-                <group colspan='4' col='4'>
-                    <group colspan="2">
-                        <separator string="Total" colspan="4"/>
-                        <field name="tot_check" colspan="4"/>
-                        <field name="lab_str" colspan="4" attrs="{'required':[('tot_check','=',True)], 'readonly':[('tot_check','=',False)],}" />
+                <group>
+                    <group string="Total">
+                        <field name="tot_check"/>
+                        <field name="lab_str"  attrs="{'required':[('tot_check','=',True)], 'readonly':[('tot_check','=',False)],}" />
                     </group>
-                    <group colspan="2">
-                        <separator string="" colspan="4"/>
-<!--
-                        <button 
-                            name="print_report" 
-                            string="Print" 
-                            type="object"
-                            icon='gtk-print'/>
-                        <button 
-                            type='special'
-                            special='cancel'
-                            string='_Cancel'
-                            icon='gtk-cancel'
-                        />
--->
+                    <group string=" " >
                     </group>
                 </group>
-
             </form>
-
             </field>
         </record>
 
@@ -120,20 +87,13 @@
                 <field name="inf_type"/>
                 <field name="columns"/>
                 <field name="display_account"/>
-                <field name="display_account_level" required="True" colspan="4"/>
+                <field name="display_account_level" required="True" />
                 <field name="fiscalyear_id"/>
-                <field name="tot_check" colspan="4"/>
-<!--
-                <button name="print_report" string="Print" type="object"/>
--->
+                <field name="tot_check" />
             </tree>
-
             </field>
         </record>
 
-
-
-
         <record model="ir.actions.act_window" id="account_financial_report_action">
             <field name="name">Account Financial Report</field>
             <field name="res_model">afr</field>
@@ -142,7 +102,6 @@
             <field name="view_mode">tree,form</field>
             <field name="help">Create your own Customized Account Financial Reports, in here, you will be able to create Templates for generating Two types of Reports: Balance Sheets and Income Statements, incluiding Analytic Ledgers. Besides, you can select within a set of choices to get better detailed report, be it that you ask it by one or several periods, by months (12 Months + YTD), or by quarters (4QRT's + YTD). Even you can get your reports in currencies other than the one set on your company.</field>
         </record>
-
         
         <menuitem 
             id="account_financial_report_menu"
@@ -152,6 +111,5 @@
             groups="account_financial_report.group_afreport"
             action="account_financial_report_action"/>
         
-        
     </data>
 </openerp>
diff --git a/account_financial_report/view/wizard.xml b/account_financial_report/view/wizard.xml
index 8f23b812..4136bfb1 100644
--- a/account_financial_report/view/wizard.xml
+++ b/account_financial_report/view/wizard.xml
@@ -7,47 +7,55 @@
             <field name="model">wizard.report</field>
             <field name="type">form</field>
             <field name="arch" type="xml">
-            <form string="Account Financial Report">
-                <group colspan="4">
-                    <group colspan="2">
-                        <separator string="Company Info" colspan="4"/>
-                        <field name="company_id" on_change='onchange_company_id(company_id)' colspan="4"/>
+                <form string="Account Financial Report" version="7.0">
+                    <header>
+                        <button 
+                            name="print_report" 
+                            string="Print" 
+                            type="object"
+                            class="oe_highlight" />
+                        or
+                        <button 
+                            type='special'
+                            special='cancel'
+                            string='_Cancel'
+                            class="oe_link" />
+                    </header>
+                <h2><label for="afr_id"/></h2>
+                <field name="afr_id"  on_change='onchange_afr_id(afr_id)' nolabel="1"/>
+                <group>
+                    <group string="Company Info">
+                        <label for="company_id"/>
+                        <field name="company_id" on_change='onchange_company_id(company_id)' nolabel="1"/>
                         <field name="currency_id" 
                         domain="[('company_id','=',company_id)]" 
-                        attrs="{'readonly':[('analytic_ledger','=',True)]}" 
-                        colspan="4"/>
+                        attrs="{'readonly':[('analytic_ledger','=',True)]}"/>
                     </group>
-                    <group colspan="2">
-                        <separator string="Report Info" colspan="4"/>
-                        <field name="afr_id"  on_change='onchange_afr_id(afr_id)'  colspan="4"/>
+                    <group string="Report Info">
                         <field 
                             name="inf_type" 
-                            colspan="4"
                             on_change='onchange_inf_type(inf_type)'
                             />
-                        <field name="columns" on_change='onchange_columns(columns,fiscalyear,periods)' colspan="4"/>
-                        <field name="analytic_ledger" colspan="4" 
+                        <field name="columns" on_change='onchange_columns(columns,fiscalyear,periods)'/>
+                        <field name="analytic_ledger"  
                         attrs="{'readonly':['|',('columns','!=','four'),('inf_type','!=','BS')],}" 
                         on_change='onchange_analytic_ledger(company_id,analytic_ledger)'
                         />
                     </group>
                 </group>
-                <group colspan='4'>
-                    <group colspan="2">
-                        <separator string="Accounts Info" colspan="4"/>
-                        <field name="display_account" required="True" colspan="4"/>
-                        <field name="display_account_level" required="True" colspan="4"/>
+                <group>
+                    <group string="Accounts Info">
+                        <field name="display_account" required="True" />
+                        <field name="display_account_level" required="True" />
                     </group>
-                    <group colspan="2">
-                        <separator string="FiscalYear Info" colspan="4"/>
+                    <group string="FiscalYear Info">
                         <field name="fiscalyear" on_change='onchange_columns(columns,fiscalyear,periods)'  domain="[('company_id','=',company_id)]"/>
                         <newline/>
                     </group>
                 </group>
-                <group colspan='4'>
-                    <group colspan="1">
-                        <separator string="Accounts to Include" colspan="4"/>
-                        <field name="account_list" nolabel="1" colspan="4" domain="[('company_id','=',company_id)]">
+                <group >
+                    <group string="Accounts to Include">
+                        <field name="account_list" nolabel="1"  domain="[('company_id','=',company_id)]">
                             <tree>
                                 <field name='code'/>
                                 <field name='name'/>
@@ -56,16 +64,12 @@
                             </tree>
                         </field>
                     </group>
-                    <group colspan="1">
-                        <separator string="Periods to Include" colspan="4"/>
+                    <group string="Periods to Include">
                         <field 
                             name="periods" 
-                            colspan="4" 
                             nolabel="1" 
                             domain="[('fiscalyear_id','=',fiscalyear),('special','=',False),('company_id','=',company_id)]"
-                             attrs="{'readonly':[('columns','in',('qtr','thirteen'))],}" 
-                             >
-                        
+                             attrs="{'readonly':[('columns','in',('qtr','thirteen'))],}">
                             <tree>
                                 <field name='code'/>
                                 <field name='name'/>
@@ -74,34 +78,19 @@
                         </field>
                     </group>
                 </group>
-                <group colspan='4'>
-                    <separator string="Account's Sign Conventions" colspan="4"/>
+                <group string="Account's Sign Conventions">
                     <label string="Please check the [ Account's Sign on Reports ] Section in the Company"/>
                 </group>
 
-                <group colspan='4' col='4'>
-                    <group colspan="2">
-                        <separator string="Total" colspan="4"/>
-                        <field name="tot_check" colspan="4"/>
-                        <field name="lab_str" colspan="4" attrs="{'required':[('tot_check','=',True)], 'readonly':[('tot_check','=',False)],}" />
+                <group>
+                    <group string="Total" >
+                        <field name="tot_check" />
+                        <field name="lab_str"  attrs="{'required':[('tot_check','=',True)], 'readonly':[('tot_check','=',False)],}" />
                     </group>
-                    <group colspan="2">
-                        <separator string="" colspan="4"/>
-                        <button 
-                            name="print_report" 
-                            string="Print" 
-                            type="object"
-                            icon='gtk-print'/>
-                        <button 
-                            type='special'
-                            special='cancel'
-                            string='_Cancel'
-                            icon='gtk-cancel'
-                        />
+                    <group string=" " >
                     </group>
                 </group>
             </form>
-
             </field>
         </record>
 
@@ -112,7 +101,7 @@
             <field name="view_type">form</field>
             <field name="view_mode">form</field>
             <field name="view_id" ref="wizard_report_view"/>
-            <field name="target">new</field>
+            <field name="target">inline</field>
         </record>