fix timestamp string representation on different platforms

realm · Jul 11, 2023 · 7ff238f · 7ff238f
1 parent 9e7503b
commit 7ff238f
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 6 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,7 +5,7 @@
 
 ### Fixed
 * <How do the end-user experience this issue? what was the impact?> ([#????](https://github.com/realm/realm-core/issues/????), since v?.?.?)
-* None.
+* Fix timestamp rapresentation when converting to json on different platforms. ([#5451](https://github.com/realm/realm-core/issues/5451)).
 
 ### Breaking changes
 * None.

diff --git a/src/realm/obj.cpp b/src/realm/obj.cpp
@@ -991,9 +991,7 @@ void out_mixed_xjson(std::ostream& out, const Mixed& val)
         }
         case type_Timestamp: {
             out << "{\"$date\": {\"$numberLong\": \"";
-            auto ts = val.get<Timestamp>();
-            int64_t timeMillis = ts.get_seconds() * 1000 + ts.get_nanoseconds() / 1000000;
-            out << timeMillis;
+            out << util::serializer::print_value(val.get<Timestamp>());
             out << "\"}}";
             break;
         }