lint

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

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 %>
+            <% active_value = action != "deactivate" %>
-            <div hidden> 
+            <div hidden>
                 <%= f.govuk_text_field :active,
                 value: active_value %>
             </div>