From 464dd4266bdc433805010b5dca428f4cb75c2a81 Mon Sep 17 00:00:00 2001
From: erdgeist <erdgeist@erdgeist.org>
Date: Fri, 31 Jul 2026 15:55:43 +0200
Subject: Group user accounts by role

Replaces the two-way admin/user split with four groups ordered by
capability: administration, Redaktion, editors, alumni. alumni takes
precedence over capability in role_group, so a retired admin appears at the
bottom rather than the top.

Forms now offer the three roles as checkboxes rather than a single admin
checkbox, with a trailing hidden blank so an empty set can be posted, and
user_params permits roles only for admins. Three create buttons prefill the
common combinations.
---
 app/controllers/users_controller.rb | 27 ++++++++++++++++++---------
 1 file changed, 18 insertions(+), 9 deletions(-)

(limited to 'app/controllers')

diff --git a/app/controllers/users_controller.rb b/app/controllers/users_controller.rb
index cb71db23..95dff220 100644
--- a/app/controllers/users_controller.rb
+++ b/app/controllers/users_controller.rb
@@ -9,14 +9,20 @@ class UsersController < ApplicationController
 
   layout 'admin'
 
+  ROLE_PRESETS = {
+    "editor"    => [],
+    "redaktion" => ["redaktion"],
+    "admin"     => ["admin", "redaktion"]
+  }.freeze
+
+  GROUP_ORDER = [:admin, :redaktion, :editor, :alumni].freeze
+
   def index
-    @users = User.order("login ASC").all.group_by do |user|
-      user.admin? ? :admin : :user
-    end
+    @users = User.order("login ASC").all.group_by(&:role_group)
   end
 
   def new
-    @user = User.new(admin: params[:admin].present?)
+    @user = User.new(:roles => ROLE_PRESETS.fetch(params[:preset], []))
   end
 
   def create
@@ -35,8 +41,7 @@ class UsersController < ApplicationController
 
   def update
     permitted = user_params
-    permitted.delete(:admin) unless current_user.is_admin?
-            
+
     if @user.update(permitted)
       flash[:notice] = t("flash.users.updated", :login => @user.login)
       redirect_to user_path(@user)
@@ -63,9 +68,13 @@ class UsersController < ApplicationController
   private
 
     def user_params
-      allowed = [:login, :email, :password, :password_confirmation]
-      allowed << :admin if current_user.admin?
-      params.fetch(:user, {}).permit(allowed)
+      permitted = params.fetch(:user, {})
+                        .permit(:login, :email, :password, :password_confirmation,
+                                :roles => [])
+      # Checkbox arrays post a leading blank from the hidden field.
+      permitted[:roles] = Array(permitted[:roles]).reject(&:blank?) if permitted.key?(:roles)
+      permitted.delete(:roles) unless current_user.is_admin?
+      permitted
     end
 
     def find_user
-- 
cgit v1.3