Arvados

Reviewing the quick Workbench fix at aa0ae05

jobuuids = object.components.select { |cname, c| c[:job] and c[:job][:uuid] }

Since jobuuids gets passed into Job.where, it looks to me like it should be an array, but select will give you a subset of the hash. Should this be object.components.map { … }.compact?

pj[:job] = job[c[:job][:uuid]]

Please check that we actually found a matching job before making this assignment. It's not guaranteed, and if we didn't, the method will crash trying to run nil[:success] a little later.

Thanks.