Skip to content

Commit 6cdbe83

Browse files
committed
MOBILE-4842 resource: Small improvements found when debugging behat
1 parent 756d487 commit 6cdbe83

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

src/addons/mod/resource/components/index/index.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -192,7 +192,6 @@ export class AddonModResourceIndexComponent extends CoreCourseModuleMainResource
192192
* Opens a file.
193193
*
194194
* @param iOSOpenFileAction Action to do in iOS.
195-
* @returns Promise resolved when done.
196195
*/
197196
async open(iOSOpenFileAction?: OpenFileAction): Promise<void> {
198197
let downloadable = await CoreCourseModulePrefetchDelegate.isModuleDownloadable(this.module, this.courseId);
@@ -212,12 +211,14 @@ export class AddonModResourceIndexComponent extends CoreCourseModuleMainResource
212211
await alert.onWillDismiss();
213212
}
214213

215-
return AddonModResourceHelper.openModuleFile(this.module, this.courseId, { iOSOpenFileAction });
214+
await AddonModResourceHelper.openModuleFile(this.module, this.courseId, { iOSOpenFileAction });
215+
216+
return;
216217
}
217218
}
218219

219220
// The resource cannot be downloaded, open the activity in browser.
220-
await CoreSites.getCurrentSite()?.openInBrowserWithAutoLogin(this.module.url || '');
221+
await CoreSites.getRequiredCurrentSite().openInBrowserWithAutoLogin(this.module.url ?? '');
221222
}
222223

223224
}

src/addons/mod/resource/services/resource-helper.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -183,7 +183,6 @@ export class AddonModResourceHelperProvider {
183183
* @param module Module where to get the contents.
184184
* @param courseId Course Id, used for completion purposes.
185185
* @param options Options to open the file.
186-
* @returns Resolved when done.
187186
*/
188187
async openModuleFile(module: CoreCourseModuleData, courseId: number, options: CoreOpenerOpenFileOptions = {}): Promise<void> {
189188
const modal = await CoreLoadings.show();

0 commit comments

Comments
 (0)