Jettison

serializeAsArray handling is incorrect.

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.1
  • Fix Version/s: 1.1
  • Labels:
    None
  • Testcase included:
    yes
  • Patch Submitted:
    Yes
  • Number of attachments :
    1

Description

The current implementation of serializeAsArray does not handle nested arrays properly. If I add "list" to be serialized as an array, I receive:

{"list": {
    "test": [{
        "foo": "bar",
        "inner": {"value": "aaaaa"}
    }],
    "string": "str"
}}

instead of what I expect:

{"list": [{
    "test": {
        "foo": "bar",
        "inner": {"value": "aaaaa"}
    },
    "string": "str"
}]}

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Kevin Conaway added a comment -

Attaching patch and test case

Show
Kevin Conaway added a comment - Attaching patch and test case
Kevin Conaway made changes -
Field Original Value New Value
Attachment jettison-46.patch [ 35486 ]
Hide
Permalink
Ryan X.Wong added a comment -

this issue is really critical.
and the project has stop developing for months.
we do really need some update....

Show
Ryan X.Wong added a comment - this issue is really critical. and the project has stop developing for months. we do really need some update....
Hide
Permalink
Dejan Bosanac added a comment -

committed in svn revision 65.

I've also built a new 1.1 snapshot with this patch included
http://snapshots.repository.codehaus.org/org/codehaus/jettison/jettison/1.1-SNAPSHOT/

Cheers
Dejan

Show
Dejan Bosanac added a comment - committed in svn revision 65. I've also built a new 1.1 snapshot with this patch included http://snapshots.repository.codehaus.org/org/codehaus/jettison/jettison/1.1-SNAPSHOT/ Cheers Dejan
Dejan Bosanac made changes -
Assignee Dan Diephouse [ dandiep ] Dejan Bosanac [ dejanb ]
Status Open [ 1 ] Resolved [ 5 ]
Fix Version/s 1.1 [ 13413 ]
Resolution Fixed [ 1 ]

People

  • Assignee:
    Dejan Bosanac
    Reporter:
    Kevin Conaway
Vote (1)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: