Allow viewing submissions without corresponding exercise
Fixes CODEOCEAN-S7
This commit is contained in:
@ -57,7 +57,7 @@ class Submission < ApplicationRecord
|
|||||||
|
|
||||||
def collect_files
|
def collect_files
|
||||||
@collect_files ||= begin
|
@collect_files ||= begin
|
||||||
ancestors = build_files_hash(exercise.files.includes(:file_type), :id)
|
ancestors = build_files_hash(exercise&.files&.includes(:file_type), :id)
|
||||||
descendants = build_files_hash(files.includes(:file_type), :file_id)
|
descendants = build_files_hash(files.includes(:file_type), :file_id)
|
||||||
ancestors.merge(descendants).values
|
ancestors.merge(descendants).values
|
||||||
end
|
end
|
||||||
@ -209,7 +209,7 @@ class Submission < ApplicationRecord
|
|||||||
private
|
private
|
||||||
|
|
||||||
def build_files_hash(files, attribute)
|
def build_files_hash(files, attribute)
|
||||||
files.map(&attribute.to_proc).zip(files).to_h
|
files&.map(&attribute.to_proc)&.zip(files)&.to_h || {}
|
||||||
end
|
end
|
||||||
|
|
||||||
def prepared_runner
|
def prepared_runner
|
||||||
|
@ -7,7 +7,7 @@
|
|||||||
|
|
||||||
h1 = @submission
|
h1 = @submission
|
||||||
|
|
||||||
= row(label: 'submission.exercise', value: link_to_if(policy(@submission.exercise).show?, @submission.exercise, @submission.exercise))
|
= row(label: 'submission.exercise', value: link_to_if(@submission.exercise && policy(@submission.exercise).show?, @submission.exercise, @submission.exercise))
|
||||||
= row(label: 'submission.contributor', value: link_to_if(policy(@submission.contributor).show?, @submission.contributor, @submission.contributor))
|
= row(label: 'submission.contributor', value: link_to_if(policy(@submission.contributor).show?, @submission.contributor, @submission.contributor))
|
||||||
= row(label: 'submission.study_group', value: link_to_if(@submission.study_group.present? && policy(@submission.study_group).show?, @submission.study_group, @submission.study_group))
|
= row(label: 'submission.study_group', value: link_to_if(@submission.study_group.present? && policy(@submission.study_group).show?, @submission.study_group, @submission.study_group))
|
||||||
= row(label: 'submission.cause', value: t("submissions.causes.#{@submission.cause}"))
|
= row(label: 'submission.cause', value: t("submissions.causes.#{@submission.cause}"))
|
||||||
|
Reference in New Issue
Block a user