From a9bf3179f189a17d8d2cf85abb57c2d88b347107 Mon Sep 17 00:00:00 2001
From: some weird guy <120821766+Mistake35@users.noreply.github.com>
Date: Mon, 4 Sep 2023 21:17:50 -0700
Subject: [PATCH] bruh

- Fixed duplicate image uploading bug (i think)
- changing community settings now redirects you, same with creating a community.
---
 closedverse_main/forms.py | 15 ++++++++++++---
 closedverse_main/views.py |  4 ++--
 static/closedverse.js     | 17 +++++++++--------
 3 files changed, 23 insertions(+), 13 deletions(-)

diff --git a/closedverse_main/forms.py b/closedverse_main/forms.py
index 01d8613..872a591 100644
--- a/closedverse_main/forms.py
+++ b/closedverse_main/forms.py
@@ -50,15 +50,24 @@ def compress_and_resize_content(image):
 # I do want to move each and every form over to here. Not only will this trivialize making new forms, this will also make it more secure or something.
 class CommunitySettingForm(forms.ModelForm):
 	description	 = forms.CharField(max_length = 2200,required=False, widget=forms.Textarea(attrs={'class': 'textarea'}))
+		
+	def __init__(self, *args, **kwargs):
+		super(CommunitySettingForm, self).__init__(*args, **kwargs)
+		# Store the initial values of the image fields
+		self.initial_ico = self.instance.ico
+		self.initial_banner = self.instance.banner
+
 	def clean_ico(self):
 		ico = self.cleaned_data.get('ico')
-		if ico:
+		# Check if the image has changed
+		if ico and ico != self.initial_ico:
 			return compress_and_resize_icon(image=ico)
 		return ico
 
-	def clean_banner(self):	
+	def clean_banner(self):    
 		banner = self.cleaned_data.get('banner')
-		if banner:
+		# Check if the image has changed
+		if banner and banner != self.initial_banner:
 			return compress_and_resize_content(image=banner)
 		return banner
 
diff --git a/closedverse_main/views.py b/closedverse_main/views.py
index d337cdf..fee0ee9 100644
--- a/closedverse_main/views.py
+++ b/closedverse_main/views.py
@@ -1013,7 +1013,7 @@ def community_tools_set(request, community):
 			return json_response(form.errors.as_text())
 		form.save()
 		AuditLog.objects.create(type=4, community=the_community, user=the_community.creator, by=request.user)
-		return HttpResponse()
+		return redirect(reverse('main:community-view', args=[the_community.id]))
 	else:
 		raise Http404()
 		
@@ -1042,7 +1042,7 @@ def community_create_action(request):
 		community.type = 3
 		community.creator = request.user
 		community.save()
-		return json_response('Community has been created, check the front page!', 'Done')
+		return redirect('/')
 	else:
 		raise Http404()
 @login_required
diff --git a/static/closedverse.js b/static/closedverse.js
index aff7c8a..7bf164a 100644
--- a/static/closedverse.js
+++ b/static/closedverse.js
@@ -3606,11 +3606,12 @@ mode_post = 0;
 	*/
 	b.router.connect("^/communities/[0-9]+/(tools)$", function(c, d, e) {
         function f(c) {
-            var d = a(this)
-              , e = d.closest("form");
+            var d = a(this),
+                e = d.closest("form");
             b.Form.isDisabled(d) || c.isDefaultPrevented() || (c.preventDefault(),
             b.Form.submit(e, d).done(function(a) {
-                b.Net.reload()
+                var communityId = window.location.pathname.split('/')[2];
+                window.location = "/communities/" + communityId;
             }))
         }
         a(document).on("click", ".apply-button", f),
@@ -3620,13 +3621,13 @@ mode_post = 0;
     }),
 	b.router.connect("^/c/create$", function(c, d, e) {
         function f(c) {
-            var d = a(this)
-              , e = d.closest("form");
+            var d = a(this),
+                e = d.closest("form");
             b.Form.isDisabled(d) || c.isDefaultPrevented() || (c.preventDefault(),
             b.Form.submit(e, d).done(function(a) {
-                b.Net.reload()
-            }))
-        }
+                window.location = "/";
+             }))
+         }
         a(document).on("click", ".apply-button", f),
         e.done(function() {
             a(document).off("click", ".apply-button", f)