From ef2a37e2bfe661b8db5f5d20b3f74c0525720962 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?R=C3=A2u=20Cao?= Date: Wed, 4 Sep 2024 13:05:17 +0200 Subject: [PATCH] Sort user services in LDAP entry Makes it predictable for programmatic comparisons (e.g. tests) --- app/models/user.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/app/models/user.rb b/app/models/user.rb index f387817..248a865 100644 --- a/app/models/user.rb +++ b/app/models/user.rb @@ -180,14 +180,14 @@ class User < ApplicationRecord def enable_service(service) current_services = services_enabled new_services = Array(service).map(&:to_s) - services = (current_services + new_services).uniq + services = (current_services + new_services).uniq.sort ldap.replace_attribute(dn, :serviceEnabled, services) end def disable_service(service) current_services = services_enabled disabled_services = Array(service).map(&:to_s) - services = (current_services - disabled_services).uniq + services = (current_services - disabled_services).uniq.sort ldap.replace_attribute(dn, :serviceEnabled, services) end