diff --git a/src/addon/mod/assign/components/index/addon-mod-assign-index.html b/src/addon/mod/assign/components/index/addon-mod-assign-index.html index 957d7ff15..fd9fd8488 100644 --- a/src/addon/mod/assign/components/index/addon-mod-assign-index.html +++ b/src/addon/mod/assign/components/index/addon-mod-assign-index.html @@ -31,7 +31,7 @@ - +

{{ 'addon.mod_assign.timeremaining' | translate }}

{{ timeRemaining }}

@@ -82,6 +82,6 @@
- + diff --git a/src/addon/mod/assign/components/index/index.ts b/src/addon/mod/assign/components/index/index.ts index f46e35bc7..5172467c7 100644 --- a/src/addon/mod/assign/components/index/index.ts +++ b/src/addon/mod/assign/components/index/index.ts @@ -38,7 +38,8 @@ export class AddonModAssignIndexComponent extends CoreCourseModuleMainActivityCo moduleName = 'assign'; assign: any; // The assign object. - canViewSubmissions: boolean; // Whether the user can view all submissions. + canViewAllSubmissions: boolean; // Whether the user can view all submissions. + canViewOwnSubmission: boolean; // Whether the user can view their own submission. timeRemaining: string; // Message about time remaining to submit. lateSubmissions: string; // Message about late submissions. showNumbers = true; // Whether to show number of submissions with each status. @@ -80,16 +81,16 @@ export class AddonModAssignIndexComponent extends CoreCourseModuleMainActivityCo // Ignore errors. }); - if (!this.canViewSubmissions) { - // User can only see his submission, log view the user submission. - this.assignProvider.logSubmissionView(this.assign.id).catch(() => { - // Ignore errors. - }); - } else { + if (this.canViewAllSubmissions) { // User can see all submissions, log grading view. this.assignProvider.logGradingView(this.assign.id).catch(() => { // Ignore errors. }); + } else if (this.canViewOwnSubmission) { + // User can only see their own submission, log view the user submission. + this.assignProvider.logSubmissionView(this.assign.id).catch(() => { + // Ignore errors. + }); } }); @@ -165,7 +166,7 @@ export class AddonModAssignIndexComponent extends CoreCourseModuleMainActivityCo return this.assignProvider.getSubmissions(this.assign.id).then((data) => { const time = this.timeUtils.timestamp(); - this.canViewSubmissions = data.canviewsubmissions; + this.canViewAllSubmissions = data.canviewsubmissions; if (data.canviewsubmissions) { @@ -206,6 +207,17 @@ export class AddonModAssignIndexComponent extends CoreCourseModuleMainActivityCo }); }); } + + // Check if the user can view their own submission. + return this.assignProvider.getSubmissionStatus(this.assign.id).then(() => { + this.canViewOwnSubmission = true; + }).catch((error) => { + this.canViewOwnSubmission = false; + + if (error.errorcode !== 'nopermission') { + return Promise.reject(error); + } + }); }); }).then(() => { // All data obtained, now fill the context menu. @@ -263,7 +275,7 @@ export class AddonModAssignIndexComponent extends CoreCourseModuleMainActivityCo if (this.assign) { promises.push(this.assignProvider.invalidateAllSubmissionData(this.assign.id)); - if (this.canViewSubmissions) { + if (this.canViewAllSubmissions) { promises.push(this.assignProvider.invalidateSubmissionStatusData(this.assign.id)); } } diff --git a/src/addon/mod/assign/providers/prefetch-handler.ts b/src/addon/mod/assign/providers/prefetch-handler.ts index e98bb8f9a..d7a86836a 100644 --- a/src/addon/mod/assign/providers/prefetch-handler.ts +++ b/src/addon/mod/assign/providers/prefetch-handler.ts @@ -64,9 +64,16 @@ export class AddonModAssignPrefetchHandler extends CoreCourseActivityPrefetchHan canUseCheckUpdates(module: any, courseId: number): boolean | Promise { // Teachers cannot use the WS because it doesn't check student submissions. return this.assignProvider.getAssignment(courseId, module.id).then((assign) => { - return this.assignProvider.getSubmissions(assign.id); - }).then((data) => { - return !data.canviewsubmissions; + return this.assignProvider.getSubmissions(assign.id).then((data) => { + if (data.canviewsubmissions) { + return false; + } + + // Check if the user can view their own submission. + return this.assignProvider.getSubmissionStatus(assign.id).then(() => { + return true; + }); + }); }).catch(() => { return false; }); @@ -322,10 +329,16 @@ export class AddonModAssignPrefetchHandler extends CoreCourseActivityPrefetchHan })); } else { // Student. - promises.push(this.assignProvider.getSubmissionStatus(assign.id, userId, false, true, false, siteId) - .then((subm) => { - return this.prefetchSubmission(assign, courseId, moduleId, subm, userId, siteId); - })); + promises.push( + this.assignProvider.getSubmissionStatus(assign.id, userId, false, true, false, siteId).then((subm) => { + return this.prefetchSubmission(assign, courseId, moduleId, subm, userId, siteId); + }).catch((error) => { + // Ignore if the user can't view their own submission. + if (error.errorcode != 'nopermission') { + return Promise.reject(error); + } + }) + ); } promises.push(this.groupsProvider.activityHasGroups(assign.cmid));