Page MenuHomePhabricator

Remove dependency on JSON extension
Closed, ResolvedPublic

Description

From bug 47431 comment #17:

If we do decide to not include a pure-PHP [JSON] implementation, I will have to make
two small changes to the installer, though that shouldn't be a big deal at
all.

(a) Adding a new environmental check for json_encode() or json_decode()
(b) Removing one of the two uses of Xml::encodeJsVar. This is for JS encoding
an array of database types that doesn't seem to be used anywhere, and which
in
any case should be alphanumeric (thus simple to encode).


Version: 1.22.0
Severity: normal
See Also:
https://bugzilla.wikimedia.org/show_bug.cgi?id=47564
https://bugzilla.wikimedia.org/show_bug.cgi?id=47431

Details

Reference
bz54774

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 2:26 AM
bzimport set Reference to bz54774.

Change 86696 had a related patch set uploaded by PleaseStand:
Installer: remove dependency on JSON extension

https://gerrit.wikimedia.org/r/86696

Change 86696 merged by jenkins-bot:
Installer: remove dependency on JSON extension

https://gerrit.wikimedia.org/r/86696