Re: [interchange] Improve application/json POST support

David Christensen <[email protected]> Tue, 30 Dec 2014 09:28:36 -0600
Newsgroups gmane.comp.web.interchange.cvs
Message-ID <62B19D8C-7870-4F99-9D3F-640F623DFABE__33482.7893566855$1420010164$gmane$org@endpoint.com>
> On Dec 29, 2014, at 5:26 PM, Peter <[email protected]> wrote:
> 
> On 12/30/2014 10:02 AM, David Christensen wrote:
>> +use JSON ();
> 
> ...
> 
>> +					$CGI::json_ref = JSON::from_json($$CGI::post_ref);
> 
> You've added in JSON as a requirement here, but it's not (last time I
> checked) part of core perl and it's not listed as a module in
> Bundle::Interchange (or even Bundle::InterchangeKitchenSink).  Please
> add in code that makes this optional so we don't break existing IC sites
> when they upgrade.

I'd checked, and I thought as of 5.14 it is core (though maybe that is the PP JSON).  Despite that, there are still plenty of supported perls where this is not the case.  I'll make the code a no-op in case JSON is not available.

> Mike, should JSON be a part of KitchenSink?

+1 (though it looks like a moot issue, as it's already done).

Regards,

David
--
David Christensen
End Point Corporation
[email protected]
785-727-1171