Merge lp:~dannf/laika/support-multiple-date-types into lp:laika

[dann frazier]

[Peter Petrakis (peter-petrakis)]

Why not just cast lpdate to str and take split from there? The lang is
duck typing after all and there isn't any other actions taken here. Now
you've got two code paths that do the same thing when:

dt = datetime.datetime.strptime(str(lpdate).split('.')[0], '%Y-%m-%dT%H:%M:%S')

will do.

[Alex Chiang (achiang)]

Thanks dannf.

review: Approve

=== modified file 'laika'
--- laika	2012-03-08 16:04:27 +0000
+++ laika	2012-04-02 16:40:25 +0000
@@ -24,11 +24,18 @@
 UTCNOW = datetime.datetime.utcnow()
 
 def lpdate2date(lpdate):
-    try:
-        dt = datetime.datetime.strptime(lpdate.split('.')[0], '%Y-%m-%dT%H:%M:%S')
-        return dt
-    except ValueError:
-        return None
+    # launchpad date type is apparently in-flux
+    if type(lpdate) is datetime.datetime:
+        return lpdate.replace(tzinfo=None)
+    elif type(lpdate) is str:
+        try:
+            print lpdate.split('.')[0]
+            dt = datetime.datetime.strptime(lpdate.split('.')[0], '%Y-%m-%dT%H:%M:%S')
+            return dt
+        except ValueError:
+            return None
+    else:
+        raise TypeError
                                  
 class LPWrap:
     '''Simple wrapper Cache-Proxy for LP objects'''