mirror of https://github.com/apache/openjpa.git
OPENJPA-2230. Set the _activated flag when processing EventListeners in super classes.
git-svn-id: https://svn.apache.org/repos/asf/openjpa/trunk@1361354 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
parent
f30d74802d
commit
5eb13dbde9
|
@ -204,6 +204,7 @@ public class LifecycleMetaData
|
|||
return _declared;
|
||||
if (_declared == null && _ignoreSups == 0) {
|
||||
_high = supMeta._high;
|
||||
_activated = true;
|
||||
return supMeta._all;
|
||||
}
|
||||
// don't hold strong refs onto redundant info
|
||||
|
|
|
@ -0,0 +1,29 @@
|
|||
/*
|
||||
* Licensed to the Apache Software Foundation (ASF) under one
|
||||
* or more contributor license agreements. See the NOTICE file
|
||||
* distributed with this work for additional information
|
||||
* regarding copyright ownership. The ASF licenses this file
|
||||
* to you under the Apache License, Version 2.0 (the
|
||||
* "License"); you may not use this file except in compliance
|
||||
* with the License. You may obtain a copy of the License at
|
||||
*
|
||||
* http://www.apache.org/licenses/LICENSE-2.0
|
||||
*
|
||||
* Unless required by applicable law or agreed to in writing,
|
||||
* software distributed under the License is distributed on an
|
||||
* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
|
||||
* KIND, either express or implied. See the License for the
|
||||
* specific language governing permissions and limitations
|
||||
* under the License.
|
||||
*/
|
||||
|
||||
package org.apache.openjpa.persistence.callbacks;
|
||||
|
||||
import javax.persistence.EntityListeners;
|
||||
import javax.persistence.MappedSuperclass;
|
||||
|
||||
@MappedSuperclass
|
||||
@EntityListeners({ListenerImpl.class})
|
||||
public abstract class EntityListenerMappedSuperClass {
|
||||
|
||||
}
|
|
@ -0,0 +1,47 @@
|
|||
/*
|
||||
* Licensed to the Apache Software Foundation (ASF) under one
|
||||
* or more contributor license agreements. See the NOTICE file
|
||||
* distributed with this work for additional information
|
||||
* regarding copyright ownership. The ASF licenses this file
|
||||
* to you under the Apache License, Version 2.0 (the
|
||||
* "License"); you may not use this file except in compliance
|
||||
* with the License. You may obtain a copy of the License at
|
||||
*
|
||||
* http://www.apache.org/licenses/LICENSE-2.0
|
||||
*
|
||||
* Unless required by applicable law or agreed to in writing,
|
||||
* software distributed under the License is distributed on an
|
||||
* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
|
||||
* KIND, either express or implied. See the License for the
|
||||
* specific language governing permissions and limitations
|
||||
* under the License.
|
||||
*/
|
||||
package org.apache.openjpa.persistence.callbacks;
|
||||
|
||||
import javax.persistence.Entity;
|
||||
import javax.persistence.Id;
|
||||
import javax.persistence.GeneratedValue;
|
||||
|
||||
@Entity
|
||||
public class MSCListenerEntity extends EntityListenerMappedSuperClass implements ListenerTestEntity {
|
||||
@Id @GeneratedValue
|
||||
private long id;
|
||||
|
||||
private int value;
|
||||
|
||||
public long getId() {
|
||||
return id;
|
||||
}
|
||||
|
||||
public void setId(long id) {
|
||||
this.id = id;
|
||||
}
|
||||
|
||||
public int getValue() {
|
||||
return value;
|
||||
}
|
||||
|
||||
public void setValue(int value) {
|
||||
this.value = value;
|
||||
}
|
||||
}
|
|
@ -0,0 +1,119 @@
|
|||
/*
|
||||
* Licensed to the Apache Software Foundation (ASF) under one
|
||||
* or more contributor license agreements. See the NOTICE file
|
||||
* distributed with this work for additional information
|
||||
* regarding copyright ownership. The ASF licenses this file
|
||||
* to you under the Apache License, Version 2.0 (the
|
||||
* "License"); you may not use this file except in compliance
|
||||
* with the License. You may obtain a copy of the License at
|
||||
*
|
||||
* http://www.apache.org/licenses/LICENSE-2.0
|
||||
*
|
||||
* Unless required by applicable law or agreed to in writing,
|
||||
* software distributed under the License is distributed on an
|
||||
* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
|
||||
* KIND, either express or implied. See the License for the
|
||||
* specific language governing permissions and limitations
|
||||
* under the License.
|
||||
*/
|
||||
package org.apache.openjpa.persistence.callbacks;
|
||||
|
||||
import org.apache.openjpa.persistence.test.SingleEMFTestCase;
|
||||
import org.apache.openjpa.persistence.OpenJPAEntityManager;
|
||||
|
||||
/*
|
||||
* Separate testcase from TestEntityListeners to avoid the declaration of
|
||||
* a System level event listener in the orm.xml file...
|
||||
*/
|
||||
public class TestMSCEntityListeners extends SingleEMFTestCase {
|
||||
|
||||
private static final int MSC_LISTENER_ENTITY = 4;
|
||||
|
||||
public void setUp() {
|
||||
setUp(CLEAR_TABLES);
|
||||
ListenerImpl.prePersistCount = 0;
|
||||
ListenerImpl.postPersistCount = 0;
|
||||
ListenerImpl.preUpdateCount = 0;
|
||||
ListenerImpl.postUpdateCount = 0;
|
||||
ListenerImpl.preRemoveCount = 0;
|
||||
ListenerImpl.postRemoveCount = 0;
|
||||
ListenerImpl.postLoadCount = 0;
|
||||
}
|
||||
|
||||
@Override
|
||||
protected String getPersistenceUnitName() {
|
||||
return "msclistener-pu";
|
||||
}
|
||||
|
||||
public void testMSCEntityListeners() {
|
||||
helper(MSC_LISTENER_ENTITY);
|
||||
}
|
||||
|
||||
public void helper(int entityListeners) {
|
||||
OpenJPAEntityManager em = emf.createEntityManager();
|
||||
try {
|
||||
em.getTransaction().begin();
|
||||
ListenerTestEntity o = null;
|
||||
switch (entityListeners) {
|
||||
case MSC_LISTENER_ENTITY:
|
||||
o = new MSCListenerEntity();
|
||||
break;
|
||||
|
||||
}
|
||||
em.persist(o);
|
||||
|
||||
assertStatus(1, 0, 0, 0, 0, 0, 0);
|
||||
|
||||
em.getTransaction().commit();
|
||||
long id = o.getId();
|
||||
em.close();
|
||||
|
||||
assertStatus(1, 1, 0, 0, 0, 0, 0);
|
||||
|
||||
em = emf.createEntityManager();
|
||||
em.getTransaction().begin();
|
||||
switch (entityListeners) {
|
||||
case MSC_LISTENER_ENTITY:
|
||||
o = em.find(MSCListenerEntity.class, id);
|
||||
break;
|
||||
|
||||
}
|
||||
assertNotNull(o);
|
||||
assertStatus(1, 1, 0, 0, 0, 0, 1);
|
||||
|
||||
o.setValue(o.getValue() + 1);
|
||||
|
||||
em.flush();
|
||||
assertStatus(1, 1, 1, 1, 0, 0, 1);
|
||||
|
||||
em.remove(o);
|
||||
assertStatus(1, 1, 1, 1, 1, 0, 1);
|
||||
|
||||
em.getTransaction().commit();
|
||||
|
||||
assertStatus(1, 1, 1, 1, 1, 1, 1);
|
||||
|
||||
em.close();
|
||||
} finally {
|
||||
if (em != null && em.getTransaction().isActive())
|
||||
em.getTransaction().rollback();
|
||||
if (em != null && em.isOpen())
|
||||
em.close();
|
||||
}
|
||||
}
|
||||
|
||||
private void assertStatus(
|
||||
int prePersist, int postPersist,
|
||||
int preUpdate, int postUpdate,
|
||||
int preRemove, int postRemove,
|
||||
int postLoad) {
|
||||
assertEquals(prePersist, ListenerImpl.prePersistCount);
|
||||
assertEquals(postPersist, ListenerImpl.postPersistCount);
|
||||
assertEquals(preUpdate, ListenerImpl.preUpdateCount);
|
||||
assertEquals(postUpdate, ListenerImpl.postUpdateCount);
|
||||
assertEquals(preRemove, ListenerImpl.preRemoveCount);
|
||||
assertEquals(postRemove, ListenerImpl.postRemoveCount);
|
||||
assertEquals(postLoad, ListenerImpl.postLoadCount);
|
||||
}
|
||||
|
||||
}
|
|
@ -111,6 +111,15 @@
|
|||
</properties>
|
||||
</persistence-unit>
|
||||
|
||||
<persistence-unit name="msclistener-pu">
|
||||
<class>org.apache.openjpa.persistence.callbacks.MSCListenerEntity</class>
|
||||
<class>org.apache.openjpa.persistence.callbacks.EntityListenerMappedSuperClass</class>
|
||||
<properties>
|
||||
<property name="openjpa.jdbc.SynchronizeMappings"
|
||||
value="buildSchema(ForeignKeys=true)"/>
|
||||
</properties>
|
||||
</persistence-unit>
|
||||
|
||||
<persistence-unit name="test-unique-constraint">
|
||||
<mapping-file>org/apache/openjpa/persistence/jdbc/unique/orm.xml</mapping-file>
|
||||
<class>org.apache.openjpa.persistence.jdbc.unique.UniqueA</class>
|
||||
|
|
Loading…
Reference in New Issue