Opened 10 years ago

Closed 10 years ago

#8434 closed defect (fixed)

[patch] [cla] PasswordValidator: "Old Password" field is always set to "" on blur

Reported by: 7twenty Owned by: Nathan Toone
Priority: high Milestone: 1.3
Component: DojoX Form Version: 1.2.3
Keywords: Cc:
Blocked By: Blocking:

Description

The PasswordValidator? control doesn't seem compatible with the _setValueAttr changes made back in August. The "Old Password" field is always set to "" on blur.

Steps to duplicate:

  1. Comment out the automated tests in dojox\form\tests\test_PasswordValidator.html line 87 (doh.run).
  2. Load dojox\form\tests\test_PasswordValidator.html in a browser
  3. In the first "Old Password" field, type in the old password (oldpw2) and press tab

Results: The field is set to "" and marked as invalid

Attachments (1)

PasswordValidator.patch​ (861 bytes) - added by 7twenty 10 years ago.

Download all attachments as: .zip

Change History (7)

Changed 10 years ago by 7twenty

Attachment: PasswordValidator.patch​ added

comment:1 Changed 10 years ago by 7twenty

I have attached a patch that fixes the issue.

comment:2 Changed 10 years ago by Nathan Toone

Owner: changed from anonymous to Nathan Toone

CLA has been sent in, but does not yet show up on the contributors list.

comment:3 Changed 10 years ago by Nathan Toone

Milestone: tbd → 1.3
Summary: PasswordValidator: "Old Password" field is always set to "" on blur → [patch]PasswordValidator: "Old Password" field is always set to "" on blur

comment:4 Changed 10 years ago by bill

Component: General → DojoX Form
Summary: [patch]PasswordValidator: "Old Password" field is always set to "" on blur → [patch] [cla] PasswordValidator: "Old Password" field is always set to "" on blur

comment:5 Changed 10 years ago by Nathan Toone

so has this cla been received, then?

comment:6 Changed 10 years ago by Nathan Toone

Resolution: → fixed
Status: new → closed

(In [16622]) Fixes #8434 - thanks for the patch, CLA received

Note: See TracTickets for help on using tickets.