Merge lp:~jeffery9/ocb-addons/ocb-addons into lp:ocb-addons

Proposed by jeffery chen fan
Status: Work in progress
Proposed branch: lp:~jeffery9/ocb-addons/ocb-addons
Merge into: lp:ocb-addons
Diff against target: 31 lines (+4/-3)
2 files modified
crm/crm_phonecall_view.xml (+3/-3)
crm/wizard/crm_phonecall_to_phonecall_view.xml (+1/-0)
To merge this branch: bzr merge lp:~jeffery9/ocb-addons/ocb-addons
Reviewer Review Type Date Requested Status
Yannick Vaucher @ Camptocamp code review, no test Approve
Holger Brunn (Therp) Needs Fixing
Review via email: mp+193883@code.launchpad.net

Description of the change

fix bug#1247018

To post a comment you must log in.
lp:~jeffery9/ocb-addons/ocb-addons updated
9638. By jeffery chen fan

MERGE

Revision history for this message
Holger Brunn (Therp) (hbrunn) wrote :

Please link the bug to your branch for simpler tracking.

Further, only #28 seems to actually fix the bug, so please remove the UI changes included in crm_phonecall_view.xml

review: Needs Fixing
Revision history for this message
Holger Brunn (Therp) (hbrunn) wrote :

setting to work in progress due to inactivity

Revision history for this message
Yannick Vaucher @ Camptocamp (yvaucher-c2c) wrote :
review: Needs Information
Revision history for this message
Alexis de Lattre (alexis-via) wrote :

I confirm that the addition of
<field name="context">{'default_state': 'open'}</field>
in crm/wizard/crm_phonecall_to_phonecall_view.xml
fixes the bug.

The other changes are not related to the bug.

Revision history for this message
Yannick Vaucher @ Camptocamp (yvaucher-c2c) wrote :

Please remove other changes in crm/crm_phonecall_view.xml

Otherwise LGTM

review: Approve (code review, no test)

Unmerged revisions

9638. By jeffery chen fan

MERGE

9637. By jeffery chen fan

fix bug#1247018

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'crm/crm_phonecall_view.xml'
2--- crm/crm_phonecall_view.xml 2012-10-23 16:05:04 +0000
3+++ crm/crm_phonecall_view.xml 2013-11-06 02:31:18 +0000
4@@ -133,12 +133,12 @@
5 <field name="partner_id"
6 on_change="onchange_partner_id(partner_id)"/>
7 <field name="partner_phone"
8- invisible="1"/>
9+ />
10 <field name="user_id"/>
11 <field name="categ_id" widget="selection"
12 domain="[('object_id.model', '=', 'crm.phonecall')]"
13- invisible="1"/>
14- <field name="state" invisible="1"/>
15+ />
16+ <field name="state"/>
17 <field name="create_date" invisible="1"/>
18 <field name="opportunity_id" invisible="1" on_change="on_change_opportunity(opportunity_id)"/>
19 <button string="Schedule Other Call"
20
21=== modified file 'crm/wizard/crm_phonecall_to_phonecall_view.xml'
22--- crm/wizard/crm_phonecall_to_phonecall_view.xml 2012-10-23 16:05:04 +0000
23+++ crm/wizard/crm_phonecall_to_phonecall_view.xml 2013-11-06 02:31:18 +0000
24@@ -36,6 +36,7 @@
25 <field name="view_mode">form</field>
26 <field name="view_id" ref="phonecall_to_phonecall_view"/>
27 <field name="target">new</field>
28+ <field name="context">{'default_state': 'open'}</field>
29 </record>
30
31 </data>