Bug #64874 [Opn->Csd]: json_decode handles whitespace and case-sensitivity incorrectly

From: Date: Mon, 18 Nov 2013 22:07:15 +0000
Subject: Bug #64874 [Opn->Csd]: json_decode handles whitespace and case-sensitivity incorrectly
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-182817@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=64874&edit=1

 ID:                 64874
 Updated by:         ajf@php.net
 Reported by:        chrivers at iversen-net dot dk
 Summary:            json_decode handles whitespace and case-sensitivity
                     incorrectly
-Status:             Open
+Status:             Closed
 Type:               Bug
 Package:            JSON related
 Operating System:   All
 PHP Version:        5.4.15
-Assigned To:        
+Assigned To:        ajf
 Block user comment: N
 Private report:     N

 New Comment:

The fix for this bug has been committed.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.

 For Windows:

http://windows.php.net/snapshots/
 
Thank you for the report, and for helping us make PHP better.




Previous Comments:
------------------------------------------------------------------------
[2013-11-14 20:01:15] ajf at ajf dot me

The whitespace bit's fixed in 5.5.6, and will be in the next 5.4 release too.

Though it matters little since Debian, Ubuntu, Fedora and so-on do not use the built-in JSON ext.

------------------------------------------------------------------------
[2013-11-11 00:38:27] ajf at ajf dot me

My request to fix the whitespace portion of this bug was merged into the PHP-5.4 branch (and hence
will be in the next minor 5.4 and 5.5 releases).

Now I simply need to try and get the case fix bit into 5.6 (it breaks BC so it can't be in 5.4
or 5.5, I bet): https://github.com/php/php-src/pull/457

------------------------------------------------------------------------
[2013-10-16 12:27:25] ajf at ajf dot me

The pull request to fix the bug was split into two parts. The first is the backwards-compatible part
(whitespace fix), which should go into 5.4 and 5.5:

https://github.com/php/php-src/pull/456

The second is to fix the non-backwards-compatible part (case), which should go into 5.6:

https://github.com/php/php-src/pull/457

------------------------------------------------------------------------
[2013-09-11 10:38:41] ajf at ajf dot me

Submitted pull request: https://github.com/php/php-src/pull/436

------------------------------------------------------------------------
[2013-09-09 17:05:34] cmbecker69 at gmx dot de

There is a RFC[1] to replace the current JSON implementation with
the jsonc PECL extension, so "fixing" that implementation may not 
be reasonable anyway.

[1] <https://wiki.php.net/rfc/free-json-parser>

------------------------------------------------------------------------


The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at

    https://bugs.php.net/bug.php?id=64874


-- 
Edit this bug report at https://bugs.php.net/bug.php?id=64874&edit=1


Thread (10 messages)

« previous php.bugs (#182817) next »