Skip to content

Commit 98f08cc

Browse files
Removing a redundancy/ beatification for ensure a CbrainFileList read access at task submission, resolves #1057
1 parent 7e21f12 commit 98f08cc

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

BrainPortal/lib/cbrain_task_generators/templates/portal.rb.erb

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -729,8 +729,9 @@ class CbrainTask::<%= name %> < <%= (descriptor['custom'] || {})['cbrain:inherit
729729
next value
730730
end
731731

732-
unless (file = Userfile.find_accessible_by_user(value, self.user, :access_requested => file_access) rescue nil ||
733-
CbrainFileList.find_accessible_by_user(value, self.user, :access_requested => :read) rescue nil )
732+
file = Userfile.find_accessible_by_user(value, self.user, :access_requested => file_access) rescue nil
733+
file ||= CbrainFileList.find_accessible_by_user(value, self.user, :access_requested => :read) rescue nil
734+
unless file
734735
params_errors.add(name, ": cannot find userfile (ID #{value})")
735736
next value
736737
end

0 commit comments

Comments
 (0)