Exam mode: Enable changing working time in exam edit screen

src/main/java/de/tum/in/www1/artemis/web/rest/ExamResource.java

@@ -190,54 +190,57 @@
     @PutMapping("courses/{courseId}/exams")
     @EnforceAtLeastInstructor
     public ResponseEntity<Exam> updateExam(@PathVariable Long courseId, @RequestBody Exam updatedExam) throws URISyntaxException {
         log.debug("REST request to update an exam : {}", updatedExam);
+
         if (updatedExam.getId() == null) {
             return createExam(courseId, updatedExam);
         }
 
         checkForExamConflictsElseThrow(courseId, updatedExam);
 
         examAccessService.checkCourseAndExamAccessForInstructorElseThrow(courseId, updatedExam.getId());
 
         // Make sure that the original references are preserved.
         Exam originalExam = examRepository.findByIdElseThrow(updatedExam.getId());
+        var originalExamDuration = originalExam.getDuration();
 
         // The Exam Mode cannot be changed after creation -> Compare request with version in the database
         if (updatedExam.isTestExam() != originalExam.isTestExam()) {
             throw new ConflictException("The Exam Mode cannot be changed after creation", ENTITY_NAME, "examModeMismatch");
         }
 
         // NOTE: Make sure that all references are preserved here
         updatedExam.setExerciseGroups(originalExam.getExerciseGroups());
         updatedExam.setStudentExams(originalExam.getStudentExams());
         updatedExam.setExamUsers(originalExam.getExamUsers());
 
         Channel updatedChannel = channelService.updateExamChannel(originalExam, updatedExam);
 
         Exam savedExam = examRepository.save(updatedExam);
 
+        // NOTE: We have to get exercises and groups as we need them for re-scheduling
+        Exam examWithExercises = examService.findByIdWithExerciseGroupsAndExercisesElseThrow(savedExam.getId());
+
         // We can't test dates for equality as the dates retrieved from the database lose precision. Also use instant to take timezones into account
         Comparator<ZonedDateTime> comparator = Comparator.comparing(date -> date.truncatedTo(ChronoUnit.SECONDS).toInstant());
         if (comparator.compare(originalExam.getVisibleDate(), updatedExam.getVisibleDate()) != 0
                 || comparator.compare(originalExam.getStartDate(), updatedExam.getStartDate()) != 0) {
-            // get all exercises
-            Exam examWithExercises = examService.findByIdWithExerciseGroupsAndExercisesElseThrow(savedExam.getId());
             // for all programming exercises in the exam, send their ids for scheduling
             examWithExercises.getExerciseGroups().stream().flatMap(group -> group.getExercises().stream()).filter(ProgrammingExercise.class::isInstance).map(Exercise::getId)
                     .forEach(instanceMessageSendService::sendProgrammingExerciseSchedule);
         }
 
-        if (comparator.compare(originalExam.getEndDate(), updatedExam.getEndDate()) != 0) {
-            // get all exercises
-            Exam examWithExercises = examService.findByIdWithExerciseGroupsAndExercisesElseThrow(savedExam.getId());
-            examService.scheduleModelingExercises(examWithExercises);
+        // NOTE: if the end date was changed, we need to update student exams and re-schedule exercises
+        if (!originalExam.getEndDate().equals(savedExam.getEndDate())) {
+            int workingTimeChange = savedExam.getDuration() - originalExamDuration;
+            updateStudentExamsAndRescheduleExercises(examWithExercises, originalExamDuration, workingTimeChange);
         }
 
         if (updatedChannel != null) {
             savedExam.setChannelName(updatedExam.getChannelName());
         }
 
         return ResponseEntity.ok(savedExam);
     }
 
     /**
@@ -259,9 +262,7 @@
             throw new BadRequestException();
         }
 
-        var now = now();
-
-        // We have to get exercise groups as `scheduleModelingExercises` needs them
+        // NOTE: We have to get exercise groups as `scheduleModelingExercises` needs them
         Exam exam = examService.findByIdWithExerciseGroupsAndExercisesElseThrow(examId);
         var originalExamDuration = exam.getDuration();
 
@@ -270,10 +271,18 @@
         exam.setWorkingTime(exam.getWorkingTime() + workingTimeChange);
         examRepository.save(exam);
 
+        // 2. Re-calculate the working times of all student exams
+        updateStudentExamsAndRescheduleExercises(exam, originalExamDuration, workingTimeChange);
+
+        return ResponseEntity.ok(exam);
+    }
+
+    private void updateStudentExamsAndRescheduleExercises(Exam exam, Integer originalExamDuration, Integer workingTimeChange) {
+        var now = now();
+
         User instructor = userRepository.getUser();
 
-        // 2. Re-calculate the working times of all student exams
-        var studentExams = studentExamRepository.findByExamId(examId);
+        var studentExams = studentExamRepository.findByExamId(exam.getId());
         for (var studentExam : studentExams) {
             Integer originalStudentWorkingTime = studentExam.getWorkingTime();
             int originalTimeExtension = originalStudentWorkingTime - originalExamDuration;
@@ -288,7 +297,9 @@
                 int adjustedWorkingTime = Math.max(newNormalWorkingTime + timeAdjustment, 0);
                 studentExam.setWorkingTime(adjustedWorkingTime);
             }
+            // TODO: probably batch these updates?
             var savedStudentExam = studentExamRepository.save(studentExam);
+
             // NOTE: if the exam is already visible, notify the student about the working time change
             if (now.isAfter(exam.getVisibleDate())) {
                 examLiveEventsService.createAndSendWorkingTimeUpdateEvent(savedStudentExam, savedStudentExam.getWorkingTime(), originalStudentWorkingTime, true, instructor);
@@ -300,12 +311,10 @@
             instanceMessageSendService.sendRescheduleAllStudentExams(exam.getId());
         }
 
+        // NOTE: potentially re-schedule clustering of modeling submissions (in case Compass is active)
         if (now.isBefore(examDateService.getLatestIndividualExamEndDate(exam))) {
-            // potentially re-schedule clustering of modeling submissions (in case Compass is active)
             examService.scheduleModelingExercises(exam);
         }
-
-        return ResponseEntity.ok(exam);
     }
 
     /**

src/main/webapp/app/exam/manage/exam-management-resolve.service.ts

@@ -8,6 +8,27 @@ import { ActivatedRouteSnapshot, Resolve } from '@angular/router';
 import { Observable, filter, map, of } from 'rxjs';
 import { HttpResponse } from '@angular/common/http';
 import { StudentExamWithGradeDTO } from 'app/exam/exam-scores/exam-score-dtos.model';
+import { Course } from 'app/entities/course.model';
+import { CourseManagementService } from 'app/course/manage/course-management.service';
+import { catchError } from 'rxjs/operators';
+
+@Injectable({ providedIn: 'root' })
+export class CourseResolve implements Resolve<Course | null> {
+    constructor(private courseManagementService: CourseManagementService) {}
+
+    resolve(route: ActivatedRouteSnapshot): Observable<Course | null> {
+        const courseId = route.params['courseId'];
+
+        if (courseId) {
+            return this.courseManagementService.find(courseId).pipe(
+                map((response) => response.body),
+                catchError(() => of(null)),
+            );
+        }
+
+        return of(null);
+    }
+}
 
 @Injectable({ providedIn: 'root' })
 export class ExamResolve implements Resolve<Exam> {

src/main/webapp/app/exam/manage/exam-management.route.ts

@@ -53,7 +53,7 @@ import { OrionTutorAssessmentComponent } from 'app/orion/assessment/orion-tutor-
 import { isOrion } from 'app/shared/orion/orion';
 import { FileUploadExerciseManagementResolve } from 'app/exercises/file-upload/manage/file-upload-exercise-management-resolve.service';
 import { ModelingExerciseResolver } from 'app/exercises/modeling/manage/modeling-exercise-resolver.service';
-import { ExamResolve, ExerciseGroupResolve, StudentExamResolve } from 'app/exam/manage/exam-management-resolve.service';
+import { CourseResolve, ExamResolve, ExerciseGroupResolve, StudentExamResolve } from 'app/exam/manage/exam-management-resolve.service';
 import { BonusComponent } from 'app/grading-system/bonus/bonus.component';
 import { SuspiciousBehaviorComponent } from 'app/exam/manage/suspicious-behavior/suspicious-behavior.component';
 import { SuspiciousSessionsOverviewComponent } from 'app/exam/manage/suspicious-behavior/suspicious-sessions-overview/suspicious-sessions-overview.component';
@@ -73,6 +73,7 @@ export const examManagementRoute: Routes = [
         component: ExamUpdateComponent,
         resolve: {
             exam: ExamResolve,
+            course: CourseResolve,
         },
         data: {
             authorities: [Authority.INSTRUCTOR, Authority.ADMIN],
@@ -85,6 +86,7 @@ export const examManagementRoute: Routes = [
         component: ExamUpdateComponent,
         resolve: {
             exam: ExamResolve,
+            course: CourseResolve,
         },
         data: {
             authorities: [Authority.INSTRUCTOR, Authority.ADMIN],
@@ -113,6 +115,7 @@ export const examManagementRoute: Routes = [
         component: ExamUpdateComponent,
         resolve: {
             exam: ExamResolve,
+            course: CourseResolve,
         },
         data: {
             authorities: [Authority.INSTRUCTOR, Authority.ADMIN],

src/main/webapp/app/exam/manage/exams/exam-checklist-component/exam-edit-workingtime-dialog/exam-edit-working-time-dialog.component.html

@@ -11,16 +11,14 @@ <h4 class="modal-title">
                 name="workingTimeSeconds"
                 durationLabelText="artemisApp.examManagement.editWorkingTime.label"
                 [(ngModel)]="workingTimeSeconds"
+                [allowNegative]="true"
                 [showRelative]="false"
                 [disabled]="isLoading"
             />
+        </div>
 
-            <p
-                class="mb-0"
-                *ngIf="absoluteWorkingTimeDuration"
-                jhiTranslate="artemisApp.examManagement.editWorkingTime.duration"
-                [translateValues]="{ duration: absoluteWorkingTimeDuration }"
-            ></p>
+        <div class="bg-light rounded-2 p-3 mt-3">
+            <jhi-working-time-change *ngIf="newWorkingTime && oldWorkingTime" [newWorkingTime]="newWorkingTime" [oldWorkingTime]="oldWorkingTime" />
         </div>
 
         <hr />

src/main/webapp/app/exam/manage/exams/exam-checklist-component/exam-edit-workingtime-dialog/exam-edit-working-time-dialog.component.ts

@@ -1,12 +1,11 @@
-import { Component, EventEmitter, Input, Output } from '@angular/core';
 import { HttpResponse } from '@angular/common/http';
+import { Component, EventEmitter, Input, Output } from '@angular/core';
 import { NgbActiveModal } from '@ng-bootstrap/ng-bootstrap';
 import { faBan, faCheck, faSpinner } from '@fortawesome/free-solid-svg-icons';
 
 import { Exam } from 'app/entities/exam.model';
 import { ExamManagementService } from 'app/exam/manage/exam-management.service';
-import { normalWorkingTime } from 'app/exam/participate/exam.utils';
-import dayjs from 'dayjs/esm';
+import { examWorkingTime } from 'app/exam/participate/exam.utils';
 
 @Component({
     selector: 'jhi-edit-working-time-dialog',
@@ -27,11 +26,12 @@ export class ExamEditWorkingTimeDialogComponent {
 
     workingTimeSeconds = 0;
 
-    get absoluteWorkingTimeDuration() {
-        const currentWorkingTimeSeconds = normalWorkingTime(this.exam);
-        if (!currentWorkingTimeSeconds) return undefined;
-        const duration = dayjs.duration(currentWorkingTimeSeconds + this.workingTimeSeconds, 'seconds');
-        return [duration.asHours() >= 1 ? `${Math.floor(duration.asHours())} h` : null, duration.format('m [min] s [s]')].filter(Boolean).join(' ');
+    get oldWorkingTime() {
+        return examWorkingTime(this.exam);
+    }
+
+    get newWorkingTime() {
+        return this.oldWorkingTime ? this.oldWorkingTime + this.workingTimeSeconds : undefined;
     }
 
     constructor(

src/main/webapp/app/exam/manage/exams/exam-checklist-component/exam-edit-workingtime-dialog/exam-edit-working-time.component.ts

@@ -5,8 +5,8 @@ import dayjs from 'dayjs/esm';
 import { Subscription, from } from 'rxjs';
 
 import { Exam } from 'app/entities/exam.model';
-import { ExamEditWorkingTimeDialogComponent } from 'app/exam/manage/exams/exam-checklist-component/exam-edit-workingtime-dialog/exam-edit-working-time-dialog.component';
 import { AlertService } from 'app/core/util/alert.service';
+import { ExamEditWorkingTimeDialogComponent } from './exam-edit-working-time-dialog.component';
 
 @Component({
     selector: 'jhi-exam-edit-working-time',

src/main/webapp/app/exam/manage/exams/exam-update.component.html

@@ -1,6 +1,6 @@
 <div class="row justify-content-center">
     <div class="col">
-        <form name="editForm" role="form" #editForm="ngForm" (ngSubmit)="save()">
+        <form name="editForm" role="form" #editForm="ngForm" (ngSubmit)="handleSubmit()">
             <div *ngIf="!exam.id; else edit" class="d-flex align-items-center">
                 <h4 *ngIf="!isImport">{{ 'artemisApp.examManagement.createExam' | artemisTranslate }}</h4>
                 <h4 *ngIf="isImport">{{ 'artemisApp.examManagement.importExam' | artemisTranslate }}</h4>
@@ -56,45 +56,61 @@ <h5 class="pb-1" jhiTranslate="artemisApp.examManagement.sections.configuration"
             <hr />
             <!-- Section exam conduction (release dates and working time) -->
             <h5 class="pb-1" jhiTranslate="artemisApp.examManagement.sections.conduction">Exam Conduction</h5>
+            <div *ngIf="isOngoingExam" class="mb-3">
+                <span class="d-inline-flex" ngbTooltip="{{ 'artemisApp.examManagement.dateChange.message' | artemisTranslate }}">
+                    <fa-icon [icon]="faExclamationTriangle" class="text-warning"></fa-icon>
+                    <span class="fw-bold mx-1" jhiTranslate="artemisApp.examManagement.dateChange.label">
+                        Changing the times of an active exam will disrupt the workflow of the students!
+                    </span>
+                </span>
+            </div>
+            <ng-template #workingTimeConfirmationContent>
+                <div class="bg-light rounded-2 p-3">
+                    <jhi-working-time-change *ngIf="oldWorkingTime && newWorkingTime" [oldWorkingTime]="oldWorkingTime" [newWorkingTime]="newWorkingTime" />
+                </div>
+            </ng-template>
             <div class="row mb-3">
                 <div class="col-sm-4">
                     <jhi-date-time-picker
                         labelName="{{ 'artemisApp.examManagement.visibleDate' | artemisTranslate }}"
                         labelTooltip="{{ 'artemisApp.examManagement.visibleDateTooltip' | artemisTranslate }}"
                         [(ngModel)]="exam.visibleDate"
                         [error]="!isValidVisibleDate"
+                        [startAt]="exam.visibleDate"
                         name="visibleDate"
                         id="visibleDate"
-                    ></jhi-date-time-picker>
+                    />
                 </div>
                 <div class="col-sm-4">
                     <jhi-date-time-picker
                         labelName="{{ 'artemisApp.examManagement' + (exam.testExam ? '.testExam' : '') + '.startDate' | artemisTranslate }}"
                         labelTooltip="{{ 'artemisApp.examManagement.startDateTooltip' | artemisTranslate }}"
                         [(ngModel)]="exam.startDate"
+                        (valueChange)="updateExamWorkingTime()"
                         [error]="!isValidStartDate"
-                        (valueChange)="calculateMaxWorkingTime()"
+                        [startAt]="exam.startDate || exam.visibleDate"
                         name="startDate"
                         id="startDate"
-                    ></jhi-date-time-picker>
+                    />
                 </div>
                 <div class="col-sm-4">
                     <jhi-date-time-picker
                         labelName="{{ 'artemisApp.examManagement' + (exam.testExam ? '.testExam' : '') + '.endDate' | artemisTranslate }}"
                         labelTooltip="{{ 'artemisApp.examManagement.endDateTooltip' | artemisTranslate }}"
                         [(ngModel)]="exam.endDate"
+                        (valueChange)="updateExamWorkingTime()"
                         [error]="!isValidEndDate"
-                        (valueChange)="calculateMaxWorkingTime()"
+                        [startAt]="exam.endDate || exam.startDate"
                         name="endDate"
                         id="endDate"
-                    ></jhi-date-time-picker>
+                    />
                 </div>
             </div>
             <div class="row mb-3">
                 <div class="col-sm-6">
                     <label class="form-check-label" for="workingTimeInMinutes" jhiTranslate="artemisApp.examManagement.workingTime">Working Time</label>
                     <jhi-help-icon [text]="'artemisApp.examManagement' + (exam.testExam ? '.testExam' : '') + '.workingTimeTooltip'"></jhi-help-icon>
-                    <input *ngIf="!exam.testExam" readonly type="text" class="form-control" [value]="calculateWorkingTime" />
+                    <input *ngIf="!exam.testExam" readonly type="text" class="form-control" [value]="workingTimeInMinutes" />
                     <input
                         *ngIf="exam.testExam"
                         id="workingTimeInMinutes"
@@ -104,7 +120,6 @@ <h5 class="pb-1" jhiTranslate="artemisApp.examManagement.sections.conduction">Ex
                         [customMin]="1"
                         [customMax]="maxWorkingTimeInMinutes"
                         [(ngModel)]="workingTimeInMinutes"
-                        (change)="convertWorkingTimeFromMinutesToSeconds($event)"
                     />
                 </div>
                 <div class="col-sm-6">
@@ -230,7 +245,7 @@ <h5 class="pb-1" jhiTranslate="artemisApp.examManagement.sections.text">Exam Tex
                 <jhi-markdown-editor id="confirmationEndText" class="markdown-editor" [(markdown)]="exam.confirmationEndText" [enableResize]="true"></jhi-markdown-editor>
             </div>
             <div>
-                <button type="button" class="btn btn-secondary" (click)="previousState()">
+                <button type="button" class="btn btn-secondary" (click)="resetToPreviousState()">
                     <fa-icon [icon]="faBan"></fa-icon>&nbsp;<span jhiTranslate="entity.action.cancel"> Cancel</span>
                 </button>
                 <button id="save-exam" type="submit" [disabled]="editForm.form.invalid || !isValidConfiguration || isSaving" class="btn btn-primary">
@@ -239,7 +254,7 @@ <h5 class="pb-1" jhiTranslate="artemisApp.examManagement.sections.text">Exam Tex
                 <ng-template #renderedWarning>
                     <div>{{ 'artemisApp.examManagement.reviewDatesInvalidExplanation' | artemisTranslate }}</div>
                 </ng-template>
-                <span *ngIf="!isValidExamStudentReviewEnd" class="badge bg-warning" [ngbTooltip]="renderedWarning" tooltip-placement="right auto">
+                <span *ngIf="!isValidExamStudentReviewEnd" class="badge bg-warning" [ngbTooltip]="renderedWarning" placement="right auto">
                     <span jhiTranslate="artemisApp.quizExercise.edit.warning"></span>
                 </span>
             </div>