chore: make linter happy

obcode · obcode · commit 0511969eb031 · 2025-05-06T10:55:45.000+02:00
diff --git a/cmd/info.go b/cmd/info.go
@@ -73,7 +73,7 @@ student name           --- get info for student.`,
 					log.Fatal(err)
 				}
 
-				defer f.Close()
+				defer f.Close() //nolint:errcheck
 
 				_, err2 := f.Write(formatted_data)
 
diff --git a/db/constraints.go b/db/constraints.go
@@ -325,7 +325,7 @@ func (db *DB) GetConstraints(ctx context.Context) ([]*model.Constraints, error)
 		log.Error().Err(err).Str("semester", db.semester).Str("collection", collectionConstraints).Msg("MongoDB Find")
 		return constraints, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	// TODO: replace all cur.Next with cur.All
 	for cur.Next(ctx) {
diff --git a/db/exam.go b/db/exam.go
@@ -34,7 +34,7 @@ func (db *DB) AdditionalExams(ctx context.Context) ([]*model.AdditionalExam, err
 		log.Error().Err(err).Str("semester", db.semester).Str("collection", collectionNameAdditionalExams).Msg("MongoDB Find")
 		return exams, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	for cur.Next(ctx) {
 		var exam model.AdditionalExam
@@ -146,7 +146,7 @@ func (db *DB) ExamsWithRegs(ctx context.Context) ([]*model.ExamWithRegs, error)
 		log.Error().Err(err).Str("semester", db.semester).Str("collection", collectionNameExamsWithRegs).Msg("MongoDB Find")
 		return exams, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	for cur.Next(ctx) {
 		var exam model.ExamWithRegs
@@ -179,7 +179,7 @@ func (db *DB) ExamsInPlan(ctx context.Context) ([]*model.ExamInPlan, error) {
 	if err != nil {
 		log.Error().Err(err).Msg("error while trying to find exams in plan")
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	exams := make([]*model.ExamInPlan, 0)
 
diff --git a/db/exam_cache.go b/db/exam_cache.go
@@ -43,7 +43,7 @@ func (db *DB) CachedExams(ctx context.Context) ([]*model.Exam, error) {
 		log.Error().Err(err).Msg("cannot get cached exams")
 		return nil, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	exams := make([]*model.Exam, 0)
 
diff --git a/db/exam_connected.go b/db/exam_connected.go
@@ -107,7 +107,7 @@ func (db *DB) GetConnectedExams(ctx context.Context) ([]*model.ConnectedExam, er
 		log.Error().Err(err).Str("semester", db.semester).Str("collection", collectionNameConnectedExams).Msg("MongoDB Find")
 		return nil, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	err = cur.All(ctx, &exams)
 
diff --git a/db/examgroup.go b/db/examgroup.go
@@ -114,7 +114,7 @@ func (db *DB) ExamGroups(ctx context.Context) ([]*model.ExamGroup, error) {
 		log.Error().Err(err).Str("semester", db.semester).Str("collection", collectionNameExamGroups).Msg("MongoDB Find")
 		return examGroups, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	for cur.Next(ctx) {
 		var examGroup model.ExamGroup
diff --git a/db/external_exam.go b/db/external_exam.go
@@ -27,7 +27,7 @@ func (db *DB) ExternalExams(ctx context.Context) ([]*model.ExternalExam, error)
 		log.Error().Err(err).Msg("cannot get external exams")
 		return nil, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	var result []*model.ExternalExam
 	err = cur.All(ctx, &result)
diff --git a/db/generatedExams.go b/db/generatedExams.go
@@ -40,7 +40,7 @@ func (db *DB) GetGeneratedExams(ctx context.Context) ([]*model.GeneratedExam, er
 		log.Error().Err(err).Msg("cannot get generated exams")
 		return nil, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	exams := make([]*model.GeneratedExam, 0)
 
@@ -61,7 +61,7 @@ func (db *DB) GetGeneratedExamsForExamer(ctx context.Context, examerID int) ([]*
 		log.Error().Err(err).Msg("cannot get generated exams")
 		return nil, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	exams := make([]*model.GeneratedExam, 0)
 
diff --git a/db/invigilation.go b/db/invigilation.go
@@ -109,7 +109,7 @@ func (db *DB) invigilationsForInvigilator(ctx context.Context, collectionName st
 		log.Error().Err(err).Str("collection", collectionSelfInvigilations).Msg("MongoDB Find")
 		return nil, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	invigilations := make([]*model.Invigilation, 0)
 	err = cur.All(ctx, &invigilations)
@@ -144,7 +144,7 @@ func (db *DB) GetSelfInvigilations(ctx context.Context) ([]*model.Invigilation,
 		log.Error().Err(err).Msg("cannot get invgilations")
 		return nil, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	invigilations := make([]*model.Invigilation, 0)
 	err = cur.All(ctx, &invigilations)
@@ -164,7 +164,7 @@ func (db *DB) GetOtherInvigilations(ctx context.Context) ([]*model.Invigilation,
 		log.Error().Err(err).Msg("cannot get invgilations")
 		return nil, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	invigilations := make([]*model.Invigilation, 0)
 	err = cur.All(ctx, &invigilations)
diff --git a/db/mucdai.go b/db/mucdai.go
@@ -31,7 +31,7 @@ func (db *DB) MucDaiExamsForProgram(ctx context.Context, program string) ([]*Muc
 		log.Error().Err(err).Str("program", program).Msg("cannot get exams for MUC.DAI program")
 		return nil, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	exams := make([]*MucDaiExam, 0)
 	err = cur.All(ctx, &exams)
diff --git a/db/non_zpa_exams.go b/db/non_zpa_exams.go
@@ -30,7 +30,7 @@ func (db *DB) NonZpaExams(ctx context.Context) ([]*model.ZPAExam, error) {
 		log.Error().Err(err).Msg("cannot get non zpa exams")
 		return nil, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	err = cur.All(ctx, &exams)
 	if err != nil {
diff --git a/db/nta.go b/db/nta.go
@@ -52,7 +52,7 @@ func (db *DB) Ntas(ctx context.Context) ([]*model.NTA, error) {
 		log.Error().Err(err).Str("collection", collectionNameNTAs).Msg("MongoDB Find")
 		return nil, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	ntas := make([]*model.NTA, 0)
 	err = cur.All(ctx, &ntas)
@@ -102,7 +102,7 @@ func (db *DB) NtasWithRegs(ctx context.Context) ([]*model.Student, error) {
 		log.Error().Err(err).Str("collection", "nta").Msg("MongoDB Find")
 		return nil, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	ntas := make([]*model.Student, 0)
 	err = cur.All(ctx, &ntas)
diff --git a/db/plan.go b/db/plan.go
@@ -67,7 +67,7 @@ func (db *DB) GetExamsInSlot(ctx context.Context, day int, time int) ([]*model.P
 		log.Error().Err(err).Str("collection", collectionNamePlan).Msg("MongoDB Find")
 		return nil, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	planEntries := make([]*model.PlanEntry, 0)
 
@@ -117,7 +117,7 @@ func (db *DB) PlanEntries(ctx context.Context) ([]*model.PlanEntry, error) {
 		log.Error().Err(err).Str("collection", collectionNamePlan).Msg("MongoDB Find")
 		return nil, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	planEntries := make([]*model.PlanEntry, 0)
 	err = cur.All(ctx, &planEntries)
@@ -137,7 +137,7 @@ func (db *DB) PlannedAncodes(ctx context.Context) ([]*model.PlanEntry, error) {
 		log.Error().Err(err).Str("collection", collectionNamePlan).Msg("cannot get plan ancode entries")
 		return nil, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	planEntries := make([]*model.PlanEntry, 0)
 	err = cur.All(ctx, &planEntries)
diff --git a/db/plannedExams.go b/db/plannedExams.go
@@ -23,7 +23,7 @@ func (db *DB) ExamsInSlot(ctx context.Context, day int, time int) ([]*model.Exam
 		log.Error().Err(err).Str("collection", collectionNamePlan).Msg("MongoDB Find")
 		return nil, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	exams := make([]*model.ExamInPlan, 0)
 	err = cur.All(ctx, &exams)
@@ -46,7 +46,7 @@ func (db *DB) PlannedExamsByMainExamer(ctx context.Context, examerID int) ([]*mo
 		log.Error().Err(err).Str("collection", collectionNamePlan).Msg("MongoDB Find")
 		return nil, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	exams := make([]*model.ExamInPlan, 0)
 	err = cur.All(ctx, &exams)
diff --git a/db/primuss_exam.go b/db/primuss_exam.go
@@ -134,7 +134,7 @@ func (db *DB) getPrimussExams(ctx context.Context, program string) ([]*model.Pri
 		log.Error().Err(err).Str("semester", db.semester).Str("program", program).Msg("MongoDB Find")
 		return exams, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	err = cur.All(ctx, &exams)
 
diff --git a/db/primuss_studentregs.go b/db/primuss_studentregs.go
@@ -22,7 +22,7 @@ func (db *DB) GetPrimussStudentRegsForProgrammAncode(ctx context.Context, progra
 		log.Error().Err(err).Int("ancode", ancode).Str("program", program).Msg("MongoDB Find (studentregs)")
 		return nil, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	var studentRegs []*model.StudentReg
 
@@ -45,7 +45,7 @@ func (db *DB) GetPrimussStudentRegsPerAncode(ctx context.Context, program string
 		log.Error().Err(err).Str("semester", db.semester).Str("program", program).Msg("MongoDB Find (studentregs)")
 		return studentRegs, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	for cur.Next(ctx) {
 		var studentReg model.StudentReg
@@ -90,7 +90,7 @@ func (db *DB) GetPrimussStudentRegsPerStudent(ctx context.Context, program strin
 		log.Error().Err(err).Str("semester", db.semester).Str("program", program).Msg("MongoDB Find (studentregs)")
 		return studentRegs, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	for cur.Next(ctx) {
 		var studentReg model.StudentReg
@@ -129,7 +129,7 @@ func (db *DB) StudentRegsForProgram(ctx context.Context, program string) ([]*mod
 		log.Error().Err(err).Str("semester", db.semester).Str("program", program).Msg("MongoDB Find (studentregs)")
 		return studentRegs, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	for cur.Next(ctx) {
 		var studentReg model.StudentReg
@@ -283,7 +283,7 @@ func (db *DB) GetRegsWithErrors(ctx context.Context) ([]*model.RegWithError, err
 		log.Error().Err(err).Str("semester", db.semester).Msg("MongoDB Find (reg with errors)")
 		return regWithErrors, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	for cur.Next(ctx) {
 		var regWithError model.RegWithError
diff --git a/db/rooms.go b/db/rooms.go
@@ -47,7 +47,7 @@ func (db *DB) Rooms(ctx context.Context) ([]*model.Room, error) {
 		log.Error().Err(err).Str("collection", collectionRooms).Msg("MongoDB Find")
 		return nil, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	rooms := make([]*model.Room, 0)
 	err = cur.All(ctx, &rooms)
diff --git a/db/studentregs.go b/db/studentregs.go
@@ -22,7 +22,7 @@ func (db *DB) GetStudentRegsPerAncodePlanned(ctx context.Context) ([]*model.Stud
 		log.Error().Err(err).Str("semester", db.semester).Msg("MongoDB Find")
 		return nil, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	for cur.Next(ctx) {
 		var studentReg model.StudentRegsPerAncode
@@ -56,7 +56,7 @@ func (db *DB) StudentRegsPerStudentPlanned(ctx context.Context) ([]*model.Studen
 		log.Error().Err(err).Str("semester", db.semester).Msg("MongoDB Find")
 		return nil, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	studentRegs := make([]*model.Student, 0)
 
@@ -78,7 +78,7 @@ func (db *DB) StudentRegsPerStudentAll(ctx context.Context) ([]*model.StudentReg
 		log.Error().Err(err).Str("semester", db.semester).Msg("MongoDB Find")
 		return nil, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	studentRegs := make([]*model.StudentRegsPerStudent, 0)
 
@@ -113,7 +113,7 @@ func (db *DB) StudentByMtknr(ctx context.Context, mtknr string, ntas map[string]
 			log.Error().Err(err).Str("collection", collectionName).Str("mtkntr", mtknr).
 				Msg("error while searching for student in collection")
 		}
-		defer cur.Close(ctx)
+		defer cur.Close(ctx) //nolint:errcheck
 
 		var results []*model.StudentReg
 
@@ -196,7 +196,7 @@ func (db *DB) StudentsByName(ctx context.Context, regex string) ([]*model.Studen
 			log.Error().Err(err).Str("collection", collectionName).Str("regex", regex).
 				Msg("error while searching for student in collection")
 		}
-		defer cur.Close(ctx)
+		defer cur.Close(ctx) //nolint:errcheck
 
 		for cur.Next(ctx) {
 			mtknr := cur.Current.Lookup("MTKNR").StringValue()
diff --git a/db/zpa.go b/db/zpa.go
@@ -56,7 +56,7 @@ func (db *DB) getTeachers(ctx context.Context, predicate func(model.Teacher) boo
 		log.Error().Err(err).Str("semester", db.semester).Str("collection", "teachers").Msg("MongoDB Find")
 		return teachers, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	for cur.Next(ctx) {
 		var teacher model.Teacher
diff --git a/db/zpa_exams.go b/db/zpa_exams.go
@@ -25,7 +25,7 @@ func (db *DB) GetZPAExams(ctx context.Context) ([]*model.ZPAExam, error) {
 		log.Error().Err(err).Str("semester", db.semester).Str("collection", "zpaexams").Msg("MongoDB Find")
 		return exams, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	addedAncodes, err := db.GetAddedAncodes(ctx)
 	if err != nil {
@@ -285,7 +285,7 @@ func (db *DB) getZpaAncodesPlannedOrNot(ctx context.Context, toPlan *bool) (*set
 		log.Error().Err(err).Interface("toPlan", toPlan).Msg("cannot get zpa exams to plan")
 		return nil, err
 	}
-	defer cur.Close(ctx)
+	defer cur.Close(ctx) //nolint:errcheck
 
 	result := make([]*ExamToPlanType, 0)
 
diff --git a/graph/mutation.resolvers.go b/graph/mutation.resolvers.go
diff --git a/graph/query.resolvers.go b/graph/query.resolvers.go
diff --git a/plexams/export.go b/plexams/export.go
diff --git a/plexams/generateplan/algorithm.go b/plexams/generateplan/algorithm.go
diff --git a/plexams/ics.go b/plexams/ics.go
diff --git a/plexams/plan.go b/plexams/plan.go
diff --git a/plexams/roomsPrepare.go b/plexams/roomsPrepare.go
diff --git a/plexams/roomsPrepareOld.go b/plexams/roomsPrepareOld.go
diff --git a/zpa/request.go b/zpa/request.go
diff --git a/zpa/zpa.go b/zpa/zpa.go

Original file line number	Diff line number	Diff line change
@@ -73,7 +73,7 @@ student name --- get info for student.`,
`73`	`73`	`log.Fatal(err)`
`74`	`74`	`}`
`75`	`75`
`76`		`- defer f.Close()`
	`76`	`+ defer f.Close() //nolint:errcheck`
`77`	`77`
`78`	`78`	`_, err2 := f.Write(formatted_data)`
`79`	`79`
Original file line number	Diff line number	Diff line change
`@@ -325,7 +325,7 @@ func (db DB) GetConstraints(ctx context.Context) ([]model.Constraints, error)`
`325`	`325`	`log.Error().Err(err).Str("semester", db.semester).Str("collection", collectionConstraints).Msg("MongoDB Find")`
`326`	`326`	`return constraints, err`
`327`	`327`	`}`
`328`		`- defer cur.Close(ctx)`
	`328`	`+ defer cur.Close(ctx) //nolint:errcheck`
`329`	`329`
`330`	`330`	`// TODO: replace all cur.Next with cur.All`
`331`	`331`	`for cur.Next(ctx) {`
Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ func (db DB) CachedExams(ctx context.Context) ([]model.Exam, error) {`
`43`	`43`	`log.Error().Err(err).Msg("cannot get cached exams")`
`44`	`44`	`return nil, err`
`45`	`45`	`}`
`46`		`- defer cur.Close(ctx)`
	`46`	`+ defer cur.Close(ctx) //nolint:errcheck`
`47`	`47`
`48`	`48`	`exams := make([]*model.Exam, 0)`
`49`	`49`
Original file line number	Diff line number	Diff line change
`@@ -107,7 +107,7 @@ func (db DB) GetConnectedExams(ctx context.Context) ([]model.ConnectedExam, er`
`107`	`107`	`log.Error().Err(err).Str("semester", db.semester).Str("collection", collectionNameConnectedExams).Msg("MongoDB Find")`
`108`	`108`	`return nil, err`
`109`	`109`	`}`
`110`		`- defer cur.Close(ctx)`
	`110`	`+ defer cur.Close(ctx) //nolint:errcheck`
`111`	`111`
`112`	`112`	`err = cur.All(ctx, &exams)`
`113`	`113`
Original file line number	Diff line number	Diff line change
`@@ -114,7 +114,7 @@ func (db DB) ExamGroups(ctx context.Context) ([]model.ExamGroup, error) {`
`114`	`114`	`log.Error().Err(err).Str("semester", db.semester).Str("collection", collectionNameExamGroups).Msg("MongoDB Find")`
`115`	`115`	`return examGroups, err`
`116`	`116`	`}`
`117`		`- defer cur.Close(ctx)`
	`117`	`+ defer cur.Close(ctx) //nolint:errcheck`
`118`	`118`
`119`	`119`	`for cur.Next(ctx) {`
`120`	`120`	`var examGroup model.ExamGroup`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ func (db DB) ExternalExams(ctx context.Context) ([]model.ExternalExam, error)`
`27`	`27`	`log.Error().Err(err).Msg("cannot get external exams")`
`28`	`28`	`return nil, err`
`29`	`29`	`}`
`30`		`- defer cur.Close(ctx)`
	`30`	`+ defer cur.Close(ctx) //nolint:errcheck`
`31`	`31`
`32`	`32`	`var result []*model.ExternalExam`
`33`	`33`	`err = cur.All(ctx, &result)`
Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,7 @@ func (db DB) GetGeneratedExams(ctx context.Context) ([]model.GeneratedExam, er`
`40`	`40`	`log.Error().Err(err).Msg("cannot get generated exams")`
`41`	`41`	`return nil, err`
`42`	`42`	`}`
`43`		`- defer cur.Close(ctx)`
	`43`	`+ defer cur.Close(ctx) //nolint:errcheck`
`44`	`44`
`45`	`45`	`exams := make([]*model.GeneratedExam, 0)`
`46`	`46`
`@@ -61,7 +61,7 @@ func (db DB) GetGeneratedExamsForExamer(ctx context.Context, examerID int) ([]`
`61`	`61`	`log.Error().Err(err).Msg("cannot get generated exams")`
`62`	`62`	`return nil, err`
`63`	`63`	`}`
`64`		`- defer cur.Close(ctx)`
	`64`	`+ defer cur.Close(ctx) //nolint:errcheck`
`65`	`65`
`66`	`66`	`exams := make([]*model.GeneratedExam, 0)`
`67`	`67`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ func (db DB) MucDaiExamsForProgram(ctx context.Context, program string) ([]Muc`
`31`	`31`	`log.Error().Err(err).Str("program", program).Msg("cannot get exams for MUC.DAI program")`
`32`	`32`	`return nil, err`
`33`	`33`	`}`
`34`		`- defer cur.Close(ctx)`
	`34`	`+ defer cur.Close(ctx) //nolint:errcheck`
`35`	`35`
`36`	`36`	`exams := make([]*MucDaiExam, 0)`
`37`	`37`	`err = cur.All(ctx, &exams)`