From b4e4c378346335ef60deec4f720b325b6479fede Mon Sep 17 00:00:00 2001
From: Dylan Corrales <DeathCamel58@gmail.com>
Date: Tue, 4 Nov 2025 17:15:35 -0500
Subject: [PATCH] Rules: Alphabetize `<select>`s in new rule form (#280)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-authored-by: Juan José Mata <juanjo.mata@gmail.com>
---
 app/models/rule/action_executor/set_transaction_category.rb | 2 +-
 app/models/rule/action_executor/set_transaction_merchant.rb | 2 +-
 app/models/rule/action_executor/set_transaction_tags.rb     | 2 +-
 app/models/rule/condition_filter/transaction_merchant.rb    | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/app/models/rule/action_executor/set_transaction_category.rb b/app/models/rule/action_executor/set_transaction_category.rb
index 6360e45a2..2da95bbac 100644
--- a/app/models/rule/action_executor/set_transaction_category.rb
+++ b/app/models/rule/action_executor/set_transaction_category.rb
@@ -4,7 +4,7 @@ class Rule::ActionExecutor::SetTransactionCategory < Rule::ActionExecutor
   end
 
   def options
-    family.categories.pluck(:name, :id)
+    family.categories.alphabetically.pluck(:name, :id)
   end
 
   def execute(transaction_scope, value: nil, ignore_attribute_locks: false)
diff --git a/app/models/rule/action_executor/set_transaction_merchant.rb b/app/models/rule/action_executor/set_transaction_merchant.rb
index f343a79f7..f9693ddac 100644
--- a/app/models/rule/action_executor/set_transaction_merchant.rb
+++ b/app/models/rule/action_executor/set_transaction_merchant.rb
@@ -4,7 +4,7 @@ class Rule::ActionExecutor::SetTransactionMerchant < Rule::ActionExecutor
   end
 
   def options
-    family.merchants.pluck(:name, :id)
+    family.merchants.alphabetically.pluck(:name, :id)
   end
 
   def execute(transaction_scope, value: nil, ignore_attribute_locks: false)
diff --git a/app/models/rule/action_executor/set_transaction_tags.rb b/app/models/rule/action_executor/set_transaction_tags.rb
index d74029ca1..f317f9602 100644
--- a/app/models/rule/action_executor/set_transaction_tags.rb
+++ b/app/models/rule/action_executor/set_transaction_tags.rb
@@ -4,7 +4,7 @@ class Rule::ActionExecutor::SetTransactionTags < Rule::ActionExecutor
   end
 
   def options
-    family.tags.pluck(:name, :id)
+    family.tags.alphabetically.pluck(:name, :id)
   end
 
   def execute(transaction_scope, value: nil, ignore_attribute_locks: false)
diff --git a/app/models/rule/condition_filter/transaction_merchant.rb b/app/models/rule/condition_filter/transaction_merchant.rb
index db1522268..581db1cdf 100644
--- a/app/models/rule/condition_filter/transaction_merchant.rb
+++ b/app/models/rule/condition_filter/transaction_merchant.rb
@@ -4,7 +4,7 @@ class Rule::ConditionFilter::TransactionMerchant < Rule::ConditionFilter
   end
 
   def options
-    family.assigned_merchants.pluck(:name, :id)
+    family.assigned_merchants.alphabetically.pluck(:name, :id)
   end
 
   def prepare(scope)