diff --git a/Duplicati/Library/Main/Operation/RestoreHandler.cs b/Duplicati/Library/Main/Operation/RestoreHandler.cs index 76ada16f2..ededa1dd6 100644 --- a/Duplicati/Library/Main/Operation/RestoreHandler.cs +++ b/Duplicati/Library/Main/Operation/RestoreHandler.cs @@ -462,6 +462,8 @@ namespace Duplicati.Library.Main.Operation Logging.Log.WriteInformationMessage(LOGTAG, "RestoreFailures", "Failed to restore {0} files, additionally the following files failed to download, which may be the cause:{1}{2}", fileErrors, Environment.NewLine, string.Join(Environment.NewLine, brokenFiles)); else if (fileErrors > 0) Logging.Log.WriteInformationMessage(LOGTAG, "RestoreFailures", "Failed to restore {0} files", fileErrors); + else if (result.FilesRestored == 0) + Logging.Log.WriteWarningMessage(LOGTAG, "NoFilesRestored", null, "Restore completed without errors but no files were restored"); // Drop the temp tables database.DropRestoreTable(); diff --git a/Duplicati/Library/Main/ResultClasses.cs b/Duplicati/Library/Main/ResultClasses.cs index f003e2377..5de6be1ea 100644 --- a/Duplicati/Library/Main/ResultClasses.cs +++ b/Duplicati/Library/Main/ResultClasses.cs @@ -610,7 +610,7 @@ namespace Duplicati.Library.Main { if (Errors != null && Errors.Any()) return ParsedResultType.Error; - else if ((Warnings != null && Warnings.Any()) || FilesRestored == 0) + else if (Warnings != null && Warnings.Any()) return ParsedResultType.Warning; else return ParsedResultType.Success; diff --git a/Duplicati/Server/Runner.cs b/Duplicati/Server/Runner.cs index 867272b42..9a3c8b5b1 100644 --- a/Duplicati/Server/Runner.cs +++ b/Duplicati/Server/Runner.cs @@ -780,7 +780,7 @@ namespace Duplicati.Server ); } } - else if (result.ParsedResult != Library.Interface.ParsedResultType.Success && (result.Warnings.Any() || result.Errors.Any())) + else if (result.ParsedResult != Library.Interface.ParsedResultType.Success) { var type = result.ParsedResult == Library.Interface.ParsedResultType.Warning ? NotificationType.Warning