database/patches: special case during epoch/time conversion

sagemathinc · Mar 31, 2022 · a57cd96 · a57cd96
1 parent b7949a5
commit a57cd96
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/src/packages/database/postgres-blobs.coffee b/src/packages/database/postgres-blobs.coffee
@@ -811,7 +811,12 @@ exports.extend_PostgreSQL = (ext) -> class PostgreSQL extends ext
                     opts.cb(err)
                 else
                     for p in patches
-                        p.time = new Date(p.epoch)
+                        # TODO why using epoch and then converting to Date, why not just taking time?
+                        # Besides that: @hsy noticed in development that p.epoch could be a string, resulting in an invalid date.
+                        if typeof p.epoch == 'string'
+                            p.time = new Date(parseInt(p.epoch))
+                        else
+                            p.time = new Date(p.epoch)
                         delete p.epoch
                     opts.cb(undefined, patches)