Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.0-M2
    • Fix Version/s: 2.0.0-M2
    • Component/s: jpa
    • Labels:
      None

      Description

      This task is to add support for basic lifecycle validation capabilities in the kernel and pluggable validation in the persistence layer.

        Issue Links

          Activity

          Hide
          techhusky Jeremy Bauer added a comment -

          Committed rev 1 under revision 776009. This rev provides a means for lifecycle-based validation by extending the LifecycleEventManager with a ValidatingLifecycleEventManager. The validating version uses a pluggable validation implementation to determine if validation should take place and then to validate the entity. A simple JSR-303 based validation impl wrapper is also included. The config code to enable validation (use the validating lifecycle event manager and plug in the validation impl) will be added under a separate item.

          Show
          techhusky Jeremy Bauer added a comment - Committed rev 1 under revision 776009. This rev provides a means for lifecycle-based validation by extending the LifecycleEventManager with a ValidatingLifecycleEventManager. The validating version uses a pluggable validation implementation to determine if validation should take place and then to validate the entity. A simple JSR-303 based validation impl wrapper is also included. The config code to enable validation (use the validating lifecycle event manager and plug in the validation impl) will be added under a separate item.
          Hide
          ppoddar@apache.org Pinaki Poddar added a comment -

          Is the following a bug in ValidatingLifeCycleManager and its correction?

          @@ -75,7 +74,7 @@

          • return _validator.validating(source, LifecycleEvent.BEFORE_PERSIST) ||
            + return _validator.validating(source, LifecycleEvent.BEFORE_UPDATE) ||

          @@ -84,7 +83,7 @@

          • return _validator.validating(source, LifecycleEvent.BEFORE_UPDATE) ||
            + return _validator.validating(source, LifecycleEvent.BEFORE_PERSIST) ||
          Show
          ppoddar@apache.org Pinaki Poddar added a comment - Is the following a bug in ValidatingLifeCycleManager and its correction? @@ -75,7 +74,7 @@ return _validator.validating(source, LifecycleEvent.BEFORE_PERSIST) || + return _validator.validating(source, LifecycleEvent.BEFORE_UPDATE) || @@ -84,7 +83,7 @@ return _validator.validating(source, LifecycleEvent.BEFORE_UPDATE) || + return _validator.validating(source, LifecycleEvent.BEFORE_PERSIST) ||
          Hide
          ppoddar@apache.org Pinaki Poddar added a comment -

          1. ValidatingLifeCycleEventManager : The JavaDoc for constructor is incorrect.

          2. ValidatingLifeCycleEventManager : Carries some auto-generated method headers

          3. Validator interface JavaDoc is incorrect.

          Show
          ppoddar@apache.org Pinaki Poddar added a comment - 1. ValidatingLifeCycleEventManager : The JavaDoc for constructor is incorrect. 2. ValidatingLifeCycleEventManager : Carries some auto-generated method headers 3. Validator interface JavaDoc is incorrect.

            People

            • Assignee:
              techhusky Jeremy Bauer
              Reporter:
              techhusky Jeremy Bauer
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Due:
                Created:
                Updated:
                Resolved:

                Development