[issue14570] Document json "sort_keys" parameter properly

Andrew Svetlov added the comment: Fixed, thanks again! -- resolution: -> fixed stage: patch review -> committed/rejected status: open -> closed ___ Python tracker ___ __

[issue14570] Document json "sort_keys" parameter properly

Roundup Robot added the comment: New changeset 7784008e9ade by Andrew Svetlov in branch '2.7': Issue #14570: Document json sort_keys parameter properly. http://hg.python.org/cpython/rev/7784008e9ade -- ___ Python tracker

[issue14570] Document json "sort_keys" parameter properly

Chris Rebert added the comment: Patch for 2.7. This assumes that changing the parameter notation is permissible. -- Added file: http://bugs.python.org/file27765/issue14570-2.7.diff ___ Python tracker __

[issue14570] Document json "sort_keys" parameter properly

Andrew Svetlov added the comment: Thanks, Chris! I have pushed the patch for 3.2-3.4 It cannot be applied to 2.7, please make separate patch for this version. -- nosy: +asvetlov ___ Python tracker _

[issue14570] Document json "sort_keys" parameter properly

Roundup Robot added the comment: New changeset 40aedc7da30f by Andrew Svetlov in branch '3.2': Issue #14570: Document json sort_keys parameter properly. http://hg.python.org/cpython/rev/40aedc7da30f New changeset 090484ccba7d by Andrew Svetlov in branch '3.3': Merge issue #14570: Document json s

[issue14570] Document json "sort_keys" parameter properly

Chris Rebert added the comment: Per Chris Jerdonek, here's a combined patch that also makes the docs changes comply with the antiquated line length limit. -- Added file: http://bugs.python.org/file27759/issue14570.diff ___ Python tracker

[issue14570] Document json "sort_keys" parameter properly

Chris Jerdonek added the comment: Also, while not strictly necessary, it is more customary and convenient to provide a single patch file for all files. -- ___ Python tracker ___

[issue14570] Document json "sort_keys" parameter properly

Chris Jerdonek added the comment: I would recommend making the added and modified lines not exceed 79 characters in both files. Also, for the .rst file, you can use slashes to break lines as shown in the following example: http://hg.python.org/cpython/file/09011896374d/Doc/library/argparse.rs

[issue14570] Document json "sort_keys" parameter properly

Changes by Ezio Melotti : -- nosy: +petri.lehtinen stage: needs patch -> patch review ___ Python tracker ___ ___ Python-bugs-list mail

[issue14570] Document json "sort_keys" parameter properly

Chris Rebert added the comment: Any reactions? The patch is pretty straightforward... -- ___ Python tracker ___ ___ Python-bugs-list m

[issue14570] Document json "sort_keys" parameter properly

Chris Rebert added the comment: Attached updated patches. -- Added file: http://bugs.python.org/file27091/json_init.patch ___ Python tracker ___ _

[issue14570] Document json "sort_keys" parameter properly

Changes by Chris Rebert : Added file: http://bugs.python.org/file27090/json.rst.patch ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue14570] Document json "sort_keys" parameter properly

Chris Rebert added the comment: Erm, the patch seems to have some ANSI terminal coloring escape code junk (e.g. "") in it... -- nosy: +cvrebert ___ Python tracker ___ ___

[issue14570] Document json "sort_keys" parameter properly

Michal Nowikowski added the comment: Attached a patch. To preserve current order of arguments in dumps/dump functions sort_keys argument has been added to the end of arguments just before **kw. -- keywords: +patch Added file: http://bugs.python.org/file25427/json-sort-keys.patch _

[issue14570] Document json "sort_keys" parameter properly

Michal Nowikowski added the comment: In json module there are dump/dumps methods which internally instantiate encoder class JSONEncoder (or some other user-defined encoder clas). They look as follows: json.dump(obj, fp, skipkeys=False, ensure_ascii=True, check_ci

[issue14570] Document json "sort_keys" parameter properly

Changes by Ezio Melotti : -- nosy: +ezio.melotti ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.py

[issue14570] Document json "sort_keys" parameter properly

New submission from Nick Coghlan : The json "sort_keys" parameter is actually supported as an argument to dump() and dumps() (and is used that way in the examples), but is only documented as an argument to the JSONEncoder constructor. -- assignee: docs@python components: Documentation