← Back to team overview

credativ team mailing list archive

[Bug 897662] Re: Weekly timesheet created on monday covers two weeks

 

Hi Amit,

well, maybe I was not clear enough but the bug only occurs when you
create a timesheet on Monday! You created a timesheet on Wednesday and
then it works as expected.

The reason for this is that the relativedelta in the code selects the
first Monday from today() a week ago. On a Monday, the result is Monday
last week. On a Wednesday, the result is last Monday.

Cheers,
Stefan.

-- 
You received this bug notification because you are a member of OpenERP
Framework Experts, which is subscribed to OpenERP Addons.
https://bugs.launchpad.net/bugs/897662

Title:
  Weekly timesheet created on monday covers two weeks

Status in OpenERP Addons (modules):
  Incomplete

Bug description:
  Hi,

  Set the company's timesheet range to 'Week'. Create a timesheet on
  Monday. The new timesheet covers a period of two weeks, starting from
  the Monday a week before.

  Culprit is (datetime.today() + relativedelta(weekday=0, weeks=-1) in
  hr_timesheet_sheet/hr_timesheet_sheet.py:hr_timesheet_sheet.default_date_to().

  >>> from datetime import datetime
  >>> from dateutil.relativedelta import relativedelta
  >>> datetime(2011, 11, 28, 13, 38, 3, 560694) +  relativedelta(weekday=0, weeks=-1)
  datetime.datetime(2011, 11, 21, 13, 38, 3, 560694)

  Instead, use (datetime.today() + relativedelta(weekday=0, days=-6).

  >>> datetime(2011, 11, 28, 13, 38, 3, 560694) +  relativedelta(weekday=0, days=-6)
  datetime.datetime(2011, 11, 28, 13, 38, 3, 560694)

  Reproducable in 6.0 and trunk.

  Cheers,
  Stefan.

To manage notifications about this bug go to:
https://bugs.launchpad.net/openobject-addons/+bug/897662/+subscriptions


References