Project

General

Profile

Actions

Patch #33117

open

Redirect to users_path instead of edit_user_path in order to avoid confusion

Added by Go MAEDA over 4 years ago. Updated over 4 years ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
Accounts / authentication
Target version:
-
Start date:
Due date:
% Done:

0%

Estimated time:

Description

Currently, a user is redirected to edit_user_path (/users/*/edit) after creating a user. However, I think it is better to be redirected to user_path (/users) for the following reasons:

  • Inconsistent. There are some exceptions but most forms in Admin page shows index page after an object is created
  • Confusing. Today I observed that an admin mistakenly rewrote the user that has just created. It is because they misunderstood that the form is /users/new, but it was actually /users/*/edit
  • I cannot find any use cases that showing edit_user_path is better than users_path
diff --git a/app/controllers/users_controller.rb b/app/controllers/users_controller.rb
index 2fb297874..d3e853b43 100644
--- a/app/controllers/users_controller.rb
+++ b/app/controllers/users_controller.rb
@@ -122,7 +122,7 @@ class UsersController < ApplicationController
             attrs = {:generate_password => @user.generate_password }
             redirect_to new_user_path(:user => attrs)
           else
-            redirect_to edit_user_path(@user)
+            redirect_to users_path
           end
         }
         format.api  { render :action => 'show', :status => :created, :location => user_url(@user) }

Files

33117.patch (1.13 KB) 33117.patch Go MAEDA, 2020-05-04 02:52

Related issues

Related to Redmine - Feature #32783: Redirect to index page instead of edit page after creating a new custom fieldClosedGo MAEDA

Actions
Actions

Also available in: Atom PDF