Skip Menu |

This queue is for tickets about the Catalyst-Action-REST CPAN distribution.

Report information
The Basics
Id: 119910
Status: open
Priority: 0/
Queue: Catalyst-Action-REST

People
Owner: Nobody in particular
Requestors: ZHENYZ [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 1.20
Fixed in: (no value)



Subject: Improve JSON serializer
For the JSON serializer, I want to have features like pretty, canonical. So it's better to improve the serializer in below aspects, so that I don't have to hack my way through it: * Support options in the serializer. * Truely support JSON::MaybeXS. (Now it uses JSON::MaybeXS but always uses JSON. )
On Thu Jan 19 03:22:17 2017, ZHENYZ wrote: Show quoted text
> For the JSON serializer, I want to have features like pretty, > canonical. So it's better to improve the serializer in below aspects, > so that I don't have to hack my way through it: > > * Support options in the serializer. > * Truely support JSON::MaybeXS. (Now it uses JSON::MaybeXS but always > uses JSON. )
Patch?
On 2017-01-19 00:22:17, ZHENYZ wrote: Show quoted text
> * Truely support JSON::MaybeXS. (Now it uses JSON::MaybeXS but always > uses JSON. )
I'm not sure what you mean by this. Catalyst::Action::Serialize::JSON calls a *subroutine* called JSON(), but this is an imported sub from JSON::MaybeXS that returns the package name of the underlying implementation being used -- which is one of JSON::PP, Cpanel::JSON::XS or JSON::XS -- it is *never* JSON (i.e. JSON.pm).