StAXEncoder.encode(XMLStreamReader xmlStream) does not handle comments properly

Brought to you by: daniel_peintner, jopso

#32 StAXEncoder.encode(XMLStreamReader xmlStream) does not handle comments properly

Milestone: v1.0_(example)

Status: closed-fixed

Owner: Daniel Peintner

Labels: None

Priority: 6

Updated: 2015-03-31

Created: 2015-03-30

Creator: Anonymous

Private: No

If I try to encode an XMLStreamReader using the StAXEncoder.encode(XMLStreamReader xmlStream), the switch statement doesn't properly handle comments - it encodes the comment as characters so when I decoded the characters are added without and opening . Using the StAXEncoder.encode(XMLEventreader xmlEvent) this is handled properly. Please fix this as it breaks my XML and all schema validation fails.

Discussion

Anonymous - 2015-03-30

...

without 

...

ie. the comment control characters. That was removed from the Sourceforge comment, sorry.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Daniel Peintner - 2015-03-30

Hi!

Thanks for pointing us to this issue. The fix has been implemented.
https://sourceforge.net/p/exificient/code/545/

-- Daniel

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Daniel Peintner - 2015-03-30

status: open --> pending

assigned_to: Daniel Peintner
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Anonymous - 2015-03-31

Perfect, do you have an estimate on when this will be released?

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Daniel Peintner - 2015-03-31

EXIficient 0.9.4 has been released this month... there will be some weeks till to the next release.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Anonymous - 2015-03-31

Ok, thanks for the info.

/Fredrik

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Daniel Peintner - 2015-03-31

status: pending --> closed-fixed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Anonymous