Merge lp:~openerp-dev/openerp-web/7.0-improve-gantt-bth into lp:openerp-web/7.0

[Bhumi Thakkar (Open ERP)]

Hello,

       In gantt view, if value is '0' or '0.00' then return. for string it is done if (!tmp) return but for integer or float it does not work.

In Project => Projects => in Gantt View displayed records which have '0' or '0.00' value.

In IE Browser, Get error because of null values records
"Invalid Argument" also not displayed create button. if records displayed with zero value.

Thanks.

[Christophe Matthieu (OpenERP) (chm-openerp)]

Hi,
Why "tmp < 1" ?
What about the value contained between 0 and 1 ?
Thanks

review: Needs Fixing

Unmerged revisions

3709. By Bhumi Thakkar (Open ERP) on 2013-01-22

[IMP] Improve code for value is less than 1 then return.

=== modified file 'addons/web_gantt/static/src/js/gantt.js'
--- addons/web_gantt/static/src/js/gantt.js	2012-12-28 10:31:54 +0000
+++ addons/web_gantt/static/src/js/gantt.js	2013-01-22 10:02:32 +0000
@@ -155,7 +155,7 @@
                 } else { // we assume date_duration is defined
                     var tmp = instance.web.format_value(task[self.fields_view.arch.attrs.date_delay],
                         self.fields[self.fields_view.arch.attrs.date_delay]);
-                    if (!tmp)
+                    if (!tmp || tmp < 1)
                         return;
                     task_stop = task_start.clone().addMilliseconds(tmp * 60 * 60 * 1000);
                 }