OpenERP Community Backports (Web)

many2many trees don't resepect create, delete, edit attributes

  1. Series 7.0
  2. Bug #1179839
Bug #1179839 reported by Holger Brunn (Therp)
16
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Odoo Server (MOVED TO GITHUB)
Confirmed
Medium
OpenERP's Framework R&D
Odoo Web (MOVED TO GITHUB)
Fix Committed
Medium
Nicolas Vanhoren (OpenERP)
OpenERP Community Backports (Server)
Status tracked in 7.0
7.0
Fix Committed
Undecided
Unassigned
OpenERP Community Backports (Web)
Status tracked in 7.0
7.0
Fix Committed
Undecided
Unassigned

Bug Description

The treeview for many2many fields disables checking for those attributes in
http://bazaar.launchpad.net/~openerp/openerp-web/7.0/view/head:/addons/web/static/src/js/view_form.js#L4369

Am I missing something or can we just drop this function?

Related branches

lp:~therp-nl/openerp-web/lp1179839
Approved for merging into lp:openerp-web/7.0
Xavier (Open ERP) (community): Approve
Holger Brunn (Therp) (community): Needs Resubmitting
Diff: 18 lines (+7/-1)
1 file modified
addons/web/static/src/js/view_form.js (+7/-1)
lp:~therp-nl/ocb-web/lp1179839
On hold for merging into lp:ocb-web
Stefan Rijnhart (Opener): Needs Information
Diff: 11 lines (+0/-1)
1 file modified
addons/web/static/src/js/view_form.js (+0/-1)
lp:~openerp-dev/openerp-web/trunk-bug-1179839-bth
Rejected for merging into lp:openerp-web
OpenERP R&D Web Team: Pending requested
Diff: 11 lines (+0/-1)
1 file modified
addons/web/static/src/js/view_form.js (+0/-1)
lp:~therp-nl/openobject-server/lp1179839
OpenERP Core Team: Pending requested
Diff: 13 lines (+3/-0)
1 file modified
openerp/osv/orm.py (+3/-0)
lp:~therp-nl/ocb-server/7.0-lp1179839-many2many_attributes_in_client
On hold for merging into lp:ocb-server
Stefan Rijnhart (Opener): Approve
Diff: 13 lines (+3/-0)
1 file modified
openerp/osv/orm.py (+3/-0)
Revision history for this message
Amit Bhavsar (Open ERP) (amb-openerp) wrote :

Hello Holger,

I have checked your Issue with latest revision. But It's seems fine. That's why I have attached screen capture for your reference. so wold you please check It again and notify us where you have faced the problem.

Thanks and waiting for your reply!

Changed in openerp-web:
status: New → Incomplete
Revision history for this message
Holger Brunn (Therp) (hbrunn) wrote :

To clarify:
I want to _disable_ this buttons. So in my own module, I write
<field name="my_many2many_field">
<tree create="false">
....
</tree>
</field>

And this only works with the merge linked.

Revision history for this message
Amit Bhavsar (Open ERP) (amb-openerp) wrote :

Hello Holger,

Yes, I have faced the same problem while apply this scenario.
Nice caught..!!!

Thank you.

Changed in openerp-web:
status: Incomplete → Confirmed
importance: Undecided → Medium
assignee: nobody → OpenERP R&D Web Team (openerp-dev-web)
Bhumi Thakkar (Open ERP) (bth-openerp)
Changed in openerp-web:
status: Confirmed → In Progress
Revision history for this message
Bhumi Thakkar (Open ERP) (bth-openerp) wrote :

Hello,

       It has been fixed in https://code.launchpad.net/~openerp-dev/openerp-web/trunk-bug-1179839-bth branch.

Thanks.

Changed in openerp-web:
status: In Progress → Fix Committed
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Hi Bhumi Thakkar, did you see the attached proposal from Therp? Very similar to yours ;-)

Nicolas Vanhoren (OpenERP) (niv-openerp)
Changed in openerp-web:
assignee: OpenERP R&D Web Team (openerp-dev-web) → Nicolas (OpenERP) (niv-openerp)
Twinkle Christian(OpenERP) (tch-openerp)
Changed in openobject-server:
assignee: nobody → OpenERP's Framework R&D (openerp-dev-framework)
importance: Undecided → Medium
status: New → Confirmed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.