Javascript returning a string as a regex for jquery validate add method

Question

I have this jquery validate add method below with a specific regex.

What I want is to create a method that returns the regex, like in my second example, but the second example doesn't work. I get some error that says 'Object doesn't support property or method 'test'

if (scope.countryCode == "DE") {
  $.validator.addMethod('PostalCodeError',
    function(value) {
      return /^(?!01000|99999)(0[1-9]\d{3}|[1-9]\d{4})$/.test(value);
    }, 'Please enter a valid German postal code.');

  $("#PostalCode").rules("add", {
    PostalCodeError: true
  });
}

I want something like this below

$.validator.addMethod('PostalCodeError',
  function(value) {
    return GetCountryRegex().test(value);
  }, 'Please enter a valid postal code.');

$("#PostalCode").rules("add", {
  PostalCodeError: true
});


function GetCountryRegex() {
  if (scope.countryCode == "DE") {
    return '/^(?!01000|99999)(0[1-9]\d{3}|[1-9]\d{4})$/';
  }
  if (scope.countryCode == "AT") {
    return '/^\\d{4}$/';
  }
}

note that /regexp/ and '/regexp/' are 2 different things (the second is a string). Try with return new RegExp('regexp'); — Kaddath
– Kaddath, Commented Dec 19, 2018 at 8:20
I removed the Code Snippets. They are completely pointless if you don't include the proper HTML/CSS/JavaScript/plugins to make a live demo. — Sparky
– Sparky, Commented Dec 19, 2018 at 17:56

Matthew Herbst · Accepted Answer · 2018-12-19 08:22:13Z

3

So, in GetCountryRegex you're not actually returning RegEx, your returning strings.

Use new RegExp on the returned value to convert the strings to RegExp:

function GetCountryRegex() {
  if (scope.countryCode == "DE") {
    return '/^(?!01000|99999)(0[1-9]\d{3}|[1-9]\d{4})$/';
  }
  if (scope.countryCode == "AT") {
    return '/^\\d{4}$/';
  }
}

var regExp = new RegExp(GetCountryRegex());
regExp.test(...);

answered Dec 19, 2018 at 8:22

Matthew Herbst

32.3k27 gold badges92 silver badges141 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Javascript returning a string as a regex for jquery validate add method

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related