From bf3d30f0ca1b35b415201a858fed0d4454668f6c Mon Sep 17 00:00:00 2001
From: "Ajayvir S." <ajayvirsingh11@hotmail.com>
Date: Thu, 9 Jan 2025 23:27:45 +0100
Subject: [PATCH] fix scrolling

---
 ...giarism-cases-instructor-view.component.html |  2 +-
 ...lagiarism-cases-instructor-view.component.ts | 17 ++++++++++-------
 2 files changed, 11 insertions(+), 8 deletions(-)

diff --git a/src/main/webapp/app/course/plagiarism-cases/instructor-view/plagiarism-cases-instructor-view.component.html b/src/main/webapp/app/course/plagiarism-cases/instructor-view/plagiarism-cases-instructor-view.component.html
index c66ac40c1588..d34a834a6b17 100644
--- a/src/main/webapp/app/course/plagiarism-cases/instructor-view/plagiarism-cases-instructor-view.component.html
+++ b/src/main/webapp/app/course/plagiarism-cases/instructor-view/plagiarism-cases-instructor-view.component.html
@@ -11,7 +11,7 @@ <h4 jhiTranslate="artemisApp.plagiarism.cases.pageSubtitle"></h4>
             </div>
         </div>
         @for (exercise of exercisesWithPlagiarismCases; track exercise.id; let exerciseIndex = $index) {
-            <div #plagExerciseElement  class="card mb-2">
+            <div #plagExerciseElement class="card mb-2" [id]="'exercise-with-plagiarism-case-' + exercise.id">
                 <div class="card-header">
                     <div class="row">
                         <div class="col-3">
diff --git a/src/main/webapp/app/course/plagiarism-cases/instructor-view/plagiarism-cases-instructor-view.component.ts b/src/main/webapp/app/course/plagiarism-cases/instructor-view/plagiarism-cases-instructor-view.component.ts
index a7e5568b90f9..c07cf5a32fc4 100644
--- a/src/main/webapp/app/course/plagiarism-cases/instructor-view/plagiarism-cases-instructor-view.component.ts
+++ b/src/main/webapp/app/course/plagiarism-cases/instructor-view/plagiarism-cases-instructor-view.component.ts
@@ -17,7 +17,6 @@ import { AlertService } from 'app/core/util/alert.service';
 export class PlagiarismCasesInstructorViewComponent implements OnInit {
     courseId: number;
     examId?: number;
-    exerciseId?: number;
     plagiarismCases: PlagiarismCase[] = [];
     groupedPlagiarismCases: GroupedPlagiarismCases;
     exercisesWithPlagiarismCases: Exercise[] = [];
@@ -36,9 +35,12 @@ export class PlagiarismCasesInstructorViewComponent implements OnInit {
         private route: ActivatedRoute,
         private alertService: AlertService,
     ) {
+        // effect needs to be in constructor context, due to the possibility of ngOnInit being called from a non-injection
+        //context
         effect(() => {
-            if (this.exerciseId) {
-                this.scrollToExercise();
+            const exerciseId = Number(this.route.snapshot.queryParamMap?.get('exerciseId'));
+            if (exerciseId) {
+                this.scrollToExerciseAfterViewInit(exerciseId);
             }
         });
     }
@@ -46,8 +48,6 @@ export class PlagiarismCasesInstructorViewComponent implements OnInit {
     ngOnInit(): void {
         this.courseId = Number(this.route.snapshot.paramMap.get('courseId'));
         this.examId = Number(this.route.snapshot.paramMap.get('examId'));
-        this.exerciseId = Number(this.route.snapshot.queryParamMap?.get('exerciseId'));
-
         const plagiarismCasesForInstructor$ = this.examId
             ? this.plagiarismCasesService.getExamPlagiarismCasesForInstructor(this.courseId, this.examId)
             : this.plagiarismCasesService.getCoursePlagiarismCasesForInstructor(this.courseId);
@@ -60,8 +60,11 @@ export class PlagiarismCasesInstructorViewComponent implements OnInit {
         });
     }
 
-    scrollToExercise() {
-        const element = this.exerciseWithPlagCasesElements().find((elem) => elem.nativeElement.id === 'exercise-with-plagiarism-case-' + this.exerciseId);
+    /**
+     * scroll to the exercise with
+     */
+    scrollToExerciseAfterViewInit(exerciseId: number) {
+        const element = this.exerciseWithPlagCasesElements().find((elem) => elem.nativeElement.id === 'exercise-with-plagiarism-case-' + exerciseId);
         if (element) {
             element.nativeElement.scrollIntoView({
                 behavior: 'smooth',