Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

Commit

Permalink
feat(filter.date): use mediumDate as default
Browse files Browse the repository at this point in the history
Breaking change!

Previously the default was fullDate.
  • Loading branch information
IgorMinar committed Oct 20, 2011
1 parent e175db3 commit c6c3949
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
4 changes: 2 additions & 2 deletions src/filters.js
Original file line number Diff line number Diff line change
Expand Up @@ -318,7 +318,7 @@ var GET_TIME_ZONE = /[A-Z]{3}(?![+\-])/,
* @param {(Date|number|string)} date Date to format either as Date object, milliseconds (string or
* number) or ISO 8601 extended datetime string (yyyy-MM-ddTHH:mm:ss.SSSZ).
* @param {string=} format Formatting rules (see Description). If not specified,
* `fullDate` is used.
* `mediumDate` is used.
* @returns {string} Formatted string or the input if input is not recognized as date/millis.
*
* @example
Expand Down Expand Up @@ -349,7 +349,7 @@ angularFilter.date = function(date, format) {
parts = [],
fn, match;

format = format || 'fullDate'
format = format || 'mediumDate'
format = $locale.DATETIME_FORMATS[format] || format;
if (isString(date)) {
if (NUMBER_STRING.test(date)) {
Expand Down
10 changes: 5 additions & 5 deletions test/FiltersSpec.js
Original file line number Diff line number Diff line change
Expand Up @@ -225,13 +225,13 @@ describe('filter', function() {
});

it('should do basic filter', function() {
expect(date(noon)).toEqual(date(noon, 'fullDate'));
expect(date(noon, '')).toEqual(date(noon, 'fullDate'));
expect(date(noon)).toEqual(date(noon, 'mediumDate'));
expect(date(noon, '')).toEqual(date(noon, 'mediumDate'));
});

it('should accept number or number string representing milliseconds as input', function() {
expect(date(noon.getTime())).toEqual(date(noon.getTime(), 'fullDate'));
expect(date(noon.getTime() + "")).toEqual(date(noon.getTime() + "", 'fullDate'));
expect(date(noon.getTime())).toEqual(date(noon.getTime(), 'mediumDate'));
expect(date(noon.getTime() + "")).toEqual(date(noon.getTime() + "", 'mediumDate'));
});

it('should accept various format strings', function() {
Expand Down Expand Up @@ -297,7 +297,7 @@ describe('filter', function() {
it('should be able to parse ISO 8601 dates/times using', function() {
var isoString = '2010-09-03T05:05:08.872Z';
expect(date(isoString)).
toEqual(date(isoString, 'fullDate'));
toEqual(date(isoString, 'mediumDate'));
});

it('should parse format ending with non-replaced string', function() {
Expand Down

0 comments on commit c6c3949

Please sign in to comment.