From 8aa38d28cd1e903a96c422dfaa2506a8a5e23a69 Mon Sep 17 00:00:00 2001
From: Kat <katrina@madetech.com>
Date: Mon, 30 May 2022 12:47:08 +0100
Subject: [PATCH] lint

---
 app/controllers/users_controller.rb                         | 4 ++--
 .../{toggle-active.html.erb => toggle_active.html.erb}      | 6 +++---
 2 files changed, 5 insertions(+), 5 deletions(-)
 rename app/views/users/{toggle-active.html.erb => toggle_active.html.erb} (86%)

diff --git a/app/controllers/users_controller.rb b/app/controllers/users_controller.rb
index a2c9228e6..aead10683 100644
--- a/app/controllers/users_controller.rb
+++ b/app/controllers/users_controller.rb
@@ -88,7 +88,7 @@ class UsersController < ApplicationController
 
   def deactivate
     if current_user.can_toggle_active?(@user)
-      render "toggle-active", locals: { action: "deactivate" }
+      render "toggle_active", locals: { action: "deactivate" }
     else
       redirect_to user_path(@user)
     end
@@ -96,7 +96,7 @@ class UsersController < ApplicationController
 
   def reactivate
     if current_user.can_toggle_active?(@user)
-      render "toggle-active", locals: { action: "reactivate" }
+      render "toggle_active", locals: { action: "reactivate" }
     else
       redirect_to user_path(@user)
     end
diff --git a/app/views/users/toggle-active.html.erb b/app/views/users/toggle_active.html.erb
similarity index 86%
rename from app/views/users/toggle-active.html.erb
rename to app/views/users/toggle_active.html.erb
index eaee4881d..9a7258042 100644
--- a/app/views/users/toggle-active.html.erb
+++ b/app/views/users/toggle_active.html.erb
@@ -2,7 +2,7 @@
 
 <div class="govuk-grid-row">
     <%= form_for(@user, as: :user, html: { method: :patch }) do |f| %>
-        <div class="govuk-grid-column-two-thirds-from-desktop"> 
+        <div class="govuk-grid-column-two-thirds-from-desktop">
             <h1 class="govuk-heading-l">
                 <span class="govuk-caption-l"><%= @user.name %></span>
                 Are you sure you want to <%= action %> this user?
@@ -11,8 +11,8 @@
                 <p>Deactivating this user will mean they can no longer access this service to submit CORE data.</p>
                 <p>Any logs this user has already submitted will not be affected.</p>
             <% end %>
-            <% active_value = action == "deactivate" ? false : true %>
-            <div hidden> 
+            <% active_value = action != "deactivate" %>
+            <div hidden>
                 <%= f.govuk_text_field :active,
                 value: active_value %>
             </div>