From 8302dddcb0b32199b7177011c8fff83b0d951508 Mon Sep 17 00:00:00 2001
From: Vincent Renaville <vincent.renaville@camptocamp.com>
Date: Tue, 12 Mar 2013 13:33:24 +0100
Subject: [PATCH 1/4] [ADD] tax analysis

---
 account_tax_analysis/__init__.py              |  21 ++++
 account_tax_analysis/__openerp__.py           |  37 ++++++
 account_tax_analysis/account_tax_analysis.py  |  51 +++++++++
 .../account_tax_analysis_view.xml             | 106 ++++++++++++++++++
 4 files changed, 215 insertions(+)
 create mode 100644 account_tax_analysis/__init__.py
 create mode 100644 account_tax_analysis/__openerp__.py
 create mode 100644 account_tax_analysis/account_tax_analysis.py
 create mode 100644 account_tax_analysis/account_tax_analysis_view.xml

diff --git a/account_tax_analysis/__init__.py b/account_tax_analysis/__init__.py
new file mode 100644
index 000000000..cf3e05bcf
--- /dev/null
+++ b/account_tax_analysis/__init__.py
@@ -0,0 +1,21 @@
+# -*- coding: utf-8 -*-
+##############################################################################
+#
+#    Author Vincent Renaville. Copyright 2013 Camptocamp SA
+#
+#    This program is free software: you can redistribute it and/or modify
+#    it under the terms of the GNU Affero General Public License as
+#    published by the Free Software Foundation, either version 3 of the
+#    License, or (at your option) any later version.
+#
+#    This program is distributed in the hope that it will be useful,
+#    but WITHOUT ANY WARRANTY; without even the implied warranty of
+#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#    GNU Affero General Public License for more details.
+#
+#    You should have received a copy of the GNU Affero General Public License
+#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
+#
+##############################################################################
+import account_tax_analysis
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
\ No newline at end of file
diff --git a/account_tax_analysis/__openerp__.py b/account_tax_analysis/__openerp__.py
new file mode 100644
index 000000000..68bd7184d
--- /dev/null
+++ b/account_tax_analysis/__openerp__.py
@@ -0,0 +1,37 @@
+# -*- coding: utf-8 -*-
+##############################################################################
+#
+#    Author Vincent Renaville. Copyright 2013 Camptocamp SA
+#
+#    This program is free software: you can redistribute it and/or modify
+#    it under the terms of the GNU Affero General Public License as
+#    published by the Free Software Foundation, either version 3 of the
+#    License, or (at your option) any later version.
+#
+#    This program is distributed in the hope that it will be useful,
+#    but WITHOUT ANY WARRANTY; without even the implied warranty of
+#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#    GNU Affero General Public License for more details.
+#
+#    You should have received a copy of the GNU Affero General Public License
+#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
+#
+##############################################################################
+{
+    "name" : "Tax analysis",
+    "version" : "1.0",
+    "depends" : ["base", "account"],
+    "author" : "CamptoCamp SA",
+    "category": 'Accounting & Finance',
+    "description": """Provide with details of your tax form (Allows you to group by move lines by tax account)
+    """,
+    'website': 'http://www.openerp.com',
+    'init_xml': [],
+    'update_xml': [
+        'account_tax_analysis_view.xml',
+    ],
+    'demo_xml': [],
+    'installable': True,
+    'active': False,
+}
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
diff --git a/account_tax_analysis/account_tax_analysis.py b/account_tax_analysis/account_tax_analysis.py
new file mode 100644
index 000000000..44301413d
--- /dev/null
+++ b/account_tax_analysis/account_tax_analysis.py
@@ -0,0 +1,51 @@
+# -*- coding: utf-8 -*-
+##############################################################################
+#
+#    Author Vincent Renaville. Copyright 2013 Camptocamp SA
+#
+#    This program is free software: you can redistribute it and/or modify
+#    it under the terms of the GNU Affero General Public License as
+#    published by the Free Software Foundation, either version 3 of the
+#    License, or (at your option) any later version.
+#
+#    This program is distributed in the hope that it will be useful,
+#    but WITHOUT ANY WARRANTY; without even the implied warranty of
+#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#    GNU Affero General Public License for more details.
+#
+#    You should have received a copy of the GNU Affero General Public License
+#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
+#
+##############################################################################
+
+from osv import orm, osv, fields
+from tools.translate import _
+
+class account_tax_declaration_analysis(orm.TransientModel):
+    _name = 'account.vat.declaration.analysis'
+    _description = 'Account Vat Declaration'
+    _columns = {
+        'fiscalyear_id': fields.many2one('account.fiscalyear', 'Fiscalyear', help='Fiscalyear', required=True),
+        'period_list': fields.many2many('account.period', 'account_tax_period_rel',
+        'tax_analysis', 'period_id', 'Period _list',required=True),
+
+    }
+
+    def create_vat(self, cr, uid, ids, context=None):
+        mod_obj =self.pool.get('ir.model.data')
+        action_obj = self.pool.get('ir.actions.act_window')
+        domain = []
+        data = self.read(cr, uid, ids, [])[0]
+        period_list = data['period_list']
+        if period_list :
+            domain = [('period_id','in',period_list)]
+        else:
+            raise osv.except_osv(_('No period defined'),_("You must selected period "))
+        ##
+        actions = mod_obj.get_object_reference(cr, uid, 'account_tax_analysis', 'action_view_tax_analysis')
+        id_action = actions and actions[1] or False
+        action_mod = action_obj.read(cr,uid,id_action)
+        action_mod['domain'] = domain
+        
+        return action_mod
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
\ No newline at end of file
diff --git a/account_tax_analysis/account_tax_analysis_view.xml b/account_tax_analysis/account_tax_analysis_view.xml
new file mode 100644
index 000000000..1973c7595
--- /dev/null
+++ b/account_tax_analysis/account_tax_analysis_view.xml
@@ -0,0 +1,106 @@
+<openerp>
+    <data>
+
+
+        <record id="view_account_move_line_filter_vat_analysis" model="ir.ui.view">
+            <field name="name">Journal Items Tax</field>
+            <field name="model">account.move.line</field>
+            <field name="type">search</field>
+             <field eval="32" name="priority"/>
+            <field name="arch" type="xml">
+                <search string="Search Journal Items">
+                    <group>
+                        <separator orientation="vertical"/>
+                        <filter icon="terp-document-new" string="Unposted" domain="[('move_id.state','=','draft')]" help="Unposted Journal Items"/>
+                        <filter name="posted" icon="terp-camera_test" string="Posted" domain="[('move_id.state','=','posted')]" help="Posted Journal Items"/>
+                        <separator orientation="vertical"/>
+                        <field name="date"/>
+                        <field name="account_id"/>
+                    </group>
+                    <newline/>
+                    <newline/>
+                    <group expand="1" string="Group By...">
+                        <filter string="Partner" icon="terp-partner" domain="[]" context="{'group_by':'partner_id'}"/>
+                        <separator orientation="vertical"/>
+                        <filter string="Account"  icon="terp-folder-green" context="{'group_by':'account_id'}"/>
+                        <separator orientation="vertical"/>
+                        <filter string="Period" icon="terp-go-month" domain="[]" context="{'group_by':'period_id'}"/>
+                        <filter string="Tax account" icon="terp-go-month" domain="[]" context="{'group_by':'tax_code_id'}"/>
+                    </group>
+                    <newline/>
+                </search>
+            </field>
+        </record>
+
+        <record id="view_move_line_tree_tax_analysis" model="ir.ui.view">
+            <field name="name">account.move.line.tree</field>
+            <field name="model">account.move.line</field>
+            <field name="type">tree</field>
+            <field eval="16" name="priority"/>
+            <field name="arch" type="xml">
+                <tree  string="Journal Items" editable="top" on_write="on_create_write">
+                    <field name="tax_code_id" />
+                    <field name="name"/>
+                    <field name="ref"/>
+                    <field name="period_id"/>
+                    <field name="date"/>
+                    <field name="partner_id" />
+                    <field name="account_id" domain="[('journal_id','=',journal_id), ('company_id', '=', company_id)]"/>
+                    <field name="debit" sum="Total debit"/>
+                    <field name="credit" sum="Total credit"/>
+                    <field name="tax_amount" sum="Total Taxe"/>
+                    <field name="amount_currency"  />
+                    <field name="currency_id"  />
+                    
+                </tree>
+            </field>
+        </record>
+
+
+        <record id="action_view_tax_analysis" model="ir.actions.act_window">
+            <field name="name">Recurring Models</field>
+            <field name="res_model">account.move.line</field>
+            <field name="view_type">form</field>
+            <field name="view_mode">tree</field>
+            <field name="search_view_id" ref="view_account_move_line_filter_vat_analysis"/>
+            <field name="view_id" ref="view_move_line_tree_tax_analysis"/>
+        </record>
+
+        <record id="view_account_vat_declaration_analysis" model="ir.ui.view">
+            <field name="name">Account Vat Declaration</field>
+            <field name="model">account.vat.declaration.analysis</field>
+            <field name="type">form</field>
+            <field name="arch" type="xml">
+            <form string="Taxes Report">
+                <separator string="Taxes Report" colspan="4"/>
+                <newline/>
+                <field name="fiscalyear_id"/>
+                <separator string="Periods"  colspan="4"/>
+                <field name="period_list"  domain="[('fiscalyear_id', '=', fiscalyear_id)]"/>
+                <group col="2" colspan="4">
+                    <button icon='gtk-cancel' special="cancel"  string="Cancel" />
+                    <button name="create_vat" string="Show tax lines" colspan="1" type="object" icon="gtk-ok"/>
+                </group>
+            </form>
+            </field>
+        </record>
+
+        <record id="action_account_vat_declaration_analysis" model="ir.actions.act_window">
+            <field name="name">Account Vat Declaration</field>
+            <field name="type">ir.actions.act_window</field>
+            <field name="res_model">account.vat.declaration.analysis</field>
+            <field name="view_type">form</field>
+            <field name="view_mode">form</field>
+            <field name="target">new</field>
+            <field name="help">This menu print a VAT declaration based on invoices or payments. You can select one or several periods of the fiscal year. Information required for a tax declaration is automatically generated by OpenERP from invoices (or payments, in some countries). This data is updated in real time. That’s very useful because it enables you to preview at any time the tax that you owe at the start and end of the month or quarter.</field>
+        </record>
+
+        <menuitem
+            name="Taxes Analysis"
+            parent="account.menu_tax_report"
+            action="action_account_vat_declaration_analysis"
+            id="menu_account_vat_declaration_analysis"
+            icon="STOCK_PRINT"/>
+    </data>
+    
+</openerp>
\ No newline at end of file

From 35e43e409ac488342045fbc8bc1c4aa7cb0c2e58 Mon Sep 17 00:00:00 2001
From: unknown <nicolas.bessi@camptocamp.com>
Date: Thu, 14 Mar 2013 15:31:56 +0100
Subject: [PATCH 2/4] [FIX] coding standards

---
 account_tax_analysis/__init__.py              |   4 +-
 account_tax_analysis/__openerp__.py           |  41 ++--
 account_tax_analysis/account_tax_analysis.py  |  27 ++-
 .../account_tax_analysis_view.xml             | 213 ++++++++++--------
 4 files changed, 156 insertions(+), 129 deletions(-)

diff --git a/account_tax_analysis/__init__.py b/account_tax_analysis/__init__.py
index cf3e05bcf..0fe7493b0 100644
--- a/account_tax_analysis/__init__.py
+++ b/account_tax_analysis/__init__.py
@@ -17,5 +17,5 @@
 #    along with this program.  If not, see <http://www.gnu.org/licenses/>.
 #
 ##############################################################################
-import account_tax_analysis
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
\ No newline at end of file
+from . import account_tax_analysis
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
diff --git a/account_tax_analysis/__openerp__.py b/account_tax_analysis/__openerp__.py
index 68bd7184d..2f349b27c 100644
--- a/account_tax_analysis/__openerp__.py
+++ b/account_tax_analysis/__openerp__.py
@@ -17,21 +17,28 @@
 #    along with this program.  If not, see <http://www.gnu.org/licenses/>.
 #
 ##############################################################################
-{
-    "name" : "Tax analysis",
-    "version" : "1.0",
-    "depends" : ["base", "account"],
-    "author" : "CamptoCamp SA",
-    "category": 'Accounting & Finance',
-    "description": """Provide with details of your tax form (Allows you to group by move lines by tax account)
-    """,
-    'website': 'http://www.openerp.com',
-    'init_xml': [],
-    'update_xml': [
-        'account_tax_analysis_view.xml',
-    ],
-    'demo_xml': [],
-    'installable': True,
-    'active': False,
-}
+{"name": "Tax analysis",
+ "version": "1.0",
+ "depends": ["base", "account"],
+ "author": "CamptoCamp SA",
+ "category": 'Accounting & Finance',
+ "description": """
+ Tax analysis view
+=================
+
+This add-on is a must if you want to be able to validate your VAT form.
+
+Thanks to a new menu 'Accounting / Tax / Tax analysis'
+you are able to group accounting entries by Taxes (VAT codes)
+and/or financial accounts.
+
+This way you will find easily differences you may see between
+the OpenERP tax report and what you see in your books.""",
+ "website": "http://www.camptocamp.com",
+ "init_xml": [],
+ "update_xml": ["account_tax_analysis_view.xml"],
+ "demo_xml": [],
+ "installable": True,
+ "active": False,
+ }
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
diff --git a/account_tax_analysis/account_tax_analysis.py b/account_tax_analysis/account_tax_analysis.py
index 44301413d..426487a8b 100644
--- a/account_tax_analysis/account_tax_analysis.py
+++ b/account_tax_analysis/account_tax_analysis.py
@@ -17,35 +17,40 @@
 #    along with this program.  If not, see <http://www.gnu.org/licenses/>.
 #
 ##############################################################################
-
 from osv import orm, osv, fields
 from tools.translate import _
 
+
 class account_tax_declaration_analysis(orm.TransientModel):
     _name = 'account.vat.declaration.analysis'
     _description = 'Account Vat Declaration'
     _columns = {
-        'fiscalyear_id': fields.many2one('account.fiscalyear', 'Fiscalyear', help='Fiscalyear', required=True),
+        'fiscalyear_id': fields.many2one('account.fiscalyear', 'Fiscalyear',
+                                         help='Fiscalyear to look on', required=True),
+
         'period_list': fields.many2many('account.period', 'account_tax_period_rel',
-        'tax_analysis', 'period_id', 'Period _list',required=True),
+                                        'tax_analysis', 'period_id',
+                                        'Period _list', required=True),
 
     }
 
     def create_vat(self, cr, uid, ids, context=None):
-        mod_obj =self.pool.get('ir.model.data')
+        mod_obj = self.pool.get('ir.model.data')
         action_obj = self.pool.get('ir.actions.act_window')
         domain = []
         data = self.read(cr, uid, ids, [])[0]
         period_list = data['period_list']
-        if period_list :
-            domain = [('period_id','in',period_list)]
+        if period_list:
+            domain = [('period_id', 'in', period_list)]
         else:
-            raise osv.except_osv(_('No period defined'),_("You must selected period "))
+            raise osv.except_osv(_('No period defined'), _("You must selected period "))
         ##
-        actions = mod_obj.get_object_reference(cr, uid, 'account_tax_analysis', 'action_view_tax_analysis')
+        actions = mod_obj.get_object_reference(cr, uid,
+                                               'account_tax_analysis', 'action_view_tax_analysis')
         id_action = actions and actions[1] or False
-        action_mod = action_obj.read(cr,uid,id_action)
+        action_mod = action_obj.read(cr, uid, id_action)
         action_mod['domain'] = domain
-        
+
         return action_mod
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
\ No newline at end of file
+
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
diff --git a/account_tax_analysis/account_tax_analysis_view.xml b/account_tax_analysis/account_tax_analysis_view.xml
index 1973c7595..53f2f8dc8 100644
--- a/account_tax_analysis/account_tax_analysis_view.xml
+++ b/account_tax_analysis/account_tax_analysis_view.xml
@@ -1,106 +1,121 @@
 <openerp>
-    <data>
+  <data>
+    <record id="view_account_move_line_filter_vat_analysis" model="ir.ui.view">
+      <field name="name">Journal Items Tax</field>
+      <field name="model">account.move.line</field>
+      <field name="type">search</field>
+      <field eval="32" name="priority"/>
+      <field name="arch" type="xml">
+        <search string="Search Journal Items">
+          <group>
+            <separator orientation="vertical"/>
+            <filter icon="terp-document-new"
+                    string="Unposted"
+                    domain="[('move_id.state','=','draft')]"
+                    help="Unposted Journal Items"/>
+            <filter name="posted"
+                    icon="terp-camera_test"
+                    string="Posted"
+                    domain="[('move_id.state','=','posted')]"
+                    help="Posted Journal Items"/>
+            <separator orientation="vertical"/>
+            <field name="date"/>
+            <field name="account_id"/>
+          </group>
+          <newline/>
+          <newline/>
+          <group expand="1" string="Group By...">
+            <filter string="Partner" icon="terp-partner"
+                    domain="[]"
+                    context="{'group_by': 'partner_id'}"/>
+            <separator orientation="vertical"/>
+            <filter string="Account"
+                    icon="terp-folder-green"
+                    context="{'group_by': 'account_id'}"/>
+            <separator orientation="vertical"/>
+            <filter string="Period"
+                    icon="terp-go-month"
+                    domain="[]"
+                    context="{'group_by': 'period_id'}"/>
+            <filter string="Tax account"
+                    icon="terp-go-month"
+                    domain="[]"
+                    context="{'group_by': 'tax_code_id'}"/>
+          </group>
+          <newline/>
+        </search>
+      </field>
+    </record>
+
+    <record id="view_move_line_tree_tax_analysis" model="ir.ui.view">
+      <field name="name">account.move.line.tree</field>
+      <field name="model">account.move.line</field>
+      <field name="type">tree</field>
+      <field eval="16" name="priority"/>
+      <field name="arch" type="xml">
+        <tree  string="Journal Items" editable="top" on_write="on_create_write">
+          <field name="tax_code_id" />
+          <field name="name"/>
+          <field name="ref"/>
+          <field name="period_id"/>
+          <field name="date"/>
+          <field name="partner_id" />
+          <field name="account_id" domain="[('journal_id','=',journal_id), ('company_id', '=', company_id)]"/>
+          <field name="debit" sum="Total debit"/>
+          <field name="credit" sum="Total credit"/>
+          <field name="tax_amount" sum="Total Taxe"/>
+          <field name="amount_currency"  />
+          <field name="currency_id"  />
+
+        </tree>
+      </field>
+    </record>
 
 
-        <record id="view_account_move_line_filter_vat_analysis" model="ir.ui.view">
-            <field name="name">Journal Items Tax</field>
-            <field name="model">account.move.line</field>
-            <field name="type">search</field>
-             <field eval="32" name="priority"/>
-            <field name="arch" type="xml">
-                <search string="Search Journal Items">
-                    <group>
-                        <separator orientation="vertical"/>
-                        <filter icon="terp-document-new" string="Unposted" domain="[('move_id.state','=','draft')]" help="Unposted Journal Items"/>
-                        <filter name="posted" icon="terp-camera_test" string="Posted" domain="[('move_id.state','=','posted')]" help="Posted Journal Items"/>
-                        <separator orientation="vertical"/>
-                        <field name="date"/>
-                        <field name="account_id"/>
-                    </group>
-                    <newline/>
-                    <newline/>
-                    <group expand="1" string="Group By...">
-                        <filter string="Partner" icon="terp-partner" domain="[]" context="{'group_by':'partner_id'}"/>
-                        <separator orientation="vertical"/>
-                        <filter string="Account"  icon="terp-folder-green" context="{'group_by':'account_id'}"/>
-                        <separator orientation="vertical"/>
-                        <filter string="Period" icon="terp-go-month" domain="[]" context="{'group_by':'period_id'}"/>
-                        <filter string="Tax account" icon="terp-go-month" domain="[]" context="{'group_by':'tax_code_id'}"/>
-                    </group>
-                    <newline/>
-                </search>
-            </field>
-        </record>
+    <record id="action_view_tax_analysis" model="ir.actions.act_window">
+      <field name="name">Recurring Models</field>
+      <field name="res_model">account.move.line</field>
+      <field name="view_type">form</field>
+      <field name="view_mode">tree</field>
+      <field name="search_view_id" ref="view_account_move_line_filter_vat_analysis"/>
+      <field name="view_id" ref="view_move_line_tree_tax_analysis"/>
+    </record>
 
-        <record id="view_move_line_tree_tax_analysis" model="ir.ui.view">
-            <field name="name">account.move.line.tree</field>
-            <field name="model">account.move.line</field>
-            <field name="type">tree</field>
-            <field eval="16" name="priority"/>
-            <field name="arch" type="xml">
-                <tree  string="Journal Items" editable="top" on_write="on_create_write">
-                    <field name="tax_code_id" />
-                    <field name="name"/>
-                    <field name="ref"/>
-                    <field name="period_id"/>
-                    <field name="date"/>
-                    <field name="partner_id" />
-                    <field name="account_id" domain="[('journal_id','=',journal_id), ('company_id', '=', company_id)]"/>
-                    <field name="debit" sum="Total debit"/>
-                    <field name="credit" sum="Total credit"/>
-                    <field name="tax_amount" sum="Total Taxe"/>
-                    <field name="amount_currency"  />
-                    <field name="currency_id"  />
-                    
-                </tree>
-            </field>
-        </record>
+    <record id="view_account_vat_declaration_analysis" model="ir.ui.view">
+      <field name="name">Account Vat Declaration</field>
+      <field name="model">account.vat.declaration.analysis</field>
+      <field name="type">form</field>
+      <field name="arch" type="xml">
+        <form string="Taxes Report">
+          <separator string="Taxes Report" colspan="4"/>
+          <newline/>
+          <field name="fiscalyear_id"/>
+          <separator string="Periods"  colspan="4"/>
+          <field name="period_list"  domain="[('fiscalyear_id', '=', fiscalyear_id)]"/>
+          <group col="2" colspan="4">
+            <button icon='gtk-cancel' special="cancel"  string="Cancel" />
+            <button name="create_vat" string="Show tax lines" colspan="1" type="object" icon="gtk-ok"/>
+          </group>
+        </form>
+      </field>
+    </record>
 
+    <record id="action_account_vat_declaration_analysis" model="ir.actions.act_window">
+      <field name="name">Account Vat Declaration</field>
+      <field name="type">ir.actions.act_window</field>
+      <field name="res_model">account.vat.declaration.analysis</field>
+      <field name="view_type">form</field>
+      <field name="view_mode">form</field>
+      <field name="target">new</field>
+      <field name="help">This menu print a VAT declaration based on invoices or payments. You can select one or several periods of the fiscal year. Information required for a tax declaration is automatically generated by OpenERP from invoices (or payments, in some countries). This data is updated in real time. That’s very useful because it enables you to preview at any time the tax that you owe at the start and end of the month or quarter.</field>
+    </record>
 
-        <record id="action_view_tax_analysis" model="ir.actions.act_window">
-            <field name="name">Recurring Models</field>
-            <field name="res_model">account.move.line</field>
-            <field name="view_type">form</field>
-            <field name="view_mode">tree</field>
-            <field name="search_view_id" ref="view_account_move_line_filter_vat_analysis"/>
-            <field name="view_id" ref="view_move_line_tree_tax_analysis"/>
-        </record>
+    <menuitem
+        name="Taxes Analysis"
+        parent="account.menu_tax_report"
+        action="action_account_vat_declaration_analysis"
+        id="menu_account_vat_declaration_analysis"
+        icon="STOCK_PRINT"/>
 
-        <record id="view_account_vat_declaration_analysis" model="ir.ui.view">
-            <field name="name">Account Vat Declaration</field>
-            <field name="model">account.vat.declaration.analysis</field>
-            <field name="type">form</field>
-            <field name="arch" type="xml">
-            <form string="Taxes Report">
-                <separator string="Taxes Report" colspan="4"/>
-                <newline/>
-                <field name="fiscalyear_id"/>
-                <separator string="Periods"  colspan="4"/>
-                <field name="period_list"  domain="[('fiscalyear_id', '=', fiscalyear_id)]"/>
-                <group col="2" colspan="4">
-                    <button icon='gtk-cancel' special="cancel"  string="Cancel" />
-                    <button name="create_vat" string="Show tax lines" colspan="1" type="object" icon="gtk-ok"/>
-                </group>
-            </form>
-            </field>
-        </record>
-
-        <record id="action_account_vat_declaration_analysis" model="ir.actions.act_window">
-            <field name="name">Account Vat Declaration</field>
-            <field name="type">ir.actions.act_window</field>
-            <field name="res_model">account.vat.declaration.analysis</field>
-            <field name="view_type">form</field>
-            <field name="view_mode">form</field>
-            <field name="target">new</field>
-            <field name="help">This menu print a VAT declaration based on invoices or payments. You can select one or several periods of the fiscal year. Information required for a tax declaration is automatically generated by OpenERP from invoices (or payments, in some countries). This data is updated in real time. That’s very useful because it enables you to preview at any time the tax that you owe at the start and end of the month or quarter.</field>
-        </record>
-
-        <menuitem
-            name="Taxes Analysis"
-            parent="account.menu_tax_report"
-            action="action_account_vat_declaration_analysis"
-            id="menu_account_vat_declaration_analysis"
-            icon="STOCK_PRINT"/>
-    </data>
-    
-</openerp>
\ No newline at end of file
+  </data>
+</openerp>

From 6d412709d08eae318728ecd908ce4758f641442f Mon Sep 17 00:00:00 2001
From: unknown <nicolas.bessi@camptocamp.com>
Date: Thu, 14 Mar 2013 15:34:35 +0100
Subject: [PATCH 3/4] [FIX] minor cleanup

---
 account_tax_analysis/account_tax_analysis.py | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/account_tax_analysis/account_tax_analysis.py b/account_tax_analysis/account_tax_analysis.py
index 426487a8b..18b806489 100644
--- a/account_tax_analysis/account_tax_analysis.py
+++ b/account_tax_analysis/account_tax_analysis.py
@@ -38,19 +38,19 @@ class account_tax_declaration_analysis(orm.TransientModel):
         mod_obj = self.pool.get('ir.model.data')
         action_obj = self.pool.get('ir.actions.act_window')
         domain = []
-        data = self.read(cr, uid, ids, [])[0]
+        data = self.read(cr, uid, ids, [], context=context)[0]
         period_list = data['period_list']
         if period_list:
             domain = [('period_id', 'in', period_list)]
         else:
-            raise osv.except_osv(_('No period defined'), _("You must selected period "))
-        ##
+            raise osv.except_osv(_('No period defined'),
+                                 _("You must selected period "))
         actions = mod_obj.get_object_reference(cr, uid,
-                                               'account_tax_analysis', 'action_view_tax_analysis')
-        id_action = actions and actions[1] or False
+                                               'account_tax_analysis',
+                                               'action_view_tax_analysis')
+        id_action = actions[1] if actions else False
         action_mod = action_obj.read(cr, uid, id_action)
         action_mod['domain'] = domain
-
         return action_mod
 
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

From 71bee226dfe031ad404d9c59deaa03722e27477c Mon Sep 17 00:00:00 2001
From: unknown <nicolas.bessi@camptocamp.com>
Date: Thu, 14 Mar 2013 15:35:09 +0100
Subject: [PATCH 4/4] [FIX] white space

---
 account_tax_analysis/account_tax_analysis.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/account_tax_analysis/account_tax_analysis.py b/account_tax_analysis/account_tax_analysis.py
index 18b806489..e5368d975 100644
--- a/account_tax_analysis/account_tax_analysis.py
+++ b/account_tax_analysis/account_tax_analysis.py
@@ -31,7 +31,6 @@ class account_tax_declaration_analysis(orm.TransientModel):
         'period_list': fields.many2many('account.period', 'account_tax_period_rel',
                                         'tax_analysis', 'period_id',
                                         'Period _list', required=True),
-
     }
 
     def create_vat(self, cr, uid, ids, context=None):