Merge pull request #4694 from scampi/undefined-date

missing date patch

src/ui/public/stringify/__tests__/_date.js

@@ -0,0 +1,20 @@
+describe('Date Format', function () {
+  var fieldFormats;
+  var expect = require('expect.js');
+  var ngMock = require('ngMock');
+
+  beforeEach(ngMock.module('kibana'));
+  beforeEach(ngMock.inject(function (Private) {
+    fieldFormats = Private(require('ui/registry/field_formats'));
+  }));
+
+  it('decoding an undefined or null date should return an empty string', function () {
+    var DateFormat = fieldFormats.getType('date');
+    var date = new DateFormat({
+      pattern: 'dd-MM-yyyy'
+    });
+    expect(date.convert(null)).to.be('');
+    expect(date.convert(undefined)).to.be('');
+  });
+
+});

src/ui/public/stringify/__tests__/index.js

@@ -4,4 +4,5 @@ describe('Stringify Component', function () {
   require('./_source');
   require('./_string');
   require('./_url');
+  require('./_date');
 });

src/ui/public/stringify/types/Date.js

@@ -45,6 +45,9 @@ define(function (require) {
       if (this._memoizedPattern !== pattern) {
         this._memoizedPattern = pattern;
         this._memoizedConverter = _.memoize(function converter(val) {
+          if (val === null || val === undefined) {
+            return '';
+          }
           return moment(val).format(pattern);
         });
       }