Opened 7 years ago

Last modified 2 years ago

#15121 assigned defect

[patch] "allowNamed" does not work when using with dojox.validate.regexp.emailAddress

Reported by: phil0135 Owned by: dylan
Priority: undecided Milestone: 1.15
Component: Dojox Version: 1.6.0
Keywords: Cc:
Blocked By: Blocking:

Description

The flag "allowNamed" is ignored when using it for validating e-mail addresses.

The problem is in file dojox.validate.regexp.js
Ln 114:

 if(flags.allowNamed){ hostNameRE += "|^[^-][a-zA-Z0-9_-]*"; }

This works fine, if you are validating host names, but can never succeed for email addresses.

Change to:

if(flags.allowNamed){ hostNameRE += "|[^-][a-zA-Z0-9_-]*"; }

works fine when using in email addresses. But I have no idea if it produces side effects. But since also flag "allowLocal" is not defined by starting/ending metatag it should be fine.

regards, Philipp

Attachments (1)

FilterDefDialog.js (1.1 KB) - added by phil0135 6 years ago.
Patch adjusts operands when using setFilter

Download all attachments as: .zip

Change History (3)

Changed 6 years ago by phil0135

Attachment: FilterDefDialog.js added

Patch adjusts operands when using setFilter

comment:1 Changed 3 years ago by dylan

Milestone: tbd1.12
Owner: changed from Adam Peller to dylan
Status: newassigned
Summary: "allowNamed" does not work when using with dojox.validate.regexp.emailAddress[patch] "allowNamed" does not work when using with dojox.validate.regexp.emailAddress

comment:2 Changed 2 years ago by dylan

Milestone: 1.131.15

Ticket planning... move current 1.13 tickets out to 1.15 to make it easier to move tickets into the 1.13 milestone.

Note: See TracTickets for help on using tickets.