From 0946a1497a0865185ad943d0d82a21f71e171d31 Mon Sep 17 00:00:00 2001
From: Josh Pigford <josh@joshpigford.com>
Date: Thu, 1 May 2025 19:07:50 -0500
Subject: [PATCH] Add conditional rendering for account links in transfers view

- Implemented checks for the existence of accounts before rendering links in the transfers partial.
- Added error messaging for missing accounts to improve user feedback and prevent broken links.
---
 app/views/transfers/_account_links.html.erb | 22 +++++++++++++++++++--
 1 file changed, 20 insertions(+), 2 deletions(-)

diff --git a/app/views/transfers/_account_links.html.erb b/app/views/transfers/_account_links.html.erb
index 33390b31..738416c6 100644
--- a/app/views/transfers/_account_links.html.erb
+++ b/app/views/transfers/_account_links.html.erb
@@ -1,7 +1,25 @@
 <%# locals: (transfer:, is_inflow: false) %>
 <div class="flex items-center gap-1">
   <% first_account, second_account = is_inflow ? [transfer.to_account, transfer.from_account] : [transfer.from_account, transfer.to_account] %>
-  <%= link_to first_account.name, account_path(first_account, tab: "activity"), class: "hover:underline", data: { turbo_frame: "_top" } %>
+
+  <%# Check if first_account exists before creating link %>
+  <% if first_account %>
+    <%= link_to first_account.name, account_path(first_account, tab: "activity"), class: "hover:underline", data: { turbo_frame: "_top" } %>
+  <% else %>
+    <span class="text-warning text-xs italic" title="Transfer ID: <%= transfer.id %>">
+      Data Error: Missing account
+    </span>
+  <% end %>
+
+  <%# Use icon helper per conventions %>
   <%= icon(is_inflow ? "arrow-left" : "arrow-right", size: "sm") %>
-  <%= link_to second_account.name, account_path(second_account, tab: "activity"), class: "hover:underline", data: { turbo_frame: "_top" } %>
+
+  <%# Check if second_account exists before creating link %>
+  <% if second_account %>
+    <%= link_to second_account.name, account_path(second_account, tab: "activity"), class: "hover:underline", data: { turbo_frame: "_top" } %>
+  <% else %>
+    <span class="text-warning text-xs italic" title="Transfer ID: <%= transfer.id %>">
+      Data Error: Missing account
+    </span>
+  <% end %>
 </div>