Merge lp:~openerp-dev/openobject-addons/7.0-opw-587622-mma into lp:openobject-addons/7.0

Proposed by Mayur Maheshwari(OpenERP)
Status: Work in progress
Proposed branch: lp:~openerp-dev/openobject-addons/7.0-opw-587622-mma
Merge into: lp:openobject-addons/7.0
Diff against target: 26 lines (+5/-5)
1 file modified
account_followup/account_followup_customers.xml (+5/-5)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/7.0-opw-587622-mma
Reviewer Review Type Date Requested Status
Naresh(OpenERP) (community) Needs Fixing
Review via email: mp+152393@code.launchpad.net

Description of the change

Hello

Go to Sale,In search view Group By.. label is displayed two times

This branch fix this isse

Thanks
Mayur

To post a comment you must log in.
Revision history for this message
Csaba TOTH (tsabi) wrote :

Just a small suggestion: at line 22 you can use "'" instead of """. It will be more readable:

<xpath expr="//filter[@string='Country']" position="after">

thanks,
Csaba

Revision history for this message
Naresh(OpenERP) (nch-openerp) wrote :

Hi Mayur,

The diff should be improved for readability as stated by Csaba TOTH.

Thanks,
Naresh Soni
OpenERP Enterprise Services

review: Needs Fixing

Unmerged revisions

8828. By Mayur Maheshwari(OpenERP)

[FIX]account_followup: set serch view fix duplicate group by

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'account_followup/account_followup_customers.xml'
2--- account_followup/account_followup_customers.xml 2013-02-26 15:28:03 +0000
3+++ account_followup/account_followup_customers.xml 2013-03-08 12:58:22 +0000
4@@ -39,7 +39,7 @@
5 <field name="model">res.partner</field>
6 <field name="inherit_id" ref="base.view_res_partner_filter"/>
7 <field name="arch" type="xml">
8- <search string="Search Partner" position="inside">
9+ <field name="parent_id" postion="after">
10 <group string="Follow-up">
11 <filter string="Partners with Overdue Credits" domain="[('payment_amount_overdue', '>', 0.0)]" name="credits"/>
12 <separator/>
13@@ -48,10 +48,10 @@
14 <filter string="No Responsible" domain="[('payment_responsible_id', '=', False)]"/>
15 <filter string="My Follow-ups" domain="[('payment_responsible_id','=', uid)]" name="my"/>
16 </group>
17- <group expand="1" string="Group By...">
18- <filter string="Follow-up Responsible" context="{'group_by':'payment_responsible_id'}"/>
19- </group>
20- </search>
21+ </field>
22+ <xpath expr="//filter[@string=&quot;Country&quot;]" position="after">
23+ <filter string="Follow-up Responsible" context="{'group_by':'payment_responsible_id'}"/>
24+ </xpath>
25 </field>
26 </record>
27