Any plans to add support replacer functionality?

Question

Any plans to add support replacer functionality?

Mika83AC opened this issue 4 years ago · 14 comments

Mika83AC commented 4 years ago

Hello eveyone,

I'd like to know if there are plans to add the native JSON.stringifys replacer() functionality like:

JSON.stringify(input, (key, value) => value instanceof Set ? [...value] : value)

So it would also be possible to stringify objects which contains Set()s, Map()s, and so on.

Regards,
Michael

Answer 1 · 2021-03-12T09:05:06.000Z

There aren't any plans to do so. That being said, I think it might be possible to add support for those natively to objects and arrays respectively.

Answer 2 · 2021-03-12T09:05:17.000Z

Would you like to send a Pull Request to address this issue? Remember to add unit tests.

Answer 3 · 2021-05-25T19:16:04.000Z

Hey, @mcollina,

I am a great fan of Fastify and now I am actively using it on my projects. Please assign this task to me. It will be my pleasure to be a part of such a great product and organisation.

I will also include unit tests. I like this issue and I really want to contribute to it.

Answer 4 · 2021-05-25T20:08:15.000Z

@onbit-syn go for it!

Answer 5 · 2022-09-05T20:50:31.000Z

@onbit-uchenik Is Anubhav still working on this issue? If not, I am more than happy to help! 😌

Answer 6 · 2022-09-05T22:05:54.000Z

I dont think so. I you want to provide a PR, just do it.

Answer 7 · 2022-09-06T13:12:41.000Z

@mcollina

I think it might be possible to add support for those natively to objects and arrays respectively.

Do you mean we do the following:

Set & WeakSet: treat them as an object
Map & WeakMap: treat them as an array

Answer 8 · 2022-09-06T16:52:54.000Z

You can do Set and Map, not their weak counterparts.

Answer 9 · 2022-09-12T16:37:08.000Z

Wanna confirm what we want. Is the type of Set in the schema set or object? The same question to Map too.

Answer 10 · 2022-09-12T16:42:02.000Z

Imho Set is Array and Map is object

Answer 11 · 2022-09-13T14:08:31.000Z

Indeed, yes.

Answer 12 · 2022-10-07T15:43:33.000Z

I didn't expect this issue would be this complicated. I have closed my PR #545. Feel free to work on this issue if you are interested!

Answer 13 · 2022-12-28T18:05:29.000Z

sooo, can we get the replacer? I have a use case where I would like to do a kind of monkey-patch;

const thing = {
   properties: [
       {
           id: 1
       },
       {
           id: 2
       }
   ],
   ref: 1
}

const str = JSON.stringify(thing, (key, value) => {
    if(key === "ref") return thing.properties.find(p => p.id === value);
    else return value;
})

Answer 14 · 2022-12-28T18:37:41.000Z

After two years, this is unlikeky to happen. I won't be opposed to a PR, but there seem no interest in implementing this by its maintainers or users.