Merge pull request #1049 from TransactionProcessing/copilot/codacy-issue-d9c3f423d4bb9476d626d1e156475960-redu

Reduce cyclomatic complexity of `OnPost` in Ciba/Consent.cshtml.cs

Author: StuartFerguson

SecurityService.UserInterface/Pages/Ciba/Consent.cshtml.cs

@@ -100,19 +100,7 @@ public async Task<IActionResult> OnPost()
             return RedirectToPage("/Home/Error/Index");
         }
 
-        CompleteBackchannelLoginRequest result = null;
-
-        // user clicked 'no' - send back the standard 'access_denied' response
-        if (Input?.Button == "no") {
-            result = await HandleNoButtonClicked(request);
-        }
-        // user clicked 'yes' - validate the data
-        else if (Input?.Button == "yes") {
-            result = await HandleYesButtonClicked(request);
-        }
-        else {
-            ModelState.AddModelError("", ConsentOptions.InvalidSelectionErrorMessage);
-        }
+        CompleteBackchannelLoginRequest result = await HandleButtonClickAsync(request);
 
         if (result != null) {
             // communicate outcome of consent back to identityserver
@@ -125,6 +113,20 @@ public async Task<IActionResult> OnPost()
         return Page();
     }
 
+    private async Task<CompleteBackchannelLoginRequest> HandleButtonClickAsync(BackchannelUserLoginRequest request)
+    {
+        switch (Input?.Button)
+        {
+            case "no":
+                return await HandleNoButtonClicked(request);
+            case "yes":
+                return await HandleYesButtonClicked(request);
+            default:
+                ModelState.AddModelError("", ConsentOptions.InvalidSelectionErrorMessage);
+                return null;
+        }
+    }
+
     private async Task<ViewModel> BuildViewModelAsync(string id, InputModel model = null)
     {
         var request = await _interaction.GetLoginRequestByInternalIdAsync(id);