From 80161e1b9130185e656ed2865488d9c732850129 Mon Sep 17 00:00:00 2001
From: Stanislav Romanov <kaermorchen@gmail.com>
Date: Thu, 1 Sep 2022 13:02:37 +0300
Subject: [PATCH] Move edit form to edit route

---
 app/components/icons/file-document-outline.js   |  1 +
 .../admin/indexes/item/documents/document.js    | 13 +------------
 .../indexes/item/documents/document/edit.js     | 14 ++++++++++++++
 .../indexes/item/documents/document/index.js    |  3 +++
 app/router.js                                   |  4 +++-
 .../indexes/item/documents/document/edit.js     | 12 ++++++++++++
 .../indexes/item/documents/document/index.js    |  3 +++
 .../admin/indexes/item/documents/document.hbs   |  6 +-----
 .../indexes/item/documents/document/edit.hbs    |  5 +++++
 .../indexes/item/documents/document/index.hbs   |  7 +++++++
 .../admin/indexes/item/documents/index.hbs      |  8 ++++++++
 .../item/documents/document/edit-test.js        | 17 +++++++++++++++++
 .../item/documents/document/index-test.js       | 17 +++++++++++++++++
 .../item/documents/document/edit-test.js        | 16 ++++++++++++++++
 .../item/documents/document/index-test.js       | 16 ++++++++++++++++
 15 files changed, 124 insertions(+), 18 deletions(-)
 create mode 100644 app/components/icons/file-document-outline.js
 create mode 100644 app/controllers/admin/indexes/item/documents/document/edit.js
 create mode 100644 app/controllers/admin/indexes/item/documents/document/index.js
 create mode 100644 app/routes/admin/indexes/item/documents/document/edit.js
 create mode 100644 app/routes/admin/indexes/item/documents/document/index.js
 create mode 100644 app/templates/admin/indexes/item/documents/document/edit.hbs
 create mode 100644 app/templates/admin/indexes/item/documents/document/index.hbs
 create mode 100644 tests/unit/controllers/admin/indexes/item/documents/document/edit-test.js
 create mode 100644 tests/unit/controllers/admin/indexes/item/documents/document/index-test.js
 create mode 100644 tests/unit/routes/admin/indexes/item/documents/document/edit-test.js
 create mode 100644 tests/unit/routes/admin/indexes/item/documents/document/index-test.js

diff --git a/app/components/icons/file-document-outline.js b/app/components/icons/file-document-outline.js
new file mode 100644
index 0000000..237a50d
--- /dev/null
+++ b/app/components/icons/file-document-outline.js
@@ -0,0 +1 @@
+export { FileDocumentOutline as default } from 'ember-mdi';
diff --git a/app/controllers/admin/indexes/item/documents/document.js b/app/controllers/admin/indexes/item/documents/document.js
index 5b293f6..85f9c80 100644
--- a/app/controllers/admin/indexes/item/documents/document.js
+++ b/app/controllers/admin/indexes/item/documents/document.js
@@ -1,14 +1,3 @@
 import Controller from '@ember/controller';
-import { action } from '@ember/object';
 
-export default class AdminIndexesItemDocumentsDocumentController extends Controller {
-  @action
-  save(event) {
-    event.preventDefault();
-
-    const formData = new FormData(event.currentTarget);
-    const data = formData.get('document');
-
-    this.model.index.saveDocument(data);
-  }
-}
+export default class AdminIndexesItemDocumentsDocumentController extends Controller {}
diff --git a/app/controllers/admin/indexes/item/documents/document/edit.js b/app/controllers/admin/indexes/item/documents/document/edit.js
new file mode 100644
index 0000000..c57f5d3
--- /dev/null
+++ b/app/controllers/admin/indexes/item/documents/document/edit.js
@@ -0,0 +1,14 @@
+import Controller from '@ember/controller';
+import { action } from '@ember/object';
+
+export default class AdminIndexesItemDocumentsDocumentEditController extends Controller {
+  @action
+  save(event) {
+    event.preventDefault();
+
+    const formData = new FormData(event.currentTarget);
+    const data = formData.get('document');
+
+    this.model.index.saveDocument(data);
+  }
+}
diff --git a/app/controllers/admin/indexes/item/documents/document/index.js b/app/controllers/admin/indexes/item/documents/document/index.js
new file mode 100644
index 0000000..c11fed5
--- /dev/null
+++ b/app/controllers/admin/indexes/item/documents/document/index.js
@@ -0,0 +1,3 @@
+import Controller from '@ember/controller';
+
+export default class AdminIndexesItemDocumentsDocumentIndexController extends Controller {}
diff --git a/app/router.js b/app/router.js
index 76296b9..5a34f09 100644
--- a/app/router.js
+++ b/app/router.js
@@ -14,7 +14,9 @@ Router.map(function () {
         this.route('info');
         this.route('edit');
         this.route('documents', function () {
-          this.route('document', { path: '/:document_id' });
+          this.route('document', { path: '/:document_id' }, function () {
+            this.route('edit');
+          });
           this.route('new');
         });
       });
diff --git a/app/routes/admin/indexes/item/documents/document/edit.js b/app/routes/admin/indexes/item/documents/document/edit.js
new file mode 100644
index 0000000..9d35917
--- /dev/null
+++ b/app/routes/admin/indexes/item/documents/document/edit.js
@@ -0,0 +1,12 @@
+import Route from '@ember/routing/route';
+// import { hash } from 'rsvp';
+
+export default class AdminIndexesItemDocumentsDocumentEditRoute extends Route {
+  // model({ document_id }) {
+  //   const index = this.modelFor('admin.indexes.item');
+  //   return hash({
+  //     index,
+  //     document: index.getDocument(document_id),
+  //   });
+  // }
+}
diff --git a/app/routes/admin/indexes/item/documents/document/index.js b/app/routes/admin/indexes/item/documents/document/index.js
new file mode 100644
index 0000000..1a56359
--- /dev/null
+++ b/app/routes/admin/indexes/item/documents/document/index.js
@@ -0,0 +1,3 @@
+import Route from '@ember/routing/route';
+
+export default class AdminIndexesItemDocumentsDocumentIndexRoute extends Route {}
diff --git a/app/templates/admin/indexes/item/documents/document.hbs b/app/templates/admin/indexes/item/documents/document.hbs
index f65b58a..c24cd68 100644
--- a/app/templates/admin/indexes/item/documents/document.hbs
+++ b/app/templates/admin/indexes/item/documents/document.hbs
@@ -1,5 +1 @@
-<form {{on 'submit' this.save}}>
-  <InputTextarea @name='document' @value={{stringify @model.document}} />
-
-  <Button type='submit' @text='Save' @style='primary' />
-</form>
+{{outlet}}
diff --git a/app/templates/admin/indexes/item/documents/document/edit.hbs b/app/templates/admin/indexes/item/documents/document/edit.hbs
new file mode 100644
index 0000000..f65b58a
--- /dev/null
+++ b/app/templates/admin/indexes/item/documents/document/edit.hbs
@@ -0,0 +1,5 @@
+<form {{on 'submit' this.save}}>
+  <InputTextarea @name='document' @value={{stringify @model.document}} />
+
+  <Button type='submit' @text='Save' @style='primary' />
+</form>
diff --git a/app/templates/admin/indexes/item/documents/document/index.hbs b/app/templates/admin/indexes/item/documents/document/index.hbs
new file mode 100644
index 0000000..dcc3448
--- /dev/null
+++ b/app/templates/admin/indexes/item/documents/document/index.hbs
@@ -0,0 +1,7 @@
+{{#each-in @model.document as |key value|}}
+  {{#if (eq '_geo' key)}}
+    {{! TODO: add map }}
+  {{else}}
+    <FormElement @label={{capitalize key}} @value={{stringify value}} />
+  {{/if}}
+{{/each-in}}
diff --git a/app/templates/admin/indexes/item/documents/index.hbs b/app/templates/admin/indexes/item/documents/index.hbs
index 532dfc7..b2dc4bc 100644
--- a/app/templates/admin/indexes/item/documents/index.hbs
+++ b/app/templates/admin/indexes/item/documents/index.hbs
@@ -46,6 +46,14 @@
           @route='admin.indexes.item.documents.document'
           @models={{array @model.index.uid (get data @model.index.primaryKey)}}
           class="inline-flex items-center"
+        >
+          <Icons::FileDocumentOutline class='mr-1' />
+          View
+        </LinkTo>
+        <LinkTo
+          @route='admin.indexes.item.documents.document.edit'
+          @models={{array @model.index.uid (get data @model.index.primaryKey)}}
+          class="inline-flex items-center"
         >
           <Icons::PencilOutline class='mr-1' />
           Edit
diff --git a/tests/unit/controllers/admin/indexes/item/documents/document/edit-test.js b/tests/unit/controllers/admin/indexes/item/documents/document/edit-test.js
new file mode 100644
index 0000000..b632539
--- /dev/null
+++ b/tests/unit/controllers/admin/indexes/item/documents/document/edit-test.js
@@ -0,0 +1,17 @@
+import { module, test } from 'qunit';
+import { setupTest } from 'meiliadmin/tests/helpers';
+
+module(
+  'Unit | Controller | admin/indexes/item/documents/document/edit',
+  function (hooks) {
+    setupTest(hooks);
+
+    // TODO: Replace this with your real tests.
+    test('it exists', function (assert) {
+      let controller = this.owner.lookup(
+        'controller:admin/indexes/item/documents/document/edit'
+      );
+      assert.ok(controller);
+    });
+  }
+);
diff --git a/tests/unit/controllers/admin/indexes/item/documents/document/index-test.js b/tests/unit/controllers/admin/indexes/item/documents/document/index-test.js
new file mode 100644
index 0000000..da08b21
--- /dev/null
+++ b/tests/unit/controllers/admin/indexes/item/documents/document/index-test.js
@@ -0,0 +1,17 @@
+import { module, test } from 'qunit';
+import { setupTest } from 'meiliadmin/tests/helpers';
+
+module(
+  'Unit | Controller | admin/indexes/item/documents/document/index',
+  function (hooks) {
+    setupTest(hooks);
+
+    // TODO: Replace this with your real tests.
+    test('it exists', function (assert) {
+      let controller = this.owner.lookup(
+        'controller:admin/indexes/item/documents/document/index'
+      );
+      assert.ok(controller);
+    });
+  }
+);
diff --git a/tests/unit/routes/admin/indexes/item/documents/document/edit-test.js b/tests/unit/routes/admin/indexes/item/documents/document/edit-test.js
new file mode 100644
index 0000000..a6e3a95
--- /dev/null
+++ b/tests/unit/routes/admin/indexes/item/documents/document/edit-test.js
@@ -0,0 +1,16 @@
+import { module, test } from 'qunit';
+import { setupTest } from 'meiliadmin/tests/helpers';
+
+module(
+  'Unit | Route | admin/indexes/item/documents/document/edit',
+  function (hooks) {
+    setupTest(hooks);
+
+    test('it exists', function (assert) {
+      let route = this.owner.lookup(
+        'route:admin/indexes/item/documents/document/edit'
+      );
+      assert.ok(route);
+    });
+  }
+);
diff --git a/tests/unit/routes/admin/indexes/item/documents/document/index-test.js b/tests/unit/routes/admin/indexes/item/documents/document/index-test.js
new file mode 100644
index 0000000..a3c12ee
--- /dev/null
+++ b/tests/unit/routes/admin/indexes/item/documents/document/index-test.js
@@ -0,0 +1,16 @@
+import { module, test } from 'qunit';
+import { setupTest } from 'meiliadmin/tests/helpers';
+
+module(
+  'Unit | Route | admin/indexes/item/documents/document/index',
+  function (hooks) {
+    setupTest(hooks);
+
+    test('it exists', function (assert) {
+      let route = this.owner.lookup(
+        'route:admin/indexes/item/documents/document/index'
+      );
+      assert.ok(route);
+    });
+  }
+);