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

Check ancestors for published culture

Browse Source 
Fixed issue where if you were 3 levels deep and published only 1 culture, warning wouldn't fire
This commit is contained in:
Nikolaj Geisle
2021-10-15 11:47:48 +02:00
parent cc74e3f4fd
commit 48003e0b70
2 changed files with 116 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
src/Umbraco.Web.BackOffice/Controllers/ContentController.cs
View File

@@ -1456,7 +1456,7 @@ namespace Umbraco.Cms.Web.BackOffice.Controllers
return;
}
var publishedCultures = persistedContent.PublishedCultures.ToList();
var publishedCultures = GetPublishedCulturesFromAncestors(persistedContent).ToList();
// If only a single culture is published we shouldn't have any routing issues
if (publishedCultures.Count < 2)
{
@@ -1583,6 +1583,27 @@ namespace Umbraco.Cms.Web.BackOffice.Controllers
return true;
}
private IEnumerable<string> GetPublishedCulturesFromAncestors(IContent content)
{
if (content.ParentId == -1)
{
return content.PublishedCultures;
}
HashSet<string> publishedCultures = new ();
publishedCultures.UnionWith(content.PublishedCultures);
IEnumerable<int> ancestorIds = content.GetAncestorIds();
foreach (var id in ancestorIds)
{
IEnumerable<string> cultures = _contentService.GetById(id).PublishedCultures;
publishedCultures.UnionWith(cultures);
}
return publishedCultures;
}
/// <summary>
/// Adds a generic culture error for use in displaying the culture validation error in the save/publish/etc... dialogs
/// </summary>