Merge lp:~openerp-dev/openerp-web/trunk-bug-1197323-psa into lp:openerp-web

Proposed by Paramjit Singh Sahota(OpenERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openerp-web/trunk-bug-1197323-psa
Merge into: lp:openerp-web
Diff against target: 23 lines (+8/-4)
1 file modified
addons/web_calendar/static/src/js/calendar.js (+8/-4)
To merge this branch: bzr merge lp:~openerp-dev/openerp-web/trunk-bug-1197323-psa
Reviewer Review Type Date Requested Status
OpenERP R&D Web Team Pending
Review via email: mp+173133@code.launchpad.net

Description of the change

Hello,

Fixed the issue when we switch to any calendar view where date_stop was not declared than itz will show the traceback **AttributeError: 'NoneType' object has no attribute 'split'**.

Thankz You.

To post a comment you must log in.

Unmerged revisions

3770. By Paramjit Singh Sahota(OpenERP)

[FIX] [trunk/7.0]OpenERP Server Error when clicking on 'Order Planning'

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'addons/web_calendar/static/src/js/calendar.js'
--- addons/web_calendar/static/src/js/calendar.js 2013-06-28 09:07:06 +0000
+++ addons/web_calendar/static/src/js/calendar.js 2013-07-05 06:03:26 +0000
@@ -399,11 +399,15 @@
399 var A = format(this.range_start.clone().addDays(-6));399 var A = format(this.range_start.clone().addDays(-6));
400 var B = format(this.range_stop.clone().addDays(6));400 var B = format(this.range_stop.clone().addDays(6));
401 var domain = [401 var domain = [
402 '|', '|',402 '&', [this.date_start, '>=', A], [this.date_start, '<=', B]
403 '&', [this.date_start, '>=', A], [this.date_start, '<=', B],
404 '&', [this.date_stop, '>=', A], [this.date_stop, '<=', B],
405 '&', [this.date_start, '<', A], [this.date_stop, '>', B]
406 ];403 ];
404 if (this.date_stop) {
405 domain.push(
406 '&', [this.date_stop, '>=', A], [this.date_stop, '<=', B],
407 '&', [this.date_start, '<', A], [this.date_stop, '>', B]
408 );
409 domain.unshift("|", "|");
410 }
407 domain.concat(this.last_search[0].slice(0))411 domain.concat(this.last_search[0].slice(0))
408 return domain;412 return domain;
409 },413 },