diff options
author | Saturnino Abril <saturnino.abril@gmail.com> | 2017-08-05 02:00:23 +0800 |
---|---|---|
committer | Christopher Speller <crspeller@gmail.com> | 2017-08-04 11:00:23 -0700 |
commit | 2c8a5ffd971f00648e7bc5f48993fc187f3179f2 (patch) | |
tree | 2376bd6ae34e9e21cdf57e240e62ad67492df654 /webapp/components/file_upload.jsx | |
parent | c973608ccb47373b60b333617f04bde8a3978521 (diff) | |
download | chat-2c8a5ffd971f00648e7bc5f48993fc187f3179f2.tar.gz chat-2c8a5ffd971f00648e7bc5f48993fc187f3179f2.tar.bz2 chat-2c8a5ffd971f00648e7bc5f48993fc187f3179f2.zip |
[PLT-2407] Improve ordering of uploaded attachments (#7022)
* improve ordering of uploaded attachments
* use LocalizationStore.getLocale() when comparing file names
Diffstat (limited to 'webapp/components/file_upload.jsx')
-rw-r--r-- | webapp/components/file_upload.jsx | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/webapp/components/file_upload.jsx b/webapp/components/file_upload.jsx index 0b32ab66e..0eb12a097 100644 --- a/webapp/components/file_upload.jsx +++ b/webapp/components/file_upload.jsx @@ -72,6 +72,8 @@ class FileUpload extends React.Component { } uploadFiles(files) { + const sortedFiles = Utils.sortFilesByName(files); + // clear any existing errors this.props.onUploadError(null); @@ -83,9 +85,9 @@ class FileUpload extends React.Component { // keep track of how many files have been too large const tooLargeFiles = []; - for (let i = 0; i < files.length && numUploads < uploadsRemaining; i++) { - if (files[i].size > global.mm_config.MaxFileSize) { - tooLargeFiles.push(files[i]); + for (let i = 0; i < sortedFiles.length && numUploads < uploadsRemaining; i++) { + if (sortedFiles[i].size > global.mm_config.MaxFileSize) { + tooLargeFiles.push(sortedFiles[i]); continue; } @@ -93,8 +95,8 @@ class FileUpload extends React.Component { const clientId = Utils.generateId(); const request = uploadFile( - files[i], - files[i].name, + sortedFiles[i], + sortedFiles[i].name, channelId, clientId, this.fileUploadSuccess.bind(this, channelId), @@ -111,7 +113,7 @@ class FileUpload extends React.Component { } const {formatMessage} = this.props.intl; - if (files.length > uploadsRemaining) { + if (sortedFiles.length > uploadsRemaining) { this.props.onUploadError(formatMessage(holders.limited, {count: Constants.MAX_UPLOAD_FILES})); } else if (tooLargeFiles.length > 1) { var tooLargeFilenames = tooLargeFiles.map((file) => file.name).join(', '); |