Uploaded image for project: 'Struts 2'
  1. Struts 2
  2. WW-2923

Allow validator parameters to be set via OGNL rather than hard coding in XML

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 2.1.2
    • 2.3.12
    • XML Validators
    • None
    • Standard

    • Patch

    Description

      I have been experimenting with a softer way to perform validations. I like the idea of externalizing the validations in XML, however, it would be more flexible if the parameters could be "set" using matches from the ValueStack rather than requiring them to be hardcoded in the XML. I have drilled into the validator creation and believe that the ValidatorConfig.Builder(ValidatorConfig cfg) might be the place to check for ${foo}. There are many other places in the framework where this string is searched for and its content replaced by using an OGNL search.

      In my current use case, I am fetching valid ranges from a database according to user inputs. These varying integers are what I would like to use as my min and max values. I realize I could use an expression type validation or just code the validation inside my action. However, it doesn't seem natural to be forced into an alternative technique with the ValueStack being so central to the framework. It seems that if the value stack can be used for the validator message "Age must be between ${min} and ${max}" it should also be available for fetching constraints.

      <field name="age">
      	<field-validator type="int">
      		<param name="min">${minLimit}</param>
      		<param name="max">${maxLimit}</param>
      		<message>Age must be between ${minLimit} and ${maxLimit}</message>
      	</field-validator>
      </field>
      

      Peace,
      Scott

      Attachments

        There are no Sub-Tasks for this issue.

        Activity

          People

            lukaszlenart Lukasz Lenart
            stanlick Scott Stanlick
            Votes:
            1 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: