Uploaded image for project: 'MyFaces Core'
  1. MyFaces Core
  2. MYFACES-1892

validator element in faces-config should support nested attribute and property definitions

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Won't Fix
    • 1.2.3
    • None
    • General
    • None

    Description

      As shown in this dtd:
      http://java.sun.com/dtd/web-facesconfig_1_1.dtd

      the validator element in a faces-config.xml file should support nested attribute and property elements:

      <validator>
      <validator-id>xyValidtor</validator-name>
      <validator-class>com.xy.XyValidator</validator-class>
      <property>
      <property-name>length</property-name>
      <property-class>java.lang.Integer</property>
      <default-value>40</default-value>
      </property>
      </validator>

      However this appears to never have been implemented in either 1.1.x or 1.2.x of core; only validator-id and validator-class are supported. Note that the equivalent feature exists for converters, and does appear to have been implemented.

      See the digester rules registered in the constructor for class
      org.apache.myfaces.config.impl.digester.DigesterFacesConfigUnmarshallerImpl

      Reported by Joerg (superjoerch at gmx.de) on the myfaces user list,

      Attachments

        1. MYFACES-1892.patch
          34 kB
          Dora Rajappan
        2. 1892impl4_attribute.patch
          3 kB
          Dora Rajappan
        3. 1892impl3_attribute.patch
          1 kB
          Dora Rajappan
        4. 1892impl2_attribute.patch
          7 kB
          Dora Rajappan
        5. 1892impl1_attribute.patch
          34 kB
          Dora Rajappan
        6. 1892api2_attribute.patch
          13 kB
          Dora Rajappan
        7. 1892api1_attribute.patch
          2 kB
          Dora Rajappan

        Activity

          People

            Unassigned Unassigned
            skitching Simon Kitching
            Votes:
            1 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: