From 66f214a51bf49b2018d3be9f4fc8f37845b1e914 Mon Sep 17 00:00:00 2001
From: Edward Betts <edward@4angle.com>
Date: Fri, 12 Nov 2021 12:17:15 +0000
Subject: [PATCH] Don't call session.commit in get_item function.

---
 matcher/api.py | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/matcher/api.py b/matcher/api.py
index 6a30f89..8841d1c 100644
--- a/matcher/api.py
+++ b/matcher/api.py
@@ -655,12 +655,7 @@ def find_osm_candidates(item, limit=80, max_distance=450, names=None):
 
 def get_item(item_id):
     item = model.Item.query.get(item_id)
-    if item:
-        return item
-    item = get_and_save_item(f"Q{item_id}")
-    database.session.add(item)
-    database.session.commit()
-    return item
+    return item or get_and_save_item(f"Q{item_id}")
 
 
 def get_item_street_addresses(item):