Fix week_of, quarter_of, year_of calculation in my_activity.py

my_activity.py

Index: my_activity.py
diff --git a/my_activity.py b/my_activity.py
index 66b5c2da7e2520b8462550675382e03ebdea119d..540af78d18bf89ea0af1c4ba8abcb834611ad105 100755
--- a/my_activity.py
+++ b/my_activity.py
@@ -142,18 +142,25 @@ def username(email):
   return email and email.split('@', 1)[0]
 
 
+def datetime_to_midnight(date):
+  return date - timedelta(hours=date.hour, minutes=date.minute,
+                          seconds=date.second, microseconds=date.microsecond)
+
+
 def get_quarter_of(date):
-  begin = date - relativedelta(months=(date.month % 3) - 1, days=(date.day - 1))
+  begin = (datetime_to_midnight(date) -

[M-A Ruel, 2012/10/22 18:43:14]

Why not date.date() or simply use datetime.date throughout?

[cjhopman, 2012/10/22 18:56:21]

I didn't use date.date() here because later on we expect it to be a datetime,
and as far as I know doing date.date() and then converting back to datetime is
really no simpler than subtracting out the time.

I think the best thing to do would be to use datetime.date throughout and almost
made that change instead of this. Really, that is what begin/end are used as
throughout.

I sort of wanted to keep this fix simple, but if you'd rather see the change to
use datetime.date throughout I'd be happy to do that.

[M-A Ruel, 2012/10/22 19:09:09]

I prefer simple code to simple fixes.

Also I don't see either why you are using the module relativedelta, it's easy to
do manually. That's a huge burden to ask others to install a random python
module.

Anyhow, while I wouldn't have written the code this way, I don't mind as I'm not
supporting this code anyway so lgtm.

+           relativedelta(months=(date.month % 3) - 1, days=(date.day - 1)))
   return begin, begin + relativedelta(months=3)
 
 
 def get_year_of(date):
-  begin = date - relativedelta(months=(date.month - 1), days=(date.day - 1))
+  begin = (datetime_to_midnight(date) -
+           relativedelta(months=(date.month - 1), days=(date.day - 1)))
   return begin, begin + relativedelta(years=1)
 
 
 def get_week_of(date):
-  begin = date - timedelta(days=date.weekday())
+  begin = (datetime_to_midnight(date) - timedelta(days=date.weekday()))
   return begin, begin + timedelta(days=7)