From c24e592e3e6ad2311818b099af8662e803ab3873 Mon Sep 17 00:00:00 2001
From: justcool393 <justcool393@gmail.com>
Date: Fri, 9 Dec 2022 20:44:34 -0600
Subject: [PATCH] fix propertey and api

---
 files/classes/sub.py | 2 +-
 files/routes/subs.py | 5 ++---
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/files/classes/sub.py b/files/classes/sub.py
index 2cb6db151..7df62c48e 100644
--- a/files/classes/sub.py
+++ b/files/classes/sub.py
@@ -44,7 +44,7 @@ class Sub(Base):
 	@property
 	@lazy
 	def banner_urls(self):
-		if self.bannerurls: return [f"{SITE_FULL}{banner}" for banner in self.bannerurl]
+		if self.bannerurls: return [f"{SITE_FULL}{banner}" for banner in self.bannerurls]
 		return []
 
 	@lazy
diff --git a/files/routes/subs.py b/files/routes/subs.py
index f1f4acf02..fe6edd478 100644
--- a/files/routes/subs.py
+++ b/files/routes/subs.py
@@ -463,9 +463,8 @@ def upload_sub_banner(v:User, sub:str):
 	bannerurl = process_image(name, v, resize=1200)
 
 	if bannerurl:
-		if sub.bannerurl:
-			os.remove(sub.bannerurl)
-		sub.bannerurl = bannerurl
+		if sub.bannerurls:
+			sub.bannerurls.add(bannerurl)
 		g.db.add(sub)
 
 	ma = SubAction(