From 49f7a9e4d5f63740212540c5a486004a58b0ed44 Mon Sep 17 00:00:00 2001
From: SammCheese <sammy@sammcheese.net>
Date: Sun, 12 Feb 2023 22:29:27 +0100
Subject: [PATCH] minor code optimizations

---
 src/components/Settings.tsx | 5 ++++-
 src/utils.ts                | 4 ++--
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/src/components/Settings.tsx b/src/components/Settings.tsx
index 9e63a46..a436986 100644
--- a/src/components/Settings.tsx
+++ b/src/components/Settings.tsx
@@ -28,7 +28,10 @@ export function Settings() {
         placeholder="password, placeholder, test"
         value={passwords.join(", ")}
         onChange={(e: string) => {
-          InvSettings.set("passwords", e.split(", "));
+          InvSettings.set(
+            "passwords",
+            e.split(",").map((s) => s.trim()),
+          );
           setPasswords(e.split(", "));
         }}
       />
diff --git a/src/utils.ts b/src/utils.ts
index 1a59d34..3fae291 100644
--- a/src/utils.ts
+++ b/src/utils.ts
@@ -146,12 +146,12 @@ export function isCorrectPassword(result: string): boolean {
 // eslint-disable-next-line @typescript-eslint/require-await
 export async function iteratePasswords(message: DiscordMessage): Promise<string | false> {
   const passwords = InvSettings.get("passwords", []);
-  if (!message || !passwords || !passwords.length) return false;
+  if (!message?.content || !passwords?.length) return false;
 
   let { content } = message;
 
   // we use an extra variable so we dont have to edit the message content directly
-  if (message.content.match(/^\W/)) content = `d ${message.content}d`;
+  if (/^\W/.test(message.content)) content = `d ${message.content}d`;
 
   for (let i = 0; i < passwords.length; i++) {
     const result = decrypt(content, passwords[i], false);