Merge pull request #377 from dpalou/MOBILE-1412

Mobile 1412

Author: jleyva

www/addons/mod_resource/controllers/index.js

@@ -71,7 +71,7 @@ angular.module('mm.addons.mod_resource')
                 });
             } else if ($mmaModResource.isDisplayedInline(module)) {
                 var downloadFailed = false;
-                $mmaModResource.downloadAllContent(module).catch(function(err) {
+                return $mmaModResource.downloadAllContent(module).catch(function(err) {
                     // Mark download as failed but go on since the main files could have been downloaded.
                     downloadFailed = true;
                 }).finally(function() {
@@ -97,14 +97,14 @@ angular.module('mm.addons.mod_resource')
                 $scope.mode = 'external';
 
                 $scope.open = function() {
-                    var modal = $mmUtil.showModalLoading('mm.core.downloading', true);
+                    var modal = $mmUtil.showModalLoading();
 
                     $mmaModResource.openFile(module.contents, module.id).then(function() {
                         $mmaModResource.logView(module.instance).then(function() {
                             $mmCourse.checkModuleCompletion(courseid, module.completionstatus);
                         });
                     }).catch(function(error) {
-                        if (error) {
+                        if (error && typeof error == 'string') {
                             $mmUtil.showErrorModal(error);
                         } else {
                             $mmUtil.showErrorModal('mma.mod_resource.errorwhileloadingthecontent', true);

www/addons/mod_resource/services/resource.js

@@ -374,12 +374,17 @@ angular.module('mm.addons.mod_resource')
             return $q.reject();
         }
 
-        var url = contents[0].fileurl,
+        var files = [contents[0]],
+            siteId = $mmSite.getId(),
+            revision = $mmFilepool.getRevisionFromFileList(files),
+            timeMod = $mmFilepool.getTimemodifiedFromFileList(files),
             promise;
 
         if ($mmFS.isAvailable()) {
             // The file system is available.
-            promise = $mmFilepool.downloadUrl($mmSite.getId(), url, false, mmaModResourceComponent, moduleId);
+            promise = $mmFilepool.downloadPackage(siteId, files, mmaModResourceComponent, moduleId, revision, timeMod).then(function() {
+                return $mmFilepool.getUrlByUrl(siteId, contents[0].fileurl, mmaModResourceComponent, moduleId, timeMod);
+            });
         } else {
             // We use the live URL.
             promise = $q.when($mmSite.fixPluginfileURL(url));

www/core/lib/util.js

@@ -457,7 +457,30 @@ angular.module('mm.core')
          */
         self.showErrorModal = function(errorMessage, needsTranslate, autocloseTime) {
             var errorKey = 'mm.core.error',
-                langKeys = [errorKey];
+                langKeys = [errorKey],
+                matches;
+
+            if (angular.isObject(errorMessage)) {
+                // We received an object instead of a string. Search for common properties.
+                if (typeof errorMessage.content != 'undefined') {
+                    errorMessage = errorMessage.content;
+                } else if (typeof errorMessage.body != 'undefined') {
+                    errorMessage = errorMessage.body;
+                } else if (typeof errorMessage.message != 'undefined') {
+                    errorMessage = errorMessage.message;
+                } else if (typeof errorMessage.error != 'undefined') {
+                    errorMessage = errorMessage.error;
+                } else {
+                    // No common properties found, just stringify it.
+                    errorMessage = JSON.stringify(errorMessage);
+                }
+
+                // Try to remove tokens from the contents.
+                matches = errorMessage.match(/token"?[=|:]"?(\w*)/, '');
+                if (matches && matches[1]) {
+                    errorMessage = errorMessage.replace(new RegExp(matches[1], 'g'), 'secret');
+                }
+            }
 
             if (needsTranslate) {
                 langKeys.push(errorMessage);