Making content app work show hide in manifest work for user roles. for example:
"show": [
"+role/admin"
]
51 lines
1.7 KiB
C#
51 lines
1.7 KiB
C#
using System;
|
|
using System.Collections.Generic;
|
|
using System.Linq;
|
|
using Umbraco.Core;
|
|
using Umbraco.Core.Composing;
|
|
using Umbraco.Core.Models.ContentEditing;
|
|
using Umbraco.Core.Logging;
|
|
using Umbraco.Core.Models.Membership;
|
|
|
|
namespace Umbraco.Web.ContentApps
|
|
{
|
|
public class ContentAppDefinitionCollection : BuilderCollectionBase<IContentAppDefinition>
|
|
{
|
|
private readonly ILogger _logger;
|
|
|
|
public ContentAppDefinitionCollection(IEnumerable<IContentAppDefinition> items, ILogger logger)
|
|
: base(items)
|
|
{
|
|
_logger = logger;
|
|
}
|
|
|
|
public IEnumerable<ContentApp> GetContentAppsFor(object o, IEnumerable<IReadOnlyUserGroup> userGroups=null)
|
|
{
|
|
var currentUser = UmbracoContext.Current.Security.CurrentUser;
|
|
var roles = currentUser.Groups;
|
|
var apps = this.Select(x => x.GetContentAppFor(o, roles)).WhereNotNull().OrderBy(x => x.Weight).ToList();
|
|
|
|
var aliases = new HashSet<string>();
|
|
List<string> dups = null;
|
|
|
|
foreach (var app in apps)
|
|
{
|
|
if (aliases.Contains(app.Alias))
|
|
(dups ?? (dups = new List<string>())).Add(app.Alias);
|
|
else
|
|
aliases.Add(app.Alias);
|
|
}
|
|
|
|
if (dups != null)
|
|
{
|
|
// dying is not user-friendly, so let's write to log instead, and wish people read logs...
|
|
|
|
//throw new InvalidOperationException($"Duplicate content app aliases found: {string.Join(",", dups)}");
|
|
_logger.Warn<ContentAppDefinitionCollection>($"Duplicate content app aliases found: {string.Join(",", dups)}");
|
|
}
|
|
|
|
return apps;
|
|
}
|
|
}
|
|
}
|