MyFaces Commons
  1. MyFaces Commons
  2. MFCOMMONS-26

validateRegExpr doesnt support value binding in patter property

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.0.1
    • Labels:
      None

      Description

      As from version 1.1.7 validateRegExpr tag desn't support value binding in pattern field.

      ValidateRegExprTag.java
      Version 1.1.6 has:
      protected Validator createValidator() throws JspException {

      FacesContext facesContext = FacesContext.getCurrentInstance();
      setValidatorId(RegExprValidator.VALIDATOR_ID);
      RegExprValidator validator = (RegExprValidator)super.createValidator();
      if (_pattern != null)
      {
      if (UIComponentTag.isValueReference(_pattern))

      { ValueBinding vb = facesContext.getApplication().createValueBinding(_pattern); validator.setPattern(new String(vb.getValue(facesContext).toString())); }

      else

      { validator.setPattern(_pattern); }

      }

      return validator;
      }

      Version 1.1.7 has:
      protected Validator createValidator() throws JspException {

      FacesContext facesContext = FacesContext.getCurrentInstance();
      setValidatorId("org.apache.myfaces.validator.RegExpr");
      org.apache.myfaces.custom.regexprvalidator.RegExprValidator validator = (org.apache.myfaces.custom.regexprvalidator.RegExprValidator)super.createValidator();
      if (_pattern != null)

      { validator.setPattern(_pattern); }

      return validator;
      }

        Activity

        Hide
        Leonardo Uribe added a comment -

        rev 668968, fields that read regular expressions like pattern should be literalOnly, in other words does not read ValueExpressions, because some regular expressions are not readed correctly if ValueExpression is activated.

        Show
        Leonardo Uribe added a comment - rev 668968, fields that read regular expressions like pattern should be literalOnly, in other words does not read ValueExpressions, because some regular expressions are not readed correctly if ValueExpression is activated.
        Hide
        Leonardo Uribe added a comment -

        Added patternExpression property.

        Show
        Leonardo Uribe added a comment - Added patternExpression property.

          People

          • Assignee:
            Leonardo Uribe
            Reporter:
            Tomasz Bech
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development