fix(FileUploader): respect multiple prop in default file uploader (ca…

…rbon-design-system#5492)
joshblack · Mar 10, 2020 · f126171 · f126171
1 parent 7ccbc21
commit f126171
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/packages/react/src/components/FileUploader/FileUploader.js b/packages/react/src/components/FileUploader/FileUploader.js
@@ -343,10 +343,14 @@ export default class FileUploader extends Component {
 
   handleChange = evt => {
     evt.stopPropagation();
+    const filenames = Array.prototype.map.call(
+      evt.target.files,
+      file => file.name
+    );
     this.setState({
-      filenames: this.state.filenames.concat(
-        Array.prototype.map.call(evt.target.files, file => file.name)
-      ),
+      filenames: this.props.multiple
+        ? this.state.filenames.concat(filenames)
+        : filenames,
     });
     if (this.props.onChange) {
       this.props.onChange(evt);

diff --git a/packages/react/src/components/FileUploader/stories/drop-container.js b/packages/react/src/components/FileUploader/stories/drop-container.js
@@ -76,7 +76,7 @@ function ExampleDropContainerApp(props) {
       }));
       props.multiple
         ? setFiles([...files, ...newFiles])
-        : setFiles([...files, newFiles[0]]);
+        : setFiles([newFiles[0]]);
       newFiles.forEach(uploadFile);
     },
     [files, props.multiple]