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

MOBILE-2905 Fix bug with NaN appearing for size

main
sam marshall 2019-05-16 16:10:37 +01:00
parent e63b1ff8ab
commit 3e9326d477
1 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -61,6 +61,7 @@ export class AddonStorageManagerCourseStoragePage {
section.totalSize = 0; section.totalSize = 0;
section.modules.forEach((module) => { section.modules.forEach((module) => {
module.parentSection = section; module.parentSection = section;
module.totalSize = 0;
// Note: This function only gets the size for modules which are downloadable. // 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. // 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. // 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. // Most modules which have large files are downloadable, so I think this is sufficient.
const promise = this.prefetchDelegate.getModuleDownloadedSize(module, this.course.id). const promise = this.prefetchDelegate.getModuleDownloadedSize(module, this.course.id).
then((size) => { then((size) => {
module.totalSize = size; // There are some cases where the return from this is not a valid number.
section.totalSize += size; if (!isNaN(size)) {
this.totalSize += size; module.totalSize = Number(size);
section.totalSize += size;
this.totalSize += size;
}
}); });
allPromises.push(promise); allPromises.push(promise);
}); });