Opened 11 years ago

Closed 11 years ago

#6964 closed enhancement (fixed)

Allow jsonPathStore.dump() to print pretty

Reported by: dante Owned by: dylan
Priority: high Milestone: 1.2
Component: General Version: 1.1.1
Keywords: Cc: Dustin Machi
Blocked By: Blocking:

Description

The dojox.data.jsonPathStore has a [very useful] .dump() method, though in using it, I immediately wanted to print "pretty" to trap the data to a pipe (from rhino / AIR, etc)

I ended up just modifying ~635: 

return dojo.toJson(data, options.pretty || false)

allowing safe passage of a "pretty" boolean to the dump() call params, retaining expected behavior if null.

I am suggesting this as an enhancement.

Attachments (1)

jsonPath.patch​ (593 bytes) - added by dante 11 years ago.
my patch

Download all attachments as: .zip

Change History (3)

Changed 11 years ago by dante

Attachment: jsonPath.patch​ added

my patch

comment:1 Changed 11 years ago by dylan

Owner: changed from kriszyp to dylan
Status: new → assigned

comment:2 Changed 11 years ago by dylan

Resolution: → fixed
Status: assigned → closed

(In [14012]) fixes #6964, allow jsonPathStore.dump() to print pretty, thanks dante, \!strict

Note: See TracTickets for help on using tickets.