[COMPASS-5711] JSON editor displays and edits Int64 as Int32 Created: 08/Apr/22  Updated: 29/Oct/23  Resolved: 12/Apr/22

Status: Closed
Project: Compass
Component/s: CRUD
Affects Version/s: 1.31.0, 1.31.1
Fix Version/s: 1.31.2

Type: Bug Priority: Major - P3
Reporter: Anna Henningsen Assignee: Anna Henningsen
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: PNG File Screenshot from 2022-04-08 14-54-00.png    
Issue Links:
Related
related to COMPASS-5710 Inserting documents replaces Int64 wi... Closed
Documentation Changes: Not Needed
Sprint: Iteration Yonkers

 Description   

 

  • Run db.coll.insertOne({value: Long.fromNumber(1)}) in the shell
  • Edit document in Compass – Compass does not retain the Long/Int64-ness and also saves back the document as Int32


 Comments   
Comment by Githook User [ 19/Apr/22 ]

Author:

{'name': 'Anna Henningsen', 'email': 'anna@addaleax.net', 'username': 'addaleax'}

Message: fix: prefer half-relaxed EJSON mode (#2971)

Fix various EJSON handling bugs by aligning our EJSON handling
a bit more closely. This fixes regressions introduced by me
in e654e1d63977 (COMPASS-5710, COMPASS-5711 aka
https://github.com/mongodb-js/compass/issues/2960) and another
long-standing export bug (COMPASS-4694).

Specifically, use `

{ relaxed: false }

` when parsing values
into `HadronDocument`, and use `

{ relaxed: false }

` but
undo the strict conversion for `$numberInt` and `$numberDouble`.
This is a mode that we may eventually want to upstream
into the Node.js driver’s `bson` package.
Branch: COMPASS-5519-update-collection-header-styles
https://github.com/mongodb-js/compass/commit/47e5bfae7ee7b3424901c01bf66da21a0a9096e4

Comment by Githook User [ 19/Apr/22 ]

Author:

{'name': 'Anna Henningsen', 'email': 'anna@addaleax.net', 'username': 'addaleax'}

Message: fix: prefer half-relaxed EJSON mode (#2971)

Fix various EJSON handling bugs by aligning our EJSON handling
a bit more closely. This fixes regressions introduced by me
in e654e1d63977 (COMPASS-5710, COMPASS-5711 aka
https://github.com/mongodb-js/compass/issues/2960) and another
long-standing export bug (COMPASS-4694).

Specifically, use `

{ relaxed: false }

` when parsing values
into `HadronDocument`, and use `

{ relaxed: false }

` but
undo the strict conversion for `$numberInt` and `$numberDouble`.
This is a mode that we may eventually want to upstream
into the Node.js driver’s `bson` package.
Branch: clustered-collections
https://github.com/mongodb-js/compass/commit/47e5bfae7ee7b3424901c01bf66da21a0a9096e4

Comment by Githook User [ 14/Apr/22 ]

Author:

{'name': 'Anna Henningsen', 'email': 'anna@addaleax.net', 'username': 'addaleax'}

Message: fix: prefer half-relaxed EJSON mode (#2971)

Fix various EJSON handling bugs by aligning our EJSON handling
a bit more closely. This fixes regressions introduced by me
in e654e1d63977 (COMPASS-5710, COMPASS-5711 aka
https://github.com/mongodb-js/compass/issues/2960) and another
long-standing export bug (COMPASS-4694).

Specifically, use `

{ relaxed: false }

` when parsing values
into `HadronDocument`, and use `

{ relaxed: false }

` but
undo the strict conversion for `$numberInt` and `$numberDouble`.
This is a mode that we may eventually want to upstream
into the Node.js driver’s `bson` package.
Branch: COMPASS-5669-aggregate-results
https://github.com/mongodb-js/compass/commit/47e5bfae7ee7b3424901c01bf66da21a0a9096e4

Comment by Githook User [ 14/Apr/22 ]

Author:

{'name': 'Anna Henningsen', 'email': 'anna@addaleax.net', 'username': 'addaleax'}

Message: fix: prefer half-relaxed EJSON mode (#2971)

Fix various EJSON handling bugs by aligning our EJSON handling
a bit more closely. This fixes regressions introduced by me
in e654e1d63977 (COMPASS-5710, COMPASS-5711 aka
https://github.com/mongodb-js/compass/issues/2960) and another
long-standing export bug (COMPASS-4694).

Specifically, use `

{ relaxed: false }

` when parsing values
into `HadronDocument`, and use `

{ relaxed: false }

` but
undo the strict conversion for `$numberInt` and `$numberDouble`.
This is a mode that we may eventually want to upstream
into the Node.js driver’s `bson` package.
Branch: remove-metrics
https://github.com/mongodb-js/compass/commit/47e5bfae7ee7b3424901c01bf66da21a0a9096e4

Comment by Githook User [ 13/Apr/22 ]

Author:

{'name': 'Anna Henningsen', 'email': 'anna@addaleax.net', 'username': 'addaleax'}

Message: fix: prefer half-relaxed EJSON mode (#2971)

Fix various EJSON handling bugs by aligning our EJSON handling
a bit more closely. This fixes regressions introduced by me
in e654e1d63977 (COMPASS-5710, COMPASS-5711 aka
https://github.com/mongodb-js/compass/issues/2960) and another
long-standing export bug (COMPASS-4694).

Specifically, use `

{ relaxed: false }

` when parsing values
into `HadronDocument`, and use `

{ relaxed: false }

` but
undo the strict conversion for `$numberInt` and `$numberDouble`.
This is a mode that we may eventually want to upstream
into the Node.js driver’s `bson` package.
Branch: COMPASS-5671-update-error-banners-to-lg
https://github.com/mongodb-js/compass/commit/47e5bfae7ee7b3424901c01bf66da21a0a9096e4

Comment by Githook User [ 13/Apr/22 ]

Author:

{'name': 'Anna Henningsen', 'email': 'anna@addaleax.net', 'username': 'addaleax'}

Message: fix: prefer half-relaxed EJSON mode (#2971)

Fix various EJSON handling bugs by aligning our EJSON handling
a bit more closely. This fixes regressions introduced by me
in e654e1d63977 (COMPASS-5710, COMPASS-5711 aka
https://github.com/mongodb-js/compass/issues/2960) and another
long-standing export bug (COMPASS-4694).

Specifically, use `

{ relaxed: false }

` when parsing values
into `HadronDocument`, and use `

{ relaxed: false }

` but
undo the strict conversion for `$numberInt` and `$numberDouble`.
This is a mode that we may eventually want to upstream
into the Node.js driver’s `bson` package.
Branch: COMPASS-5543
https://github.com/mongodb-js/compass/commit/47e5bfae7ee7b3424901c01bf66da21a0a9096e4

Comment by Githook User [ 12/Apr/22 ]

Author:

{'name': 'Anna Henningsen', 'email': 'anna@addaleax.net', 'username': 'addaleax'}

Message: fix: prefer half-relaxed EJSON mode (#2971)

Fix various EJSON handling bugs by aligning our EJSON handling
a bit more closely. This fixes regressions introduced by me
in e654e1d63977 (COMPASS-5710, COMPASS-5711 aka
https://github.com/mongodb-js/compass/issues/2960) and another
long-standing export bug (COMPASS-4694).

Specifically, use `

{ relaxed: false }

` when parsing values
into `HadronDocument`, and use `

{ relaxed: false }

` but
undo the strict conversion for `$numberInt` and `$numberDouble`.
This is a mode that we may eventually want to upstream
into the Node.js driver’s `bson` package.
Branch: 5717-dev
https://github.com/mongodb-js/compass/commit/47e5bfae7ee7b3424901c01bf66da21a0a9096e4

Comment by Githook User [ 12/Apr/22 ]

Author:

{'name': 'Anna Henningsen', 'email': 'anna@addaleax.net', 'username': 'addaleax'}

Message: fix: prefer half-relaxed EJSON mode (#2971)

Fix various EJSON handling bugs by aligning our EJSON handling
a bit more closely. This fixes regressions introduced by me
in e654e1d63977 (COMPASS-5710, COMPASS-5711 aka
https://github.com/mongodb-js/compass/issues/2960) and another
long-standing export bug (COMPASS-4694).

Specifically, use `

{ relaxed: false }

` when parsing values
into `HadronDocument`, and use `

{ relaxed: false }

` but
undo the strict conversion for `$numberInt` and `$numberDouble`.
This is a mode that we may eventually want to upstream
into the Node.js driver’s `bson` package.
Branch: 1.31-releases
https://github.com/mongodb-js/compass/commit/47e5bfae7ee7b3424901c01bf66da21a0a9096e4

Comment by Githook User [ 12/Apr/22 ]

Author:

{'name': 'Anna Henningsen', 'email': 'anna@addaleax.net', 'username': 'addaleax'}

Message: fix: prefer half-relaxed EJSON mode (#2971)

Fix various EJSON handling bugs by aligning our EJSON handling
a bit more closely. This fixes regressions introduced by me
in e654e1d63977 (COMPASS-5710, COMPASS-5711 aka
https://github.com/mongodb-js/compass/issues/2960) and another
long-standing export bug (COMPASS-4694).

Specifically, use `

{ relaxed: false }

` when parsing values
into `HadronDocument`, and use `

{ relaxed: false }

` but
undo the strict conversion for `$numberInt` and `$numberDouble`.
This is a mode that we may eventually want to upstream
into the Node.js driver’s `bson` package.
Branch: COMPASS-5610-move-maxtime-ms
https://github.com/mongodb-js/compass/commit/47e5bfae7ee7b3424901c01bf66da21a0a9096e4

Comment by Githook User [ 12/Apr/22 ]

Author:

{'name': 'Anna Henningsen', 'email': 'anna@addaleax.net', 'username': 'addaleax'}

Message: fix: prefer half-relaxed EJSON mode (#2971)

Fix various EJSON handling bugs by aligning our EJSON handling
a bit more closely. This fixes regressions introduced by me
in e654e1d63977 (COMPASS-5710, COMPASS-5711 aka
https://github.com/mongodb-js/compass/issues/2960) and another
long-standing export bug (COMPASS-4694).

Specifically, use `

{ relaxed: false }

` when parsing values
into `HadronDocument`, and use `

{ relaxed: false }

` but
undo the strict conversion for `$numberInt` and `$numberDouble`.
This is a mode that we may eventually want to upstream
into the Node.js driver’s `bson` package.
Branch: compass-5655-replace-react-dnd
https://github.com/mongodb-js/compass/commit/47e5bfae7ee7b3424901c01bf66da21a0a9096e4

Comment by Githook User [ 12/Apr/22 ]

Author:

{'name': 'Anna Henningsen', 'email': 'anna@addaleax.net', 'username': 'addaleax'}

Message: fix: prefer half-relaxed EJSON mode (#2971)

Fix various EJSON handling bugs by aligning our EJSON handling
a bit more closely. This fixes regressions introduced by me
in e654e1d63977 (COMPASS-5710, COMPASS-5711 aka
https://github.com/mongodb-js/compass/issues/2960) and another
long-standing export bug (COMPASS-4694).

Specifically, use `

{ relaxed: false }

` when parsing values
into `HadronDocument`, and use `

{ relaxed: false }

` but
undo the strict conversion for `$numberInt` and `$numberDouble`.
This is a mode that we may eventually want to upstream
into the Node.js driver’s `bson` package.
Branch: main
https://github.com/mongodb-js/compass/commit/47e5bfae7ee7b3424901c01bf66da21a0a9096e4

Comment by Githook User [ 11/Apr/22 ]

Author:

{'name': 'Anna Henningsen', 'email': 'anna@addaleax.net', 'username': 'addaleax'}

Message: fix: prefer half-relaxed EJSON mode

Fix various EJSON handling bugs by aligning our EJSON handling
a bit more closely. This fixes regressions introduced by me
in e654e1d63977 (COMPASS-5710, COMPASS-5711 aka
https://github.com/mongodb-js/compass/issues/2960) and another
long-standing export bug (COMPASS-4694).

Specifically, use `

{ relaxed: false }

` when parsing values
into `HadronDocument`, and use `

{ relaxed: false }

` but
undo the strict conversion for `$numberInt` and `$numberDouble`.
This is a mode that we may eventually want to upstream
into the Node.js driver’s `bson` package.
Branch: ejson-fixes
https://github.com/mongodb-js/compass/commit/f7e519d8964297a4223ba6b1e8d80671a7923d8c

Generated at Wed Feb 07 22:40:33 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.