Ticket #5411 (closed defect: fixed)

Opened 11 months ago

Last modified 11 months ago

dojox.regexp.host uses dojox.regexp.integer, which has been removed

Reported by: guest Owned by: peller
Priority: normal Milestone: 1.1
Component: Dojox Version: 1.0
Severity: normal Keywords:
Cc:

Description

The dojo.regexp.host function, which is called by dojox.validate.isUrl relies on a function which has been removed. I think we're supposed to use dojo.number._integerRegexp instead. Patch below... hope it's correct:

===================================================================
--- regexp.js   (revision 11819)
+++ regexp.js   (working copy)
@@ -153,7 +153,7 @@
        var domainNameRE = "([0-9a-zA-Z]([-0-9a-zA-Z]{0,61}[0-9a-zA-Z])?\.)+" + dojox.regexp.tld(flags);
 
        // port number RE
-       var portRE = ( flags.allowPort ) ? "(\:" + dojox.regexp.integer({signed: false}) + ")?" : "";
+       var portRE = ( flags.allowPort ) ? "(\:" + dojo.number._integerRegexp({signed: false}) + ")?" : "";
 
        // build host RE
        var hostNameRE = domainNameRE;

Change History

Changed 11 months ago by peller

nah, this is a non-formatted number, so a simple d+ ought to do it.

Changed 11 months ago by peller

  • status changed from new to closed
  • resolution set to fixed

(In [11854]) Remove obsolete regexp.integer reference with d+ Fixes #5411

Changed 11 months ago by peller

  • milestone changed from 1.0.3 to 1.1

unit tests are still b0rken. See #4350

Note: See TracTickets for help on using tickets.