Change By: Keiji Suzuki (29/Jul/13 3:04 AM)
Comment: Hi Ivan,

This patches produce another bugs.

There is no method to set a default value in getProperty() unlike getBooleanProperty() and so on. 
In the signature Configuration.getProperty(String, String), the second String parameter means 
a property in the module specified by  the first String parameter.

If you set a default value, you should be the following for example in FileUploadRequest.java

tempDir = ConfigurationManager.getProperty("upload.temp.dir") != null 
    ? ConfigurationManager.getProperty("upload.temp.dir")  : System.getProperty("java.io.tmpdir");

instead of

tempDir = ConfigurationManager.getProperty("upload.temp.dir", System.getProperty("java.io.tmpdir")); 
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
------------------------------------------------------------------------------
See everything from the browser to the database with AppDynamics
Get end-to-end visibility with application monitoring from AppDynamics
Isolate bottlenecks and diagnose root cause in seconds.
Start your free trial of AppDynamics Pro today!
http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk
_______________________________________________
Dspace-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/dspace-devel

Reply via email to