Tapestry 5
  1. Tapestry 5
  2. TAP5-1319

Need to support property paths for validated field

    Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 5.2
    • Fix Version/s: None
    • Component/s: tapestry-beanvalidator
    • Labels:
      None

      Description

      Current implementation does not support property paths for beans, while the Bean Validation API and Hibernate Validator support this feature.

      In attachment my own implementation of BeanFieldValidator.java with support this feature as well as isRequired() method.

      1. ImprovedBeanFieldValidator.java
        7 kB
        Vladimir Velikiy
      2. propertyPathSupport.diff
        7 kB
        Vladimir Velikiy
      3. RequiredContext.java
        0.2 kB
        Vladimir Velikiy
      4. RequiredContextImpl.java
        0.5 kB
        Vladimir Velikiy

        Activity

        Hide
        Vladimir Velikiy added a comment -

        No, this patch allow property traversal for subobjects fields in property paths. For example: organization.address.city.

        Show
        Vladimir Velikiy added a comment - No, this patch allow property traversal for subobjects fields in property paths. For example: organization.address.city.
        Hide
        Dustin Woods added a comment -

        Does this patch allow property traversal for those marked with @Valid or is that a different issue?

        Show
        Dustin Woods added a comment - Does this patch allow property traversal for those marked with @Valid or is that a different issue?
        Hide
        Vladimir Velikiy added a comment -

        Patch for property path support attached

        Show
        Vladimir Velikiy added a comment - Patch for property path support attached
        Hide
        Igor Drobiazko added a comment -

        If you want any changes to be applied to the code base you should provid a patch.

        Show
        Igor Drobiazko added a comment - If you want any changes to be applied to the code base you should provid a patch.

          People

          • Assignee:
            Unassigned
            Reporter:
            Vladimir Velikiy
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:

              Development