Fixes #4542 - Add ability to override upload location

Author: mnriem

extension/fileupload/src/main/java/cloud/piranha/extension/fileupload/FileUploadMultiPartInitializer.java

@@ -44,8 +44,10 @@
  * </p>
  *
  * <ol>
- * <li>Sets the MultiPartManager to an instance of FileUploadMultiPartManager.</li>
- * <li>Adds the JakartaFileCleaner listener that cleans up the temporary files.</li>
+ * <li>Sets the MultiPartManager to an instance of
+ * FileUploadMultiPartManager.</li>
+ * <li>Adds the JakartaFileCleaner listener that cleans up the temporary
+ * files.</li>
  * </ol>
  *
  * @author Manfred Riem (mriem@manorrock.com)
@@ -62,13 +64,17 @@ public class FileUploadMultiPartInitializer implements ServletContainerInitializ
      */
     public FileUploadMultiPartInitializer() {
     }
-    
+
     @Override
     public void onStartup(Set<Class<?>> classes, ServletContext servletContext) throws ServletException {
         WebApplication webApplication = (WebApplication) servletContext;
-        LOGGER.log(TRACE, "Setting the MultiPartManager");
+        if (LOGGER.isLoggable(TRACE)) {
+            LOGGER.log(TRACE, "Setting the MultiPartManager");
+        }
         webApplication.getManager().setMultiPartManager(new FileUploadMultiPartManager());
-        LOGGER.log(TRACE, "Adding the listener used to cleanup temporary files");
+        if (LOGGER.isLoggable(TRACE)) {
+            LOGGER.log(TRACE, "Adding the listener used to cleanup temporary files");
+        }
         webApplication.addListener("org.apache.commons.fileupload2.jakarta.servlet6.JakartaFileCleaner");
     }
 }

extension/fileupload/src/main/java/cloud/piranha/extension/fileupload/FileUploadMultiPartManager.java

@@ -156,7 +156,7 @@ private synchronized JakartaServletFileUpload setupFileUpload(WebApplication web
              * If the multipart config has a location use it. If it is relative
              * use the TEMPDIR as the parent directory.
              */
-            if (multipartConfig.getLocation() != null && multipartConfig.getLocation().isEmpty()) {
+            if (multipartConfig.getLocation() != null && !multipartConfig.getLocation().isEmpty()) {
                 File location = new File(multipartConfig.getLocation());
                 if (!location.isAbsolute()) {
                     location = ((File) webApplication.getAttribute(TEMPDIR)).toPath().resolve(location.toPath()).toFile();