openerp-community-reviewer team mailing list archive

["Yannick Vaucher @ Camptocamp" <yannick.vaucher@xxxxxxxxxxxxxx>]

Review: Needs Fixing

Hello,

I think you can even use float_compare instead of float_is_zero. That way you can do the whole teste in one if.

http://bazaar.launchpad.net/~openerp/openobject-server/7.0/view/head:/openerp/tools/float_utils.py#L99


if float_compare(float_end_time, float_start_time, precision_rounding=0.0000001) == -1:

Cheers.
-- 
https://code.launchpad.net/~agilebg/hr-timesheet/imp_hr_attendance_analysis_roundings/+merge/206493
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:hr-timesheet.