Actions
Defect #35731
closed
Password and Confirmation fields are marked as required when editing a user
Start date:
Due date:
% Done:
0%
Estimated time:
Resolution:
Fixed
Affected version:
Description
Password and Confirmation are marked as required in UsersController#edit page. However, in actuality, those fields are only required when creating a new user, not when editing a user.
One of my customers misunderstood that the fields are required and they always set a new password when they edit a user's information.
"*" marks that indicate Password and Confirmation fields are required should be removed when editing a user in order to avoid such confusion.
Files
Updated by Go MAEDA over 2 years ago
The following patch fixes the issue.
diff --git a/app/views/users/_form.html.erb b/app/views/users/_form.html.erb
index 4b030d920..0d50d3d56 100644
--- a/app/views/users/_form.html.erb
+++ b/app/views/users/_form.html.erb
@@ -32,13 +32,13 @@
<% end %>
<div id="password_fields" style="<%= 'display:none;' if @user.auth_source %>">
<p>
- <%= f.password_field :password, :required => true, :size => 25 %>
+ <%= f.password_field :password, :required => @user.new_record?, :size => 25 %>
<em class="info"><%= l(:text_caracters_minimum, :count => Setting.password_min_length) %></em>
<% if Setting.password_required_char_classes.any? %>
<em class="info"><%= l(:text_characters_must_contain, :character_classes => Setting.password_required_char_classes.collect{|c| l("label_password_char_class_#{c}")}.join(", ")) %></em>
<% end %>
</p>
- <p><%= f.password_field :password_confirmation, :required => true, :size => 25 %></p>
+ <p><%= f.password_field :password_confirmation, :required => @user.new_record?, :size => 25 %></p>
<p><%= f.check_box :generate_password %></p>
<p><%= f.check_box :must_change_passwd %></p>
</div>
Updated by Go MAEDA over 2 years ago
- File 35731.patch 35731.patch added
- Target version set to Candidate for next minor release
Added test code.
Updated by
Updated by Go MAEDA over 2 years ago
- Target version changed from Candidate for next minor release to 4.1.5
Setting the target version to 4.1.5.
Updated by Go MAEDA over 2 years ago
- Status changed from New to Closed
- Assignee set to Go MAEDA
- Resolution set to Fixed
Committed the patch.
Actions