From 7b235e232d5c3c98856228a783e76d9a59ab5e8c Mon Sep 17 00:00:00 2001
From: Nikolai R Kristiansen <nikolaik@gmail.com>
Date: Sun, 5 Jan 2020 17:44:11 +0100
Subject: [PATCH] =?UTF-8?q?=F0=9F=90=9B=20Fix=20some=20bugs?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 fabfile.py                                                 | 1 +
 .../static/mailadmin/src/components/AliasListCreate.js     | 2 +-
 .../static/mailadmin/src/components/EmailsTextarea.js      | 7 +++++--
 mailadmin/static/mailadmin/src/components/MainArea.js      | 5 +++--
 4 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/fabfile.py b/fabfile.py
index 029dcd6..261dc66 100644
--- a/fabfile.py
+++ b/fabfile.py
@@ -14,6 +14,7 @@ def deploy(c):
 
     with c.cd(project_path), c.prefix('source {}/venv/bin/activate'.format(project_path)):
         c.run('git pull')  # Get source
+        c.run('pip install -U pip')
         c.run('pip install -r requirements.txt')  # install deps in virtualenv
         with c.cd('mailadmin/static/mailadmin'):  # install and compile frontend deps
             c.run('npm i')
diff --git a/mailadmin/static/mailadmin/src/components/AliasListCreate.js b/mailadmin/static/mailadmin/src/components/AliasListCreate.js
index 681c898..8228a02 100644
--- a/mailadmin/static/mailadmin/src/components/AliasListCreate.js
+++ b/mailadmin/static/mailadmin/src/components/AliasListCreate.js
@@ -71,7 +71,7 @@ const AliasListCreate = ({ domain, lists, orgUnits }) => {
               <span className="input-group-text">{`@${domain.name}`}</span>
             </div>
           </div>
-          <EmailsTextarea source={listName} domainId={domain.id} create={shouldCreate} />
+          <EmailsTextarea source={listName} domainId={domain.id} create={shouldCreate} setName={setName} />
         </div>
       </div>
       {/* <div className="new-list-result"></div> */}
diff --git a/mailadmin/static/mailadmin/src/components/EmailsTextarea.js b/mailadmin/static/mailadmin/src/components/EmailsTextarea.js
index 1784c2d..026222f 100644
--- a/mailadmin/static/mailadmin/src/components/EmailsTextarea.js
+++ b/mailadmin/static/mailadmin/src/components/EmailsTextarea.js
@@ -5,7 +5,7 @@ import { parseEmails, notify } from '../utils';
 import { CREATE_ALIASES_MUTATION } from '../mutations';
 import { GET_ALIASES } from '../queries';
 
-const EmailsTextarea = ({ source, domainId, create = false }) => {
+const EmailsTextarea = ({ source, domainId, create = false, setName = null }) => {
   const [rawAliases, setRawAliases] = useState('');
   const [createAliases] = useMutation(CREATE_ALIASES_MUTATION);
 
@@ -19,6 +19,9 @@ const EmailsTextarea = ({ source, domainId, create = false }) => {
         variables: { input },
         update: (cache, { data }) => {
           setRawAliases('');
+          if (setName) {
+            setName('');
+          }
           const { createAliasesResponse: newAliases } = data;
           const { aliases: aliasesFromCache } = cache.readQuery({ query: GET_ALIASES });
           cache.writeQuery({
@@ -35,7 +38,7 @@ const EmailsTextarea = ({ source, domainId, create = false }) => {
         console.error(errors);
       }
     },
-    [aliases, create, createAliases, domainId, source]
+    [aliases, create, createAliases, domainId, setName, source]
   );
   return (
     <>
diff --git a/mailadmin/static/mailadmin/src/components/MainArea.js b/mailadmin/static/mailadmin/src/components/MainArea.js
index 1f41b75..8e205c6 100644
--- a/mailadmin/static/mailadmin/src/components/MainArea.js
+++ b/mailadmin/static/mailadmin/src/components/MainArea.js
@@ -27,7 +27,8 @@ const MainArea = ({ query, selectedOrgUnit, orgUnits }) => {
     'source'
   );
 
-  const listPrefixes = selectedOrgUnit && orgUnits.find(({ id }) => id === selectedOrgUnit).prefixesRegex;
+  const selectedOrgUnitObj = selectedOrgUnit && orgUnits.find(({ id }) => id === selectedOrgUnit);
+  const listPrefixes = selectedOrgUnitObj && selectedOrgUnitObj.prefixesRegex;
   const listRegex = listPrefixes && new RegExp(listPrefixes);
 
   return (
@@ -42,7 +43,7 @@ const MainArea = ({ query, selectedOrgUnit, orgUnits }) => {
           }
           // Only render lists which matches the org units prefixes
           if (listRegex && !listRegex.test(list)) {
-            return null; // FIXME: use styles to show/hide ?
+            return null;
           }
           return <AliasList key={list} list={list} aliases={aliases} domain={domain} query={query} />;
         })}