[trunk/7.0] res.partner - use_parent_address allows to change parent address
Bug #1160425 reported by
Yannick Vaucher @ Camptocamp
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Odoo Server (MOVED TO GITHUB) |
Fix Released
|
Medium
|
OpenERP's Framework R&D | ||
Bug Description
If a partner has a parent_id defined, the "use_parent_
However, this option is dangerous as per default it set it to true.
Address should only be editable if the checkbox "use_parent_
Related branches
lp:~openerp-dev/openobject-server/7.0-fix-contact-company-handling
- Joël Grand-Guillaume @ camptocamp (community): Needs Fixing (test, no code review)
- hbto [Vauxoo] http://www.vauxoo.com (community): Needs Fixing
- Moisés López - http://www.vauxoo.com (community): Needs Fixing
- Nhomar - Vauxoo (community): Needs Fixing
- Stefan Rijnhart (Opener) (community): Disapprove
- Yannick Vaucher @ Camptocamp (community): Needs Fixing (code, test)
- OpenERP Core Team: Pending requested
-
Diff: 727 lines (+465/-95)4 files modifiedopenerp/addons/base/res/res_partner.py (+209/-76)
openerp/addons/base/res/res_partner_view.xml (+45/-18)
openerp/addons/base/tests/test_base.py (+210/-0)
openerp/addons/base/tests/test_expression.py (+1/-1)
| Changed in openobject-server: | |
| assignee: | nobody → OpenERP's Framework R&D (openerp-dev-framework) |
| importance: | Undecided → Low |
| status: | New → Confirmed |
Revision history for this message
| Olivier Dony (Odoo) (odo-openerp) wrote | #1 |
| Changed in openobject-server: | |
| importance: | Low → Medium |
| milestone: | none → 7.0 |
| status: | Confirmed → Fix Committed |
Revision history for this message
| Olivier Dony (Odoo) (odo-openerp) wrote | #2 |
| Changed in openobject-server: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
The fact that use_parent_address is enabled by default (and hidden from the view) can be quite confusing. The proposed fix is part of the larger effort to fix contacts and addresses handling: see the description of the merge proposal for more details: https:/