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
387
387
|> Repo . insert ( )
388
388
else
389
389
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 )
396
392
|> Map . put ( :display_order , index )
397
393
398
394
% { 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 )
400
397
|> Repo . one ( )
401
398
402
399
if question_params . type != Atom . to_string ( type ) do
@@ -427,23 +424,23 @@ defmodule Cadet.Assessments do
427
424
|> elem ( 1 )
428
425
|> elem ( 1 ) ) . data . id
429
426
430
- # check if assessment already exists
431
- if ! assessment_id do
432
- false
433
- else
427
+ if assessment_id do
434
428
open_date = Repo . get ( Assessment , assessment_id ) . open_at
435
429
# check if assessment is already opened
436
430
if Timex . after? ( open_date , Timex . now ( ) ) do
437
431
false
438
432
else
439
433
existing_questions_count =
440
- where ( Question , [ q ] , q . assessment_id == ^ assessment_id )
434
+ Question
435
+ |> where ( [ q ] , q . assessment_id == ^ assessment_id )
441
436
|> Repo . all ( )
442
437
|> Enum . count ( )
443
438
444
439
new_questions_count = Enum . count ( questions_params )
445
440
existing_questions_count != new_questions_count
446
441
end
442
+ else
443
+ false
447
444
end
448
445
end
449
446
You can’t perform that action at this time.
0 commit comments