From 974e0964c0ba963c3c321fdf550da9f2526c5067 Mon Sep 17 00:00:00 2001 From: Italo Matos Date: Sun, 4 Jan 2026 10:11:09 -0300 Subject: [PATCH] Refactor: Replace find_by(id:) with find() for cleaner code Replace verbose find_by(id: ...) calls with the more concise find() method across Card model and Storage::Tracked concern. --- app/models/card.rb | 2 +- app/models/concerns/storage/tracked.rb | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/app/models/card.rb b/app/models/card.rb index dbb45f184e..fee8852dab 100644 --- a/app/models/card.rb +++ b/app/models/card.rb @@ -122,7 +122,7 @@ def set_default_title end def handle_board_change - old_board = account.boards.find_by(id: board_id_before_last_save) + old_board = account.boards.find(board_id_before_last_save) transaction do update! column: nil diff --git a/app/models/concerns/storage/tracked.rb b/app/models/concerns/storage/tracked.rb index 745ef66922..0480acde65 100644 --- a/app/models/concerns/storage/tracked.rb +++ b/app/models/concerns/storage/tracked.rb @@ -29,7 +29,7 @@ def board_transfer? end def track_board_transfer - old_board = Board.find_by(id: attribute_in_database(:board_id)) + old_board = Board.find(attribute_in_database(:board_id)) records = storage_transfer_records.compact return if records.empty?