projects
/
arvados.git
/ commitdiff
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
d3a9348
)
20300: Fix where(nil, ...) case.
author
Tom Clegg <tom@curii.com>
Thu, 7 Sep 2023 13:39:34 +0000
(09:39 -0400)
committer
Tom Clegg <tom@curii.com>
Fri, 29 Sep 2023 14:14:14 +0000
(10:14 -0400)
Rails 5 accepted where(nil, ...) as a no-op, but in Rails 6 that's an
error.
Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom@curii.com>
services/api/app/models/arvados_model.rb
patch
|
blob
|
history
diff --git
a/services/api/app/models/arvados_model.rb
b/services/api/app/models/arvados_model.rb
index cf9e2c277f7e6364078e8fa43a82732985823b3e..c909e47cef7ff10e4e4b01cc5221713c7d78a542 100644
(file)
--- a/
services/api/app/models/arvados_model.rb
+++ b/
services/api/app/models/arvados_model.rb
@@
-464,6
+464,7
@@
class ArvadosModel < ApplicationRecord
end
end
+ return self if sql_conds == nil
self.where(sql_conds,
user_uuids: all_user_uuids.collect{|c| c["target_uuid"]},
permission_link_classes: ['permission'])