Abdera
  1. Abdera
  2. ABDERA-114

Missing coma in JSON serilaizer for element content

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.3.0
    • Fix Version/s: 0.4.0
    • Labels:
      None

      Description

      The JSON serialization misses a coma when writing a tag child with mixed content (text-tag-text). Here is an example of output:

      "children":["There is a train wreck coming.\u00a0 And it has nothing to do with whether the content is \u201cmore\u201d programmable or not.\u00a0 Long, long, long before it gets to that point, one needs to be able to reliably (and in a ",{
      "name":"a",
      "attributes":

      { "href":"http://en.wikipedia.org/wiki/Non-repudiation" }

      ,
      "children":["non-reputable"
      ]
      }" manner) determine whether or not the information is consumable at all."

      The last list is missing a comma just before " manner.

      Looking at the code it actually looks like a simple type, patch to come.

      1. patch.txt
        0.7 kB
        Matthieu Riou

        Activity

        Matthieu Riou created issue -
        Hide
        Matthieu Riou added a comment -

        Patch to properly add comma after element inclusion in a mixed content text.

        Show
        Matthieu Riou added a comment - Patch to properly add comma after element inclusion in a mixed content text.
        Matthieu Riou made changes -
        Field Original Value New Value
        Attachment patch.txt [ 12375201 ]
        Hide
        James M Snell added a comment -

        Checked in. Thank you!

        Show
        James M Snell added a comment - Checked in. Thank you!
        James M Snell made changes -
        Resolution Fixed [ 1 ]
        Fix Version/s 0.4.0 [ 12312825 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        12h 1m 1 James M Snell 11/Feb/08 16:50

          People

          • Assignee:
            Unassigned
            Reporter:
            Matthieu Riou
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development