From ee541d567f6df7c5c43db9a60edbaabd1158d695 Mon Sep 17 00:00:00 2001
From: jessikitty <jess.rogerson.29@outlook.com>
Date: Wed, 22 Apr 2026 09:51:34 +1000
Subject: [PATCH] Downgrade to C# 5 - remove interpolation, null-conditional,
 expression-bodied members

---
 WebHandler/ADCompareWebHandler.cs | 55 ++++++++++++++++++-------------
 1 file changed, 33 insertions(+), 22 deletions(-)

diff --git a/WebHandler/ADCompareWebHandler.cs b/WebHandler/ADCompareWebHandler.cs
index 090f3f5..723f6a0 100644
--- a/WebHandler/ADCompareWebHandler.cs
+++ b/WebHandler/ADCompareWebHandler.cs
@@ -12,9 +12,9 @@ namespace Disco.Plugins.ADCompare.WebHandler
     {
         public override ActionResult ExecuteAction(string ActionName)
         {
-            switch (ActionName?.ToLower())
+            var action = ActionName != null ? ActionName.ToLower() : "";
+            switch (action)
             {
-                case null:
                 case "":
                 case "index":
                     return Index();
@@ -58,28 +58,39 @@ namespace Disco.Plugins.ADCompare.WebHandler
             foreach (var r in summary.Results.Where(r => !r.IsMatch))
             {
                 sb.AppendLine(string.Join(",",
-                    Csv(r.SerialNumber), Csv(r.ComputerName),
-                    Csv(r.DiscoAssignedUserId), Csv(r.DiscoAssignedUserDisplayName),
-                    Csv(r.ADManagedByUserId), Csv(r.ADManagedByDisplayName),
-                    r.IsMatch.ToString(), Csv(r.MismatchReason)));
+                    CsvEscape(r.SerialNumber), CsvEscape(r.ComputerName),
+                    CsvEscape(r.DiscoAssignedUserId), CsvEscape(r.DiscoAssignedUserDisplayName),
+                    CsvEscape(r.ADManagedByUserId), CsvEscape(r.ADManagedByDisplayName),
+                    r.IsMatch.ToString(), CsvEscape(r.MismatchReason)));
             }
+            var fileName = "AD_ManagedBy_Compare_" + DateTime.Now.ToString("yyyyMMdd_HHmmss") + ".csv";
             HostController.Response.Headers.Add("Content-Disposition",
-                $"attachment; filename=\"AD_ManagedBy_Compare_{DateTime.Now:yyyyMMdd_HHmmss}.csv\"");
-            return new ContentResult { Content = sb.ToString(), ContentType = "text/csv", ContentEncoding = Encoding.UTF8 };
+                "attachment; filename=\"" + fileName + "\"");
+            return new ContentResult
+            {
+                Content = sb.ToString(),
+                ContentType = "text/csv",
+                ContentEncoding = Encoding.UTF8
+            };
         }
 
-        private string Csv(string v) => $"\"{(v ?? "").Replace("\"", "\"\"")}\"";
+        private string CsvEscape(string v)
+        {
+            if (v == null) v = "";
+            return "\"" + v.Replace("\"", "\"\"") + "\"";
+        }
 
         private string BuildDashboardHtml()
         {
             var pluginUrl = Url.Action(MVC.API.Disco.Plugin.PluginWebAction(Manifest.Id, null));
-            return $@"
+
+            return @"
 <div id='adcompare'>
     <h2><i class='fa fa-exchange'></i> AD Compare &mdash; Device Managed By</h2>
     <p>Compares the AD computer <strong>Managed By</strong> field against the Disco <strong>Assigned User</strong>.</p>
     <div style='margin:15px 0;'>
         <button id='btnRun' class='btn btn-primary' onclick='runCompare()'><i class='fa fa-refresh'></i> Run Comparison</button>
-        <button id='btnExport' class='btn btn-default' onclick='location.href=""{pluginUrl}/Export""' style='margin-left:10px;' disabled><i class='fa fa-download'></i> Export Mismatches CSV</button>
+        <button id='btnExport' class='btn btn-default' onclick='location.href=""" + pluginUrl + @"/Export""' style='margin-left:10px;' disabled><i class='fa fa-download'></i> Export Mismatches CSV</button>
         <span id='loading' style='display:none;margin-left:15px;'><i class='fa fa-spinner fa-spin'></i> Querying AD for all devices&hellip;</span>
     </div>
     <div id='summary' style='display:none;margin:15px 0;padding:15px;background:#f5f5f5;border-radius:4px;'>
@@ -106,30 +117,30 @@ namespace Disco.Plugins.ADCompare.WebHandler
 </div>
 <script>
 var data=[];
-function runCompare(){{
+function runCompare(){
     $('#loading').show();$('#btnRun').prop('disabled',true);$('#tbl').hide();
-    $.getJSON('{pluginUrl}/Compare',function(d){{
+    $.getJSON('" + pluginUrl + @"/Compare',function(d){
         data=d.Results;
         $('#sTotal').text(d.TotalDevices);$('#sMatch').text(d.DevicesMatched);
         $('#sMismatch').text(d.DevicesMismatched);$('#sNotAD').text(d.DevicesNotInAD);
         $('#sNoMgr').text(d.DevicesNoManagedBy);$('#sNoAssign').text(d.DevicesNoAssignment);
         $('#summary').show();$('#btnExport').prop('disabled',false);
         renderTable();$('#tbl').show();
-    }}).fail(function(x){{alert('Error: '+x.statusText);}})
-    .always(function(){{$('#loading').hide();$('#btnRun').prop('disabled',false);}});
-}}
-function renderTable(){{
+    }).fail(function(x){alert('Error: '+x.statusText);})
+    .always(function(){$('#loading').hide();$('#btnRun').prop('disabled',false);});
+}
+function renderTable(){
     var mo=$('#chkMismatchOnly').is(':checked'),q=$('#txtFilter').val().toLowerCase(),tb=$('#tbody');tb.empty();
-    data.forEach(function(r){{
+    data.forEach(function(r){
         if(mo&&r.IsMatch)return;
-        if(q){{var hay=(r.SerialNumber+' '+r.ComputerName+' '+r.DiscoAssignedUserId+' '+r.ADManagedByUserId+' '+(r.DiscoAssignedUserDisplayName||'')+' '+(r.ADManagedByDisplayName||'')).toLowerCase();if(hay.indexOf(q)<0)return;}}
+        if(q){var hay=(r.SerialNumber+' '+r.ComputerName+' '+r.DiscoAssignedUserId+' '+r.ADManagedByUserId+' '+(r.DiscoAssignedUserDisplayName||'')+' '+(r.ADManagedByDisplayName||'')).toLowerCase();if(hay.indexOf(q)<0)return;}
         var icon=r.IsMatch?'<i class=""fa fa-check text-success""></i>':(!r.FoundInAD?'<i class=""fa fa-times text-danger""></i>':'<i class=""fa fa-exclamation-triangle text-warning""></i>');
         var discoUser=r.DiscoAssignedUserId?(esc(r.DiscoAssignedUserId)+(r.DiscoAssignedUserDisplayName?'<br/><small class=""text-muted"">'+esc(r.DiscoAssignedUserDisplayName)+'</small>':'')):'<em class=""text-muted"">Not assigned</em>';
         var adUser=r.ADManagedByUserId?(esc(r.ADManagedByUserId)+(r.ADManagedByDisplayName?'<br/><small class=""text-muted"">'+esc(r.ADManagedByDisplayName)+'</small>':'')):(r.FoundInAD?'<em class=""text-muted"">Empty</em>':'<em class=""text-muted"">N/A</em>');
         tb.append('<tr class=""'+(r.IsMatch?'':'warning')+'"">'+'<td>'+icon+'</td><td>'+esc(r.SerialNumber)+'</td><td>'+esc(r.ComputerName||'')+'</td><td>'+discoUser+'</td><td>'+adUser+'</td><td>'+esc(r.MismatchReason||'')+'</td></tr>');
-    }});
-}}
-function esc(t){{if(!t)return'';var d=document.createElement('div');d.appendChild(document.createTextNode(t));return d.innerHTML;}}
+    });
+}
+function esc(t){if(!t)return'';var d=document.createElement('div');d.appendChild(document.createTextNode(t));return d.innerHTML;}
 </script>";
         }
     }