fix(#1113): postgres parse date value from string or date object

[Venipa]

#1113 since this issue is still open and i recently started using pglite with electron i got this issue.

fixed it by just returning a new date that allows a Date and a string argument without changing much.

[zfben]

when will this request merge?

[Venipa]

@Angelelz any update on this issue? #1113

[pranshuchittora]

This issue is a major blocker, I can confirm that this change fixes this issue. Not sure about any side-effects or edge cases.
In case anyone needs patch via patch-package

patches/drizzle-orm+0.38.2.patch

diff --git a/node_modules/drizzle-orm/pg-core/columns/timestamp.cjs b/node_modules/drizzle-orm/pg-core/columns/timestamp.cjs
index f558f6c..32185fa 100644
--- a/node_modules/drizzle-orm/pg-core/columns/timestamp.cjs
+++ b/node_modules/drizzle-orm/pg-core/columns/timestamp.cjs
@@ -58,7 +58,7 @@ class PgTimestamp extends import_common.PgColumn {
     return new Date(this.withTimezone ? value : value + "+0000");
   };
   mapToDriverValue = (value) => {
-    return value.toISOString();
+      return new Date(value).toISOString();
   };
 }
 class PgTimestampStringBuilder extends import_date_common.PgDateColumnBaseBuilder {
diff --git a/node_modules/drizzle-orm/pg-core/columns/timestamp.js b/node_modules/drizzle-orm/pg-core/columns/timestamp.js
index 5f793f0..b455b04 100644
--- a/node_modules/drizzle-orm/pg-core/columns/timestamp.js
+++ b/node_modules/drizzle-orm/pg-core/columns/timestamp.js
@@ -31,7 +31,7 @@ class PgTimestamp extends PgColumn {
     return new Date(this.withTimezone ? value : value + "+0000");
   };
   mapToDriverValue = (value) => {
-    return value.toISOString();
+    return new Date(value).toISOString();
   };
 }
 class PgTimestampStringBuilder extends PgDateColumnBaseBuilder {

[koitoror]

Its a major blocker ..

[pranshuchittora]

This issue is a major blocker, I can confirm that this change fixes this issue. Not sure about any side-effects or edge cases. In case anyone needs patch via patch-package

patches/drizzle-orm+0.38.2.patch

diff --git a/node_modules/drizzle-orm/pg-core/columns/timestamp.cjs b/node_modules/drizzle-orm/pg-core/columns/timestamp.cjs
index f558f6c..32185fa 100644
--- a/node_modules/drizzle-orm/pg-core/columns/timestamp.cjs
+++ b/node_modules/drizzle-orm/pg-core/columns/timestamp.cjs
@@ -58,7 +58,7 @@ class PgTimestamp extends import_common.PgColumn {
     return new Date(this.withTimezone ? value : value + "+0000");
   };
   mapToDriverValue = (value) => {
-    return value.toISOString();
+      return new Date(value).toISOString();
   };
 }
 class PgTimestampStringBuilder extends import_date_common.PgDateColumnBaseBuilder {
diff --git a/node_modules/drizzle-orm/pg-core/columns/timestamp.js b/node_modules/drizzle-orm/pg-core/columns/timestamp.js
index 5f793f0..b455b04 100644
--- a/node_modules/drizzle-orm/pg-core/columns/timestamp.js
+++ b/node_modules/drizzle-orm/pg-core/columns/timestamp.js
@@ -31,7 +31,7 @@ class PgTimestamp extends PgColumn {
     return new Date(this.withTimezone ? value : value + "+0000");
   };
   mapToDriverValue = (value) => {
-    return value.toISOString();
+    return new Date(value).toISOString();
   };
 }
 class PgTimestampStringBuilder extends PgDateColumnBaseBuilder {

This diff should work, but we need to adapt the types too, because when I run this patch, I am seeing issues with the drizzle-zod schema validation. Not sure how much related this is w.r.t. to this PR.

[EntropyValley]

Will this be patched into the other cores as well? I'm seeing a similar issue with mysql.