WW-5461 Extends UploadedFile with inputName field

lukaszlenart · lukaszlenart · commit 9613325152bf · 2024-09-03T08:28:42.000+02:00
diff --git a/apps/showcase/src/main/java/org/apache/struts2/showcase/fileupload/FileUploadAction.java b/apps/showcase/src/main/java/org/apache/struts2/showcase/fileupload/FileUploadAction.java
@@ -37,6 +37,7 @@ public class FileUploadAction extends ActionSupport implements UploadedFilesAwar
     private String fileName;
     private String caption;
     private String originalName;
+    private String inputName;
 
     public String input() throws Exception {
         return SUCCESS;
@@ -58,6 +59,10 @@ public String getOriginalName() {
         return originalName;
     }
 
+    public String getInputName() {
+        return inputName;
+    }
+
     public Object getUploadedFile() {
         return uploadedFile.getContent();
     }
@@ -85,5 +90,6 @@ public void withUploadedFiles(List<UploadedFile> uploadedFiles) {
         this.fileName = uploadedFile.getName();
         this.contentType = uploadedFile.getContentType();
         this.originalName = uploadedFile.getOriginalName();
+        this.inputName = uploadedFile.getInputName();
     }
 }
diff --git a/apps/showcase/src/main/webapp/WEB-INF/fileupload/upload-success.jsp b/apps/showcase/src/main/webapp/WEB-INF/fileupload/upload-success.jsp
@@ -1,51 +1,52 @@
 <!--
 /*
 * Licensed to the Apache Software Foundation (ASF) under one
-* or more contributor license agreements.  See the NOTICE file
+* or more contributor license agreements. See the NOTICE file
 * distributed with this work for additional information
-* regarding copyright ownership.  The ASF licenses this file
+* regarding copyright ownership. The ASF licenses this file
 * to you under the Apache License, Version 2.0 (the
 * "License"); you may not use this file except in compliance
-* with the License.  You may obtain a copy of the License at
+* with the License. You may obtain a copy of the License at
 *
-*  http://www.apache.org/licenses/LICENSE-2.0
+* http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing,
 * software distributed under the License is distributed on an
 * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
-* KIND, either express or implied.  See the License for the
+* KIND, either express or implied. See the License for the
 * specific language governing permissions and limitations
 * under the License.
 */
 -->
 <%@ page
-	language="java"
-	contentType="text/html; charset=UTF-8"
-    pageEncoding="UTF-8"%>
+        language="java"
+        contentType="text/html; charset=UTF-8"
+        pageEncoding="UTF-8" %>
 <%@ taglib prefix="s" uri="/struts-tags" %>
 <html>
 <head>
-	<title>Struts2 Showcase - Fileupload sample</title>
+    <title>Struts2 Showcase - Fileupload sample</title>
 </head>
 
 <body>
 <div class="page-header">
-	<h1>Fileupload sample</h1>
+    <h1>Fileupload sample</h1>
 </div>
 
 <div class="container-fluid">
-	<div class="row">
-		<div class="col-md-12">
-			<ul>
-		        <li>ContentType: <s:property value="contentType" /></li>
-		        <li>FileName: <s:property value="fileName" /></li>
-		        <li>Original FileName: <s:property value="originalName" /></li>
-		        <li>File: <s:property value="uploadedFile" /></li>
-				<li>Size: <s:property value="uploadSize" /></li>
-				<li>Caption: <s:property value="caption" /></li>
-	        </ul>
-		</div>
-	</div>
+    <div class="row">
+        <div class="col-md-12">
+            <ul>
+                <li>ContentType: <s:property value="contentType"/></li>
+                <li>FileName: <s:property value="fileName"/></li>
+                <li>Original FileName: <s:property value="originalName"/></li>
+                <li>File: <s:property value="uploadedFile"/></li>
+                <li>Caption: <s:property value="caption"/></li>
+                <li>Size: <s:property value="uploadSize"/></li>
+                <li>Input name: <s:property value="inputName"/></li>
+            </ul>
+        </div>
+    </div>
 </div>
 
 </body>
diff --git a/apps/showcase/src/test/java/it/org/apache/struts2/showcase/FileUploadTest.java b/apps/showcase/src/test/java/it/org/apache/struts2/showcase/FileUploadTest.java
@@ -24,13 +24,19 @@
 import org.htmlunit.html.HtmlInput;
 import org.htmlunit.html.HtmlPage;
 import org.htmlunit.html.HtmlSubmitInput;
+import org.htmlunit.html.DomElement;
 import org.junit.Test;
 
 import java.io.File;
 import java.io.FileWriter;
 import java.security.SecureRandom;
 
 import static org.assertj.core.api.Assertions.assertThat;
+import static org.assertj.core.api.Assertions.assertThatThrownBy;
+import static org.junit.Assert.assertNotNull;
+
+import java.nio.file.Files;
+import java.nio.file.Paths;
 
 public class FileUploadTest {
 
@@ -59,7 +65,8 @@ public void testSimpleFileUpload() throws Exception {
                     "ContentType: text/plain",
                     "Original FileName: " + tempFile.getName(),
                     "Caption: some caption",
-                    "Size: 12"
+                    "Size: 12",
+                    "Input name: upload"
             );
         }
     }
diff --git a/core/src/main/java/org/apache/struts2/dispatcher/multipart/JakartaMultiPartRequest.java b/core/src/main/java/org/apache/struts2/dispatcher/multipart/JakartaMultiPartRequest.java
@@ -118,6 +118,7 @@ protected void processFileField(DiskFileItem item) {
                     .create(item.getPath().toFile())
                     .withOriginalName(item.getName())
                     .withContentType(item.getContentType())
+                    .withInputName(item.getFieldName())
                     .build();
             values.add(uploadedFile);
         }
diff --git a/core/src/main/java/org/apache/struts2/dispatcher/multipart/JakartaStreamMultiPartRequest.java b/core/src/main/java/org/apache/struts2/dispatcher/multipart/JakartaStreamMultiPartRequest.java
@@ -251,6 +251,7 @@ protected void createUploadedFile(FileItemInput fileItemInput, File file) {
                 .create(file)
                 .withOriginalName(fileName)
                 .withContentType(fileItemInput.getContentType())
+                .withInputName(fileItemInput.getFieldName())
                 .build();
 
         if (uploadedFiles.containsKey(fieldName)) {
diff --git a/core/src/main/java/org/apache/struts2/dispatcher/multipart/StrutsUploadedFile.java b/core/src/main/java/org/apache/struts2/dispatcher/multipart/StrutsUploadedFile.java
@@ -25,11 +25,13 @@ public class StrutsUploadedFile implements UploadedFile {
     private final File file;
     private final String contentType;
     private final String originalName;
+    private final String inputName;
 
-    private StrutsUploadedFile(File file, String contentType, String originalName) {
+    private StrutsUploadedFile(File file, String contentType, String originalName, String inputName) {
         this.file = file;
         this.contentType = contentType;
         this.originalName = originalName;
+        this.inputName = inputName;
     }
 
     @Override
@@ -72,18 +74,25 @@ public String getOriginalName() {
         return originalName;
     }
 
+    @Override
+    public String getInputName() {
+        return inputName;
+    }
+
     @Override
     public String toString() {
         return "StrutsUploadedFile{" +
-                "contentType='" + contentType + '\'' +
-                ", originalName='" + originalName + '\'' +
-                '}';
+            "contentType='" + contentType + '\'' +
+            ", originalName='" + originalName + '\'' +
+            ", inputName='" + inputName + '\'' +
+            '}';
     }
 
     public static class Builder {
         private final File file;
         private String contentType;
         private String originalName;
+        private String inputName;
 
         private Builder(File file) {
             this.file = file;
@@ -103,8 +112,13 @@ public Builder withOriginalName(String originalName) {
             return this;
         }
 
+        public Builder withInputName(String inputName) {
+            this.inputName = inputName;
+            return this;
+        }
+
         public UploadedFile build() {
-            return new StrutsUploadedFile(this.file, this.contentType, this.originalName);
+            return new StrutsUploadedFile(this.file, this.contentType, this.originalName, this.inputName);
         }
     }
 }
diff --git a/core/src/main/java/org/apache/struts2/dispatcher/multipart/UploadedFile.java b/core/src/main/java/org/apache/struts2/dispatcher/multipart/UploadedFile.java
@@ -65,4 +65,13 @@ public interface UploadedFile extends Serializable {
      */
     String getContentType();
 
+    /**
+     * Represents a name of the input file, eg.:
+     * "myFile" in case of <input type="file" name="myFile">
+     *
+     * @return name of the input file field
+     * @since 6.7.0
+     */
+    String getInputName();
+
 }
diff --git a/core/src/test/java/org/apache/struts2/interceptor/ActionFileUploadInterceptorTest.java b/core/src/test/java/org/apache/struts2/interceptor/ActionFileUploadInterceptorTest.java
@@ -91,6 +91,11 @@ public String getOriginalName() {
         public String getContentType() {
             return null;
         }
+
+        @Override
+        public String getInputName() {
+            return null;
+        }
     };
 
     private MockHttpServletRequest request;

Original file line number	Diff line number	Diff line change
`@@ -37,6 +37,7 @@ public class FileUploadAction extends ActionSupport implements UploadedFilesAwar`
`37`	`37`	`private String fileName;`
`38`	`38`	`private String caption;`
`39`	`39`	`private String originalName;`
	`40`	`+ private String inputName;`
`40`	`41`
`41`	`42`	`public String input() throws Exception {`
`42`	`43`	`return SUCCESS;`
`@@ -58,6 +59,10 @@ public String getOriginalName() {`
`58`	`59`	`return originalName;`
`59`	`60`	`}`
`60`	`61`
	`62`	`+ public String getInputName() {`
	`63`	`+ return inputName;`
	`64`	`+ }`
	`65`	`+`
`61`	`66`	`public Object getUploadedFile() {`
`62`	`67`	`return uploadedFile.getContent();`
`63`	`68`	`}`
`@@ -85,5 +90,6 @@ public void withUploadedFiles(List<UploadedFile> uploadedFiles) {`
`85`	`90`	`this.fileName = uploadedFile.getName();`
`86`	`91`	`this.contentType = uploadedFile.getContentType();`
`87`	`92`	`this.originalName = uploadedFile.getOriginalName();`
	`93`	`+ this.inputName = uploadedFile.getInputName();`
`88`	`94`	`}`
`89`	`95`	`}`
Original file line number	Diff line number	Diff line change
`@@ -118,6 +118,7 @@ protected void processFileField(DiskFileItem item) {`
`118`	`118`	`.create(item.getPath().toFile())`
`119`	`119`	`.withOriginalName(item.getName())`
`120`	`120`	`.withContentType(item.getContentType())`
	`121`	`+ .withInputName(item.getFieldName())`
`121`	`122`	`.build();`
`122`	`123`	`values.add(uploadedFile);`
`123`	`124`	`}`