From 0bc1fa8aa4ea5393193cc5c44adbf7eee24cd395 Mon Sep 17 00:00:00 2001 From: Gerhard Schlager Date: Thu, 20 Dec 2018 12:54:02 +0100 Subject: [PATCH] FEATURE: Don't create PM for successful automatic backups --- lib/backup_restore/backuper.rb | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/lib/backup_restore/backuper.rb b/lib/backup_restore/backuper.rb index 4fdd1948f1a..49c4252ffeb 100644 --- a/lib/backup_restore/backuper.rb +++ b/lib/backup_restore/backuper.rb @@ -276,18 +276,18 @@ module BackupRestore end def notify_user + return if @success && @user.id == Discourse::SYSTEM_USER_ID + log "Notifying '#{@user.username}' of the end of the backup..." status = @success ? :backup_succeeded : :backup_failed - post = SystemMessage.create_from_system_user(@user, status, - logs: Discourse::Utils.pretty_logs(@logs) + post = SystemMessage.create_from_system_user( + @user, status, logs: Discourse::Utils.pretty_logs(@logs) ) - if !@success && @user.id == Discourse::SYSTEM_USER_ID + if @user.id == Discourse::SYSTEM_USER_ID post.topic.invite_group(@user, Group[:admins]) end - - post rescue => ex log "Something went wrong while notifying user.", ex end