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

MOBILE-2478 lti: Fix LTI icon in Windows apps

Browse Source
This commit is contained in:
Dani Palou 2018-07-16 15:53:39 +02:00
parent e26c8f9de4
commit 3a31ef1806
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/addon/mod/lti/providers/module-handler.ts
View File

@ -96,7 +96,10 @@ export class AddonModLtiModuleHandler implements CoreCourseModuleHandler {
const icon = ltiData.secureicon || ltiData.icon; const icon = ltiData.secureicon || ltiData.icon;
if (icon) { if (icon) {
const siteId = this.sitesProvider.getCurrentSiteId(); const siteId = this.sitesProvider.getCurrentSiteId();
this.filepoolProvider.downloadUrl(siteId, icon, false, AddonModLtiProvider.COMPONENT, module.id).then((url) => { this.filepoolProvider.downloadUrl(siteId, icon, false, AddonModLtiProvider.COMPONENT, module.id).then(() => {
// Get the internal URL.
return this.filepoolProvider.getSrcByUrl(siteId, icon, AddonModLtiProvider.COMPONENT, module.id);
}).then((url) => {
data.icon = url; data.icon = url;
}).catch(() => { }).catch(() => {
// Error downloading. If we're online we'll set the online url. // Error downloading. If we're online we'll set the online url.