From a631ceb223392244d6241c5bb44712ef802f0c98 Mon Sep 17 00:00:00 2001
From: Iwein Fuld
Date: Thu, 30 Aug 2012 06:39:52 +0200
Subject: fix(dateFilter): make timezone optional

Makes the time zone optional in the date filter

Problem with the current R_ISO8601_STR regex was that the time was optional, but the zone was not.
This results in the filter not formatting local date times, which it could easily do.

For example:
2012-08-30 -> formatted
2012-08-30T06:06:06.123Z -> formatted
2012-08-30T06:06:06.123 -> NOT formatted

A simple change in the regex fixes this. Arguably this is closer to the ISO8601 spec which specifies
local dates being in the "current time zone" and not requiring a Z. In any case it behaves more like
a user would expect.
---
 test/ng/filter/filtersSpec.js | 3 +++
 1 file changed, 3 insertions(+)

(limited to 'test')

diff --git a/test/ng/filter/filtersSpec.js b/test/ng/filter/filtersSpec.js
index 9e0a408a..883e91f6 100644
--- a/test/ng/filter/filtersSpec.js
+++ b/test/ng/filter/filtersSpec.js
@@ -258,6 +258,9 @@ describe('filters', function() {
 
       expect(date('20030910T033203-0930', format)).toEqual('2003-09 03');
 
+      //no timezone
+      expect(date('2003-09-10T13:02:03.000', format)).toEqual('2003-09 03');
+
       //no millis
       expect(date('2003-09-10T13:02:03Z', format)).toEqual('2003-09 03');
 
-- 
cgit v1.2.3