Merge pull request #42 from NikRimington/patch-1

Warren Buckley · web-flow · commit bc6df56ba25b · 2022-10-11T13:01:37.000+01:00
Add null check on BackofficeUser access
diff --git a/Our.Umbraco.TagHelpers/EditLinkTagHelper.cs b/Our.Umbraco.TagHelpers/EditLinkTagHelper.cs
@@ -52,7 +52,7 @@ public override void Process(TagHelperContext context, TagHelperOutput output)
 
             // Check if the user is logged in to the backoffice
             // and they have access to the content section
-            if (_backofficeUserAccessor.BackofficeUser.IsAllowedToSeeEditLink())
+            if (_backofficeUserAccessor?.BackofficeUser != null && _backofficeUserAccessor.BackofficeUser.IsAllowedToSeeEditLink())
             {
                 // Try & get Umbraco Current Node int ID (Only do this if ContentId has NOT been set)
                 if (_umbracoContextAccessor.TryGetUmbracoContext(out var umbracoContext) && ContentId == int.MinValue)
@@ -150,4 +150,4 @@ private static string GetOuterElementStyles(
             return outerStyles.ToString();
         }
     }
-}
+}

Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ public override void Process(TagHelperContext context, TagHelperOutput output)`
`52`	`52`
`53`	`53`	`// Check if the user is logged in to the backoffice`
`54`	`54`	`// and they have access to the content section`
`55`		`- if (_backofficeUserAccessor.BackofficeUser.IsAllowedToSeeEditLink())`
	`55`	`+ if (_backofficeUserAccessor?.BackofficeUser != null && _backofficeUserAccessor.BackofficeUser.IsAllowedToSeeEditLink())`
`56`	`56`	`{`
`57`	`57`	`// Try & get Umbraco Current Node int ID (Only do this if ContentId has NOT been set)`
`58`	`58`	`if (_umbracoContextAccessor.TryGetUmbracoContext(out var umbracoContext) && ContentId == int.MinValue)`
`@@ -150,4 +150,4 @@ private static string GetOuterElementStyles(`
`150`	`150`	`return outerStyles.ToString();`
`151`	`151`	`}`
`152`	`152`	`}`
`153`		`-}`
	`153`	`+}`