From b64d2b7478a9b1db8b4ce0bf5a6cead4df592ca2 Mon Sep 17 00:00:00 2001
From: TekH <h.tek@digitaldata.works>
Date: Fri, 6 Mar 2026 09:50:33 +0100
Subject: [PATCH] Refactor: cache IsReadAndConfirm() result in variable

Store envelope.IsReadAndConfirm() in isReadAndConfirm variable to avoid redundant calls and improve code readability when setting ViewData["Title"]. No change to logic or behavior.
---
 EnvelopeGenerator.Web/Views/Envelope/ShowEnvelope.cshtml | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/EnvelopeGenerator.Web/Views/Envelope/ShowEnvelope.cshtml b/EnvelopeGenerator.Web/Views/Envelope/ShowEnvelope.cshtml
index 6e850e11..7bdf3fd1 100644
--- a/EnvelopeGenerator.Web/Views/Envelope/ShowEnvelope.cshtml
+++ b/EnvelopeGenerator.Web/Views/Envelope/ShowEnvelope.cshtml
@@ -25,9 +25,11 @@
     if (ViewData["IsReadOnly"] is bool isReadOnly_bool)
         isReadOnly = isReadOnly_bool;
 
+    var isReadAndConfirm = envelope.IsReadAndConfirm();
+
     ViewData["Title"] = isReadOnly
         ? _localizer.ViewDoc()
-        : envelope.IsReadAndConfirm()
+        : isReadAndConfirm
             ? _localizer.ConfirmDoc()
             : _localizer.SignDoc();
 }