From f0816853da21126b7ee8f77cad2a817aa2a9a12f Mon Sep 17 00:00:00 2001
From: Aevann1 <randomname42029@gmail.com>
Date: Sat, 30 Jul 2022 00:43:25 +0200
Subject: [PATCH] make ban_reason logic a little better

---
 files/classes/user.py          | 3 ++-
 files/routes/admin.py          | 2 --
 files/templates/ban_modal.html | 2 +-
 files/templates/userpage.html  | 4 ++--
 4 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/files/classes/user.py b/files/classes/user.py
index 798717bb7c..6b09055353 100644
--- a/files/classes/user.py
+++ b/files/classes/user.py
@@ -662,7 +662,8 @@ class User(Base):
 		elif self.discord_id: remove_user(self)
 
 		self.is_banned = admin.id if admin else AUTOJANNY_ID
-		if reason: self.ban_reason = reason
+		if reason and len(reason) <= 256:
+			self.ban_reason = reason
 
 
 
diff --git a/files/routes/admin.py b/files/routes/admin.py
index 68e4c8826f..c58de5455a 100644
--- a/files/routes/admin.py
+++ b/files/routes/admin.py
@@ -1009,8 +1009,6 @@ def ban_user(user_id, v):
 	if reason.startswith("/") and '\\' not in reason: 
 		reason = f'<a href="{reason.split()[0]}">{reason}</a>'
 
-	if len(reason) > 256: reason = reason
-
 	user.ban(admin=v, reason=reason, days=days)
 
 	if request.values.get("alts"):
diff --git a/files/templates/ban_modal.html b/files/templates/ban_modal.html
index a5a9860cba..ce4b9ba937 100644
--- a/files/templates/ban_modal.html
+++ b/files/templates/ban_modal.html
@@ -15,7 +15,7 @@
 					<input type="hidden" name="formkey" value="{{v.formkey}}">
 
 					<label for="ban-modal-link">Public ban reason (optional)</label>
-					<textarea autocomplete="off" maxlength="64" name="reason" form="banModalForm" class="form-control" id="ban-modal-link" aria-label="With textarea" placeholder="Enter reason"></textarea>
+					<textarea autocomplete="off" maxlength="256" name="reason" form="banModalForm" class="form-control" id="ban-modal-link" aria-label="With textarea" placeholder="Enter reason"></textarea>
 
 					<label for="days" class="mt-3">Duration days</label>
 					<input autocomplete="off" type="number" step="any" name="days" id="days" class="form-control" placeholder="leave blank for permanent">
diff --git a/files/templates/userpage.html b/files/templates/userpage.html
index 65727fef00..dde645b45f 100644
--- a/files/templates/userpage.html
+++ b/files/templates/userpage.html
@@ -290,7 +290,7 @@
 							<form action="/ban_user/{{u.id}}" method="post" action="">
 								<input type="hidden" name="formkey" value="{{v.formkey}}">
 								<input type="hidden" name="redir" value="true">
-								<input autocomplete="off" style="font-size:11px" type="text" class="form-control" name="reason" placeholder="Ban Reason" oninput="document.getElementById('user-ban-submit').disabled=false">
+								<input autocomplete="off" style="font-size:11px" type="text" class="form-control" maxlength="256" name="reason" placeholder="Ban Reason" oninput="document.getElementById('user-ban-submit').disabled=false">
 								<input autocomplete="off" style="font-size:11px" type="number" step="any" class="form-control" name="days" placeholder="Days (blank = permanent)">
 								<div class="custom-control custom-checkbox">
 									<input autocomplete="off" type="checkbox" id="alts-2-desktop" class="custom-control-input" name="alts" value="1">
@@ -624,7 +624,7 @@
 							<form action="/ban_user/{{u.id}}/" method="post">
 								<input type="hidden" name="formkey" value="{{v.formkey}}">
 								<input type="hidden" name="redir" value="true">
-								<input autocomplete="off" style="font-size:11px" type="text" class="form-control" name="reason" placeholder="Ban Reason" oninput="document.getElementById('user-ban-submit2').disabled=false">
+								<input autocomplete="off" style="font-size:11px" type="text" class="form-control" maxlength="256" name="reason" placeholder="Ban Reason" oninput="document.getElementById('user-ban-submit2').disabled=false">
 								<input autocomplete="off" style="font-size:11px" type="number" step="any" class="form-control" name="days" placeholder="Days (blank = permanent)">
 								<br >
 								<div class="custom-control custom-checkbox">