groovy

JsonOutput.toJson(object) does not handle char[] correctly

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.8.0, 1.8.1
  • Fix Version/s: 1.8.2, 1.9-beta-3
  • Component/s: groovy-jdk
  • Labels:
    None
  • Testcase included:
    yes
  • Number of attachments :
    3

Description

Currently

println new JsonBuilder(['a','b','c','d'] as char[]).toString()

will output

[{},{},{},{}]

since the primitive class char has no properties (other than class).

My solution is to call toString on the object if there are no properties left. I have attached an updated JsonOutput.groovy and a fixed test.

  1. Text File
    GROOVY-4984.patch
    28/Aug/11 10:01 AM
    1 kB
    Seth Miller
  2. File
    JsonOutput.groovy
    23/Aug/11 12:30 PM
    6 kB
    Seth Miller
  3. File
    JsonOutputTest.groovy
    23/Aug/11 12:30 PM
    9 kB
    Seth Miller

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Seth Miller added a comment -

Here is a patch file with my changes, if that helps.

Show
Seth Miller added a comment - Here is a patch file with my changes, if that helps.
Hide
Permalink
Guillaume Laforge added a comment -

I've got a slightly different view on this issue.
I think char[] should be represented as a String, ie. ['a','b','c','d'] as char[] would be "abcd".

Show
Guillaume Laforge added a comment - I've got a slightly different view on this issue. I think char[] should be represented as a String, ie. ['a','b','c','d'] as char[] would be "abcd".
Hide
Permalink
Seth Miller added a comment -

I'm afraid I would have to disagree. If someone has gone through the trouble of dealing with a char array, it is likely that they wanted to work with the individual chars. At least this was the case for me.

Show
Seth Miller added a comment - I'm afraid I would have to disagree. If someone has gone through the trouble of dealing with a char array, it is likely that they wanted to work with the individual chars. At least this was the case for me.
Hide
Permalink
Guillaume Laforge added a comment -

Ok fair enough.

Show
Guillaume Laforge added a comment - Ok fair enough.
Hide
Permalink
Guillaume Laforge added a comment -

Implemented as a JSON array of single-character strings

Show
Guillaume Laforge added a comment - Implemented as a JSON array of single-character strings

People

  • Assignee:
    Guillaume Laforge
    Reporter:
    Seth Miller
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: