Merge lp:~serpent-consulting-services/openobject-addons/6.1-960872-SerpentCS into lp:openobject-addons/6.1

Proposed by Serpent Consulting Services
Status: Approved
Approved by: Rifakat Husen (OpenERP)
Approved revision: 6748
Proposed branch: lp:~serpent-consulting-services/openobject-addons/6.1-960872-SerpentCS
Merge into: lp:openobject-addons/6.1
Diff against target: 36 lines (+4/-3)
1 file modified
account/account_view.xml (+4/-3)
To merge this branch: bzr merge lp:~serpent-consulting-services/openobject-addons/6.1-960872-SerpentCS
Reviewer Review Type Date Requested Status
Rifakat Husen (OpenERP) (community) Approve
Review via email: mp+102967@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Rifakat Husen (OpenERP) (rha-openerp) wrote :

Hi Serpent Consulting Services,

It works.
Thanks for your fix.

Regards,
Rifakat Haradwala

review: Approve
Revision history for this message
Serpent Consulting Services (serpent-consulting-services) wrote :

Rifakat(OpenERP),

Thanks for the approval.

Waiting for this fix getting merged.
This looks like a simple one technically, but is really a show stopper for many accountants.

Regards,
Serpent Consulting Services.

Revision history for this message
Serpent Consulting Services (serpent-consulting-services) wrote :

Urge to get this upon 6.1 !

Unmerged revisions

6748. By Serpent Consulting Services

[FIX] Account : Corrected the Views of Account.period in order to fix the bug of states.State field should be in view if other fields depend on it.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'account/account_view.xml'
--- account/account_view.xml 2012-02-14 11:07:04 +0000
+++ account/account_view.xml 2012-04-21 10:47:20 +0000
@@ -22,12 +22,13 @@
22 </group>22 </group>
23 <separator colspan="4" string="Periods"/>23 <separator colspan="4" string="Periods"/>
24 <field colspan="4" name="period_ids" nolabel="1" widget="one2many_list">24 <field colspan="4" name="period_ids" nolabel="1" widget="one2many_list">
25 <form string="Period">25 <form string="Periods">
26 <field name="name"/>26 <field name="name"/>
27 <field name="code"/>27 <field name="code"/>
28 <field name="date_start"/>28 <field name="date_start"/>
29 <field name="date_stop"/>29 <field name="date_stop"/>
30 <field name="special"/>30 <field name="special"/>
31 <field name="state" invisible="1"/>
31 </form>32 </form>
32 </field>33 </field>
33 <separator colspan="4" string="States"/>34 <separator colspan="4" string="States"/>
@@ -92,7 +93,7 @@
92 <field name="model">account.period</field>93 <field name="model">account.period</field>
93 <field name="type">form</field>94 <field name="type">form</field>
94 <field name="arch" type="xml">95 <field name="arch" type="xml">
95 <form string="Period">96 <form string="Periods">
96 <field name="name"/>97 <field name="name"/>
97 <field name="code"/>98 <field name="code"/>
98 <field name="date_start"/>99 <field name="date_start"/>
@@ -111,7 +112,7 @@
111 <field name="model">account.period</field>112 <field name="model">account.period</field>
112 <field name="type">tree</field>113 <field name="type">tree</field>
113 <field name="arch" type="xml">114 <field name="arch" type="xml">
114 <tree colors="blue:state == 'draft';gray:state == 'done' " string="Period">115 <tree colors="blue:state == 'draft';gray:state == 'done' " string="Periods">
115 <field name="name"/>116 <field name="name"/>
116 <field name="code"/>117 <field name="code"/>
117 <field name="date_start"/>118 <field name="date_start"/>