lint: Rubocop

OpenSourcePolitics · Jan 7, 2025 · 1eff300 · 1eff300
1 parent e5bdaa1
commit 1eff300
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 8 deletions.
diff --git a/app/jobs/clear_duplicated_half_signup_users_job.rb b/app/jobs/clear_duplicated_half_signup_users_job.rb
@@ -42,12 +42,10 @@ def clear_data(users, clear_user_accounts)
     users.each do |user|
       if user.email.include?("quick_auth")
         soft_delete_user(user, "HalfSignup duplicated account")
+      elsif clear_user_accounts
+        clear_account_phone_number(user)
       else
-        if clear_user_accounts
-          clear_account_phone_number(user)
-        else
-          decidim_user_dup_accounts << user
-        end
+        decidim_user_dup_accounts << user
       end
     end
 
@@ -97,7 +95,6 @@ def clear_account_phone_number(user)
       user.save(validate: false)
     end
 
-
     log! "User (ID/#{user.id} email/#{user.email}) has been cleaned"
   end
 

diff --git a/app/jobs/concerns/decidim/logging.rb b/app/jobs/concerns/decidim/logging.rb
@@ -10,10 +10,10 @@ def log!(msg, level = :warn)
       case level
       when :info
         Rails.logger.info msg
-        stdout_logger.info msg unless Rails.env.production?
+        stdout_logger.info msg
       else
         Rails.logger.warn msg
-        stdout_logger.warn msg unless Rails.env.production?
+        stdout_logger.warn msg
       end
     end