|
||||||||||||||||
|
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
I think 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"));