Merge lp:~cci-team/openobject-addons/cci_crm_fix-invoice_id-redundacy into lp:~cci-team/openobject-addons/addons-cci

Proposed by qdp (OpenERP)
Status: Needs review
Proposed branch: lp:~cci-team/openobject-addons/cci_crm_fix-invoice_id-redundacy
Merge into: lp:~cci-team/openobject-addons/addons-cci
Diff against target: 27 lines (+4/-2)
2 files modified
cci_crm/cci_crm.py (+3/-1)
cci_crm/cci_crm_view.xml (+1/-1)
To merge this branch: bzr merge lp:~cci-team/openobject-addons/cci_crm_fix-invoice_id-redundacy
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+120574@code.launchpad.net

This proposal supersedes a proposal from 2012-06-28.

Description of the change

Fix an ambiguous name on crm.case.

n cci_crm, we add an 'invoice_id'. Or in 'event', the object 'event.registration' inherits from crm.case but had already an 'invoice_id' name. This 'event.registration' has two fields 'invoice_id' : one on crm.case and one on 'event.registration'. This ambuguity causes some problems to show or use this field.

To correct this, I've changed the 'invoice_id' on cci_crm to 'sponsor_invoice_id' because this is the real usage of this field : to link a case to the invoice of a sponsor and only for this (on screen it appears in 'sponsor' part of view).

To post a comment you must log in.
Revision history for this message
qdp (OpenERP) (qdp) wrote : Posted in a previous version of this proposal

Hello Philmer,

mmh i think you wrongly choose the branch aimed by your proposal: this is for addons trunk. Let me choose the right target.

See you soon,
Quentin

Revision history for this message
qdp (OpenERP) (qdp) wrote :

to the reviewer: i just changed the target from addons/trunk to this branch.

Unmerged revisions

150. By Philippe Vandermeer <philmer@philmer-laptop2>

[FIX] ambiguous invoice_id name with module cci_event renamed to sponsor_invoice_id

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'cci_crm/cci_crm.py'
--- cci_crm/cci_crm.py 2011-07-14 13:37:12 +0000
+++ cci_crm/cci_crm.py 2012-08-21 14:06:20 +0000
@@ -65,7 +65,9 @@
65 _columns = {65 _columns = {
66 # 'meeting_id' : fields.many2one('meeting.confidential.info','Meeting confidential'),66 # 'meeting_id' : fields.many2one('meeting.confidential.info','Meeting confidential'),
67 'group_id' : fields.many2one('res.groups','Group'),67 'group_id' : fields.many2one('res.groups','Group'),
68 'invoice_id' : fields.many2one('account.invoice','Invoice'),68 # from 'invoice_id' to 'sponsor_invoice_id' to eliminate ambiguity with object 'cci_event.registration'
69 # using same name and inheriting from crm.case (inherits)
70 'sponsor_invoice_id' : fields.many2one('account.invoice','Invoice'),
69 'event_ids' : fields.many2many('event.event','case_event_rel','case_id','event_id','Events'),71 'event_ids' : fields.many2many('event.event','case_event_rel','case_id','event_id','Events'),
70 }72 }
7173
7274
=== modified file 'cci_crm/cci_crm_view.xml'
--- cci_crm/cci_crm_view.xml 2011-07-14 13:37:12 +0000
+++ cci_crm/cci_crm_view.xml 2012-08-21 14:06:20 +0000
@@ -39,7 +39,7 @@
39 <page string="Sponsoring">39 <page string="Sponsoring">
40 <separator string="Events" colspan="4"/>40 <separator string="Events" colspan="4"/>
41 <field name="event_ids" colspan="4" nolabel="1"/>41 <field name="event_ids" colspan="4" nolabel="1"/>
42 <field name="invoice_id" invisible="1"/>42 <field name="sponsor_invoice_id" invisible="1"/>
43 </page>43 </page>
44 </notebook>44 </notebook>
45 </field>45 </field>

Subscribers

People subscribed via source and target branches

to all changes: