Skip to content

Commit 08a2805

Browse files
committed
MOBILE-1554 prefetch: Fix bug in prefetch download error
1 parent 6c9586a commit 08a2805

File tree

6 files changed

+12
-11
lines changed

6 files changed

+12
-11
lines changed

www/addons/mod_book/services/handlers.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ angular.module('mm.addons.mod_book')
6868
e.preventDefault();
6969
e.stopPropagation();
7070
var size = $mmaModBookPrefetchHandler.getDownloadSize(module);
71-
$mmCourseHelper.prefetchModule($mmaModBook, module, size, false);
71+
$mmCourseHelper.prefetchModule($scope, $mmaModBook, module, size, false);
7272
}
7373
};
7474

@@ -80,7 +80,7 @@ angular.module('mm.addons.mod_book')
8080
e.preventDefault();
8181
e.stopPropagation();
8282
var size = $mmaModBookPrefetchHandler.getDownloadSize(module);
83-
$mmCourseHelper.prefetchModule($mmaModBook, module, size, true);
83+
$mmCourseHelper.prefetchModule($scope, $mmaModBook, module, size, true);
8484
}
8585
};
8686

www/addons/mod_folder/services/handlers.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ angular.module('mm.addons.mod_folder')
7373

7474
// Check size and show confirmation if needed.
7575
var size = $mmaModFolderPrefetchHandler.getDownloadSize(module);
76-
$mmCourseHelper.prefetchModule($mmaModFolder, module, size, false);
76+
$mmCourseHelper.prefetchModule($scope, $mmaModFolder, module, size, false);
7777
}
7878

7979
downloadBtn = {

www/addons/mod_imscp/services/handlers.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ angular.module('mm.addons.mod_imscp')
7474
e.preventDefault();
7575
e.stopPropagation();
7676
var size = $mmaModImscpPrefetchHandler.getDownloadSize(module);
77-
$mmCourseHelper.prefetchModule($mmaModImscp, module, size, false);
77+
$mmCourseHelper.prefetchModule($scope, $mmaModImscp, module, size, false);
7878
}
7979
};
8080

@@ -86,7 +86,7 @@ angular.module('mm.addons.mod_imscp')
8686
e.preventDefault();
8787
e.stopPropagation();
8888
var size = $mmaModImscpPrefetchHandler.getDownloadSize(module);
89-
$mmCourseHelper.prefetchModule($mmaModImscp, module, size, true);
89+
$mmCourseHelper.prefetchModule($scope, $mmaModImscp, module, size, true);
9090
}
9191
};
9292

www/addons/mod_page/services/handlers.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ angular.module('mm.addons.mod_page')
6868
e.preventDefault();
6969
e.stopPropagation();
7070
var size = $mmaModPagePrefetchHandler.getDownloadSize(module);
71-
$mmCourseHelper.prefetchModule($mmaModPage, module, size, false);
71+
$mmCourseHelper.prefetchModule($scope, $mmaModPage, module, size, false);
7272
}
7373
};
7474

@@ -80,7 +80,7 @@ angular.module('mm.addons.mod_page')
8080
e.preventDefault();
8181
e.stopPropagation();
8282
var size = $mmaModPagePrefetchHandler.getDownloadSize(module);
83-
$mmCourseHelper.prefetchModule($mmaModPage, module, size, true);
83+
$mmCourseHelper.prefetchModule($scope, $mmaModPage, module, size, true);
8484
}
8585
};
8686

www/addons/mod_resource/services/handlers.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ angular.module('mm.addons.mod_resource')
6868
e.preventDefault();
6969
e.stopPropagation();
7070
var size = $mmaModResourcePrefetchHandler.getDownloadSize(module);
71-
$mmCourseHelper.prefetchModule($mmaModResource, module, size, false);
71+
$mmCourseHelper.prefetchModule($scope, $mmaModResource, module, size, false);
7272
}
7373
};
7474

@@ -80,7 +80,7 @@ angular.module('mm.addons.mod_resource')
8080
e.preventDefault();
8181
e.stopPropagation();
8282
var size = $mmaModResourcePrefetchHandler.getDownloadSize(module);
83-
$mmCourseHelper.prefetchModule($mmaModResource, module, size, true);
83+
$mmCourseHelper.prefetchModule($scope, $mmaModResource, module, size, true);
8484
}
8585
};
8686

www/core/components/course/services/helper.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -314,13 +314,14 @@ angular.module('mm.core.course')
314314
* @module mm.core.course
315315
* @ngdoc method
316316
* @name $mmCourseHelper#prefetchModule
317+
* @param {Object} scope Scope.
317318
* @param {Object} service Service implementing 'invalidateContent' and 'prefetchContent'.
318319
* @param {Object} module Module to download.
319320
* @param {Number} size Size of the module.
320321
* @param {Boolean} refresh True if refreshing, false otherwise.
321322
* @return {Promise} Promise resolved when downloaded.
322323
*/
323-
self.prefetchModule = function(service, module, size, refresh) {
324+
self.prefetchModule = function(scope, service, module, size, refresh) {
324325
// Show confirmation if needed.
325326
return $mmUtil.confirmDownloadSize(size).then(function() {
326327
// Invalidate content if refreshing and download the data.
@@ -329,7 +330,7 @@ angular.module('mm.core.course')
329330
// Ignore errors.
330331
}).then(function() {
331332
return service.prefetchContent(module).catch(function() {
332-
if (!$scope.$$destroyed) {
333+
if (!scope.$$destroyed) {
333334
$mmUtil.showErrorModal('mm.core.errordownloading', true);
334335
}
335336
});

0 commit comments

Comments
 (0)