Merge lp:~openerp-dev/openerp-web/7.0-opw-585938-cpa into lp:openerp-web/7.0

Proposed by Chirag Patel (OpenERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openerp-web/7.0-opw-585938-cpa
Merge into: lp:openerp-web/7.0
Diff against target: 14 lines (+1/-3)
1 file modified
addons/web_calendar/static/src/js/calendar.js (+1/-3)
To merge this branch: bzr merge lp:~openerp-dev/openerp-web/7.0-opw-585938-cpa
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+148690@code.launchpad.net

Description of the change

Hello,

Demo: Sales > Quotations > Calendar View
1) Click on any date to create new quotations.
2) Fill data

Observed: After filled data in popup window all buttons can't work.

In name field default set "/" but get_event_data method pass default_name with "" in context.
name field is readonly and required.
When click on any button first it's try to save but required field is blank.
So invalid form can't save and all button can't work.

Thanks

To post a comment you must log in.

Unmerged revisions

3761. By Chirag Patel (OpenERP)

[FIX] Fixed name field value changed from calendar.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'addons/web_calendar/static/src/js/calendar.js'
2--- addons/web_calendar/static/src/js/calendar.js 2013-02-04 14:38:04 +0000
3+++ addons/web_calendar/static/src/js/calendar.js 2013-02-15 12:17:27 +0000
4@@ -338,9 +338,7 @@
5 return r;
6 },
7 get_event_data: function(event_obj) {
8- var data = {
9- name: event_obj.text
10- };
11+ var data = {};
12 data[this.date_start] = instance.web.datetime_to_str(event_obj.start_date);
13 if (this.date_stop) {
14 data[this.date_stop] = instance.web.datetime_to_str(event_obj.end_date);