From 5f39a0b2fa48f3a78022da38212d682604b07954 Mon Sep 17 00:00:00 2001 From: Tom Clegg Date: Thu, 20 Jun 2024 16:41:23 -0400 Subject: [PATCH] 20640: Use reject instead of select for readability. Arvados-DCO-1.1-Signed-off-by: Tom Clegg --- .../api/app/controllers/arvados/v1/schema_controller.rb | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/services/api/app/controllers/arvados/v1/schema_controller.rb b/services/api/app/controllers/arvados/v1/schema_controller.rb index 2e61017393..1e4429cdbb 100644 --- a/services/api/app/controllers/arvados/v1/schema_controller.rb +++ b/services/api/app/controllers/arvados/v1/schema_controller.rb @@ -402,12 +402,12 @@ class Arvados::V1::SchemaController < ApplicationController discovery[:resources]['computed_permissions'][:methods].select! do |method| method == :list end - discovery[:resources]['computed_permissions'][:methods][:list][:parameters].select! do |param| - ![:cluster_id, :bypass_federation, :offset].include?(param) + discovery[:resources]['computed_permissions'][:methods][:list][:parameters].reject! do |param| + [:cluster_id, :bypass_federation, :offset].include?(param) end discovery[:schemas]['ComputedPermission'].delete(:uuidPrefix) - discovery[:schemas]['ComputedPermission'][:properties].select! do |prop| - ![:uuid, :etag].include?(prop) + discovery[:schemas]['ComputedPermission'][:properties].reject! do |prop| + [:uuid, :etag].include?(prop) end # The 'replace_files' option is implemented in lib/controller, -- 2.30.2