jsakuda / jupiter-eclipse-plugin

Automatically exported from code.google.com/p/jupiter-eclipse-plugin
1 stars 1 forks source link

Using signs like <, > in description causes it to be cut #58

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Go to Individual Phase of some review
2. Add a review issue where description contains some '<' signs, e.g. a
part of code:
        if (x < 0) {
            System.out.println("Positive");
        } else if (x > 0) {
            System.out.println("Negative");
        }
3. Go to Team Phase - the description is just "if (x ", still the .review
file contains the full description
4. Change the resolution of the issue to e.g. "Valid Needs Fixing", now in
the .review description is also cut.

What is the expected output? What do you see instead?

The expected is that the description is not cut

What version of the product are you using? On what operating system?

Jupiter 3.4.1.1 with Eclipse 3.3.1.1 on Windows XP.

Original issue reported on code.google.com by dstank...@gmail.com on 3 Dec 2008 at 9:41

GoogleCodeExporter commented 8 years ago
I'm going to guess that this happens because Jupiter stores data in XML.  And 
having
< and > signs causes a problem for the XML.  The characters need to be 
converted into
> and <.

Scheduling for the next release.

Original comment by jsakuda on 3 Dec 2008 at 7:23

GoogleCodeExporter commented 8 years ago
Jupiter correctly converts it to > and < for the first time (when it's added in
Individual Phase), but this information is lost somehow later.

Original comment by dstank...@gmail.com on 4 Dec 2008 at 6:25

GoogleCodeExporter commented 8 years ago
Issue 55 has been merged into this issue.

Original comment by jsakuda on 18 Jan 2009 at 3:26

GoogleCodeExporter commented 8 years ago
Codehaus StAX splits up the text when there are special characters even when it 
is set to return text as one 
block.  Worked around this and issue should be fixed.

Original comment by jsakuda on 18 Jan 2009 at 6:49