MyFaces Extensions Validator
  1. MyFaces Extensions Validator
  2. EXTVAL-105

AbstractValidationInterceptor: call to processAfterValidation() even if object was not validated

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.2.3, 2.0.3, 1.1.3
    • Fix Version/s: 1.2.4, 2.0.4, 1.1.4
    • Component/s: Core
    • Labels:
      None

      Description

      Is that correct: in case the uiComponent should not be processed and in consequence is not validated, the processing after validation is executed!?

      org.apache.myfaces.extensions.validator.core.interceptor.AbstractValidationInterceptor:

      try
      {

          1. if(processComponent(uiComponent)) ###
            {
            convertedObject = transformValueForValidation(convertedObject);

      if(validateValue(convertedObject) &&
      processBeforeValidation(facesContext, uiComponent, convertedObject))

      { processValidation(facesContext, uiComponent, convertedObject); }

      }
      }
      catch (ValidatorException e)
      {
      try

      { //ViolationSeverityInterpreter might decide that it isn't an exception ExtValUtils.tryToThrowValidatorExceptionForComponent(uiComponent, e.getFacesMessage(), e); }

      catch (ValidatorException finalException)

      { throw new ConverterException(e.getFacesMessage(), e); }

      }
      finally

      { ### processAfterValidation(facesContext, uiComponent, convertedObject); ### resetRendererInterceptorProperties(uiComponent); }

        Activity

        Hide
        Tom M. added a comment - - edited

        Fixed by Gerhard Petracek - SVN r961924

        Show
        Tom M. added a comment - - edited Fixed by Gerhard Petracek - SVN r961924

          People

          • Assignee:
            Unassigned
            Reporter:
            Tom M.
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development