BUG: tools.parse_time_string does not parse intraday strings correctly …

…#2306

pandas/tseries/index.py

@@ -1539,7 +1539,7 @@ def _to_m8(key):
 def _str_to_dt_array(arr, offset=None, dayfirst=None, yearfirst=None):
     def parser(x):
         result = parse_time_string(x, offset, dayfirst=dayfirst,
-                                   yearfirst=None)
+                                   yearfirst=yearfirst)
         return result[0]
 
     arr = np.asarray(arr, dtype=object)

pandas/tseries/tests/test_timeseries.py

@@ -246,6 +246,10 @@ def _check_rng(rng):
         _check_rng(rng_eastern)
         _check_rng(rng_utc)
 
+    def test_ctor_str_intraday(self):
+        rng = DatetimeIndex(['1-1-2000 00:00:01'])
+        self.assert_(rng[0].second == 1)
+
     def test_series_ctor_plus_datetimeindex(self):
         rng = date_range('20090415', '20090519', freq='B')
         data = dict((k, 1) for k in rng)

pandas/tseries/tools.py

@@ -235,18 +235,13 @@ def parse_time_string(arg, freq=None, dayfirst=None, yearfirst=None):
 
     repl = {}
     reso = 'year'
-    stopped = False
+
     for attr in ["year", "month", "day", "hour",
                  "minute", "second", "microsecond"]:
-        can_be_zero = ['hour', 'minute', 'second', 'microsecond']
         value = getattr(parsed, attr)
         if value is not None and value != 0:  # or attr in can_be_zero):
             repl[attr] = value
-            if not stopped:
-                reso = attr
-        else:
-            stopped = True
-            break
+            reso = attr
     ret = default.replace(**repl)
     return ret, parsed, reso  # datetime, resolution