[ARVADOS] updated: 2.1.0-1223-g402e69f6e

Git user git at public.arvados.org
Tue Aug 24 13:58:08 UTC 2021


Summary of changes:
 services/api/lib/record_filters.rb | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

       via  402e69f6e55dce4e11d354c3ca708b8e536c124b (commit)
      from  388713bd42fb20bae6f628c50f4ad1e3ba067b5b (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 402e69f6e55dce4e11d354c3ca708b8e536c124b
Author: Tom Clegg <tom at curii.com>
Date:   Tue Aug 24 09:57:10 2021 -0400

    17994: Update comment.
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom at curii.com>

diff --git a/services/api/lib/record_filters.rb b/services/api/lib/record_filters.rb
index 337197c2c..8f4244f5b 100644
--- a/services/api/lib/record_filters.rb
+++ b/services/api/lib/record_filters.rb
@@ -255,9 +255,12 @@ module RecordFilters
                 raise ArgumentError.new("Invalid element #{operand.inspect} in operand for #{operator.inspect} operator (operand must be a string or array of strings)")
               end
             end
-            q = operand.map { |s| ActiveRecord::Base.connection.quote(s) }.join(',')
             # We use jsonb_exists_all(a,b) instead of "a ?& b" because
-            # the pg gem thinks "?" is a bind var.
+            # the pg gem thinks "?" is a bind var. And we use string
+            # interpolation instead of param_out because the pg gem
+            # flattens param_out / doesn't support passing arrays as
+            # bind vars.
+            q = operand.map { |s| ActiveRecord::Base.connection.quote(s) }.join(',')
             cond_out << "jsonb_exists_all(#{attr_table_name}.#{attr}, array[#{q}])"
           else
             raise ArgumentError.new("Invalid operator '#{operator}'")

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list