chore: Sonarcloud complaints

DFE-Digital · Oct 1, 2024 · eef3cb5 · eef3cb5
1 parent 90d064a
commit eef3cb5
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/src/Dfe.ContentSupport.Web/Models/Mapped/RichTextContentItem.cs b/src/Dfe.ContentSupport.Web/Models/Mapped/RichTextContentItem.cs
@@ -11,6 +11,6 @@ public class RichTextContentItem : CsContentItem
     public string? Value { get; set; } = null;
     public List<string> Tags { get; set; } = [];
 
-    public virtual bool HaveNoContent => Content.Count == 0 || Content.All(content => content.IsEmptyContent);
+    public virtual bool HaveNoContent => Content.Count == 0 || Content.TrueForAll(content => content.IsEmptyContent);
     public virtual bool IsEmptyContent => HaveNoContent && string.IsNullOrEmpty(Value);
 }
diff --git a/src/Dfe.ContentSupport.Web/Models/Mapped/Standard/EmbeddedEntry.cs b/src/Dfe.ContentSupport.Web/Models/Mapped/Standard/EmbeddedEntry.cs
@@ -16,6 +16,6 @@ public EmbeddedEntry()
     public RichTextContentItem? RichText { get; set; }
     public CustomComponent? CustomComponent { get; set; }
 
-    public override bool HaveNoContent => Content.Count == 0 || Content.All(content => content.IsEmptyContent);
-    public override bool IsEmptyContent => base.IsEmptyContent && (RichText == null || (RichText != null && RichText.IsEmptyContent));
+    public override bool HaveNoContent => Content.Count == 0 || Content.TrueForAll(content => content.IsEmptyContent);
+    public override bool IsEmptyContent => base.IsEmptyContent && (RichText == null || RichText.IsEmptyContent);
 }
diff --git a/src/Dfe.ContentSupport.Web/Views/Shared/RichText/_HyperLink.cshtml b/src/Dfe.ContentSupport.Web/Views/Shared/RichText/_HyperLink.cshtml
@@ -6,10 +6,10 @@
 }
 else
 {
-    var textContainsOpenInNewTab = @Model.Content[0].Value.ToLower().Contains("(opens in new tab)");
+    var textContainsOpenInNewTab = Model.Content[0].Value?.ToLower().Contains("(opens in new tab)") ?? false;
     var linkClass = textContainsOpenInNewTab ? "opens-in-new-tab" : "";
     if(textContainsOpenInNewTab){
-        Model.Content[0].Value =  @Model.Content[0].Value.Replace(" (opens in new tab)", "");
+        Model.Content[0].Value = Model.Content[0].Value?.Replace(" (opens in new tab)", "") ?? "";
     }
 
     <a href="@Model.Uri" target="_blank" rel="noopener noreferrer" class="@linkClass">@Model.Content[0].Value</a>