yv01p/Umbraco-CMS
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fixes notifications when publishing fails and warnings are issues - these were not being displayed

Browse Source
This commit is contained in:
Shannon
2019-06-07 13:13:52 +10:00
parent 9f346e6ee2
commit 9b36bc4097
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/Umbraco.Web/Editors/ContentController.cs
View File

@@ -741,7 +741,7 @@ namespace Umbraco.Web.Editors
//global notifications
AddMessageForPublishStatus(publishStatus, globalNotifications, successfulCultures);
//variant specific notifications
foreach (var c in successfulCultures)
foreach (var c in successfulCultures ?? Array.Empty<string>())
AddMessageForPublishStatus(publishStatus, notifications.GetOrCreate(c), successfulCultures);
}
break;
@@ -762,7 +762,7 @@ namespace Umbraco.Web.Editors
//global notifications
AddMessageForPublishStatus(publishStatus, globalNotifications, successfulCultures);
//variant specific notifications
foreach (var c in successfulCultures)
foreach (var c in successfulCultures ?? Array.Empty<string>())
AddMessageForPublishStatus(publishStatus, notifications.GetOrCreate(c), successfulCultures);
}
break;
@@ -1143,7 +1143,7 @@ namespace Umbraco.Web.Editors
var publishStatus = Services.ContentService.SaveAndPublishBranch(contentItem.PersistedContent, force, userId: Security.CurrentUser.Id);
// TODO: Deal with multiple cancellations
wasCancelled = publishStatus.Any(x => x.Result == PublishResultType.FailedPublishCancelledByEvent);
successfulCultures = Array.Empty<string>();
successfulCultures = null; //must be null! this implies invariant
return publishStatus;
}