Merge lp:~openerp-dev/openobject-addons/6.1-opw-575267-bth into lp:openobject-addons/6.1

Proposed by Bhumi Thakkar (Open ERP)
Status: Approved
Approved by: Naresh(OpenERP)
Approved revision: 6868
Proposed branch: lp:~openerp-dev/openobject-addons/6.1-opw-575267-bth
Merge into: lp:openobject-addons/6.1
Diff against target: 12 lines (+1/-1)
1 file modified
survey/wizard/ (+1/-1)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/6.1-opw-575267-bth
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email:

Description of the change


   In WEB Open any survey You find extra unused field.

1. Tools => Survey => Define Surveys => Surveys.
2. Open any record.
3. click on any button (test survey/answer survey/edit survey)

Observed: "extra textbox/field is displyed"
Expected: "unnecessary field should not displayed."

In file, field should add 'modifiers':'{"invisible":true}'.


To post a comment you must log in.
Revision history for this message
Naresh(OpenERP) (nch-openerp) wrote :


This bug was qualified as Already Fixed on Trunk (means that it was already fixed and merged in Trunk). If this Merge Proposal could not be merged in v6.1 at the release of v7.0, it will be closed.

Naresh Soni

Unmerged revisions

6868. By Bhumi Thakkar (Open ERP)

[FIX] Modifier invisible is added in field.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'survey/wizard/'
2--- survey/wizard/ 2012-01-31 13:36:57 +0000
3+++ survey/wizard/ 2012-07-02 13:15:24 +0000
4@@ -162,7 +162,7 @@
6 if wiz_id:
7 fields["wizardid_" + str(wiz_id)] = {'type':'char', 'size' : 255, 'string':"", 'views':{}}
8- etree.SubElement(xml_form, 'field', {'invisible':'1','name': "wizardid_" + str(wiz_id),'default':str(lambda *a: 0)})
9+ etree.SubElement(xml_form, 'field', {'invisible':'1','name': "wizardid_" + str(wiz_id),'default':str(lambda *a: 0),'modifiers':'{"invisible":true}'})
11 if pag_rec.note:
12 xml_group = etree.SubElement(xml_form, 'group', {'col': '1', 'colspan': '4'})