From 14fd952f940c2f896d11b635481fcb6ed53b6316 Mon Sep 17 00:00:00 2001
From: floppydiskette <git@diskfloppy.me>
Date: Tue, 13 Aug 2024 17:05:22 +0100
Subject: [PATCH] Actually do it in a not broken way

---
 app/helpers/home_helper.rb    | 8 +++++---
 app/views/home/index.html.erb | 2 +-
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/app/helpers/home_helper.rb b/app/helpers/home_helper.rb
index 7e43352..cbd4ff9 100644
--- a/app/helpers/home_helper.rb
+++ b/app/helpers/home_helper.rb
@@ -6,7 +6,7 @@ module HomeHelper
     stdout, stderr, status = Open3.capture3('getent', 'passwd')
     return [] unless status.success?
     valid_users = stdout.split("\n").select { |user| user.start_with?(/\w/) }.map(&:split).map(&:first)
-    valid_users.map do |user|
+    return valid_users.map do |user|
       user.split(':')[0]
     end
   end
@@ -18,6 +18,8 @@ module HomeHelper
     end
   end
 
-  users = list_all_users
-  users_with_public_html(users)
+  def get_members
+    users = list_all_users
+    users_with_public_html(users)
+  end
 end
diff --git a/app/views/home/index.html.erb b/app/views/home/index.html.erb
index 897fa33..ca5468a 100644
--- a/app/views/home/index.html.erb
+++ b/app/views/home/index.html.erb
@@ -7,7 +7,7 @@
 <div class="section">
   <p>Members</p>
   <ul class="members">
-    <% list_all_users.each do |user| %>
+    <% get_members.each do |user| %>
       <li><a href="/~<%= user %>"><%= user %></a></li>
     <% end %>
   </ul>