File tree Expand file tree Collapse file tree 1 file changed +9
-12
lines changed Expand file tree Collapse file tree 1 file changed +9
-12
lines changed Original file line number Diff line number Diff line change @@ -387,16 +387,13 @@ defmodule Cadet.Assessments do
387387 |> Repo . insert ( )
388388 else
389389 params =
390- if ! question_params . max_xp do
391- question_params
392- |> Map . put ( :max_xp , 0 )
393- else
394- question_params
395- end
390+ question_params
391+ |> Map . put_new ( :max_xp , 0 )
396392 |> Map . put ( :display_order , index )
397393
398394 % { id: question_id , type: type } =
399- where ( Question , [ q ] , q . display_order == ^ index and q . assessment_id == ^ id )
395+ Question
396+ |> where ( [ q ] , q . display_order == ^ index and q . assessment_id == ^ id )
400397 |> Repo . one ( )
401398
402399 if question_params . type != Atom . to_string ( type ) do
@@ -427,23 +424,23 @@ defmodule Cadet.Assessments do
427424 |> elem ( 1 )
428425 |> elem ( 1 ) ) . data . id
429426
430- # check if assessment already exists
431- if ! assessment_id do
432- false
433- else
427+ if assessment_id do
434428 open_date = Repo . get ( Assessment , assessment_id ) . open_at
435429 # check if assessment is already opened
436430 if Timex . after? ( open_date , Timex . now ( ) ) do
437431 false
438432 else
439433 existing_questions_count =
440- where ( Question , [ q ] , q . assessment_id == ^ assessment_id )
434+ Question
435+ |> where ( [ q ] , q . assessment_id == ^ assessment_id )
441436 |> Repo . all ( )
442437 |> Enum . count ( )
443438
444439 new_questions_count = Enum . count ( questions_params )
445440 existing_questions_count != new_questions_count
446441 end
442+ else
443+ false
447444 end
448445 end
449446
You can’t perform that action at this time.
0 commit comments