From 498ecdf68b2d373c44c0f38875118b69838acba3 Mon Sep 17 00:00:00 2001
From: insects <mail@liv.nrw>
Date: Tue, 11 Mar 2025 01:12:30 +0100
Subject: [PATCH] feat: check for password existence

---
 app/assets/stylesheets/application.css   | 11 +++++++++-
 app/controllers/instance_controller.rb   | 28 ++++++++++++++----------
 app/javascript/list.js                   | 22 +++++++++++++++++++
 app/javascript/save_password.js          | 12 ++++++++++
 app/views/instance/_list.html.erb        |  3 +--
 app/views/instance/set_password.html.erb |  6 +++++
 app/views/instance/show.html.erb         |  6 ++++-
 7 files changed, 73 insertions(+), 15 deletions(-)
 create mode 100644 app/javascript/list.js
 create mode 100644 app/views/instance/set_password.html.erb

diff --git a/app/assets/stylesheets/application.css b/app/assets/stylesheets/application.css
index f3f3189..d23263d 100644
--- a/app/assets/stylesheets/application.css
+++ b/app/assets/stylesheets/application.css
@@ -84,7 +84,7 @@ small.badge {
     border-color: #63f0fd;
 }
 
-button.action {
+.action button {
     width: 100%;
     height: 100%;
     border: 0;
@@ -103,6 +103,15 @@ button.action:hover {
     cursor: pointer;
 }
 
+.action {
+    display: none;
+}
+
+.action.shown {
+    display: block;
+    height: 100%;
+}
+
 section .meta {
     padding-left: 10px;
     padding-top: 10px;
diff --git a/app/controllers/instance_controller.rb b/app/controllers/instance_controller.rb
index 35b226b..3ebd248 100644
--- a/app/controllers/instance_controller.rb
+++ b/app/controllers/instance_controller.rb
@@ -6,7 +6,9 @@ class InstanceController < ApplicationController
     password = Nanoid.generate(size: 3, alphabet: "0123456789")
     instance = Instance.new(zone: zone, public_id: public_id, name: name, password: password)
     if instance.save
-      redirect_to(show_instance_path(public_id: public_id))
+      @id = instance.public_id
+      @password = instance.password
+      render "set_password"
     end
   end
 
@@ -15,21 +17,25 @@ class InstanceController < ApplicationController
   end
 
   def pop
-    instance_id, nm = pop_instance_params
+    instance_id, nm, pwd = pop_instance_params
     parent_instance = Instance.find_by(public_id: instance_id)
-    pop = Pop.new(instance_id: parent_instance.id, name: nm)
-    if pop.save
-      @instance = Instance.includes(:pops).find_by(public_id: instance_id)
-      render partial: "list", locals: { instance: @instance }
+    if parent_instance.password == pwd
+      pop = Pop.new(instance_id: parent_instance.id, name: nm)
+      if pop.save
+        @instance = Instance.includes(:pops).find_by(public_id: instance_id)
+        render partial: "list", locals: { instance: @instance }
+      end
     end
   end
 
   def reset
-    instance_id, nm = pop_instance_params
+    instance_id, nm, pwd = pop_instance_params
     parent_instance = Instance.find_by(public_id: instance_id)
-    Pop.delete_by(instance_id: parent_instance.id, name: nm)
-    @instance = Instance.includes(:pops).find_by(public_id: instance_id)
-    render partial: "list", locals: { instance: @instance }
+    if parent_instance.password == pwd
+      Pop.delete_by(instance_id: parent_instance.id, name: nm)
+      @instance = Instance.includes(:pops).find_by(public_id: instance_id)
+      render partial: "list", locals: { instance: @instance }
+    end
   end
 
   private
@@ -43,6 +49,6 @@ class InstanceController < ApplicationController
   end
 
   def pop_instance_params
-    params.expect(:instance, :nm)
+    params.expect(:instance, :nm, :pwd)
   end
 end
diff --git a/app/javascript/list.js b/app/javascript/list.js
new file mode 100644
index 0000000..7f83b03
--- /dev/null
+++ b/app/javascript/list.js
@@ -0,0 +1,22 @@
+function checkPwd() {
+    const ls = window.localStorage;
+    const id = document.getElementById("public_id").dataset.content;
+    const pwd = ls.getItem(`ecoffee-pwd:${id}`);
+    if (pwd) {
+        const to_show = document.querySelectorAll(".needs_pwd");
+        to_show.forEach(el => {
+            el.classList.add("shown");
+        });
+        const buttons = document.querySelectorAll(".action button");
+        buttons.forEach(btn => {
+          const oldUrl= btn.getAttribute("hx-post");
+          btn.setAttribute("hx-post", `${oldUrl}&pwd=${pwd}`);
+        });
+    }
+}
+
+checkPwd();
+
+document.addEventListener("htmx:afterRequest", evt => {
+    checkPwd();
+})
diff --git a/app/javascript/save_password.js b/app/javascript/save_password.js
index e69de29..eb400c8 100644
--- a/app/javascript/save_password.js
+++ b/app/javascript/save_password.js
@@ -0,0 +1,12 @@
+const id = document.getElementById("public_id");
+const pwd = document.getElementById("password");
+const ls = window.localStorage;
+
+if (id && pwd) {
+    ls.setItem(`ecoffee-pwd:${id.dataset.content}`, pwd.dataset.content);
+}
+
+let loc = new URL(window.location.href);
+loc.search = "";
+loc.pathname = `/${id.dataset.content}`;
+window.location = loc;
diff --git a/app/views/instance/_list.html.erb b/app/views/instance/_list.html.erb
index c3c569d..6c30549 100644
--- a/app/views/instance/_list.html.erb
+++ b/app/views/instance/_list.html.erb
@@ -35,7 +35,7 @@
           </span>
         <% end %>
       </div>
-      <div class="button">
+      <div class="action needs_pwd">
         <% if is_popped %>
           <button
             class="action reset"
@@ -49,7 +49,6 @@
             hx-target="#nm-list"
           >Pop</button>
         <% end %>
-
       </div>
     </section>
   <% end %>
diff --git a/app/views/instance/set_password.html.erb b/app/views/instance/set_password.html.erb
new file mode 100644
index 0000000..33ac011
--- /dev/null
+++ b/app/views/instance/set_password.html.erb
@@ -0,0 +1,6 @@
+<h1>Redirecting...</h1>
+
+<div id="public_id" data-content="<%= @id %>"></div>
+<div id="password" data-content="<%= @password %>"></div>
+
+<%= javascript_include_tag "save_password" %>
diff --git a/app/views/instance/show.html.erb b/app/views/instance/show.html.erb
index d16acd9..ab0a23c 100644
--- a/app/views/instance/show.html.erb
+++ b/app/views/instance/show.html.erb
@@ -1,9 +1,13 @@
+<div id="public_id" data-content="<%= @instance.public_id %>"></div>
 <div hx-get="" hx-trigger="every 30s" hx-swap="innerHTML" hx-target="this">
   <header>
-    <img src="/icon.png" width="50" alt="eureka.coffee logo" />
+    <%= link_to root_path do %><img src="/icon.png" width="50" alt="eureka.coffee logo" /><% end %>
     <h1><span class="muted">instance</span> <%= @instance.name %></h1>
     <%= render partial: "zone_img", locals: { zone: @instance.zone, alt: @instance.zone, title: @instance.zone.upcase_first } %>
   </header>
 
   <%= render partial: "list", locals: { instance: @instance } %>
+
+  <%= javascript_include_tag "list" %>
 </div>
+