From ec145456e280fc9bc44bdb8e269c63a8248af5e1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Alberto=20Miedes=20Garc=C3=A9s?= Date: Thu, 14 Jan 2021 16:50:15 +0100 Subject: [PATCH] Migrate traces to new logger --- app/models/user.rb | 2 +- app/services/visualization/common_data_service.rb | 9 ++++----- 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/app/models/user.rb b/app/models/user.rb index a5a1447dfb..b1e83d74c5 100644 --- a/app/models/user.rb +++ b/app/models/user.rb @@ -329,7 +329,7 @@ class User < Sequel::Model def delete_common_data CartoDB::Visualization::CommonDataService.new.delete_common_data_for_user(self) rescue StandardError => e - CartoDB.notify_error("Error deleting common data for user", user: self, error: e.inspect) + Rails.logger.error(message: 'Error deleting common data for user', user: self, exception: e) end def after_save diff --git a/app/services/visualization/common_data_service.rb b/app/services/visualization/common_data_service.rb index a977741e39..903804185b 100644 --- a/app/services/visualization/common_data_service.rb +++ b/app/services/visualization/common_data_service.rb @@ -190,12 +190,11 @@ module CartoDB log_error(message: "Couldn't delete, already imported", visualization: { id: visualization.id }) false else - CartoDB.notify_error( - "Couldn't delete remote visualization", - visualization: visualization.id, - error: e.inspect + Rails.logger.error( + message: "Couldn't delete remote visualization", + visualization_id: visualization.id, + exception: e ) - raise e end end end