Page MenuHomePhabricator

Some diffs are escaped thus appear in single line
Closed, ResolvedPublic

Description

Some diffs are escaped thus appear in single line

http://www.mediawiki.org/wiki/Special:Code/MediaWiki/42010

I was unable to find the dependencies though :-/ - some diffs are OK, some are like this one.


Version: unspecified
Severity: major

Details

Reference
bz16643

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 10:27 PM
bzimport set Reference to bz16643.

Don't have this locally, maybe it has to do with the proxy svn fetcher

A null byte crept into the file in that diff, which seems to confuse the crap out of the particular json_decode implementation we happen to have live.

json_decode has the lovely behavior that sometimes, without easily being able to predict why, sometimes it returns your literal string, or... a slightly decoded literal string... instead of actually decoding it, if it's invalid, or sometimes if it seems to be valid but it doesn't like it, or wtf?

Anyway, I switched the proxy from using JSON to PHP serialization, and it just passes the string through correctly now. Whee!

r44643, r44644.