Skip to content

Commit 7901ed7

Browse files
authored
Merge pull request #2446 from dpalou/MOBILE-3473
MOBILE-3473 upload: Fix big files not uploaded
2 parents 0ad21f4 + 7375102 commit 7901ed7

File tree

2 files changed

+7
-19
lines changed

2 files changed

+7
-19
lines changed

src/core/fileuploader/providers/file-handler.ts

Lines changed: 3 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -64,12 +64,10 @@ export class CoreFileUploaderFileHandler implements CoreFileUploaderHandler {
6464
* @return Data.
6565
*/
6666
getData(): CoreFileUploaderHandlerData {
67-
const isIOS = this.platform.is('ios');
68-
6967
const handler: CoreFileUploaderHandlerData = {
70-
title: isIOS ? 'core.fileuploader.more' : 'core.fileuploader.file',
68+
title: 'core.fileuploader.file',
7169
class: 'core-fileuploader-file-handler',
72-
icon: isIOS ? 'more' : 'folder',
70+
icon: 'folder',
7371
};
7472

7573
if (this.appProvider.isMobile()) {
@@ -98,7 +96,6 @@ export class CoreFileUploaderFileHandler implements CoreFileUploaderHandler {
9896

9997
input.addEventListener('change', (evt: Event) => {
10098
const file = input.files[0];
101-
let fileName;
10299

103100
input.value = ''; // Unset input.
104101
if (!file) {
@@ -113,17 +110,8 @@ export class CoreFileUploaderFileHandler implements CoreFileUploaderHandler {
113110
return;
114111
}
115112

116-
fileName = file.name;
117-
if (isIOS) {
118-
// Check the name of the file and add a timestamp if needed (take picture).
119-
const matches = fileName.match(/image\.(jpe?g|png)/);
120-
if (matches) {
121-
fileName = 'image_' + this.timeUtils.readableTimestamp() + '.' + matches[1];
122-
}
123-
}
124-
125113
// Upload the picked file.
126-
this.uploaderHelper.uploadFileObject(file, maxSize, upload, allowOffline, fileName).then((result) => {
114+
this.uploaderHelper.uploadFileObject(file, maxSize, upload, allowOffline, file.name).then((result) => {
127115
this.uploaderHelper.fileUploaded(result);
128116
}).catch((error) => {
129117
this.domUtils.showErrorModalDefault(error,

src/core/fileuploader/providers/helper.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -65,8 +65,12 @@ export class CoreFileUploaderHelperProvider {
6565
*/
6666
async chooseAndUploadFile(maxSize: number, upload?: boolean, allowOffline?: boolean, mimetypes?: string[]): Promise<any> {
6767

68+
const modal = this.domUtils.showModalLoading();
69+
6870
const result = await this.fileChooser.getFile(mimetypes ? mimetypes.join(',') : undefined);
6971

72+
modal.dismiss();
73+
7074
if (!result) {
7175
// User canceled.
7276
throw this.domUtils.createCanceledError();
@@ -87,10 +91,6 @@ export class CoreFileUploaderHelperProvider {
8791
const options = this.fileUploaderProvider.getFileUploadOptions(result.uri, result.name, result.mediaType, true);
8892

8993
if (upload) {
90-
const size = await this.fileProvider.getExternalFileSize(result.uri);
91-
92-
await this.confirmUploadFile(size, false, allowOffline);
93-
9494
return this.uploadFile(result.uri, maxSize, true, options);
9595
} else {
9696
return this.copyToTmpFolder(result.uri, false, maxSize, undefined, options);

0 commit comments

Comments
 (0)