#4770: Don't crash on deleted item for event history

Author: dorner

app/views/events/_snapshot_event_row.html.erb

@@ -11,7 +11,7 @@
       <%= link_to storage_locs.find { |i| i.id == loc.id}.name, storage_location_path(loc.id) %>
     <td>
       <% loc.items.values.each do |entry| %>
-        <%= link_to items.find { |i| i.id == entry.item_id}.name, item_path(entry.item_id) %>:
+        <%= link_to items.find { |i| i.id == entry.item_id}&.name || "Item #{entry.item_id} (deleted)", item_path(entry.item_id) %>:
         <%= entry.quantity %><br>
       <% end %>
     </td>

spec/requests/events_requests_spec.rb

@@ -47,6 +47,44 @@
         expect(response.body).not_to include("99<br>")
       end
 
+      it "should show deleted items without crashing" do
+        deleted_item = create(:item, organization: organization)
+        travel(-1.day) do
+          SnapshotEvent.create!(
+            eventable: organization,
+            organization_id: organization.id,
+            event_time: Time.zone.now,
+            data: EventTypes::Inventory.new(
+              organization_id: organization.id,
+              storage_locations: {
+                storage_location.id => EventTypes::EventStorageLocation.new(
+                  id: storage_location.id,
+                  items: {
+                    item.id => EventTypes::EventItem.new(item_id: item.id, quantity: 0),
+                    item2.id => EventTypes::EventItem.new(item_id: item2.id, quantity: 0),
+                    deleted_item.id => EventTypes::EventItem.new(item_id: deleted_item.id, quantity: 0)
+                  }
+                ),
+                storage_location2.id => EventTypes::EventStorageLocation.new(
+                  id: storage_location2.id,
+                  items: {
+                    item.id => EventTypes::EventItem.new(item_id: item.id, quantity: 0),
+                    item2.id => EventTypes::EventItem.new(item_id: item2.id, quantity: 0),
+                    deleted_item.id => EventTypes::EventItem.new(item_id: deleted_item.id, quantity: 0)
+                  }
+                )
+              }
+            )
+          )
+        end
+        deleted_id = deleted_item.id
+        deleted_item.destroy
+        subject
+        expect(response.body).to include("Item1</a>")
+        expect(response.body).to include("Item2</a>")
+        expect(response.body).to include("Item #{deleted_id} (deleted)")
+      end
+
       context "with type filter" do
         let(:params) { {format: "html", filters: {by_type: "DonationEvent"}} }