CIT/Vmeda.Online
3
0
Fork 1
forked from EVOgeek/Vmeda.Online
Code Pull Requests Activity

MOBILE-4081 storage: Improve TODO comment

Browse Source
This commit is contained in:
Pau Ferrer Ocaña 2022-06-15 11:09:32 +02:00
parent 7090f8d918
commit 7c98579097
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/addons/storagemanager/pages/course-storage/course-storage.ts
View File

@ -482,9 +482,9 @@ export class AddonStorageManagerCourseStoragePage implements OnInit, OnDestroy {
* Mark course as not downloaded. * Mark course as not downloaded.
*/ */
protected markCourseAsNotDownloaded(): void { protected markCourseAsNotDownloaded(): void {
// @TODO This is a workaround that should be more specific solving MOBILE-3305. // @TODO In order to correctly check the status of the course we should check all module statuses.
// Also should take into account all modules are not downloaded. // We are currently marking as not downloaded if size is 0 but we should take into account that
// Check after MOBILE-3188 is integrated. // resources without files can be downloaded and cached.
CoreCourse.setCourseStatus(this.courseId, CoreConstants.NOT_DOWNLOADED); CoreCourse.setCourseStatus(this.courseId, CoreConstants.NOT_DOWNLOADED);
} }
@ -506,7 +506,6 @@ export class AddonStorageManagerCourseStoragePage implements OnInit, OnDestroy {
* Confirm and prefetch a section. If the section is "all sections", prefetch all the sections. * Confirm and prefetch a section. If the section is "all sections", prefetch all the sections.
* *
* @param section Section to download. * @param section Section to download.
* @param refresh Refresh clicked (not used).
*/ */
async prefecthSection(section: AddonStorageManagerCourseSection): Promise<void> { async prefecthSection(section: AddonStorageManagerCourseSection): Promise<void> {
section.isCalculating = true; section.isCalculating = true;