EVOgeek
/
Vmeda.Online
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #1915 from sammarshallou/MOBILE-2905 

MOBILE-2905 Fix bug with NaN appearing for size
main
Juan Leyva 2019-05-16 17:49:42 +02:00 committed by GitHub
parent e7bda47d7b 3e9326d477
commit 6dd8786da6
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/addon/storagemanager/pages/course-storage/course-storage.ts
View File

@ -61,6 +61,7 @@ export class AddonStorageManagerCourseStoragePage {
section.totalSize = 0;
section.modules.forEach((module) => {
module.parentSection = section;
module.totalSize = 0;
// Note: This function only gets the size for modules which are downloadable.
// For other modules it always returns 0, even if they have downloaded some files.
// However there is no 100% reliable way to actually track the files in this case.
@ -70,9 +71,12 @@ export class AddonStorageManagerCourseStoragePage {
// Most modules which have large files are downloadable, so I think this is sufficient.
const promise = this.prefetchDelegate.getModuleDownloadedSize(module, this.course.id).
then((size) => {
module.totalSize = size;
// There are some cases where the return from this is not a valid number.
if (!isNaN(size)) {
module.totalSize = Number(size);
section.totalSize += size;
this.totalSize += size;
}
});
allPromises.push(promise);
});