Skip to content

Commit cecba45

Browse files
committed
refactor: Negate if condition
1 parent fc64d3f commit cecba45

File tree

1 file changed

+11
-11
lines changed

1 file changed

+11
-11
lines changed

lib/cadet/jobs/autograder/grading_job.ex

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -103,17 +103,8 @@ defmodule Cadet.Autograder.GradingJob do
103103
end
104104

105105
defp insert_empty_submission(%{student_id: student_id, assessment: assessment}) do
106-
if !Assessments.is_team_assessment?(assessment.id) do
107-
# Individual assessment
108-
%Submission{}
109-
|> Submission.changeset(%{
110-
student_id: student_id,
111-
assessment: assessment,
112-
status: :submitted
113-
})
114-
|> Repo.insert!()
115-
else
116-
# Team assessment, get current team if any
106+
if Assessments.is_team_assessment?(assessment.id) do
107+
# Get current team if any
117108
team =
118109
Team
119110
|> where(assessment_id: ^assessment.id)
@@ -146,6 +137,15 @@ defmodule Cadet.Autograder.GradingJob do
146137
status: :submitted
147138
})
148139
|> Repo.insert!()
140+
else
141+
# Individual assessment
142+
%Submission{}
143+
|> Submission.changeset(%{
144+
student_id: student_id,
145+
assessment: assessment,
146+
status: :submitted
147+
})
148+
|> Repo.insert!()
149149
end
150150
end
151151

0 commit comments

Comments
 (0)