Merge lp:~lepistone/openobject-italia/6.1-ondelete-restrict-lep into lp:~openobject-italia-core-devs/openobject-italia/italian-addons-6.1

Proposed by Leonardo Pistone
Status: Merged
Merged at revision: 231
Proposed branch: lp:~lepistone/openobject-italia/6.1-ondelete-restrict-lep
Merge into: lp:~openobject-italia-core-devs/openobject-italia/italian-addons-6.1
Diff against target: 41 lines (+5/-5)
1 file modified
l10n_it_base/partner/partner.py (+5/-5)
To merge this branch: bzr merge lp:~lepistone/openobject-italia/6.1-ondelete-restrict-lep
Reviewer Review Type Date Requested Status
Lorenzo Battistini Approve
Review via email: mp+161084@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Lorenzo Battistini (elbati) wrote :

Non sarebbe da aggiungere anche ai campi 'province_id' di 'res.city' e 'province' di 'res.partner.address'?

review: Needs Information
232. By Leonardo Pistone

[imp] l10n_it_base: really do that on provinces, too

Revision history for this message
Leonardo Pistone (lepistone) wrote :

Correct!

> Non sarebbe da aggiungere anche ai campi 'province_id' di 'res.city' e
> 'province' di 'res.partner.address'?

Revision history for this message
Lorenzo Battistini (elbati) wrote :

Puoi farlo anche per 7.0?

review: Approve
Revision history for this message
Leonardo Pistone (lepistone) wrote :

> Puoi farlo anche per 7.0?

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'l10n_it_base/partner/partner.py'
2--- l10n_it_base/partner/partner.py 2013-01-19 17:36:47 +0000
3+++ l10n_it_base/partner/partner.py 2013-04-26 10:13:28 +0000
4@@ -28,7 +28,7 @@
5 _description = 'Region'
6 _columns = {
7 'name': fields.char('Region Name', size=64, help='The full name of the region.', required=True),
8- 'country_id': fields.many2one('res.country', 'Country'),
9+ 'country_id': fields.many2one('res.country', 'Country', ondelete='restrict'),
10 }
11 res_region()
12
13@@ -38,7 +38,7 @@
14 _columns = {
15 'name': fields.char('Province Name', size=64, help='The full name of the province.', required=True),
16 'code': fields.char('Province Code', size=2, help='The province code in two chars.',required=True),
17- 'region': fields.many2one('res.region','Region'),
18+ 'region': fields.many2one('res.region','Region', ondelete='restrict'),
19 }
20
21 res_province()
22@@ -48,7 +48,7 @@
23 _description = 'City'
24 _columns = {
25 'name': fields.char('City', size=64, required=True),
26- 'province_id': fields.many2one('res.province','Province'),
27+ 'province_id': fields.many2one('res.province','Province', ondelete='restrict'),
28 'zip': fields.char('ZIP', size=5),
29 'phone_prefix': fields.char('Telephone Prefix' , size=16),
30 'istat_code': fields.char('ISTAT code', size=16),
31@@ -61,8 +61,8 @@
32 _inherit = 'res.partner.address'
33
34 _columns = {
35- 'province': fields.many2one('res.province', string='Province'),
36- 'region': fields.many2one('res.region', string='Region'),
37+ 'province': fields.many2one('res.province', string='Province', ondelete='restrict'),
38+ 'region': fields.many2one('res.region', string='Region', ondelete='restrict'),
39 }
40
41 def on_change_city(self, cr, uid, ids, city):

Subscribers

People subscribed via source and target branches