fix(data-types): unnecessary warning when getting data with DATE data…

…Types (#13712) * fix(data-types): unnecessary error when getting data with DATE dataTypes * fix(data-types): date stringify mariadb
sequelize · Nov 27, 2021 · 121884b · 121884b
1 parent 2d7b865
commit 121884b
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 7 deletions.
diff --git a/lib/dialects/mariadb/data-types.js b/lib/dialects/mariadb/data-types.js
@@ -54,8 +54,12 @@ module.exports = BaseTypes => {
       return this._length ? `DATETIME(${this._length})` : 'DATETIME';
     }
     _stringify(date, options) {
-      date = this._applyTimezone(date, options);
-      return date.format('YYYY-MM-DD HH:mm:ss.SSS');
+      if(_.isDate(date)){
+        date = this._applyTimezone(date, options);
+        return date.format('YYYY-MM-DD HH:mm:ss.SSS');
+      }
+
+      return date;
     }
     static parse(value, options) {
       value = value.string();

diff --git a/lib/dialects/mysql/data-types.js b/lib/dialects/mysql/data-types.js
@@ -53,12 +53,16 @@ module.exports = BaseTypes => {
       return this._length ? `DATETIME(${this._length})` : 'DATETIME';
     }
     _stringify(date, options) {
-      date = this._applyTimezone(date, options);
-      // Fractional DATETIMEs only supported on MySQL 5.6.4+
-      if (this._length) {
-        return date.format('YYYY-MM-DD HH:mm:ss.SSS');
+      if(_.isDate(date)){
+        date = this._applyTimezone(date, options);
+        // Fractional DATETIMEs only supported on MySQL 5.6.4+
+        if (this._length) {
+          return date.format('YYYY-MM-DD HH:mm:ss.SSS');
+        }
+        return date.format('YYYY-MM-DD HH:mm:ss');
       }
-      return date.format('YYYY-MM-DD HH:mm:ss');
+
+      return date;
     }
     static parse(value, options) {
       value = value.string();