Skip to content

Commit b6e7195

Browse files
authored
Merge pull request #2574 from dpalou/MOBILE-3523
MOBILE-3523 mod: Fix cannot get siteId of undefined
2 parents deb6438 + 4de10e2 commit b6e7195

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/addon/mod/data/providers/data.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -929,7 +929,7 @@ export class AddonModDataProvider {
929929
* @param options Other options.
930930
* @return Promise resolved when the action is done.
931931
*/
932-
searchEntries(dataId: number, options?: AddonModDataSearchEntriesOptions): Promise<AddonModDataEntries> {
932+
searchEntries(dataId: number, options: AddonModDataSearchEntriesOptions = {}): Promise<AddonModDataEntries> {
933933
options.groupId = options.groupId || 0;
934934
options.sort = options.sort || 0;
935935
options.order || options.order || 'DESC';

src/addon/mod/folder/providers/folder.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ export class AddonModFolderProvider {
5757
* @param options Other options.
5858
* @return Promise resolved when the book is retrieved.
5959
*/
60-
protected getFolderByKey(courseId: number, key: string, value: any, options?: CoreSitesCommonWSOptions)
60+
protected getFolderByKey(courseId: number, key: string, value: any, options: CoreSitesCommonWSOptions = {})
6161
: Promise<AddonModFolderFolder> {
6262
return this.sitesProvider.getSite(options.siteId).then((site) => {
6363
const params = {

0 commit comments

Comments
 (0)