Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.3
    • Fix Version/s: None
    • Labels:
      None
    • Environment:

      Operating System: other
      Platform: Other

      Description

      Accept /foo/bar/* matches.
      So digester.addBeanPropertySetter("foo/bar/*);

      will work with

      <foo>
      <bar>
      <propertyA>snafu</propertyA>
      <propertyB>snafu</propertyB>
      </bar>
      </foo>

      where propertyA and propertyB are bar Javabean properties.

        Activity

        Hide
        Kelvin Tan added a comment -

        Created an attachment (id=4517)
        Patch for RuleBase, EBR and required tests

        Show
        Kelvin Tan added a comment - Created an attachment (id=4517) Patch for RuleBase, EBR and required tests
        Hide
        rdonkin@apache.org added a comment -

        hi kelvin

        RulesBase is intended to be a basic implementation and so the adding the
        matching rule you propose isn't really appropriate.

        I will look at adding this to ExtendedBaseRules.

        Show
        rdonkin@apache.org added a comment - hi kelvin RulesBase is intended to be a basic implementation and so the adding the matching rule you propose isn't really appropriate. I will look at adding this to ExtendedBaseRules.
        Hide
        Kelvin Tan added a comment -

        Robert,

        The patch for EBR is included in the attachment, but the added tests in RulesBaseTestCase have to be moved to EBRTestCase.

        Show
        Kelvin Tan added a comment - Robert, The patch for EBR is included in the attachment, but the added tests in RulesBaseTestCase have to be moved to EBRTestCase.
        Hide
        rdonkin@apache.org added a comment -

        this will have to wait until after the upcoming 1.4 release

        Show
        rdonkin@apache.org added a comment - this will have to wait until after the upcoming 1.4 release
        Hide
        rdonkin@apache.org added a comment -

        digester 1.4 released

        Show
        rdonkin@apache.org added a comment - digester 1.4 released
        Hide
        rdonkin@apache.org added a comment -

        after 1.4.1 release

        Show
        rdonkin@apache.org added a comment - after 1.4.1 release
        Hide
        rdonkin@apache.org added a comment -

        digester 1.4.1 released

        Show
        rdonkin@apache.org added a comment - digester 1.4.1 released
        Hide
        rdonkin@apache.org added a comment -

        I've committed a solution based on this patch. This is based on your patch but
        extended and integrates the concept. This has been added to ExtendedBaseRules only.

        Show
        rdonkin@apache.org added a comment - I've committed a solution based on this patch. This is based on your patch but extended and integrates the concept. This has been added to ExtendedBaseRules only.

          People

          • Assignee:
            Unassigned
            Reporter:
            Kelvin Tan
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development