diff --git a/zone/trading.cpp b/zone/trading.cpp
index 3e7bb95c40..d6895c6c3b 100644
--- a/zone/trading.cpp
+++ b/zone/trading.cpp
@@ -881,7 +881,7 @@ void Client::FinishTrade(Mob* tradingWith, bool finalizer, void* event_entry, st
 					{"gold",     trade->gp},
 					{"platinum", trade->pp}
 				};
-
+				std::vector<const EQ::ItemInstance *> list(items.begin(), items.end());
 				for (EQ::ItemInstance *inst: items) {
 					if (!inst || !inst->GetItem()) {
 						continue;
@@ -889,13 +889,6 @@ void Client::FinishTrade(Mob* tradingWith, bool finalizer, void* event_entry, st
 
 					std::string item_id = fmt::format("{}", inst->GetItem()->ID);
 					handin[item_id] += inst->GetCharges();
-				}
-
-				std::vector<const EQ::ItemInstance *> list(items.begin(), items.end());
-				for (EQ::ItemInstance *inst: items) {
-					if (!inst || !inst->GetItem()) {
-						continue;
-					}
 					item_list.emplace_back(inst);
 				}