From 44018a30169544ca099860ef1e808d0a7a529400 Mon Sep 17 00:00:00 2001 From: Ari Karim Date: Tue, 27 May 2025 14:58:23 +0300 Subject: [PATCH] Refactor error logging in manager.rb to always log transition errors if logger is present, simplifying the conditional check. --- lib/workflower/manager.rb | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/lib/workflower/manager.rb b/lib/workflower/manager.rb index 8d3c108..95b75c3 100644 --- a/lib/workflower/manager.rb +++ b/lib/workflower/manager.rb @@ -63,9 +63,8 @@ module Workflower rescue Exception => e # if the log level is set to debug, we want to log the error logger = Workflower.configuration.logger - if logger.present? && logger.level == Logger::DEBUG - logger.debug("Error during transition: #{e.message}") - end + logger.debug("Error during transition: #{e.message}") if logger.present? + @calling_model.errors.add(@calling_model.workflower_state_column_name, :transition_faild) false end