Redundant error messages on user create

Bug #1445043 reported by Dimitri Mazmanov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Low
Dimitri Mazmanov

Bug Description

Two errors are shown when I skip the password confirmation field upon user creation:
1. Passwords do not match
2. Confirm Password: This field is required

Steps to reproduce
1. Create a new user.
2. Fill in user name, password, but skip the confirm password field.

I should be enough to show one error message: "This field is required"

Changed in horizon:
assignee: nobody → Dimitri Mazmanov (sorantis)
Changed in horizon:
importance: Undecided → Low
tags: added: error-reporting
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

Fix proposed to branch: master
Review: https://review.openstack.org/174469

Changed in horizon:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/174469
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=21d155395cbaa4da50de6453a845c6bdbf0b2773
Submitter: Jenkins
Branch: master

commit 21d155395cbaa4da50de6453a845c6bdbf0b2773
Author: Dimitri Mazmanov <email address hidden>
Date: Thu Apr 16 19:00:47 2015 +0200

    Fix the redundant error message during user create

    Simply check if both password and confirm_password fields are not empty.

    Change-Id: Ie04e0515f15993ed3f693d107475939604e55310
    Closes-Bug: #1445043

Changed in horizon:
status: In Progress → Fix Committed
Changed in horizon:
milestone: none → liberty-1
Changed in horizon:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in horizon:
milestone: liberty-1 → 8.0.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.