From 193ee7c00d8c84f69af2f41e0b0fe2afdfece3bb Mon Sep 17 00:00:00 2001 From: Christophe Bliard Date: Tue, 26 May 2026 16:42:57 +0200 Subject: [PATCH] Fix rubocop warnings --- .../synchronized_filters_controller.rb | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/modules/ldap_groups/app/controllers/ldap_groups/synchronized_filters_controller.rb b/modules/ldap_groups/app/controllers/ldap_groups/synchronized_filters_controller.rb index 8767ded6bfc..00792186490 100644 --- a/modules/ldap_groups/app/controllers/ldap_groups/synchronized_filters_controller.rb +++ b/modules/ldap_groups/app/controllers/ldap_groups/synchronized_filters_controller.rb @@ -13,12 +13,12 @@ module LdapGroups layout "admin" menu_item :plugin_ldap_groups + def show; end + def new @filter = SynchronizedFilter.new end - def show; end - def end; end def destroy_info @@ -80,14 +80,17 @@ module LdapGroups private def find_filter - @filter = SynchronizedFilter.find(params[:ldap_filter_id]) + @filter = SynchronizedFilter.find(params.expect(:ldap_filter_id)) end def permitted_params - params - .require(:synchronized_filter) - .permit(:filter_string, :name, :ldap_auth_source_id, :group_name_attribute, :sync_users, :base_dn, - :member_lookup_attribute) + params.expect(synchronized_filter: %i[filter_string + name + ldap_auth_source_id + group_name_attribute + sync_users + base_dn + member_lookup_attribute]) end end end