Commit 8fe5bcde authored by Micaël Bergeron's avatar Micaël Bergeron

apply feedback

parent f3de4809
...@@ -159,7 +159,11 @@ module ObjectStorage ...@@ -159,7 +159,11 @@ module ObjectStorage
raise(SanityCheckError, "Mount point #{mounted_as} not found in #{model_class}.") unless model_has_mount raise(SanityCheckError, "Mount point #{mounted_as} not found in #{model_class}.") unless model_has_mount
end end
def perform(ids, model_type, mounted_as, to_store) def perform(*args)
args_check!(args)
(ids, model_type, mounted_as, to_store) = args
@model_class = model_type.constantize @model_class = model_type.constantize
@mounted_as = mounted_as&.to_sym @mounted_as = mounted_as&.to_sym
@to_store = to_store @to_store = to_store
...@@ -179,6 +183,16 @@ module ObjectStorage ...@@ -179,6 +183,16 @@ module ObjectStorage
self.class.sanity_check!(uploads, @model_class, @mounted_as) self.class.sanity_check!(uploads, @model_class, @mounted_as)
end end
def args_check!(args)
return if args.count == 4
case args.count
when 3 then raise SanityCheckError, "Job is missing the `model_type` argument."
else
raise SanityCheckError, "Job has wrong arguments format."
end
end
def build_uploaders(uploads) def build_uploaders(uploads)
uploads.map { |upload| upload.build_uploader(@mounted_as) } uploads.map { |upload| upload.build_uploader(@mounted_as) }
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment