From cd601d8ada7a9bcf8710f8d844b1acc86542eb34 Mon Sep 17 00:00:00 2001 From: Sebastiaan Janssen Date: Mon, 30 Jul 2018 12:25:18 +0200 Subject: [PATCH] Revert whitespace changes --- .../Macros/MacroFieldEditorsResolver.cs | 146 ++++----- .../Persistence/PetaPocoExtensionsTest.cs | 18 +- .../ImageThumbnailProvider.cs | 2 +- .../umbraco/Trees/loadScripts.cs | 13 +- .../RazorDynamicNode/DynamicNodeList.cs | 293 +++++++++--------- .../BasePages/BasePage.cs | 24 +- .../BasePages/UmbracoEnsuredPage.cs | 4 +- 7 files changed, 250 insertions(+), 250 deletions(-) diff --git a/src/Umbraco.Core/Macros/MacroFieldEditorsResolver.cs b/src/Umbraco.Core/Macros/MacroFieldEditorsResolver.cs index 76b6d0295e..00aa513209 100644 --- a/src/Umbraco.Core/Macros/MacroFieldEditorsResolver.cs +++ b/src/Umbraco.Core/Macros/MacroFieldEditorsResolver.cs @@ -9,84 +9,84 @@ using Umbraco.Core.ObjectResolution; namespace Umbraco.Core.Macros { - /// - /// A resolver to return all IMacroGuiRendering objects - /// - /// - /// Much of this classes methods are based on legacy code from umbraco.editorControls.macrocontainer.MacroControlFactory - /// this code should probably be reviewed and cleaned up if necessary. - /// - internal sealed class MacroFieldEditorsResolver : LazyManyObjectsResolverBase - { - /// - /// Constructor - /// - /// - /// - /// - internal MacroFieldEditorsResolver(IServiceProvider serviceProvider, ILogger logger, Func> macroEditors) + /// + /// A resolver to return all IMacroGuiRendering objects + /// + /// + /// Much of this classes methods are based on legacy code from umbraco.editorControls.macrocontainer.MacroControlFactory + /// this code should probably be reviewed and cleaned up if necessary. + /// + internal sealed class MacroFieldEditorsResolver : LazyManyObjectsResolverBase + { + /// + /// Constructor + /// + /// + /// + /// + internal MacroFieldEditorsResolver(IServiceProvider serviceProvider, ILogger logger, Func> macroEditors) : base(serviceProvider, logger, macroEditors, ObjectLifetimeScope.Transient) - { } + { } - /// - /// Gets the implementations. - /// - public IEnumerable MacroFieldEditors - { - get - { - return Values; - } - } + /// + /// Gets the implementations. + /// + public IEnumerable MacroFieldEditors + { + get + { + return Values; + } + } - /// - /// Gets the value based on the type of control - /// - /// - /// - /// - /// This is legacy code migrated from umbraco.editorControls.macrocontainer.MacroControlFactory - /// - internal string GetValueFromMacroControl(Control macroControl) - { + /// + /// Gets the value based on the type of control + /// + /// + /// + /// + /// This is legacy code migrated from umbraco.editorControls.macrocontainer.MacroControlFactory + /// + internal string GetValueFromMacroControl(Control macroControl) + { return HttpUtility.HtmlDecode(((IMacroGuiRendering)macroControl).Value); - } + } - /// - /// This is legacy code migrated from umbraco.editorControls.macrocontainer.MacroControlFactory - /// - internal List MacroControlTypes - { - get { return InstanceTypes.ToList(); } - } + /// + /// This is legacy code migrated from umbraco.editorControls.macrocontainer.MacroControlFactory + /// + internal List MacroControlTypes + { + get { return InstanceTypes.ToList(); } + } - /// - /// Create an instance of a Macro control and return it. - /// Because the macro control uses inline client script whichs is not generated after postback - /// That's why we use the Page Picker instead of the content picker of the macro. - /// - /// - /// This is legacy code migrated from umbraco.editorControls.macrocontainer.MacroControlFactory - /// - internal Control GetMacroRenderControlByType(PersistableMacroProperty prop, string uniqueId) - { - var m = MacroControlTypes.FindLast(macroGuiCcontrol => macroGuiCcontrol.ToString() == string.Format("{0}.{1}", prop.AssemblyName, prop.TypeName)); - var instance = ServiceProvider.GetService(m) as IMacroGuiRendering; - if (instance != null) - { - if (string.IsNullOrEmpty(prop.Value) == false) - { + /// + /// Create an instance of a Macro control and return it. + /// Because the macro control uses inline client script whichs is not generated after postback + /// That's why we use the Page Picker instead of the content picker of the macro. + /// + /// + /// This is legacy code migrated from umbraco.editorControls.macrocontainer.MacroControlFactory + /// + internal Control GetMacroRenderControlByType(PersistableMacroProperty prop, string uniqueId) + { + var m = MacroControlTypes.FindLast(macroGuiCcontrol => macroGuiCcontrol.ToString() == string.Format("{0}.{1}", prop.AssemblyName, prop.TypeName)); + var instance = ServiceProvider.GetService(m) as IMacroGuiRendering; + if (instance != null) + { + if (string.IsNullOrEmpty(prop.Value) == false) + { instance.Value = HttpUtility.HtmlDecode(prop.Value); - } - var macroControl = instance as Control; - if (macroControl != null) - { - macroControl.ID = uniqueId; - return macroControl; - } - } - return null; - } + } + var macroControl = instance as Control; + if (macroControl != null) + { + macroControl.ID = uniqueId; + return macroControl; + } + } + return null; + } - } -} + } +} \ No newline at end of file diff --git a/src/Umbraco.Tests/Persistence/PetaPocoExtensionsTest.cs b/src/Umbraco.Tests/Persistence/PetaPocoExtensionsTest.cs index d4472e2f1b..d06d422d83 100644 --- a/src/Umbraco.Tests/Persistence/PetaPocoExtensionsTest.cs +++ b/src/Umbraco.Tests/Persistence/PetaPocoExtensionsTest.cs @@ -109,16 +109,16 @@ namespace Umbraco.Tests.Persistence //re-create it dbSqlServer.Execute(@"CREATE TABLE [umbracoServer]( - [id] [int] IDENTITY(1,1) NOT NULL, - [address] [nvarchar](500) NOT NULL, - [computerName] [nvarchar](255) NOT NULL, - [registeredDate] [datetime] NOT NULL CONSTRAINT [DF_umbracoServer_registeredDate] DEFAULT (getdate()), - [lastNotifiedDate] [datetime] NOT NULL, - [isActive] [bit] NOT NULL, - [isMaster] [bit] NOT NULL, + [id] [int] IDENTITY(1,1) NOT NULL, + [address] [nvarchar](500) NOT NULL, + [computerName] [nvarchar](255) NOT NULL, + [registeredDate] [datetime] NOT NULL CONSTRAINT [DF_umbracoServer_registeredDate] DEFAULT (getdate()), + [lastNotifiedDate] [datetime] NOT NULL, + [isActive] [bit] NOT NULL, + [isMaster] [bit] NOT NULL, CONSTRAINT [PK_umbracoServer] PRIMARY KEY CLUSTERED ( - [id] ASC + [id] ASC )WITH (PAD_INDEX = OFF, STATISTICS_NORECOMPUTE = OFF, IGNORE_DUP_KEY = OFF, ALLOW_ROW_LOCKS = ON, ALLOW_PAGE_LOCKS = ON) ON [PRIMARY] )"); var data = new List(); @@ -278,4 +278,4 @@ namespace Umbraco.Tests.Persistence } } } -} +} \ No newline at end of file diff --git a/src/Umbraco.Web/Media/ThumbnailProviders/ImageThumbnailProvider.cs b/src/Umbraco.Web/Media/ThumbnailProviders/ImageThumbnailProvider.cs index 3a212dddfa..bccf9ac0b6 100644 --- a/src/Umbraco.Web/Media/ThumbnailProviders/ImageThumbnailProvider.cs +++ b/src/Umbraco.Web/Media/ThumbnailProviders/ImageThumbnailProvider.cs @@ -6,7 +6,7 @@ using Umbraco.Core.ObjectResolution; namespace Umbraco.Web.Media.ThumbnailProviders { - [Weight(1000)] + [Weight(1000)] public class ImageThumbnailProvider : AbstractThumbnailProvider { protected override IEnumerable SupportedExtensions diff --git a/src/Umbraco.Web/umbraco.presentation/umbraco/Trees/loadScripts.cs b/src/Umbraco.Web/umbraco.presentation/umbraco/Trees/loadScripts.cs index 77a8575ace..c944d231d3 100644 --- a/src/Umbraco.Web/umbraco.presentation/umbraco/Trees/loadScripts.cs +++ b/src/Umbraco.Web/umbraco.presentation/umbraco/Trees/loadScripts.cs @@ -8,6 +8,7 @@ using umbraco.interfaces; using Umbraco.Core; using Umbraco.Core.IO; + namespace umbraco { [EditorBrowsable(EditorBrowsableState.Never)] @@ -26,13 +27,13 @@ namespace umbraco { Javascript.Append( @" - function openScriptEditor(id) { - UmbClientMgr.contentFrame('settings/scripts/editScript.aspx?file=' + id); - } + function openScriptEditor(id) { + UmbClientMgr.contentFrame('settings/scripts/editScript.aspx?file=' + id); + } function openScriptFolder(id) { - return false; - } - "); + return false; + } + "); } protected override string FilePath diff --git a/src/umbraco.MacroEngines/RazorDynamicNode/DynamicNodeList.cs b/src/umbraco.MacroEngines/RazorDynamicNode/DynamicNodeList.cs index 1205c2f0fd..7ddc33815a 100644 --- a/src/umbraco.MacroEngines/RazorDynamicNode/DynamicNodeList.cs +++ b/src/umbraco.MacroEngines/RazorDynamicNode/DynamicNodeList.cs @@ -7,18 +7,17 @@ using System.Linq.Dynamic; using System.Reflection; using umbraco.interfaces; using Umbraco.Core; - namespace umbraco.MacroEngines { public class DynamicNodeList : DynamicObject, IEnumerable { - public List Items; + public List Items; - [Obsolete("Use the Items property instead")] - public List get_Items() - { - return Items; - } + [Obsolete("Use the Items property instead")] + public List get_Items() + { + return Items; + } public DynamicNodeList() { @@ -65,97 +64,97 @@ namespace umbraco.MacroEngines } public override bool TryInvokeMember(InvokeMemberBinder binder, object[] args, out object result) { - var firstArg = args.FirstOrDefault(); - //this is to check for 'DocumentTypeAlias' vs 'NodeTypeAlias' for compatibility - if (firstArg != null && firstArg.ToString().InvariantStartsWith("DocumentTypeAlias")) - { - firstArg = "NodeTypeAlias" + firstArg.ToString().Substring("DocumentTypeAlias".Length); - } + var firstArg = args.FirstOrDefault(); + //this is to check for 'DocumentTypeAlias' vs 'NodeTypeAlias' for compatibility + if (firstArg != null && firstArg.ToString().InvariantStartsWith("DocumentTypeAlias")) + { + firstArg = "NodeTypeAlias" + firstArg.ToString().Substring("DocumentTypeAlias".Length); + } var name = binder.Name; - if (name == "Single") - { - string predicate = firstArg == null ? "" : firstArg.ToString(); - var values = predicate.IsNullOrWhiteSpace() ? new object[] { } : args.Skip(1).ToArray(); - var single = this.Single(predicate, values); - result = new DynamicNode(single); - return true; - } - if (name == "SingleOrDefault") - { - string predicate = firstArg == null ? "" : firstArg.ToString(); - var values = predicate.IsNullOrWhiteSpace() ? new object[] { } : args.Skip(1).ToArray(); - var single = this.SingleOrDefault(predicate, values); - if (single == null) - result = new DynamicNull(); - else - result = new DynamicNode(single); - return true; - } - if (name == "First") - { - string predicate = firstArg == null ? "" : firstArg.ToString(); - var values = predicate.IsNullOrWhiteSpace() ? new object[] { } : args.Skip(1).ToArray(); - var first = this.First(predicate, values); - result = new DynamicNode(first); - return true; - } - if (name == "FirstOrDefault") - { - string predicate = firstArg == null ? "" : firstArg.ToString(); - var values = predicate.IsNullOrWhiteSpace() ? new object[] { } : args.Skip(1).ToArray(); - var first = this.FirstOrDefault(predicate, values); - if (first == null) - result = new DynamicNull(); - else - result = new DynamicNode(first); - return true; - } - if (name == "Last") - { - string predicate = firstArg == null ? "" : firstArg.ToString(); - var values = predicate.IsNullOrWhiteSpace() ? new object[] { } : args.Skip(1).ToArray(); - var last = this.Last(predicate, values); - result = new DynamicNode(last); - return true; - } - if (name == "LastOrDefault") - { - string predicate = firstArg == null ? "" : firstArg.ToString(); - var values = predicate.IsNullOrWhiteSpace() ? new object[] { } : args.Skip(1).ToArray(); - var last = this.LastOrDefault(predicate, values); - if (last == null) - result = new DynamicNull(); - else - result = new DynamicNode(last); - return true; - } + if (name == "Single") + { + string predicate = firstArg == null ? "" : firstArg.ToString(); + var values = predicate.IsNullOrWhiteSpace() ? new object[] { } : args.Skip(1).ToArray(); + var single = this.Single(predicate, values); + result = new DynamicNode(single); + return true; + } + if (name == "SingleOrDefault") + { + string predicate = firstArg == null ? "" : firstArg.ToString(); + var values = predicate.IsNullOrWhiteSpace() ? new object[] { } : args.Skip(1).ToArray(); + var single = this.SingleOrDefault(predicate, values); + if (single == null) + result = new DynamicNull(); + else + result = new DynamicNode(single); + return true; + } + if (name == "First") + { + string predicate = firstArg == null ? "" : firstArg.ToString(); + var values = predicate.IsNullOrWhiteSpace() ? new object[] { } : args.Skip(1).ToArray(); + var first = this.First(predicate, values); + result = new DynamicNode(first); + return true; + } + if (name == "FirstOrDefault") + { + string predicate = firstArg == null ? "" : firstArg.ToString(); + var values = predicate.IsNullOrWhiteSpace() ? new object[] { } : args.Skip(1).ToArray(); + var first = this.FirstOrDefault(predicate, values); + if (first == null) + result = new DynamicNull(); + else + result = new DynamicNode(first); + return true; + } + if (name == "Last") + { + string predicate = firstArg == null ? "" : firstArg.ToString(); + var values = predicate.IsNullOrWhiteSpace() ? new object[] { } : args.Skip(1).ToArray(); + var last = this.Last(predicate, values); + result = new DynamicNode(last); + return true; + } + if (name == "LastOrDefault") + { + string predicate = firstArg == null ? "" : firstArg.ToString(); + var values = predicate.IsNullOrWhiteSpace() ? new object[] { } : args.Skip(1).ToArray(); + var last = this.LastOrDefault(predicate, values); + if (last == null) + result = new DynamicNull(); + else + result = new DynamicNode(last); + return true; + } if (name == "Where") { - string predicate = firstArg.ToString(); + string predicate = firstArg.ToString(); var values = args.Skip(1).ToArray(); result = new DynamicNodeList(this.Where(predicate, values).ToList()); return true; } if (name == "OrderBy") { - result = new DynamicNodeList(this.OrderBy(firstArg.ToString()).ToList()); - return true; - } - if (name == "Take") - { - result = new DynamicNodeList(this.Take((int)firstArg)); - return true; - } - if (name == "Skip") - { - result = new DynamicNodeList(this.Skip((int)firstArg)); + result = new DynamicNodeList(this.OrderBy(firstArg.ToString()).ToList()); return true; } + if (name == "Take") + { + result = new DynamicNodeList(this.Take((int)firstArg)); + return true; + } + if (name == "Skip") + { + result = new DynamicNodeList(this.Skip((int)firstArg)); + return true; + } if (name == "InGroupsOf") { int groupSize = 0; - if (int.TryParse(firstArg.ToString(), out groupSize)) + if (int.TryParse(firstArg.ToString(), out groupSize)) { result = this.InGroupsOf(groupSize); return true; @@ -166,7 +165,7 @@ namespace umbraco.MacroEngines if (name == "GroupedInto") { int groupCount = 0; - if (int.TryParse(firstArg.ToString(), out groupCount)) + if (int.TryParse(firstArg.ToString(), out groupCount)) { result = this.GroupedInto(groupCount); return true; @@ -176,7 +175,7 @@ namespace umbraco.MacroEngines } if (name == "GroupBy") { - result = this.GroupBy(firstArg.ToString()); + result = this.GroupBy(firstArg.ToString()); return true; } if (name == "Average" || name == "Min" || name == "Max" || name == "Sum") @@ -186,40 +185,40 @@ namespace umbraco.MacroEngines } if (name == "Union") { - if ((firstArg as IEnumerable) != null) + if ((firstArg as IEnumerable) != null) { - result = new DynamicNodeList(this.Items.Union(firstArg as IEnumerable)); + result = new DynamicNodeList(this.Items.Union(firstArg as IEnumerable)); return true; } - if ((firstArg as DynamicNodeList) != null) + if ((firstArg as DynamicNodeList) != null) { - result = new DynamicNodeList(this.Items.Union((firstArg as DynamicNodeList).Items)); + result = new DynamicNodeList(this.Items.Union((firstArg as DynamicNodeList).Items)); return true; } } if (name == "Except") { - if ((firstArg as IEnumerable) != null) + if ((firstArg as IEnumerable) != null) { - result = new DynamicNodeList(this.Items.Except(firstArg as IEnumerable, new DynamicNodeIdEqualityComparer())); + result = new DynamicNodeList(this.Items.Except(firstArg as IEnumerable, new DynamicNodeIdEqualityComparer())); return true; } - if ((firstArg as DynamicNodeList) != null) + if ((firstArg as DynamicNodeList) != null) { - result = new DynamicNodeList(this.Items.Except((firstArg as DynamicNodeList).Items, new DynamicNodeIdEqualityComparer())); + result = new DynamicNodeList(this.Items.Except((firstArg as DynamicNodeList).Items, new DynamicNodeIdEqualityComparer())); return true; } } if (name == "Intersect") { - if ((firstArg as IEnumerable) != null) + if ((firstArg as IEnumerable) != null) { - result = new DynamicNodeList(this.Items.Intersect(firstArg as IEnumerable, new DynamicNodeIdEqualityComparer())); + result = new DynamicNodeList(this.Items.Intersect(firstArg as IEnumerable, new DynamicNodeIdEqualityComparer())); return true; } - if ((firstArg as DynamicNodeList) != null) + if ((firstArg as DynamicNodeList) != null) { - result = new DynamicNodeList(this.Items.Intersect((firstArg as DynamicNodeList).Items, new DynamicNodeIdEqualityComparer())); + result = new DynamicNodeList(this.Items.Intersect((firstArg as DynamicNodeList).Items, new DynamicNodeIdEqualityComparer())); return true; } } @@ -449,12 +448,12 @@ namespace umbraco.MacroEngines var genericArgs = (new[] { this }).Concat(args); result = methodToExecute.Invoke(null, genericArgs.ToArray()); } - else if (TypeHelper.IsTypeAssignableFrom(methodToExecute.GetParameters().First().ParameterType)) - { - //if it is IQueryable, we'll need to cast Items AsQueryable - var genericArgs = (new[] { Items.AsQueryable() }).Concat(args); - result = methodToExecute.Invoke(null, genericArgs.ToArray()); - } + else if (TypeHelper.IsTypeAssignableFrom(methodToExecute.GetParameters().First().ParameterType)) + { + //if it is IQueryable, we'll need to cast Items AsQueryable + var genericArgs = (new[] { Items.AsQueryable() }).Concat(args); + result = methodToExecute.Invoke(null, genericArgs.ToArray()); + } else { var genericArgs = (new[] { Items }).Concat(args); @@ -483,52 +482,52 @@ namespace umbraco.MacroEngines return result; } - public IEnumerator GetEnumerator() - { - return Items.GetEnumerator(); - } + public IEnumerator GetEnumerator() + { + return Items.GetEnumerator(); + } - IEnumerator IEnumerable.GetEnumerator() - { - return GetEnumerator(); - } + IEnumerator IEnumerable.GetEnumerator() + { + return GetEnumerator(); + } - public T Single(string predicate, params object[] values) - { - return predicate.IsNullOrWhiteSpace() - ? ((IQueryable)Items.AsQueryable()).Single() - : Where(predicate, values).Single(); - } - public T SingleOrDefault(string predicate, params object[] values) - { - return predicate.IsNullOrWhiteSpace() - ? ((IQueryable)Items.AsQueryable()).SingleOrDefault() - : Where(predicate, values).SingleOrDefault(); - } - public T First(string predicate, params object[] values) - { - return predicate.IsNullOrWhiteSpace() - ? ((IQueryable)Items.AsQueryable()).First() - : Where(predicate, values).First(); - } - public T FirstOrDefault(string predicate, params object[] values) - { - return predicate.IsNullOrWhiteSpace() - ? ((IQueryable)Items.AsQueryable()).FirstOrDefault() - : Where(predicate, values).FirstOrDefault(); - } - public T Last(string predicate, params object[] values) - { - return predicate.IsNullOrWhiteSpace() - ? ((IQueryable)Items.AsQueryable()).Last() - : Where(predicate, values).Last(); - } - public T LastOrDefault(string predicate, params object[] values) - { - return predicate.IsNullOrWhiteSpace() - ? ((IQueryable)Items.AsQueryable()).LastOrDefault() - : Where(predicate, values).LastOrDefault(); - } + public T Single(string predicate, params object[] values) + { + return predicate.IsNullOrWhiteSpace() + ? ((IQueryable)Items.AsQueryable()).Single() + : Where(predicate, values).Single(); + } + public T SingleOrDefault(string predicate, params object[] values) + { + return predicate.IsNullOrWhiteSpace() + ? ((IQueryable)Items.AsQueryable()).SingleOrDefault() + : Where(predicate, values).SingleOrDefault(); + } + public T First(string predicate, params object[] values) + { + return predicate.IsNullOrWhiteSpace() + ? ((IQueryable)Items.AsQueryable()).First() + : Where(predicate, values).First(); + } + public T FirstOrDefault(string predicate, params object[] values) + { + return predicate.IsNullOrWhiteSpace() + ? ((IQueryable)Items.AsQueryable()).FirstOrDefault() + : Where(predicate, values).FirstOrDefault(); + } + public T Last(string predicate, params object[] values) + { + return predicate.IsNullOrWhiteSpace() + ? ((IQueryable)Items.AsQueryable()).Last() + : Where(predicate, values).Last(); + } + public T LastOrDefault(string predicate, params object[] values) + { + return predicate.IsNullOrWhiteSpace() + ? ((IQueryable)Items.AsQueryable()).LastOrDefault() + : Where(predicate, values).LastOrDefault(); + } public IQueryable Where(string predicate, params object[] values) { return ((IQueryable)Items.AsQueryable()).Where(predicate, values); diff --git a/src/umbraco.businesslogic/BasePages/BasePage.cs b/src/umbraco.businesslogic/BasePages/BasePage.cs index d16e3b658b..32629091cb 100644 --- a/src/umbraco.businesslogic/BasePages/BasePage.cs +++ b/src/umbraco.businesslogic/BasePages/BasePage.cs @@ -101,17 +101,17 @@ namespace umbraco.BasePages } } - private UrlHelper _url; - /// - /// Returns a UrlHelper - /// - /// - /// This URL helper is created without any route data and an empty request context - /// - public UrlHelper Url - { - get { return _url ?? (_url = new UrlHelper(Context.Request.RequestContext)); } - } + private UrlHelper _url; + /// + /// Returns a UrlHelper + /// + /// + /// This URL helper is created without any route data and an empty request context + /// + public UrlHelper Url + { + get { return _url ?? (_url = new UrlHelper(Context.Request.RequestContext)); } + } /// /// Returns a refernce of an instance of ClientTools for access to the pages client API @@ -308,7 +308,7 @@ namespace umbraco.BasePages Culture = ui.Culture(u) }); - LogHelper.Info("User {0} (Id: {1}) logged in", () => u.Name, () => u.Id); + LogHelper.Info("User {0} (Id: {1}) logged in", () => u.Name, () => u.Id); } diff --git a/src/umbraco.businesslogic/BasePages/UmbracoEnsuredPage.cs b/src/umbraco.businesslogic/BasePages/UmbracoEnsuredPage.cs index b19d1671ca..d5867fbc68 100644 --- a/src/umbraco.businesslogic/BasePages/UmbracoEnsuredPage.cs +++ b/src/umbraco.businesslogic/BasePages/UmbracoEnsuredPage.cs @@ -131,8 +131,8 @@ namespace umbraco.BasePages if (permissions.IndexOf(Action) > -1 && (Path.Contains("-20") || ("," + Path + ",").Contains("," + getUser().StartNodeId.ToString() + ","))) return true; - var user = getUser(); - LogHelper.Info("User {0} has insufficient permissions in UmbracoEnsuredPage: '{1}', '{2}', '{3}'", () => user.Name, () => Path, () => permissions, () => Action); + var user = getUser(); + LogHelper.Info("User {0} has insufficient permissions in UmbracoEnsuredPage: '{1}', '{2}', '{3}'", () => user.Name, () => Path, () => permissions, () => Action); return false; }