yuanshuo / jupiter-eclipse-plugin

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

Adding review issue to .js, .jsp, or .jspf fails to remember the file or line number #76

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Add review issue to a JS or JSP file "Add review issue" missing when
right clicking, so must click the + in the review table.
2. Enter type, severity, summary, and description then save.
3. Review editor shows reviewer correctly but file and line and empty. "lux
: : " instead of "lux : web/somefile.js : 21".

Adding a review issue to a .txt or .properties or .rb file works as expected.

Work around is to manually type in the file and line number in the summary
or description, but this is cumbersome.

Original issue reported on code.google.com by ash...@gmail.com on 15 Jun 2009 at 5:26

GoogleCodeExporter commented 9 years ago
What editor is associated with the JS/JSP files?  And what Eclipse are you 
using?

The other files probably open up in the text editor or an editor that extends 
the text 
editor which is why it works.  The workaround is to open the files that don't 
work 
properly in the Eclipse text editor.

The next version should fix the problem but will not be out until after Eclipse 
3.5 is 
released.

Original comment by jsakuda on 15 Jun 2009 at 11:43

GoogleCodeExporter commented 9 years ago
I am using Version: 3.4.2, Build id: M20090211-1700. You are correct, opening 
the JSP
in the text editor works correctly.

Is the fix in Eclipse or in the Jupiter plugin? Maybe I could use a development
version of the Jupiter plugin.

Original comment by ash...@gmail.com on 16 Jun 2009 at 1:39

GoogleCodeExporter commented 9 years ago
What editor is associated with the JS/JSP files?  Or which Eclipse bundle do 
you use?  
I won't know for sure if it will work for sure with your editor unless I can 
test it.

Original comment by jsakuda on 16 Jun 2009 at 7:44

GoogleCodeExporter commented 9 years ago
I am assuming the JS/JSP editor being used most likely extends the Eclipse XML 
editor.  Marking this issue blocked on issue 73.

Original comment by jsakuda on 22 Jun 2010 at 12:34

GoogleCodeExporter commented 9 years ago

Original comment by jsakuda on 22 Jun 2010 at 12:35