Opened 17 years ago
Closed 17 years ago
#737 closed defect (invalid)
Script that fixes copyright text may also add extra new line to end of files
Reported by: | Nicklas Nordborg | Owned by: | Jari Häkkinen |
---|---|---|---|
Priority: | major | Milestone: | BASE 2.4.4 |
Component: | web | Version: | |
Keywords: | Cc: |
Description
I am not sure if the extra new line at the end of www/filemanager/upload/show_exception.jsp was caused by the script or by something else. The newline appeared at revision 3675 with log message: Fixing copyright statements. Fixing svn properties.
Change History (4)
comment:1 by , 17 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:2 by , 17 years ago
The extra new line is converted into a call to out.print("\n")
when the JSP is compiled to a java servlet. In most cases this means nothing, but on the www/filemanager/upload/show_exception.jsp the last "real" statement is a throw exception
statement. Thus, the java compiler complains about an unreachable statement
(see ticket #736) and refuses to compile the JSP.
Actually, JSP:s are not so good for things that don't produce HTML and I think it would be better if the file upload was implemented as a pure servlet instead (ticket #738).
comment:3 by , 17 years ago
Milestone: | → BASE 2.4.2 |
---|
comment:4 by , 17 years ago
Resolution: | → invalid |
---|---|
Status: | assigned → closed |
This was an issue here for the affected files, but the problem belongs to the svndigest project and they have resolved it in http://trac.thep.lu.se/trac/svndigest/ticket/264
I'll add an ticket to the svndigest project and request that they check if the extra line is their problem.
Is this a problem for JSP's in general? Of course, no extra information should be added but some text editors are very eager to add an extra line in the end of an file automatically.