[jboss-jira] [JBoss JIRA] (AS7-4576) No ConstraintViolationException thrown by EntityManager persist upon pre-persist constraint violation

Scott Marlow (JIRA) jira-events at lists.jboss.org
Fri Apr 20 10:10:18 EDT 2012


    [ https://issues.jboss.org/browse/AS7-4576?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12686076#comment-12686076 ] 

Scott Marlow commented on AS7-4576:
-----------------------------------

I think the test should be adding a pre-persist entity listener instead. And seeing if the constraint happens after pre-persist has completed or not.

The actual database changes may be applied during the EntityManager.persist or EntityManager.flush or during tx commit.  The validation needs to happen after the point of pre-persist (is my read).

                
> No ConstraintViolationException thrown by EntityManager persist upon pre-persist constraint violation
> -----------------------------------------------------------------------------------------------------
>
>                 Key: AS7-4576
>                 URL: https://issues.jboss.org/browse/AS7-4576
>             Project: Application Server 7
>          Issue Type: Bug
>          Components: JPA / Hibernate
>    Affects Versions: 7.1.1.Final
>         Environment: as7.1.2-SNAPSHOT (2012-04-20) c525e538fda12bde76b44a6db4f61d55105c1cea, hibernate4
>            Reporter: Marek Schmidt
>            Assignee: Scott Marlow
>         Attachments: jboss-as-kitchensink.war
>
>
> Per JPA 2.0 spec 3.6.1.2 Requirements for Automatic Validation upon Lifecycle Events, the "persistence provider must throw the javax.validation.ConstraintViolationException" ... and the validation must occur "immediately after the point" where callback method invocations, such as PrePersist have been completed.
> Having a Member entity with some @NonNull validator present on one of its fields, the following test shows the javax.validation.ConstraintViolationException is not thrown while invoking persist, but later inside flush():
> {noformat}
>       try {
>          System.out.println("XXX: before em.persist()");
>          em.persist(new Member());
>          System.out.println("XXX: after em.persist()");
>          em.flush();
>          System.out.println("XXX: after em.flush()");
>       }
>       catch(javax.validation.ConstraintViolationException x) {
>          System.out.println("XXX: Constraint violation!");
>       }
> {noformat}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        


More information about the jboss-jira mailing list