プロジェクト

全般

プロフィール

Vote #80737

未完了

Redirect to users_path instead of edit_user_path in order to avoid confusion

Admin Redmine さんが約4年前に追加. 約4年前に更新.

ステータス:
New
優先度:
通常
担当者:
-
カテゴリ:
Accounts / authentication_7
対象バージョン:
-
開始日:
2022/05/09
期日:
進捗率:

0%

予定工数:
category_id:
7
version_id:
0
issue_org_id:
33117
author_id:
332
assigned_to_id:
0
comments:
7
status_id:
1
tracker_id:
3
plus1:
0
affected_version:
closed_on:
affected_version_id:
ステータス-->[New]

説明

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) }

journals

In early versions, the index page was displayed after creating a user, but the behavior was changed to display the edit form in r2869.
--------------------------------------------------------------------------------

--------------------------------------------------------------------------------

--------------------------------------------------------------------------------
Setting the target version to 4.2.0.
--------------------------------------------------------------------------------
Attaching a patch with an updated test.
--------------------------------------------------------------------------------
Removing from 4.2.0 because always redirecting to the first page of the list may not be appropriate.

The current patch always redirects to the first page of the users list even though a newly created user may be displayed on the other page. I felt the behavior is strange while testing.
--------------------------------------------------------------------------------

--------------------------------------------------------------------------------


related_issues

relates,Closed,32783,Redirect to index page instead of edit page after creating a new custom field

Admin Redmine さんが約4年前に更新

  • カテゴリAccounts / authentication_7 にセット

他の形式にエクスポート: Atom PDF

いいね!0
いいね!0