Skip to content

Commit 4d8ea3d

Browse files
committed
remove ability to delete user account, spammers delete themselves if they are banned and repeat their actions.
1 parent 5213ab9 commit 4d8ea3d

File tree

4 files changed

+0
-43
lines changed

4 files changed

+0
-43
lines changed

app/controllers/users_controller.rb

Lines changed: 0 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -157,17 +157,6 @@ def specialties
157157
redirect_to badge_url(@user.username)
158158
end
159159

160-
def delete_account
161-
return head(:forbidden) unless signed_in?
162-
end
163-
164-
def delete_account_confirmed
165-
user = User.find(current_user.id)
166-
user.destroy
167-
sign_out
168-
redirect_to root_url
169-
end
170-
171160
def clear_provider
172161
return head(:forbidden) unless current_user.admin?
173162

@@ -180,17 +169,6 @@ def clear_provider
180169
redirect_to(badge_url(username: @user.username))
181170
end
182171

183-
def destroy
184-
return head(:forbidden) unless current_user.admin?
185-
186-
destroy_params = params.permit(:id)
187-
188-
@user = User.find(destroy_params[:id])
189-
@user.destroy
190-
record_event('deleted account')
191-
redirect_to badge_url(@user.username)
192-
end
193-
194172
def settings
195173
if signed_in?
196174
record_event("api key requested", username: current_user.username, site: request.env["REMOTE_HOST"])

app/views/users/_show_admin_panel.slim

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,4 +22,3 @@
2222
=link_to('Clear LinkedIn!', clear_provider_path(user, :provider => 'linkedin'), :confirm => 'Are you sure?')
2323
li.admin-action
2424
=link_to('Delete Facts', clear_provider_path(user, :provider => 'facts'), :confirm => 'Are you sure?', :method => :delete)
25-
li.admin-action= link_to('Delete User', user_path(user), :confirm => 'Are you sure?', :method => :delete)

app/views/users/delete_account.html.haml

Lines changed: 0 additions & 18 deletions
This file was deleted.

config/routes.rb

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -320,8 +320,6 @@
320320
get '/settings' => 'users#edit', as: :settings
321321
get '/unsubscribe' => 'emails#unsubscribe'
322322
get '/delivered' => 'emails#delivered'
323-
get '/delete_account' => 'users#delete_account', as: :delete_account
324-
post '/delete_account_confirmed' => 'users#delete_account_confirmed', as: :delete_account_confirmed
325323

326324
resources :authentications, :usernames
327325
resources :invitations

0 commit comments

Comments
 (0)