Make JSON results more readable for debugging about grunt-phantomas HOT 9 CLOSED

Currently, the most importans JS code in the index.html is written in one line, getting longer and longer with every run.

As an example, in my index.html I have this code:

<script>var results = [{"requests":{"values":[>92.000 more characters here]</script>

This is barely maintainable by a human. It would be much better to break the code into single lines, like this:

<script>var results = [
{
    /* 2014-01-01 08:15:00 */
    "requests":{
        "values":[156,156,156,156,156,null],"sum":780,"min":156,"max":156,"median":156,"average":156
    },
    "gzipRequests":{
        "values":[37,37,37,37,37,null],"sum":185,"min":37,"max":37,"median":37,"average":37
    },
    [..]
    /* 2014-01-02 08:15:00 */
    "requests":{
        "values":[156,156,156,156,156,null],"sum":780,"min":156,"max":156,"median":156,"average":156
    },
    "gzipRequests":{
        "values":[37,37,37,37,37,null],"sum":185,"min":37,"max":37,"median":37,"average":37
    },

That is, data could be split into several lines - at least one line per phantomas run, but maybe even in several lines as shown above - preceded with the corresponding human readable date and time in a separate line as JS comment, so that one can e.g. easily remove data sets from the whole code.

from grunt-phantomas.

@stryju can think of any way to get the requested date into it? It would be perfect then. ;)

/* 2014-01-02 08:15:00 */
"requests":{
  "values":[156,156,156,156,156,null],"sum":780,"min":156,"max":156,"median":156,"average":156
},

from grunt-phantomas.

date as object prop?

results = {
  '2014-01-01 08:15:00' : {
    'requests' : ...,
    ...
  },
  '2014-01-02 08:15:00' : { ... },
  ...
}

this would still be traversable, order is still maintained (via timestamps)

from grunt-phantomas.

the timestamp indexing sounds like a new feature - beyond the scope of this bug

@Augenfeind agreed? if so, please create a new feature request - should be able to fix up a PR for that soon ;-)

from grunt-phantomas.

Better, but not completely done yet.

from grunt-phantomas.

@Augenfeind feel free to close, if you agree with @stryju and create another feature request. :)

from grunt-phantomas.

@stryju I agree!
@stefanjudis You created this issue - I can not close it ;)

from grunt-phantomas.

Haha. :)

from grunt-phantomas.

damn - it writes the formatted json files, but not the index.html ^^

will fix that :)

from grunt-phantomas.