From 1f76b16e7558f6055f3de63110d807d15504e012 Mon Sep 17 00:00:00 2001
From: Romein van Buren <romein@vburen.nl>
Date: Tue, 21 Feb 2023 08:14:52 +0100
Subject: [PATCH] Rename collection through zenity dialog

---
 .../src/organisms/connection/index.svelte     | 74 +++----------------
 frontend/wailsjs/go/app/App.d.ts              |  2 +-
 frontend/wailsjs/go/app/App.js                |  4 +-
 internal/app/app.go                           |  5 +-
 4 files changed, 16 insertions(+), 69 deletions(-)

diff --git a/frontend/src/organisms/connection/index.svelte b/frontend/src/organisms/connection/index.svelte
index dbb7a83..0cfad70 100644
--- a/frontend/src/organisms/connection/index.svelte
+++ b/frontend/src/organisms/connection/index.svelte
@@ -1,7 +1,4 @@
 <script>
-  import Icon from '$components/icon.svelte';
-  import Modal from '$components/modal.svelte';
-  import input from '$lib/actions/input';
   import busy from '$lib/stores/busy';
   import { connections } from '$lib/stores/connections';
   import { EnterText, Hosts, RenameCollection } from '$wails/go/app/App';
@@ -18,13 +15,8 @@
   export let activeCollKey = '';
 
   let hostTree;
-
   let showHostDetail = false;
   let hostDetailKey = '';
-
-  let collToRename = '';
-  let newCollKey = '';
-
   let exportInfo;
 
   async function getHosts() {
@@ -48,21 +40,17 @@
     }
   }
 
-  function openEditCollModal(collKey) {
-    newCollKey = collKey;
-    collToRename = collKey;
-  }
-
-  async function renameCollection() {
-    busy.start();
-    const ok = await RenameCollection(activeHostKey, activeDbKey, collToRename, newCollKey);
-    if (ok) {
-      activeCollKey = newCollKey;
-      collToRename = '';
-      newCollKey = '';
-      await hostTree.reload();
+  async function renameCollection(oldCollKey) {
+    const newCollKey = await EnterText('Rename collection', `Enter a new name for collection ${oldCollKey}.`, oldCollKey);
+    if (newCollKey && (newCollKey !== oldCollKey)) {
+      busy.start();
+      const ok = await RenameCollection(activeHostKey, activeDbKey, oldCollKey, newCollKey);
+      if (ok) {
+        activeCollKey = newCollKey;
+        await hostTree.reload();
+      }
+      busy.end();
     }
-    busy.end();
   }
 
   async function createCollection() {
@@ -109,7 +97,7 @@
     on:newDatabase={createDatabase}
     on:newCollection={createCollection}
     on:editHost={e => editHost(e.detail)}
-    on:renameCollection={e => openEditCollModal(e.detail)}
+    on:renameCollection={e => renameCollection(e.detail)}
     on:exportCollection={e => exportCollection(e.detail)}
     on:dumpCollection={e => dumpCollection(e.detail)}
   />
@@ -132,49 +120,9 @@
 
 <Export bind:info={exportInfo} {hosts} />
 
-{#if collToRename}
-  <Modal bind:show={collToRename} width="400px">
-    <form class="rename" on:submit|preventDefault={renameCollection}>
-      <div>Renaming collection <strong>{collToRename}</strong></div>
-      <Icon name="arr-d" />
-      <label class="field">
-        <input type="text" bind:value={newCollKey} use:input={{ autofocus: true }} spellcheck="false" />
-      </label>
-      <div class="cancelorsave">
-        <button class="btn secondary" type="button" on:click={() => collToRename = ''}>Cancel</button>
-        <button class="btn" type="submit">Save</button>
-      </div>
-    </form>
-  </Modal>
-{/if}
-
 <style>
   .tree {
     padding: 0.5rem;
     background-color: #fff;
   }
-
-  .rename {
-    text-align: center;
-    display: flex;
-    flex-direction: column;
-    gap: 0.5rem;
-    align-items: center;
-  }
-  .rename .field {
-    width: 100%;
-  }
-  .rename input {
-    text-align: center;
-    width: 100%;
-  }
-  .rename strong {
-    font-weight: 700;
-  }
-  .rename .cancelorsave {
-    display: flex;
-    gap: 0.5rem;
-    justify-content: space-between;
-    width: 100%;
-  }
 </style>
diff --git a/frontend/wailsjs/go/app/App.d.ts b/frontend/wailsjs/go/app/App.d.ts
index f0e303b..0a1d6e5 100755
--- a/frontend/wailsjs/go/app/App.d.ts
+++ b/frontend/wailsjs/go/app/App.d.ts
@@ -19,7 +19,7 @@ export function DropDatabase(arg1:string,arg2:string):Promise<boolean>;
 
 export function DropIndex(arg1:string,arg2:string,arg3:string,arg4:string):Promise<boolean>;
 
-export function EnterText(arg1:string,arg2:string):Promise<string>;
+export function EnterText(arg1:string,arg2:string,arg3:string):Promise<string>;
 
 export function Environment():Promise<app.EnvironmentInfo>;
 
diff --git a/frontend/wailsjs/go/app/App.js b/frontend/wailsjs/go/app/App.js
index e4092d6..c3f50d9 100755
--- a/frontend/wailsjs/go/app/App.js
+++ b/frontend/wailsjs/go/app/App.js
@@ -30,8 +30,8 @@ export function DropIndex(arg1, arg2, arg3, arg4) {
   return window['go']['app']['App']['DropIndex'](arg1, arg2, arg3, arg4);
 }
 
-export function EnterText(arg1, arg2) {
-  return window['go']['app']['App']['EnterText'](arg1, arg2);
+export function EnterText(arg1, arg2, arg3) {
+  return window['go']['app']['App']['EnterText'](arg1, arg2, arg3);
 }
 
 export function Environment() {
diff --git a/internal/app/app.go b/internal/app/app.go
index 186b085..f5b28a7 100644
--- a/internal/app/app.go
+++ b/internal/app/app.go
@@ -167,9 +167,8 @@ func (a *App) OpenDirectory(id, title string) string {
 	return dir
 }
 
-func (a *App) EnterText(title, info string) string {
-	println("wertyuijhgfd")
-	input, err := zenity.Entry(info, zenity.Title(title))
+func (a *App) EnterText(title, info, defaultEntry string) string {
+	input, err := zenity.Entry(info, zenity.Title(title), zenity.EntryText(defaultEntry))
 
 	if err == zenity.ErrCanceled {
 		return ""