jaculabilis/danbooru
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

users: remove confirmation when changing username.

Browse Source
This commit is contained in:
evazion
2022-03-08 03:24:12 -06:00
parent 99479dfcf2
commit 5ce724f1e2
4 changed files with 3 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
app/models/user_name_change_request.rb
View File

@@ -5,7 +5,6 @@ class UserNameChangeRequest < ApplicationRecord
belongs_to :approver, class_name: "User", optional: true belongs_to :approver, class_name: "User", optional: true
validate :not_limited, on: :create validate :not_limited, on: :create
validates :desired_name, user_name: true, confirmation: true, on: :create
validates :original_name, presence: true validates :original_name, presence: true
validates :desired_name, presence: true validates :desired_name, presence: true

2
app/policies/user_name_change_request_policy.rb
View File

@@ -10,6 +10,6 @@ class UserNameChangeRequestPolicy < ApplicationPolicy
end end
def permitted_attributes def permitted_attributes
[:desired_name, :desired_name_confirmation] [:desired_name]
end end
end end

3
app/views/user_name_change_requests/new.html.erb
View File

@@ -27,8 +27,7 @@
<%= edit_form_for(@change_request) do |f| %> <%= edit_form_for(@change_request) do |f| %>
<%= f.input :desired_name, label: "New name" %> <%= f.input :desired_name, label: "New name" %>
<%= f.input :desired_name_confirmation, label: "Confirm name" %> <%= f.submit "Submit" %>
<%= f.submit "Submit", "data-confirm": "Are you sure you want to change your name?" %>
<% end %> <% end %>
</div> </div>
</div> </div>

2
test/functional/user_name_change_requests_controller_test.rb
View File

@@ -23,7 +23,7 @@ class UserNameChangeRequestsControllerTest < ActionDispatch::IntegrationTest
context "create action" do context "create action" do
should "work" do should "work" do
post_auth user_name_change_requests_path, @user, params: { user_name_change_request: { desired_name: "zun", desired_name_confirmation: "zun" }} post_auth user_name_change_requests_path, @user, params: { user_name_change_request: { desired_name: "zun" }}
assert_redirected_to profile_path assert_redirected_to profile_path
assert_equal("zun", @user.reload.name) assert_equal("zun", @user.reload.name)