Add a @fixme comment on FormatJson::encode(); the '$isHtml' parameter's purpose and...
authorBrion Vibber <brion@users.mediawiki.org>
Sat, 5 Feb 2011 21:06:44 +0000 (21:06 +0000)
committerBrion Vibber <brion@users.mediawiki.org>
Sat, 5 Feb 2011 21:06:44 +0000 (21:06 +0000)
commitd10a408d5ffd797c1dcbd484bc3f2408627d5bae
treeef14e0617521f74b8d5ed9ab0f2c0e066ed667bb
parent95e6386122a8326c88e29a4cbe233720084327a6
Add a @fixme comment on FormatJson::encode(); the '$isHtml' parameter's purpose and function is not documented, but it seems to map to a pretty-printing parameter on Services_JSON(). If that's its purpose, it should be renamed and documented. Since it's a bare boolean parameter with no clear purpose in calls when "true" or "false" is passed bare, it probably also wants a nice legible constant value or two.
includes/json/FormatJson.php