Merge lp:~openerp-commiter/openobject-client/6.0-opw-288262-nch into lp:openobject-client/6.0

Proposed by Naresh(OpenERP)
Status: Merged
Merged at revision: 1942
Proposed branch: lp:~openerp-commiter/openobject-client/6.0-opw-288262-nch
Merge into: lp:openobject-client/6.0
Diff against target: 0 lines
To merge this branch: bzr merge lp:~openerp-commiter/openobject-client/6.0-opw-288262-nch
Reviewer Review Type Date Requested Status
Olivier Laurent (Open ERP) Pending
Review via email: mp+89207@code.launchpad.net

Description of the change

Hello,

case 288262:

The issue is only reproduced on ubuntu 11.10. To reproduce open product form go to supplier tab edit one of the supplier then click delete button on the internal o2m and the client hangs !

Regards,

To post a comment you must log in.
Revision history for this message
Xavier ALT (dex-phx) wrote :

Hi Naresh,

Pehaps should be keep on the old behaviour, i.e fallback to the 'gui.main' as parent, but ensure we only the parent once (so moving 'if not parent' check just before win.set_transition_for()). (It's also fixing the problem on my Ubuntu 12.04)

What do you think ?

Cheers,
Xavier

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

Hi Xavier,

Thanks for your review ! But I removed the check 'if not parent' from the bottom because it was duplicated. We already do a check for the parent at the start of the defination. so I think we want need any changes.

Thanks
Naresh

1931. By Naresh(OpenERP)

[MERGE WITH STABLE]

Preview Diff

Empty