diff options
| author | Iwein Fuld | 2012-08-30 06:39:52 +0200 |
|---|---|---|
| committer | Misko Hevery | 2012-09-06 16:06:23 -0700 |
| commit | 9473780e77a960ba27644ca76c2413924cc8972e (patch) | |
| tree | 10f54e3fb0bba365c9aa3190093a69b1ade62fcd /test | |
| parent | eb5fd400d34ea9c6e5ce1e76dffd9b40f412763a (diff) | |
| download | angular.js-9473780e77a960ba27644ca76c2413924cc8972e.tar.bz2 | |
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.
Diffstat (limited to 'test')
| -rw-r--r-- | test/ng/filter/filtersSpec.js | 3 |
1 files changed, 3 insertions, 0 deletions
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'); |