From e414afd95b60a7299c3ea001164962370c3831d6 Mon Sep 17 00:00:00 2001
From: Michael Housh <mhoush@houshhomeenergy.com>
Date: Wed, 8 Jan 2025 18:10:26 -0500
Subject: [PATCH] feat: Reorganizing views

---
 Resources/Views/employees.leaf                  | 10 ----------
 .../{employee-form.leaf => employees/form.leaf} |  0
 Resources/Views/employees/index.leaf            | 14 ++++++++++++++
 .../table.leaf}                                 |  0
 Resources/Views/home.leaf                       | 13 +++----------
 Resources/Views/img/trash-can.leaf              |  1 +
 Resources/Views/login.leaf                      |  2 +-
 .../{po-form.leaf => purchaseOrders/form.leaf}  |  0
 .../table.leaf}                                 |  0
 Resources/Views/users.leaf                      | 10 ----------
 .../Views/{user-form.leaf => users/form.leaf}   |  0
 Resources/Views/users/index.leaf                | 14 ++++++++++++++
 .../Views/{user-table.leaf => users/table.leaf} |  0
 .../{vendor-form.leaf => vendors/form.leaf}     |  0
 .../Views/{vendors.leaf => vendors/index.leaf}  |  4 ++--
 .../{vendor-table.leaf => vendors/table.leaf}   |  2 +-
 .../Controllers/EmployeeViewController.swift    | 14 +++++++-------
 .../App/Controllers/UsersViewController.swift   |  6 +++---
 .../App/Controllers/VendorViewController.swift  |  9 ++++-----
 Sources/App/Controllers/ViewController.swift    | 17 +++++------------
 20 files changed, 55 insertions(+), 61 deletions(-)
 delete mode 100644 Resources/Views/employees.leaf
 rename Resources/Views/{employee-form.leaf => employees/form.leaf} (100%)
 create mode 100644 Resources/Views/employees/index.leaf
 rename Resources/Views/{employee-table.leaf => employees/table.leaf} (100%)
 create mode 100644 Resources/Views/img/trash-can.leaf
 rename Resources/Views/{po-form.leaf => purchaseOrders/form.leaf} (100%)
 rename Resources/Views/{po-table.leaf => purchaseOrders/table.leaf} (100%)
 delete mode 100644 Resources/Views/users.leaf
 rename Resources/Views/{user-form.leaf => users/form.leaf} (100%)
 create mode 100644 Resources/Views/users/index.leaf
 rename Resources/Views/{user-table.leaf => users/table.leaf} (100%)
 rename Resources/Views/{vendor-form.leaf => vendors/form.leaf} (100%)
 rename Resources/Views/{vendors.leaf => vendors/index.leaf} (82%)
 rename Resources/Views/{vendor-table.leaf => vendors/table.leaf} (93%)

diff --git a/Resources/Views/employees.leaf b/Resources/Views/employees.leaf
deleted file mode 100644
index 3f7952c..0000000
--- a/Resources/Views/employees.leaf
+++ /dev/null
@@ -1,10 +0,0 @@
-<div id="home-content" class="container" #if(oob): hx-swap-oob="outerHTML" #endif>
-  <div class="container">
-    <h1>Employees</h1>
-    <br>
-    <p>Employees are who purchase orders can be generated for.</p>
-    <br>
-  </div>
-  #extend("employee-form", form)
-  #extend("employee-table")
-</div>
diff --git a/Resources/Views/employee-form.leaf b/Resources/Views/employees/form.leaf
similarity index 100%
rename from Resources/Views/employee-form.leaf
rename to Resources/Views/employees/form.leaf
diff --git a/Resources/Views/employees/index.leaf b/Resources/Views/employees/index.leaf
new file mode 100644
index 0000000..24710a0
--- /dev/null
+++ b/Resources/Views/employees/index.leaf
@@ -0,0 +1,14 @@
+#extend("home"):
+  #export("homeContent"):
+  <div id="home-content" class="container" #if(oob): hx-swap-oob="outerHTML" #endif>
+    <div class="container">
+      <h1>Employees</h1>
+      <br>
+      <p>Employees are who purchase orders can be generated for.</p>
+      <br>
+    </div>
+    #extend("employees/form", form)
+    #extend("employees/table")
+  </div>
+  #endexport
+#endextend
diff --git a/Resources/Views/employee-table.leaf b/Resources/Views/employees/table.leaf
similarity index 100%
rename from Resources/Views/employee-table.leaf
rename to Resources/Views/employees/table.leaf
diff --git a/Resources/Views/home.leaf b/Resources/Views/home.leaf
index 420c9f9..9f1e400 100644
--- a/Resources/Views/home.leaf
+++ b/Resources/Views/home.leaf
@@ -13,26 +13,22 @@
         <ul class="nav-links">
           <li>
             <a hx-get="/users"
-               hx-target="#home-content"
-               hx-swap="outerHTML"
+               hx-target="body"
                hx-push-url="true"
-               #if(route == "users"): hx-trigger="revealed" #endif
                >
               Users
             </a>
           </li>
           <li>
             <a hx-get="/employees"
-               hx-target="#home-content"
-               hx-swap="outerHTML"
+               hx-target="body"
                hx-push-url="true"
-               #if(route == "employees"): hx-trigger="revealed" #endif
                >
                Employees
             </a>
           </li>
           <li>
-            <a hx-get="/vendors?branches=true"
+            <a hx-get="/vendors"
                hx-target="body"
                hx-push-url="true"
                >
@@ -43,9 +39,6 @@
       </nav>
       </div>
       #import("homeContent")
-      <!-- <div id="home-content" class="container"> -->
-      <!--   <p>We're in!</p> -->
-      <!-- </div> -->
     </section>
   </div>
   #endexport
diff --git a/Resources/Views/img/trash-can.leaf b/Resources/Views/img/trash-can.leaf
new file mode 100644
index 0000000..f41997e
--- /dev/null
+++ b/Resources/Views/img/trash-can.leaf
@@ -0,0 +1 @@
+<img src="images/trash-can.svg" alt="Delete">
diff --git a/Resources/Views/login.leaf b/Resources/Views/login.leaf
index 9062c21..6f02767 100644
--- a/Resources/Views/login.leaf
+++ b/Resources/Views/login.leaf
@@ -6,7 +6,7 @@
       #extend("logo")
     </div>
   </header>
-  #extend("user-form")
+  #extend("users/form")
 </div>
 #endexport
 #endextend
diff --git a/Resources/Views/po-form.leaf b/Resources/Views/purchaseOrders/form.leaf
similarity index 100%
rename from Resources/Views/po-form.leaf
rename to Resources/Views/purchaseOrders/form.leaf
diff --git a/Resources/Views/po-table.leaf b/Resources/Views/purchaseOrders/table.leaf
similarity index 100%
rename from Resources/Views/po-table.leaf
rename to Resources/Views/purchaseOrders/table.leaf
diff --git a/Resources/Views/users.leaf b/Resources/Views/users.leaf
deleted file mode 100644
index 42144d2..0000000
--- a/Resources/Views/users.leaf
+++ /dev/null
@@ -1,10 +0,0 @@
-<div id="home-content" class="container">
-  <div class="container">
-    <h1>Users</h1>
-    <br>
-    <p>Users are people that can login and generate puchase orders for employees.</p>
-    <br>
-  </div>
-  #extend("user-form", form)
-  #extend("user-table")
-</div>
diff --git a/Resources/Views/user-form.leaf b/Resources/Views/users/form.leaf
similarity index 100%
rename from Resources/Views/user-form.leaf
rename to Resources/Views/users/form.leaf
diff --git a/Resources/Views/users/index.leaf b/Resources/Views/users/index.leaf
new file mode 100644
index 0000000..71f08be
--- /dev/null
+++ b/Resources/Views/users/index.leaf
@@ -0,0 +1,14 @@
+#extend("home"):
+  #export("homeContent"):
+  <div id="home-content" class="container">
+    <div class="container">
+      <h1>Users</h1>
+      <br>
+      <p>Users are people that can login and generate puchase orders for employees.</p>
+      <br>
+    </div>
+    #extend("users/form", form)
+    #extend("users/table")
+  </div>
+  #endexport
+#endextend
diff --git a/Resources/Views/user-table.leaf b/Resources/Views/users/table.leaf
similarity index 100%
rename from Resources/Views/user-table.leaf
rename to Resources/Views/users/table.leaf
diff --git a/Resources/Views/vendor-form.leaf b/Resources/Views/vendors/form.leaf
similarity index 100%
rename from Resources/Views/vendor-form.leaf
rename to Resources/Views/vendors/form.leaf
diff --git a/Resources/Views/vendors.leaf b/Resources/Views/vendors/index.leaf
similarity index 82%
rename from Resources/Views/vendors.leaf
rename to Resources/Views/vendors/index.leaf
index 017904e..7e8ba1f 100644
--- a/Resources/Views/vendors.leaf
+++ b/Resources/Views/vendors/index.leaf
@@ -6,8 +6,8 @@
       <br>
       <p>Vendors are who purchase orders can be issued for, they consist of multiple branches / locations.</p>
       <br>
-      #extend("vendor-form", form)
-      #extend("vendor-table")
+      #extend("vendors/form", form)
+      #extend("vendors/table")
     </div>
   </div>
   #endexport
diff --git a/Resources/Views/vendor-table.leaf b/Resources/Views/vendors/table.leaf
similarity index 93%
rename from Resources/Views/vendor-table.leaf
rename to Resources/Views/vendors/table.leaf
index 22f8f85..bccd6a3 100644
--- a/Resources/Views/vendor-table.leaf
+++ b/Resources/Views/vendors/table.leaf
@@ -25,7 +25,7 @@
            hx-swap="outerHTML swap:0.5s"
            hx-confirm="Are you sure you want to delete this vendor?"
            >
-          <img src="images/trash-can.svg" alt="Delete">
+          #extend("img/trash-can")
         </a>
       </td>
     </tr>
diff --git a/Sources/App/Controllers/EmployeeViewController.swift b/Sources/App/Controllers/EmployeeViewController.swift
index 8574bf5..0c536ba 100644
--- a/Sources/App/Controllers/EmployeeViewController.swift
+++ b/Sources/App/Controllers/EmployeeViewController.swift
@@ -21,14 +21,14 @@ struct EmployeeViewController: RouteCollection {
 
   @Sendable
   func index(req: Request) async throws -> View {
-    return try await req.view.render("employees", EmployeesCTX(api: api, req: req))
+    return try await req.view.render("employees/index", EmployeesCTX(api: api, req: req))
   }
 
   @Sendable
   func postEmployeeForm(req: Request) async throws -> View {
     _ = try await api.createEmployee(req: req)
     let employees = try await api.getSortedEmployees(req: req)
-    return try await req.view.render("employee-table", ["employees": employees])
+    return try await req.view.render("employees/table", ["employees": employees])
   }
 
   @Sendable
@@ -39,14 +39,14 @@ struct EmployeeViewController: RouteCollection {
     employee.active.toggle()
     try await employee.save(on: req.db)
     let employees = try await api.getSortedEmployees(req: req)
-    return try await req.view.render("employee-table", ["employees": employees])
+    return try await req.view.render("employees/table", ["employees": employees])
   }
 
   @Sendable
   func deleteEmployee(req: Request) async throws -> View {
     _ = try await api.deleteEmployee(req: req)
     let employees = try await api.getSortedEmployees(req: req)
-    return try await req.view.render("employee-table", ["employees": employees])
+    return try await req.view.render("employees/table", ["employees": employees])
   }
 
   @Sendable
@@ -54,18 +54,18 @@ struct EmployeeViewController: RouteCollection {
     guard let employee = try await Employee.find(req.parameters.get("employeeID"), on: req.db) else {
       throw Abort(.notFound)
     }
-    return try await req.view.render("employee-form", EmployeeFormCTX(employee: employee.toDTO()))
+    return try await req.view.render("employees/form", EmployeeFormCTX(employee: employee.toDTO()))
   }
 
   @Sendable
   func updateEmployee(req: Request) async throws -> View {
     _ = try await api.updateEmployee(req: req)
-    return try await req.view.render("employees", EmployeesCTX(oob: true, api: api, req: req))
+    return try await req.view.render("employees/index", EmployeesCTX(oob: true, api: api, req: req))
   }
 
   @Sendable
   func employeeForm(req: Request) async throws -> View {
-    try await req.view.render("employee-form", EmployeeFormCTX())
+    try await req.view.render("employees/form", EmployeeFormCTX())
   }
 
 }
diff --git a/Sources/App/Controllers/UsersViewController.swift b/Sources/App/Controllers/UsersViewController.swift
index 096692d..88b75d9 100644
--- a/Sources/App/Controllers/UsersViewController.swift
+++ b/Sources/App/Controllers/UsersViewController.swift
@@ -17,7 +17,7 @@ struct UserViewController: RouteCollection {
   @Sendable
   func index(req: Request) async throws -> View {
     try await req.view.render(
-      "users",
+      "users/index",
       UsersCTX(users: api.getSortedUsers(req: req))
     )
   }
@@ -25,13 +25,13 @@ struct UserViewController: RouteCollection {
   @Sendable
   func create(req: Request) async throws -> View {
     _ = try await api.createUser(req: req)
-    return try await req.view.render("user-table", ["users": api.getSortedUsers(req: req)])
+    return try await req.view.render("users/table", ["users": api.getSortedUsers(req: req)])
   }
 
   @Sendable
   func delete(req: Request) async throws -> View {
     _ = try await api.deleteUser(req: req)
-    return try await req.view.render("user-table", ["users": api.getSortedUsers(req: req)])
+    return try await req.view.render("users/table", ["users": api.getSortedUsers(req: req)])
   }
 }
 
diff --git a/Sources/App/Controllers/VendorViewController.swift b/Sources/App/Controllers/VendorViewController.swift
index 346828d..e354f36 100644
--- a/Sources/App/Controllers/VendorViewController.swift
+++ b/Sources/App/Controllers/VendorViewController.swift
@@ -17,13 +17,13 @@ struct VendorViewController: RouteCollection {
 
   @Sendable
   func index(req: Request) async throws -> View {
-    return try await req.view.render("vendors", makeCtx(req: req))
+    return try await req.view.render("vendors/index", makeCtx(req: req))
   }
 
   @Sendable
   func create(req: Request) async throws -> View {
     let ctx = try req.content.decode(CreateVendorCTX.self)
-    req.logger.info("CTX: \(ctx)")
+    req.logger.debug("CTX: \(ctx)")
     let vendor = Vendor.Create(name: ctx.name).toModel()
     try await vendor.save(on: req.db)
 
@@ -39,8 +39,7 @@ struct VendorViewController: RouteCollection {
       }
     }
 
-    // _ = try await api.createVendor(req: req)
-    return try await req.view.render("vendor-table", makeCtx(req: req))
+    return try await req.view.render("vendors/table", makeCtx(req: req))
   }
 
   @Sendable
@@ -51,7 +50,7 @@ struct VendorViewController: RouteCollection {
   @Sendable
   func update(req: Request) async throws -> View {
     _ = try await api.updateVendor(req: req)
-    return try await req.view.render("vendor-table", makeCtx(req: req, oob: true))
+    return try await req.view.render("vendors/table", makeCtx(req: req, oob: true))
   }
 
   private func makeCtx(req: Request, vendor: Vendor? = nil, oob: Bool = false) async throws -> VendorsCTX {
diff --git a/Sources/App/Controllers/ViewController.swift b/Sources/App/Controllers/ViewController.swift
index 09e9350..9dabdc3 100644
--- a/Sources/App/Controllers/ViewController.swift
+++ b/Sources/App/Controllers/ViewController.swift
@@ -37,7 +37,7 @@ struct ViewController: RouteCollection {
   }
 
   @Sendable
-  func postLogin(req: Request) async throws -> View {
+  func postLogin(req: Request) async throws -> Response {
     let content = try req.content.decode(UserForm.self)
     guard let user = try await User.query(on: req.db)
       .filter(\.$username == content.username)
@@ -62,18 +62,11 @@ struct ViewController: RouteCollection {
   }
 
   @Sendable
-  func home(req: Request) async throws -> View {
-    var route: HomeRoute?
-
-    if let loginParams = try? req.query.decode(LoginParameter.self),
-       let next = loginParams.next.split(separator: "/").last
-    {
-      route = HomeRoute(rawValue: String(next))
-    } else if let routeString = req.parameters.getCatchall().first {
-      route = HomeRoute(rawValue: routeString)
+  func home(req: Request) async throws -> Response {
+    if let loginParams = try? req.query.decode(LoginParameter.self) {
+      return req.redirect(to: loginParams.next)
     }
-
-    return try await req.view.render("home", HomeCTX(route: route))
+    return try await req.view.render("home").encodeResponse(for: req)
   }
 
   @Sendable