Author: adamw Date: 2010-07-06 02:31:40 -0400 (Tue, 06 Jul 2010) New Revision: 19902 Added: core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/AbstractSessionTest.java core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/auditedEntity/ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/auditedEntity/Person.java core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/auditedEntity/ReadEntityWhtiEntityNameTest.java core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/manyToManyAudited/ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/manyToManyAudited/Car.java core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/manyToManyAudited/Person.java core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/manyToManyAudited/ReadEntityWithAuditedManyToManyTest.java core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyAudited/ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyAudited/Car.java core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyAudited/Person.java core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyAudited/ReadEntityWithAuditedCollectionTest.java core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyNotAudited/ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyNotAudited/Car.java core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyNotAudited/Person.java core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyNotAudited/ReadEntityWithAuditedCollectionTest.java core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedAudited/ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedAudited/Car.java core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedAudited/Person.java core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedAudited/ReadEntityAssociatedAuditedTest.java core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedNotAudited/ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedNotAudited/Car.java core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedNotAudited/Person.java core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedNotAudited/ReadEntityAssociatedNotAuditedTest.java core/branches/Branch_3_5/envers/src/test/resources/hibernate.test.session-cfg.xml core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/ core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/auditedEntity/ core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/auditedEntity/mappings.hbm.xml core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/manyToManyAudited/ core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/manyToManyAudited/mappings.hbm.xml core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/oneToManyAudited/ core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/oneToManyAudited/mappings.hbm.xml core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/oneToManyNotAudited/ core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/oneToManyNotAudited/mappings.hbm.xml core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/singleAssociatedAudited/ core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/singleAssociatedAudited/mappings.hbm.xml core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/singleAssociatedNotAudited/ core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/singleAssociatedNotAudited/mappings.hbm.xml Modified: core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/AuditReader.java core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/configuration/metadata/AuditMetadataGenerator.java core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/configuration/metadata/CollectionMetadataGenerator.java core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/entities/EntityConfiguration.java core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/entities/EntityInstantiator.java core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/entities/mapper/relation/OneToOneNotOwningMapper.java core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/entities/mapper/relation/ToOneIdMapper.java core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/entities/mapper/relation/lazy/ToOneDelegateSessionImplementor.java core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/event/AuditEventListener.java core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/query/AuditQueryCreator.java core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/query/impl/AbstractAuditQuery.java core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/query/impl/EntitiesAtRevisionQuery.java core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/query/impl/RevisionsOfEntityQuery.java core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/reader/AuditReaderImpl.java core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/tools/Tools.java core/branches/Branch_3_5/envers/src/test/resources/testng.xml Log: HHH-4716: - backporting the fix from trunk - supporting entity name concept Modified: core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/AuditReader.java =================================================================== --- core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/AuditReader.java 2010-07-05 17:36:04 UTC (rev 19901) +++ core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/AuditReader.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -32,7 +32,7 @@ /** * @author Adam Warski (adam at warski dot org) - * @author Hernan Chanfreau + * @author Hern�n Chanfreau */ public interface AuditReader { /** @@ -49,6 +49,23 @@ */ <T> T find(Class<T> cls, Object primaryKey, Number revision) throws IllegalArgumentException, NotAuditedException, IllegalStateException; + + /** + * Find an entity by primary key at the given revision with the specified entityName. + * @param cls Class of the entity. + * @param entityName EntityName to find. + * @param primaryKey Primary key of the entity. + * @param revision Revision in which to get the entity. + * @return The found entity instance at the given revision (its properties may be partially filled + * if not all properties are audited) or null, if an entity with that id didn't exist at that + * revision. + * @throws IllegalArgumentException If cls or primaryKey is null or revision is less or equal to 0. + * @throws NotAuditedException When entities of the given class are not audited. + * @throws IllegalStateException If the associated entity manager is closed. + */ + <T> T find(Class<T> cls, String entityName, Object primaryKey, + Number revision) throws IllegalArgumentException, + NotAuditedException, IllegalStateException; /** * Get a list of revision numbers, at which an entity was modified. @@ -62,6 +79,21 @@ */ List<Number> getRevisions(Class<?> cls, Object primaryKey) throws IllegalArgumentException, NotAuditedException, IllegalStateException; + + /** + * Get a list of revision numbers, at which an entity was modified, looking by entityName. + * @param cls Class of the entity. + * @param entityName EntityName to find. + * @param primaryKey Primary key of the entity. + * @return A list of revision numbers, at which the entity was modified, sorted in ascending order (so older + * revisions come first). + * @throws NotAuditedException When entities of the given class are not audited. + * @throws IllegalArgumentException If cls or primaryKey is null. + * @throws IllegalStateException If the associated entity manager is closed. + */ + List<Number> getRevisions(Class<?> cls, String entityName, Object primaryKey) + throws IllegalArgumentException, NotAuditedException, + IllegalStateException; /** * Get the date, at which a revision was created. Modified: core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/configuration/metadata/AuditMetadataGenerator.java =================================================================== --- core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/configuration/metadata/AuditMetadataGenerator.java 2010-07-05 17:36:04 UTC (rev 19901) +++ core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/configuration/metadata/AuditMetadataGenerator.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -53,6 +53,7 @@ * @author Adam Warski (adam at warski dot org) * @author Sebastian Komander * @author Tomasz Bech + * @author Hern�n Chanfreau */ public final class AuditMetadataGenerator { private static final Logger log = LoggerFactory.getLogger(AuditMetadataGenerator.class); @@ -353,7 +354,7 @@ ExtendedPropertyMapper propertyMapper = null; String parentEntityName = null; - EntityConfiguration entityCfg = new EntityConfiguration(entityName, idMapper, propertyMapper, + EntityConfiguration entityCfg = new EntityConfiguration(entityName, pc.getClassName(), idMapper, propertyMapper, parentEntityName); notAuditedEntitiesConfigurations.put(entityName, entityCfg); return; @@ -427,7 +428,7 @@ addJoins(pc, propertyMapper, auditingData, pc.getEntityName(), xmlMappingData, true); // Storing the generated configuration - EntityConfiguration entityCfg = new EntityConfiguration(auditEntityName, idMapper, + EntityConfiguration entityCfg = new EntityConfiguration(auditEntityName,pc.getClassName(), idMapper, propertyMapper, parentEntityName); entitiesConfigurations.put(pc.getEntityName(), entityCfg); } Modified: core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/configuration/metadata/CollectionMetadataGenerator.java =================================================================== --- core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/configuration/metadata/CollectionMetadataGenerator.java 2010-07-05 17:36:04 UTC (rev 19901) +++ core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/configuration/metadata/CollectionMetadataGenerator.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -80,7 +80,7 @@ /** * Generates metadata for a collection-valued property. * @author Adam Warski (adam at warski dot org) - * @author Hern�n Chanfreau + * @author Hern�n Chanfreau */ public final class CollectionMetadataGenerator { private static final Logger log = LoggerFactory.getLogger(CollectionMetadataGenerator.class); @@ -209,7 +209,7 @@ // The mapper will only be used to map from entity to map, so no need to provide other details // when constructing the PropertyData. new PropertyData(auditMappedBy, null, null, null), - referencedEntityName, false); + referencingEntityName, false); // Checking if there's an index defined. If so, adding a mapper for it. if (propertyAuditingData.getPositionMappedBy() != null) { Modified: core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/entities/EntityConfiguration.java =================================================================== --- core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/entities/EntityConfiguration.java 2010-07-05 17:36:04 UTC (rev 19901) +++ core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/entities/EntityConfiguration.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -32,18 +32,22 @@ /** * @author Adam Warski (adam at warski dot org) + * @author Hern�n Chanfreau */ public class EntityConfiguration { private String versionsEntityName; - private IdMappingData idMappingData; + /** Holds the className for instantiation the configured entity */ + private String entityClassName; + private IdMappingData idMappingData; private ExtendedPropertyMapper propertyMapper; // Maps from property name private Map<String, RelationDescription> relations; private String parentEntityName; - public EntityConfiguration(String versionsEntityName, IdMappingData idMappingData, + public EntityConfiguration(String versionsEntityName, String entityClassName, IdMappingData idMappingData, ExtendedPropertyMapper propertyMapper, String parentEntityName) { this.versionsEntityName = versionsEntityName; + this.entityClassName = entityClassName; this.idMappingData = idMappingData; this.propertyMapper = propertyMapper; this.parentEntityName = parentEntityName; @@ -113,4 +117,11 @@ Iterable<RelationDescription> getRelationsIterator() { return relations.values(); } + + /** + * @return the className for the configured entity + */ + public String getEntityClassName() { + return entityClassName; + } } Modified: core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/entities/EntityInstantiator.java =================================================================== --- core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/entities/EntityInstantiator.java 2010-07-05 17:36:04 UTC (rev 19901) +++ core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/entities/EntityInstantiator.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -36,6 +36,7 @@ /** * @author Adam Warski (adam at warski dot org) + * @author Hern�n Chanfreau */ public class EntityInstantiator { private final AuditConfiguration verCfg; @@ -80,7 +81,13 @@ // If it is not in the cache, creating a new entity instance Object ret; try { - Class<?> cls = ReflectionTools.loadClass(entityName); + EntityConfiguration entCfg = verCfg.getEntCfg().get(entityName); + if(entCfg == null) { + // a relation marked as RelationTargetAuditMode.NOT_AUDITED + entCfg = verCfg.getEntCfg().getNotVersionEntityConfiguration(entityName); + } + + Class<?> cls = ReflectionTools.loadClass(entCfg.getEntityClassName()); ret = ReflectHelper.getDefaultConstructor(cls).newInstance(); } catch (Exception e) { throw new AuditException(e); Modified: core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/entities/mapper/relation/OneToOneNotOwningMapper.java =================================================================== --- core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/entities/mapper/relation/OneToOneNotOwningMapper.java 2010-07-05 17:36:04 UTC (rev 19901) +++ core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/entities/mapper/relation/OneToOneNotOwningMapper.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -31,6 +31,7 @@ import org.hibernate.envers.configuration.AuditConfiguration; import org.hibernate.envers.entities.mapper.PersistentCollectionChangeData; import org.hibernate.envers.entities.mapper.PropertyMapper; +import org.hibernate.envers.entities.EntityConfiguration; import org.hibernate.envers.entities.PropertyData; import org.hibernate.envers.exception.AuditException; import org.hibernate.envers.query.AuditEntity; @@ -44,6 +45,7 @@ /** * @author Adam Warski (adam at warski dot org) + * @author Hern�n Chanfreau */ public class OneToOneNotOwningMapper implements PropertyMapper { private String owningReferencePropertyName; @@ -66,12 +68,18 @@ return; } - Class<?> entityClass = ReflectionTools.loadClass(owningEntityName); + EntityConfiguration entCfg = verCfg.getEntCfg().get(owningEntityName); + if(entCfg == null) { + // a relation marked as RelationTargetAuditMode.NOT_AUDITED + entCfg = verCfg.getEntCfg().getNotVersionEntityConfiguration(owningEntityName); + } + Class<?> entityClass = ReflectionTools.loadClass(entCfg.getEntityClassName()); + Object value; try { - value = versionsReader.createQuery().forEntitiesAtRevision(entityClass, revision) + value = versionsReader.createQuery().forEntitiesAtRevision(entityClass, owningEntityName, revision) .add(AuditEntity.relatedId(owningReferencePropertyName).eq(primaryKey)).getSingleResult(); } catch (NoResultException e) { value = null; Modified: core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/entities/mapper/relation/ToOneIdMapper.java =================================================================== --- core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/entities/mapper/relation/ToOneIdMapper.java 2010-07-05 17:36:04 UTC (rev 19901) +++ core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/entities/mapper/relation/ToOneIdMapper.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -33,6 +33,7 @@ import org.hibernate.envers.entities.mapper.PropertyMapper; import org.hibernate.envers.entities.mapper.id.IdMapper; import org.hibernate.envers.entities.mapper.relation.lazy.ToOneDelegateSessionImplementor; +import org.hibernate.envers.entities.EntityConfiguration; import org.hibernate.envers.entities.PropertyData; import org.hibernate.envers.reader.AuditReaderImplementor; import org.hibernate.envers.tools.Tools; @@ -44,6 +45,7 @@ /** * @author Adam Warski (adam at warski dot org) + * @author Hern�n Chanfreau */ public class ToOneIdMapper implements PropertyMapper { private final IdMapper delegate; @@ -68,7 +70,7 @@ delegate.mapToMapFromEntity(newData, nonInsertableFake ? oldObj : newObj); //noinspection SimplifiableConditionalExpression - return nonInsertableFake ? false : !Tools.entitiesEqual(session, newObj, oldObj); + return nonInsertableFake ? false : !Tools.entitiesEqual(session, referencedEntityName, newObj, oldObj); } public void mapToEntityFromMap(AuditConfiguration verCfg, Object obj, Map data, Object primaryKey, @@ -85,7 +87,13 @@ if (versionsReader.getFirstLevelCache().contains(referencedEntityName, revision, entityId)) { value = versionsReader.getFirstLevelCache().get(referencedEntityName, revision, entityId); } else { - Class<?> entityClass = ReflectionTools.loadClass(referencedEntityName); + EntityConfiguration entCfg = verCfg.getEntCfg().get(referencedEntityName); + if(entCfg == null) { + // a relation marked as RelationTargetAuditMode.NOT_AUDITED + entCfg = verCfg.getEntCfg().getNotVersionEntityConfiguration(referencedEntityName); + } + + Class<?> entityClass = ReflectionTools.loadClass(entCfg.getEntityClassName()); value = versionsReader.getSessionImplementor().getFactory().getEntityPersister(referencedEntityName). createProxy((Serializable)entityId, new ToOneDelegateSessionImplementor(versionsReader, entityClass, entityId, revision, verCfg)); Modified: core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/entities/mapper/relation/lazy/ToOneDelegateSessionImplementor.java =================================================================== --- core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/entities/mapper/relation/lazy/ToOneDelegateSessionImplementor.java 2010-07-05 17:36:04 UTC (rev 19901) +++ core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/entities/mapper/relation/lazy/ToOneDelegateSessionImplementor.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -27,7 +27,6 @@ import org.hibernate.envers.configuration.AuditConfiguration; import org.hibernate.envers.entities.EntitiesConfigurations; -import org.hibernate.envers.entities.EntityConfiguration; import org.hibernate.envers.reader.AuditReaderImplementor; import org.hibernate.HibernateException; @@ -35,6 +34,7 @@ /** * @author Adam Warski (adam at warski dot org) * @author Tomasz Bech + * @author Hern�n Chanfreau */ public class ToOneDelegateSessionImplementor extends AbstractDelegateSessionImplementor { private static final long serialVersionUID = 4770438372940785488L; @@ -43,7 +43,7 @@ private final Class<?> entityClass; private final Object entityId; private final Number revision; - private EntityConfiguration notVersionedEntityConfiguration; + private EntitiesConfigurations entCfg; public ToOneDelegateSessionImplementor(AuditReaderImplementor versionsReader, Class<?> entityClass, Object entityId, Number revision, @@ -53,14 +53,15 @@ this.entityClass = entityClass; this.entityId = entityId; this.revision = revision; - EntitiesConfigurations entCfg = verCfg.getEntCfg(); - notVersionedEntityConfiguration = entCfg.getNotVersionEntityConfiguration(entityClass.getName()); + this.entCfg = verCfg.getEntCfg(); } public Object doImmediateLoad(String entityName) throws HibernateException { - if (notVersionedEntityConfiguration == null) { - return versionsReader.find(entityClass, entityId, revision); + if(entCfg.getNotVersionEntityConfiguration(entityName) == null){ + // audited relation, look up entity with envers + return versionsReader.find(entityClass, entityName, entityId, revision); } else { + // notAudited relation, look up entity with hibernate return delegate.immediateLoad(entityName, (Serializable) entityId); } } Modified: core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/event/AuditEventListener.java =================================================================== --- core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/event/AuditEventListener.java 2010-07-05 17:36:04 UTC (rev 19901) +++ core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/event/AuditEventListener.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -61,6 +61,7 @@ /** * @author Adam Warski (adam at warski dot org) + * @author Hern�n Chanfreau */ public class AuditEventListener implements PostInsertEventListener, PostUpdateEventListener, PostDeleteEventListener, PreCollectionUpdateEventListener, PreCollectionRemoveEventListener, @@ -91,7 +92,7 @@ Object oldValue = oldState == null ? null : oldState[i]; Object newValue = newState == null ? null : newState[i]; - if (!Tools.entitiesEqual(session, oldValue, newValue)) { + if (!Tools.entitiesEqual(session, relDesc.getToEntityName(), oldValue, newValue)) { // We have to generate changes both in the old collection (size decreses) and new collection // (size increases). if (newValue != null) { Modified: core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/query/AuditQueryCreator.java =================================================================== --- core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/query/AuditQueryCreator.java 2010-07-05 17:36:04 UTC (rev 19901) +++ core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/query/AuditQueryCreator.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -32,6 +32,7 @@ /** * @author Adam Warski (adam at warski dot org) + * @author Hern�n Chanfreau */ public class AuditQueryCreator { private final AuditConfiguration auditCfg; @@ -56,6 +57,22 @@ checkPositive(revision, "Entity revision"); return new EntitiesAtRevisionQuery(auditCfg, auditReaderImplementor, c, revision); } + + /** + * Creates a query, which will return entities satisfying some conditions (specified later), + * at a given revision and a given entityName. + * @param c Class of the entities for which to query. + * @param entityName EntityName of the entities for which to query. + * @param revision Revision number at which to execute the query. + * @return A query for entities at a given revision, to which conditions can be added and which + * can then be executed. The result of the query will be a list of entities (beans), unless a + * projection is added. + */ + public AuditQuery forEntitiesAtRevision(Class<?> c, String entityName, Number revision) { + checkNotNull(revision, "Entity revision"); + checkPositive(revision, "Entity revision"); + return new EntitiesAtRevisionQuery(auditCfg, auditReaderImplementor, c, entityName, revision); + } /** * Creates a query, which selects the revisions, at which the given entity was modified. @@ -80,4 +97,30 @@ public AuditQuery forRevisionsOfEntity(Class<?> c, boolean selectEntitiesOnly, boolean selectDeletedEntities) { return new RevisionsOfEntityQuery(auditCfg, auditReaderImplementor, c, selectEntitiesOnly,selectDeletedEntities); } + + /** + * Creates a query, which selects the revisions, at which the given entity was modified and with a given entityName. + * Unless an explicit projection is set, the result will be a list of three-element arrays, containing: + * <ol> + * <li>the entity instance</li> + * <li>revision entity, corresponding to the revision at which the entity was modified. If no custom + * revision entity is used, this will be an instance of {@link org.hibernate.envers.DefaultRevisionEntity}</li> + * <li>type of the revision (an enum instance of class {@link org.hibernate.envers.RevisionType})</li>. + * </ol> + * Additional conditions that the results must satisfy may be specified. + * @param c Class of the entities for which to query. + * @param entityName EntityName of the entities for which to query. + * @param selectEntitiesOnly If true, instead of a list of three-element arrays, a list of entites will be + * returned as a result of executing this query. + * @param selectDeletedEntities If true, also revisions where entities were deleted will be returned. The additional + * entities will have revision type "delete", and contain no data (all fields null), except for the id field. + * @return A query for revisions at which instances of the given entity were modified, to which + * conditions can be added (for example - a specific id of an entity of class <code>c</code>), and which + * can then be executed. The results of the query will be sorted in ascending order by the revision number, + * unless an order or projection is added. + */ + public AuditQuery forRevisionsOfEntity(Class<?> c, String entityName, boolean selectEntitiesOnly, boolean selectDeletedEntities) { + return new RevisionsOfEntityQuery(auditCfg, auditReaderImplementor, c, entityName, selectEntitiesOnly,selectDeletedEntities); + } + } Modified: core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/query/impl/AbstractAuditQuery.java =================================================================== --- core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/query/impl/AbstractAuditQuery.java 2010-07-05 17:36:04 UTC (rev 19901) +++ core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/query/impl/AbstractAuditQuery.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -50,12 +50,14 @@ /** * @author Adam Warski (adam at warski dot org) + * @author Hern�n Chanfreau */ public abstract class AbstractAuditQuery implements AuditQuery { protected EntityInstantiator entityInstantiator; protected List<AuditCriterion> criterions; protected String entityName; + protected String entityClassName; protected String versionsEntityName; protected QueryBuilder qb; @@ -67,18 +69,25 @@ protected AbstractAuditQuery(AuditConfiguration verCfg, AuditReaderImplementor versionsReader, Class<?> cls) { - this.verCfg = verCfg; - this.versionsReader = versionsReader; + this(verCfg, versionsReader, cls, cls.getName()); + } - criterions = new ArrayList<AuditCriterion>(); - entityInstantiator = new EntityInstantiator(verCfg, versionsReader); + protected AbstractAuditQuery(AuditConfiguration verCfg, + AuditReaderImplementor versionsReader, Class<?> cls, String entityName) { + this.verCfg = verCfg; + this.versionsReader = versionsReader; - entityName = cls.getName(); - versionsEntityName = verCfg.getAuditEntCfg().getAuditEntityName(entityName); + criterions = new ArrayList<AuditCriterion>(); + entityInstantiator = new EntityInstantiator(verCfg, versionsReader); - qb = new QueryBuilder(versionsEntityName, "e"); - } + entityClassName = cls.getName(); + this.entityName = entityName; + versionsEntityName = verCfg.getAuditEntCfg().getAuditEntityName( + entityName); + qb = new QueryBuilder(versionsEntityName, "e"); + } + protected List buildAndExecuteQuery() { StringBuilder querySb = new StringBuilder(); Map<String, Object> queryParamValues = new HashMap<String, Object>(); Modified: core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/query/impl/EntitiesAtRevisionQuery.java =================================================================== --- core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/query/impl/EntitiesAtRevisionQuery.java 2010-07-05 17:36:04 UTC (rev 19901) +++ core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/query/impl/EntitiesAtRevisionQuery.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -35,6 +35,7 @@ /** * @author Adam Warski (adam at warski dot org) + * @author Hern�n Chanfreau */ public class EntitiesAtRevisionQuery extends AbstractAuditQuery { private final Number revision; @@ -45,6 +46,12 @@ super(verCfg, versionsReader, cls); this.revision = revision; } + + public EntitiesAtRevisionQuery(AuditConfiguration verCfg, + AuditReaderImplementor versionsReader, Class<?> cls, String entityName, Number revision) { + super(verCfg, versionsReader, cls, entityName); + this.revision = revision; + } @SuppressWarnings({"unchecked"}) public List list() { Modified: core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/query/impl/RevisionsOfEntityQuery.java =================================================================== --- core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/query/impl/RevisionsOfEntityQuery.java 2010-07-05 17:36:04 UTC (rev 19901) +++ core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/query/impl/RevisionsOfEntityQuery.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -33,11 +33,11 @@ import org.hibernate.envers.exception.AuditException; import org.hibernate.envers.query.criteria.AuditCriterion; import org.hibernate.envers.reader.AuditReaderImplementor; - import org.hibernate.proxy.HibernateProxy; /** * @author Adam Warski (adam at warski dot org) + * @author Hern�n Chanfreau */ public class RevisionsOfEntityQuery extends AbstractAuditQuery { private final boolean selectEntitiesOnly; @@ -53,6 +53,15 @@ this.selectDeletedEntities = selectDeletedEntities; } + public RevisionsOfEntityQuery(AuditConfiguration verCfg, + AuditReaderImplementor versionsReader, Class<?> cls, String entityName, + boolean selectEntitiesOnly, boolean selectDeletedEntities) { + super(verCfg, versionsReader, cls, entityName); + + this.selectEntitiesOnly = selectEntitiesOnly; + this.selectDeletedEntities = selectDeletedEntities; + } + private Number getRevisionNumber(Map versionsEntity) { AuditEntitiesConfiguration verEntCfg = verCfg.getAuditEntCfg(); Modified: core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/reader/AuditReaderImpl.java =================================================================== --- core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/reader/AuditReaderImpl.java 2010-07-05 17:36:04 UTC (rev 19901) +++ core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/reader/AuditReaderImpl.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -46,7 +46,7 @@ /** * @author Adam Warski (adam at warski dot org) - * @author Hernan Chanfreau + * @author Hern�n Chanfreau */ public class AuditReaderImpl implements AuditReaderImplementor { private final AuditConfiguration verCfg; @@ -81,17 +81,22 @@ return firstLevelCache; } - @SuppressWarnings({"unchecked"}) public <T> T find(Class<T> cls, Object primaryKey, Number revision) throws IllegalArgumentException, NotAuditedException, IllegalStateException { + + return this.find(cls, cls.getName(), primaryKey, revision); + } + + @SuppressWarnings({"unchecked"}) + public <T> T find(Class<T> cls, String entityName, Object primaryKey, Number revision) throws + IllegalArgumentException, NotAuditedException, IllegalStateException { checkNotNull(cls, "Entity class"); + checkNotNull(entityName, "Entity name"); checkNotNull(primaryKey, "Primary key"); checkNotNull(revision, "Entity revision"); checkPositive(revision, "Entity revision"); checkSession(); - String entityName = cls.getName(); - if (!verCfg.getEntCfg().isVersioned(entityName)) { throw new NotAuditedException(entityName, entityName + " is not versioned!"); } @@ -103,7 +108,7 @@ Object result; try { // The result is put into the cache by the entity instantiator called from the query - result = createQuery().forEntitiesAtRevision(cls, revision) + result = createQuery().forEntitiesAtRevision(cls, entityName, revision) .add(AuditEntity.id().eq(primaryKey)).getSingleResult(); } catch (NoResultException e) { result = null; @@ -112,23 +117,28 @@ } return (T) result; + } + + public List<Number> getRevisions(Class<?> cls, Object primaryKey) + throws IllegalArgumentException, NotAuditedException, IllegalStateException { + + return this.getRevisions(cls, cls.getName(), primaryKey); } @SuppressWarnings({"unchecked"}) - public List<Number> getRevisions(Class<?> cls, Object primaryKey) + public List<Number> getRevisions(Class<?> cls, String entityName, Object primaryKey) throws IllegalArgumentException, NotAuditedException, IllegalStateException { // todo: if a class is not versioned from the beginning, there's a missing ADD rev - what then? checkNotNull(cls, "Entity class"); + checkNotNull(entityName, "Entity name"); checkNotNull(primaryKey, "Primary key"); checkSession(); - String entityName = cls.getName(); - if (!verCfg.getEntCfg().isVersioned(entityName)) { throw new NotAuditedException(entityName, entityName + " is not versioned!"); } - return createQuery().forRevisionsOfEntity(cls, false, true) + return createQuery().forRevisionsOfEntity(cls, entityName, false, true) .addProjection(AuditEntity.revisionNumber()) .add(AuditEntity.id().eq(primaryKey)) .getResultList(); Modified: core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/tools/Tools.java =================================================================== --- core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/tools/Tools.java 2010-07-05 17:36:04 UTC (rev 19901) +++ core/branches/Branch_3_5/envers/src/main/java/org/hibernate/envers/tools/Tools.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -32,6 +32,7 @@ /** * @author Adam Warski (adam at warski dot org) + * @author Hern�n Chanfreau */ public class Tools { public static <K,V> Map<K,V> newHashMap() { @@ -46,14 +47,14 @@ return new LinkedHashMap<K,V>(); } - public static boolean entitiesEqual(SessionImplementor session, Object obj1, Object obj2) { - Object id1 = getIdentifier(session, obj1); - Object id2 = getIdentifier(session, obj2); + public static boolean entitiesEqual(SessionImplementor session, String entityName, Object obj1, Object obj2) { + Object id1 = getIdentifier(session, entityName, obj1); + Object id2 = getIdentifier(session, entityName, obj2); return objectsEqual(id1, id2); - } + } - public static Object getIdentifier(SessionImplementor session, Object obj) { + public static Object getIdentifier(SessionImplementor session, String entityName, Object obj) { if (obj == null) { return null; } @@ -63,9 +64,9 @@ return hibernateProxy.getHibernateLazyInitializer().getIdentifier(); } + return session.getEntityPersister(entityName, obj).getIdentifier(obj, session); + } - return session.getEntityPersister( null, obj ).getIdentifier( obj, session ); - } public static Object getTargetFromProxy(SessionFactoryImplementor sessionFactoryImplementor, HibernateProxy proxy) { if (!proxy.getHibernateLazyInitializer().isUninitialized()) { Added: core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/AbstractSessionTest.java =================================================================== --- core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/AbstractSessionTest.java (rev 0) +++ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/AbstractSessionTest.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,79 @@ +package org.hibernate.envers.test; + +import java.io.File; +import java.net.URISyntaxException; +import java.net.URL; + +import org.hibernate.MappingException; +import org.hibernate.Session; +import org.hibernate.SessionFactory; +import org.hibernate.cfg.AnnotationConfiguration; +import org.hibernate.cfg.Configuration; +import org.hibernate.envers.AuditReader; +import org.hibernate.envers.AuditReaderFactory; +import org.testng.annotations.AfterClass; +import org.testng.annotations.BeforeClass; +import org.testng.annotations.BeforeMethod; + +/** + * Base class for testing envers with Session. + * + * @author Hern�n Chanfreau + * + */ +public abstract class AbstractSessionTest { + + protected Configuration config; + private SessionFactory sessionFactory; + private Session session ; + private AuditReader auditReader; + + + @BeforeClass + public void init() { + config = new AnnotationConfiguration(); + try { + URL url = Thread.currentThread().getContextClassLoader().getResource("hibernate.test.session-cfg.xml"); + config.configure(new File(url.toURI())); + this.initMappings(); + } catch (Exception e) { + // TODO Auto-generated catch block + e.printStackTrace(); + } + + sessionFactory = config.buildSessionFactory(); + } + + protected abstract void initMappings() throws MappingException, URISyntaxException ; + + + + private SessionFactory getSessionFactory(){ + return sessionFactory; + } + + + @BeforeMethod + public void newSessionFactory() { + session = getSessionFactory().openSession(); + auditReader = AuditReaderFactory.get(session); + } + + @AfterClass + public void closeSessionFactory() { + sessionFactory.close(); + } + + + protected Session getSession() { + return session; + } + + + + protected AuditReader getAuditReader() { + return auditReader; + } + +} + Added: core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/auditedEntity/Person.java =================================================================== --- core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/auditedEntity/Person.java (rev 0) +++ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/auditedEntity/Person.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,51 @@ +package org.hibernate.envers.test.entityNames.auditedEntity; + +import org.hibernate.envers.Audited; + +/** + * @author Hern�n Chanfreau + * + */ + +@Audited +public class Person { + + private long id; + + private String name; + + private int age; + + public Person(){ } + + public Person(String name, int age){ + this.name = name; + this.age = age; + } + + + public long getId() { + return id; + } + + public void setId(long id) { + this.id = id; + } + + public String getName() { + return name; + } + + public void setName(String name) { + this.name = name; + } + + public int getAge() { + return age; + } + + public void setAge(int age) { + this.age = age; + } + +} Added: core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/auditedEntity/ReadEntityWhtiEntityNameTest.java =================================================================== --- core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/auditedEntity/ReadEntityWhtiEntityNameTest.java (rev 0) +++ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/auditedEntity/ReadEntityWhtiEntityNameTest.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,99 @@ +package org.hibernate.envers.test.entityNames.auditedEntity; + +import java.io.File; +import java.net.URISyntaxException; +import java.net.URL; +import java.util.List; + +import org.hibernate.MappingException; +import org.hibernate.envers.test.AbstractSessionTest; +import org.testng.annotations.BeforeClass; +import org.testng.annotations.Test; + +/** + * @author Hern�n Chanfreau + * + */ + +public class ReadEntityWhtiEntityNameTest extends AbstractSessionTest{ + + private long id_pers1; + private long id_pers2; + private long id_pers3; + + + protected void initMappings() throws MappingException, URISyntaxException { + URL url = Thread.currentThread().getContextClassLoader().getResource("mappings/entityNames/auditedEntity/mappings.hbm.xml"); + config.addFile(new File(url.toURI())); + } + + + @BeforeClass(dependsOnMethods = "init") + public void initData() { + + newSessionFactory(); + + Person pers1 = new Person("Hernan", 28); + Person pers2 = new Person("Leandro", 29); + Person pers3 = new Person("Barba", 30); + + //REV 1 + getSession().getTransaction().begin(); + getSession().persist("Personaje",pers1); + id_pers1 = pers1.getId(); + getSession().getTransaction().commit(); + + //REV 2 + getSession().getTransaction().begin(); + pers1 = (Person)getSession().get("Personaje", id_pers1); + pers1.setAge(29); + getSession().persist("Personaje",pers1); + getSession().persist("Personaje",pers2); + id_pers2 = pers2.getId(); + getSession().getTransaction().commit(); + + //REV + getSession().getTransaction().begin(); + pers1 = (Person)getSession().get("Personaje", id_pers1); + pers1.setName("Hernan David"); + pers2 = (Person)getSession().get("Personaje", id_pers2); + pers2.setAge(30); + getSession().persist("Personaje",pers1); + getSession().persist("Personaje",pers2); + getSession().persist("Personaje",pers3); + id_pers3 = pers3.getId(); + getSession().getTransaction().commit(); + + } + + + @Test + public void testRetrieveRevisionsWithEntityName() { + List<Number> pers1Revs = getAuditReader().getRevisions(Person.class,"Personaje", id_pers1); + List<Number> pers2Revs = getAuditReader().getRevisions(Person.class,"Personaje", id_pers2); + List<Number> pers3Revs = getAuditReader().getRevisions(Person.class,"Personaje", id_pers3); + + assert(pers1Revs.size() == 3); + assert(pers2Revs.size() == 2); + assert(pers3Revs.size() == 1); + } + + @Test + public void testRetrieveAuditedEntityWithEntityName() { + Person Person1 = getAuditReader().find(Person.class, "Personaje", id_pers1, 1); + Person Person2 = getAuditReader().find(Person.class, "Personaje", id_pers1, 2); + Person Person3 = getAuditReader().find(Person.class, "Personaje", id_pers1, 3); + + System.out.println("Revision 1:"); + System.out.println(" > Name: " + Person1.getName()); + System.out.println(" > Age: " + Person1.getAge()); + System.out.println("Revision 2:"); + System.out.println(" > Name: " + Person2.getName()); + System.out.println(" > Age: " + Person2.getAge()); + System.out.println("Revision 3:"); + System.out.println(" > Name: " + Person3.getName()); + System.out.println(" > Age: " + Person3.getAge()); + } + + +} Added: core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/manyToManyAudited/Car.java =================================================================== --- core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/manyToManyAudited/Car.java (rev 0) +++ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/manyToManyAudited/Car.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,56 @@ +package org.hibernate.envers.test.entityNames.manyToManyAudited; + +import java.util.List; + +import org.hibernate.envers.Audited; + +/** + * @author Hern�n Chanfreau + * + */ + +@Audited +public class Car { + + private long id; + + private int number; + + private List<Person> owners; + + + public Car() { } + + public Car(int number, List<Person> owners) { + this.number = number; + this.owners = owners; + } + + + public long getId() { + return id; + } + + public void setId(long id) { + this.id = id; + } + + public List<Person> getOwners() { + return owners; + } + + public void setOwners(List<Person> owners) { + this.owners = owners; + } + + public int getNumber() { + return number; + } + + public void setNumber(int number) { + this.number = number; + } + + + +} Added: core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/manyToManyAudited/Person.java =================================================================== --- core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/manyToManyAudited/Person.java (rev 0) +++ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/manyToManyAudited/Person.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,63 @@ +package org.hibernate.envers.test.entityNames.manyToManyAudited; + +import java.util.List; + +import org.hibernate.envers.Audited; + +/** + * @author Hern�n Chanfreau + * + */ + +@Audited +public class Person { + + private long id; + + private String name; + + private int age; + + private List<Car> cars; + + public Person(){ } + + public Person(String name, int age){ + this.name = name; + this.age = age; + } + + + public long getId() { + return id; + } + + public void setId(long id) { + this.id = id; + } + + public String getName() { + return name; + } + + public void setName(String name) { + this.name = name; + } + + public int getAge() { + return age; + } + + public void setAge(int age) { + this.age = age; + } + + public List<Car> getCars() { + return cars; + } + + public void setCars(List<Car> cars) { + this.cars = cars; + } + +} Added: core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/manyToManyAudited/ReadEntityWithAuditedManyToManyTest.java =================================================================== --- core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/manyToManyAudited/ReadEntityWithAuditedManyToManyTest.java (rev 0) +++ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/manyToManyAudited/ReadEntityWithAuditedManyToManyTest.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,99 @@ +package org.hibernate.envers.test.entityNames.manyToManyAudited; + +import java.io.File; +import java.net.URISyntaxException; +import java.net.URL; +import java.util.ArrayList; +import java.util.List; + +import org.hibernate.MappingException; +import org.hibernate.envers.test.AbstractSessionTest; +import org.testng.annotations.BeforeClass; +import org.testng.annotations.Test; + +/** + * @author Hern�n Chanfreau + * + */ + +public class ReadEntityWithAuditedManyToManyTest extends AbstractSessionTest{ + + private long id_car1; + private long id_car2; + + private long id_pers1; + + protected void initMappings() throws MappingException, URISyntaxException { + URL url = Thread.currentThread().getContextClassLoader().getResource("mappings/entityNames/manyToManyAudited/mappings.hbm.xml"); + config.addFile(new File(url.toURI())); + } + + + @BeforeClass(dependsOnMethods = "init") + public void initData() { + + newSessionFactory(); + + Person pers1 = new Person("Hernan", 28); + Person pers2 = new Person("Leandro", 29); + Person pers3 = new Person("Barba", 32); + Person pers4 = new Person("Camomo", 15); + + //REV 1 + getSession().getTransaction().begin(); + List<Person > owners = new ArrayList<Person>(); + owners.add(pers1); + owners.add(pers2); + owners.add(pers3); + Car car1 = new Car(5, owners); + + getSession().persist(car1); + getSession().getTransaction().commit(); + id_pers1 = pers1.getId(); + id_car1 = car1.getId(); + + owners = new ArrayList<Person>(); + owners.add(pers2); + owners.add(pers3); + owners.add(pers4); + Car car2 = new Car(27, owners); + //REV 2 + getSession().getTransaction().begin(); + Person person1 = (Person)getSession().get("Personaje", id_pers1); + person1.setName("Hernan David"); + person1.setAge(40); + getSession().persist(car1); + getSession().persist(car2); + getSession().getTransaction().commit(); + id_car2 = car2.getId(); + + } + + @Test + public void testObtainManyYoManyWithEntityName() { + + Car car1 = getAuditReader().find(Car.class, id_car1, 2); + Car car2 = getAuditReader().find(Car.class, id_car2, 2); + + System.out.println(" > Car: " + car1.getNumber()); + System.out.println(" > Owners:"); + for (Person owner : car1.getOwners()) { + System.out.println(" > Name: " + owner.getName() + " - Age:" + owner.getAge()); + System.out.println(" > Cars owned:"); + for (Car ownedCar : owner.getCars()) { + System.out.println(" o Car: " + ownedCar.getNumber()); + } + } + System.out.println(" > Car: " + car2.getNumber()); + System.out.println(" > Owners:"); + for (Person owner : car2.getOwners()) { + System.out.println(" > Name: " + owner.getName() + " - Age:" + owner.getAge()); + System.out.println(" > Cars owned:"); + for (Car ownedCar : owner.getCars()) { + System.out.println(" o Car: " + ownedCar.getNumber()); + } + } + } + + +} Added: core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyAudited/Car.java =================================================================== --- core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyAudited/Car.java (rev 0) +++ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyAudited/Car.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,56 @@ +package org.hibernate.envers.test.entityNames.oneToManyAudited; + +import java.util.List; + +import org.hibernate.envers.Audited; + +/** + * @author Hern�n Chanfreau + * + */ + +@Audited +public class Car { + + private long id; + + private int number; + + private List<Person> owners; + + + public Car() { } + + public Car(int number, List<Person> owners) { + this.number = number; + this.owners = owners; + } + + + public long getId() { + return id; + } + + public void setId(long id) { + this.id = id; + } + + public List<Person> getOwners() { + return owners; + } + + public void setOwners(List<Person> owners) { + this.owners = owners; + } + + public int getNumber() { + return number; + } + + public void setNumber(int number) { + this.number = number; + } + + + +} Added: core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyAudited/Person.java =================================================================== --- core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyAudited/Person.java (rev 0) +++ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyAudited/Person.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,51 @@ +package org.hibernate.envers.test.entityNames.oneToManyAudited; + +import org.hibernate.envers.Audited; + +/** + * @author Hern�n Chanfreau + * + */ + +@Audited +public class Person { + + private long id; + + private String name; + + private int age; + + public Person(){ } + + public Person(String name, int age){ + this.name = name; + this.age = age; + } + + + public long getId() { + return id; + } + + public void setId(long id) { + this.id = id; + } + + public String getName() { + return name; + } + + public void setName(String name) { + this.name = name; + } + + public int getAge() { + return age; + } + + public void setAge(int age) { + this.age = age; + } + +} Added: core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyAudited/ReadEntityWithAuditedCollectionTest.java =================================================================== --- core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyAudited/ReadEntityWithAuditedCollectionTest.java (rev 0) +++ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyAudited/ReadEntityWithAuditedCollectionTest.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,89 @@ +package org.hibernate.envers.test.entityNames.oneToManyAudited; + +import java.io.File; +import java.net.URISyntaxException; +import java.net.URL; +import java.util.ArrayList; +import java.util.List; + +import org.hibernate.MappingException; +import org.hibernate.envers.test.AbstractSessionTest; +import org.testng.annotations.BeforeClass; +import org.testng.annotations.Test; + +/** + * @author Hern�n Chanfreau + * + */ + +public class ReadEntityWithAuditedCollectionTest extends AbstractSessionTest{ + + private long id_car1; + private long id_car2; + + private long id_pers1; + + protected void initMappings() throws MappingException, URISyntaxException { + URL url = Thread.currentThread().getContextClassLoader().getResource("mappings/entityNames/oneToManyAudited/mappings.hbm.xml"); + config.addFile(new File(url.toURI())); + } + + + @BeforeClass(dependsOnMethods = "init") + public void initData() { + + newSessionFactory(); + + Person pers1 = new Person("Hernan", 28); + Person pers2 = new Person("Leandro", 29); + Person pers3 = new Person("Barba", 32); + Person pers4 = new Person("Camomo", 15); + + List<Person > owners = new ArrayList<Person>(); + owners.add(pers1); + owners.add(pers2); + Car car1 = new Car(5, owners); + + //REV 1 + getSession().getTransaction().begin(); + getSession().persist(car1); + getSession().getTransaction().commit(); + id_pers1 = pers1.getId(); + id_car1 = car1.getId(); + + owners = new ArrayList<Person>(); + owners.add(pers2); + owners.add(pers4); + Car car2 = new Car(27, owners); + //REV 2 + getSession().getTransaction().begin(); + Person person1 = (Person)getSession().get("Personaje", id_pers1); + person1.setName("Hernan David"); + person1.setAge(40); + getSession().persist(car1); + getSession().persist(car2); + getSession().getTransaction().commit(); + id_car2 = car2.getId(); + + } + + @Test + public void testObtainCollectionWithEntityName() { + + Car car1 = getAuditReader().find(Car.class, id_car1, 2); + Car car2 = getAuditReader().find(Car.class, id_car2, 2); + + System.out.println(" > Car: " + car1.getNumber()); + System.out.println(" > Owners:"); + for (Person owner : car1.getOwners()) { + System.out.println(" > Name: " + owner.getName() + " - Age:" + owner.getAge()); + } + System.out.println(" > Car: " + car2.getNumber()); + System.out.println(" > Owners:"); + for (Person owner : car2.getOwners()) { + System.out.println(" > Name: " + owner.getName() + " - Age:" + owner.getAge()); + } + } + + +} Added: core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyNotAudited/Car.java =================================================================== --- core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyNotAudited/Car.java (rev 0) +++ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyNotAudited/Car.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,58 @@ +package org.hibernate.envers.test.entityNames.oneToManyNotAudited; + +import java.util.List; + +import org.hibernate.envers.Audited; +import org.hibernate.envers.RelationTargetAuditMode; + +/** + * @author Hern�n Chanfreau + * + */ + +public class Car { + + private long id; + + private int number; + + private List<Person> owners; + + + public Car() { } + + public Car(int number, List<Person> owners) { + this.number = number; + this.owners = owners; + } + + + public long getId() { + return id; + } + + public void setId(long id) { + this.id = id; + } + + @Audited(targetAuditMode=RelationTargetAuditMode.NOT_AUDITED) + public List<Person> getOwners() { + return owners; + } + + public void setOwners(List<Person> owners) { + this.owners = owners; + } + + @Audited + public int getNumber() { + return number; + } + + public void setNumber(int number) { + this.number = number; + } + + + +} Added: core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyNotAudited/Person.java =================================================================== --- core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyNotAudited/Person.java (rev 0) +++ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyNotAudited/Person.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,48 @@ +package org.hibernate.envers.test.entityNames.oneToManyNotAudited; + +/** + * @author Hern�n Chanfreau + * + */ + +public class Person { + + private long id; + + private String name; + + private int age; + + public Person(){ } + + public Person(String name, int age){ + this.name = name; + this.age = age; + } + + + public long getId() { + return id; + } + + public void setId(long id) { + this.id = id; + } + + public String getName() { + return name; + } + + public void setName(String name) { + this.name = name; + } + + public int getAge() { + return age; + } + + public void setAge(int age) { + this.age = age; + } + +} Added: core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyNotAudited/ReadEntityWithAuditedCollectionTest.java =================================================================== --- core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyNotAudited/ReadEntityWithAuditedCollectionTest.java (rev 0) +++ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/oneToManyNotAudited/ReadEntityWithAuditedCollectionTest.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,84 @@ +package org.hibernate.envers.test.entityNames.oneToManyNotAudited; + +import java.io.File; +import java.net.URISyntaxException; +import java.net.URL; +import java.util.ArrayList; +import java.util.List; + +import org.hibernate.MappingException; +import org.hibernate.envers.test.AbstractSessionTest; +import org.testng.annotations.BeforeClass; +import org.testng.annotations.Test; + +public class ReadEntityWithAuditedCollectionTest extends AbstractSessionTest{ + + private long id_car1; + private long id_car2; + + private long id_pers1; + + protected void initMappings() throws MappingException, URISyntaxException { + URL url = Thread.currentThread().getContextClassLoader().getResource("mappings/entityNames/oneToManyNotAudited/mappings.hbm.xml"); + config.addFile(new File(url.toURI())); + } + + + @BeforeClass(dependsOnMethods = "init") + public void initData() { + + newSessionFactory(); + + Person pers1 = new Person("Hernan", 28); + Person pers2 = new Person("Leandro", 29); + Person pers3 = new Person("Barba", 32); + Person pers4 = new Person("Camomo", 15); + + List<Person > owners = new ArrayList<Person>(); + owners.add(pers1); + owners.add(pers2); + Car car1 = new Car(5, owners); + + //REV 1 + getSession().getTransaction().begin(); + getSession().persist(car1); + getSession().getTransaction().commit(); + id_pers1 = pers1.getId(); + id_car1 = car1.getId(); + + owners = new ArrayList<Person>(); + owners.add(pers2); + owners.add(pers4); + Car car2 = new Car(27, owners); + //REV 2 + getSession().getTransaction().begin(); + Person person1 = (Person)getSession().get("Personaje", id_pers1); + person1.setName("Hernan David"); + person1.setAge(40); + getSession().persist(car1); + getSession().persist(car2); + getSession().getTransaction().commit(); + id_car2 = car2.getId(); + + } + + @Test + public void testObtainCollectionWithEntityNameAndNotAuditedMode() { + + Car car1 = getAuditReader().find(Car.class, id_car1, 2); + Car car2 = getAuditReader().find(Car.class, id_car2, 2); + + System.out.println(" > Car: " + car1.getNumber()); + System.out.println(" > Owners:"); + for (Person owner : car1.getOwners()) { + System.out.println(" > Name: " + owner.getName() + " - Age:" + owner.getAge()); + } + System.out.println(" > Car: " + car2.getNumber()); + System.out.println(" > Owners:"); + for (Person owner : car2.getOwners()) { + System.out.println(" > Name: " + owner.getName() + " - Age:" + owner.getAge()); + } + } + + +} Added: core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedAudited/Car.java =================================================================== --- core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedAudited/Car.java (rev 0) +++ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedAudited/Car.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,54 @@ +package org.hibernate.envers.test.entityNames.singleAssociatedAudited; + +import org.hibernate.envers.Audited; + +/** + * @author Hern�n Chanfreau + * + */ + +@Audited +public class Car { + + private long id; + + private int number; + + private Person owner; + + + public Car() { } + + public Car(int number, Person owner) { + this.number = number; + this.owner = owner; + } + + + public long getId() { + return id; + } + + public void setId(long id) { + this.id = id; + } + + public Person getOwner() { + return owner; + } + + public void setOwner(Person owner) { + this.owner = owner; + } + + public int getNumber() { + return number; + } + + public void setNumber(int number) { + this.number = number; + } + + + +} Added: core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedAudited/Person.java =================================================================== --- core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedAudited/Person.java (rev 0) +++ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedAudited/Person.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,51 @@ +package org.hibernate.envers.test.entityNames.singleAssociatedAudited; + +import org.hibernate.envers.Audited; + +/** + * @author Hern�n Chanfreau + * + */ + +@Audited +public class Person { + + private long id; + + private String name; + + private int age; + + public Person(){ } + + public Person(String name, int age){ + this.name = name; + this.age = age; + } + + + public long getId() { + return id; + } + + public void setId(long id) { + this.id = id; + } + + public String getName() { + return name; + } + + public void setName(String name) { + this.name = name; + } + + public int getAge() { + return age; + } + + public void setAge(int age) { + this.age = age; + } + +} Added: core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedAudited/ReadEntityAssociatedAuditedTest.java =================================================================== --- core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedAudited/ReadEntityAssociatedAuditedTest.java (rev 0) +++ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedAudited/ReadEntityAssociatedAuditedTest.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,77 @@ +package org.hibernate.envers.test.entityNames.singleAssociatedAudited; + +import java.io.File; +import java.net.URISyntaxException; +import java.net.URL; + +import org.hibernate.MappingException; +import org.hibernate.envers.test.AbstractSessionTest; +import org.testng.annotations.BeforeClass; +import org.testng.annotations.Test; + +/** + * @author Hern�n Chanfreau + * + */ + +public class ReadEntityAssociatedAuditedTest extends AbstractSessionTest{ + + private long id_car1; + private long id_car2; + + private long id_pers1; + private long id_pers2; + + + protected void initMappings() throws MappingException, URISyntaxException { + URL url = Thread.currentThread().getContextClassLoader().getResource("mappings/entityNames/singleAssociatedAudited/mappings.hbm.xml"); + config.addFile(new File(url.toURI())); + } + + + @BeforeClass(dependsOnMethods = "init") + public void initData() { + + newSessionFactory(); + + Person pers1 = new Person("Hernan", 15); + Person pers2 = new Person("Leandro", 19); + + Car car1 = new Car(1, pers1); + Car car2 = new Car(2, pers2); + + //REV 1 + getSession().getTransaction().begin(); + getSession().persist("Personaje",pers1); + getSession().persist(car1); + getSession().getTransaction().commit(); + id_car1 = car1.getId(); + id_pers1 = pers1.getId(); + + //REV 2 + getSession().getTransaction().begin(); + pers1.setAge(50); + getSession().persist("Personaje", pers1); + getSession().persist("Personaje", pers2); + getSession().persist(car2); + getSession().getTransaction().commit(); + id_car2 = car2.getId(); + id_pers2 = pers2.getId(); + + } + + @Test + public void testGetAssociationWithEntityName() { + + Person person1 = (Person)getSession().get("Personaje", id_pers1); + Car car1 = getAuditReader().find(Car.class, id_car1, 1); + Person person1_1 = car1.getOwner(); + assert(person1.getAge() != person1_1.getAge()); + + Person person2 = (Person)getSession().get("Personaje", id_pers2); + Car car2 = getAuditReader().find(Car.class, id_car2, 2); + Person person2_1 = car2.getOwner(); + assert(person2.getAge() == person2_1.getAge()); + } + +} Added: core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedNotAudited/Car.java =================================================================== --- core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedNotAudited/Car.java (rev 0) +++ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedNotAudited/Car.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,56 @@ +package org.hibernate.envers.test.entityNames.singleAssociatedNotAudited; + +import org.hibernate.envers.Audited; +import org.hibernate.envers.RelationTargetAuditMode; + +/** + * @author Hern�n Chanfreau + * + */ + +public class Car { + + private long id; + + private int number; + + private Person owner; + + + public Car() { } + + public Car(int number, Person owner) { + this.number = number; + this.owner = owner; + } + + + public long getId() { + return id; + } + + public void setId(long id) { + this.id = id; + } + + @Audited(targetAuditMode=RelationTargetAuditMode.NOT_AUDITED) + public Person getOwner() { + return owner; + } + + public void setOwner(Person owner) { + this.owner = owner; + } + + @Audited + public int getNumber() { + return number; + } + + public void setNumber(int number) { + this.number = number; + } + + + +} Added: core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedNotAudited/Person.java =================================================================== --- core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedNotAudited/Person.java (rev 0) +++ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedNotAudited/Person.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,48 @@ +package org.hibernate.envers.test.entityNames.singleAssociatedNotAudited; + +/** + * @author Hern�n Chanfreau + * + */ + +public class Person { + + private long id; + + private String name; + + private int age; + + public Person(){ } + + public Person(String name, int age){ + this.name = name; + this.age = age; + } + + + public long getId() { + return id; + } + + public void setId(long id) { + this.id = id; + } + + public String getName() { + return name; + } + + public void setName(String name) { + this.name = name; + } + + public int getAge() { + return age; + } + + public void setAge(int age) { + this.age = age; + } + +} Added: core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedNotAudited/ReadEntityAssociatedNotAuditedTest.java =================================================================== --- core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedNotAudited/ReadEntityAssociatedNotAuditedTest.java (rev 0) +++ core/branches/Branch_3_5/envers/src/test/java/org/hibernate/envers/test/entityNames/singleAssociatedNotAudited/ReadEntityAssociatedNotAuditedTest.java 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,80 @@ +package org.hibernate.envers.test.entityNames.singleAssociatedNotAudited; + +import java.io.File; +import java.net.URISyntaxException; +import java.net.URL; + +import org.hibernate.MappingException; +import org.hibernate.envers.test.AbstractSessionTest; +import org.testng.annotations.BeforeClass; +import org.testng.annotations.Test; + +/** + * @author Hern�n Chanfreau + * + */ + +public class ReadEntityAssociatedNotAuditedTest extends AbstractSessionTest{ + + private long id_car1; + private long id_car2; + + private long id_pers1; + private long id_pers2; + + + protected void initMappings() throws MappingException, URISyntaxException { + URL url = Thread.currentThread().getContextClassLoader().getResource("mappings/entityNames/singleAssociatedNotAudited/mappings.hbm.xml"); + config.addFile(new File(url.toURI())); + } + + + @BeforeClass(dependsOnMethods = "init") + public void initData() { + + newSessionFactory(); + + Person pers1 = new Person("Hernan", 15); + Person pers2 = new Person("Leandro", 19); + + Car car1 = new Car(1, pers1); + Car car2 = new Car(2, pers2); + + //REV 1 + getSession().getTransaction().begin(); + getSession().persist("Personaje",pers1); + getSession().persist(car1); + getSession().getTransaction().commit(); + id_car1 = car1.getId(); + id_pers1 = pers1.getId(); + + //REV 2 + getSession().getTransaction().begin(); + pers1.setAge(50); + getSession().persist("Personaje", pers1); + getSession().persist("Personaje", pers2); + getSession().persist(car2); + getSession().getTransaction().commit(); + id_car2 = car2.getId(); + id_pers2 = pers2.getId(); + + } + + @Test + public void testGetAssociationWithEntityNameAndNotAuditedMode() { + // persons from "actual" model + Person person1 = (Person)getSession().get("Personaje", id_pers1); + Person person2 = (Person)getSession().get("Personaje", id_pers2); + + Car car1 = getAuditReader().find(Car.class, id_car1, 1); + Car car2 = getAuditReader().find(Car.class, id_car2, 2); + + // persons from "historic" model + Person person1_1 = car1.getOwner(); + Person person2_1 = car2.getOwner(); + + assert(person1.getAge() == person1_1.getAge()); + assert(person2.getAge() == person2_1.getAge()); + } + +} Added: core/branches/Branch_3_5/envers/src/test/resources/hibernate.test.session-cfg.xml =================================================================== --- core/branches/Branch_3_5/envers/src/test/resources/hibernate.test.session-cfg.xml (rev 0) +++ core/branches/Branch_3_5/envers/src/test/resources/hibernate.test.session-cfg.xml 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,46 @@ +<?xml version='1.0' encoding='utf-8'?> +<!DOCTYPE hibernate-configuration PUBLIC + "-//Hibernate/Hibernate Configuration DTD//EN" + "http://hibernate.sourceforge.net/hibernate-configuration-3.0.dtd"> + +<hibernate-configuration> + <session-factory> + <property name="hbm2ddl.auto">create-drop</property> + + <property name="show_sql">false</property> + <property name="format_sql">false</property> + + <property name="dialect">org.hibernate.dialect.H2Dialect</property> + <property name="connection.url">jdbc:h2:mem:envers</property> + <property name="connection.driver_class">org.h2.Driver</property> + <property name="connection.username">sa</property> + <property name="connection.password"></property> + + <!--<property name="dialect">org.hibernate.dialect.MySQL5InnoDBDialect</property>--> + <!--<property name="connection.url">jdbc:mysql:///hibernate_tests?useUnicode=true&amp;characterEncoding=UTF-8</property>--> + <!--<property name="connection.driver_class">com.mysql.jdbc.Driver</property>--> + <!--<property name="connection.username">root</property>--> + <!--<property name="connection.password"></property>--> + + <!--<property name="hibernate.jdbc.batch_size">100</property>--> + + <event type="post-insert"> + <listener class="org.hibernate.envers.event.AuditEventListener" /> + </event> + <event type="post-update"> + <listener class="org.hibernate.envers.event.AuditEventListener" /> + </event> + <event type="post-delete"> + <listener class="org.hibernate.envers.event.AuditEventListener" /> + </event> + <event type="pre-collection-update"> + <listener class="org.hibernate.envers.event.AuditEventListener" /> + </event> + <event type="pre-collection-remove"> + <listener class="org.hibernate.envers.event.AuditEventListener" /> + </event> + <event type="post-collection-recreate"> + <listener class="org.hibernate.envers.event.AuditEventListener" /> + </event> + </session-factory> +</hibernate-configuration> \ No newline at end of file Added: core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/auditedEntity/mappings.hbm.xml =================================================================== --- core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/auditedEntity/mappings.hbm.xml (rev 0) +++ core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/auditedEntity/mappings.hbm.xml 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,18 @@ +<?xml version="1.0"?> +<!DOCTYPE hibernate-mapping PUBLIC "-//Hibernate/Hibernate Mapping DTD 3.0//EN" "http://hibernate.sourceforge.net/hibernate-mapping-3.0.dtd"> +<hibernate-mapping> + + <class name="org.hibernate.envers.test.entityNames.auditedEntity.Person" entity-name="Personaje"> + + <id name="id" column="ID_person" type="long" > + <generator class="native" /> + </id> + + <property name="name" type="string" length="255" + column="NAME" not-null="true" /> + + <property name="age" type="int" column="AGE"/> + + </class> + +</hibernate-mapping> \ No newline at end of file Added: core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/manyToManyAudited/mappings.hbm.xml =================================================================== --- core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/manyToManyAudited/mappings.hbm.xml (rev 0) +++ core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/manyToManyAudited/mappings.hbm.xml 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,40 @@ +<?xml version="1.0"?> +<!DOCTYPE hibernate-mapping PUBLIC "-//Hibernate/Hibernate Mapping DTD 3.0//EN" "http://hibernate.sourceforge.net/hibernate-mapping-3.0.dtd"> +<hibernate-mapping> + + <class name="org.hibernate.envers.test.entityNames.manyToManyAudited.Person" + entity-name="Personaje"> + + <id name="id" column="ID_PERSON" type="long"> + <generator class="native" /> + </id> + + <property name="name" type="string" length="255" column="NAME" + not-null="true" /> + + <property name="age" type="int" column="AGE" /> + + <bag name="cars" inverse="true" table="PERSON_CAR" > + <key column="ID_CAR" /> + <many-to-many + class="org.hibernate.envers.test.entityNames.manyToManyAudited.Car" column="ID_PERSON"/> + </bag> + + </class> + + <class name="org.hibernate.envers.test.entityNames.manyToManyAudited.Car"> + + <id name="id" column="ID_CAR" type="long"> + <generator class="native" /> + </id> + + <property name="number" type="int" not-null="true" /> + + <bag name="owners" cascade="all" table="PERSON_CAR" > + <key column="ID_PERSON" /> + <many-to-many entity-name="Personaje" column="ID_CAR"/> + </bag> + + </class> + +</hibernate-mapping> \ No newline at end of file Added: core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/oneToManyAudited/mappings.hbm.xml =================================================================== --- core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/oneToManyAudited/mappings.hbm.xml (rev 0) +++ core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/oneToManyAudited/mappings.hbm.xml 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,48 @@ +<?xml version="1.0"?> +<!DOCTYPE hibernate-mapping PUBLIC "-//Hibernate/Hibernate Mapping DTD 3.0//EN" "http://hibernate.sourceforge.net/hibernate-mapping-3.0.dtd"> +<hibernate-mapping> + + <class name="org.hibernate.envers.test.entityNames.oneToManyAudited.Person" entity-name="Personaje"> + + <id name="id" column="ID_PERSON" type="long" > + <generator class="native" /> + </id> + + <property name="name" type="string" length="255" + column="NAME" not-null="true" /> + + <property name="age" type="int" column="AGE"/> + + </class> + + <class name="org.hibernate.envers.test.entityNames.oneToManyAudited.Car" > + + <id name="id" column="ID_CAR" type="long"> + <generator class="native" /> + </id> + + <property name="number" type="int" column="int" not-null="true" /> + + <bag name="owners" cascade="save-update"> + <key column="ID_GARAGE" not-null="true" update="false" /> + <one-to-many entity-name="Personaje"/> + </bag> + + </class> + +<!-- <class name="org.hibernate.envers.test.entityNames.oneToManyAudited.Garage" >--> +<!----> +<!-- <id name="id" column="ID_GARAGE" type="long">--> +<!-- <generator class="native" />--> +<!-- </id>--> +<!----> +<!-- <property name="name" type="int" column="int" not-null="true" />--> +<!----> +<!-- <bag name="cars" cascade="save-update">--> +<!-- <key column="ID_GARAGE" not-null="true" update="false" />--> +<!-- <one-to-many class="org.hibernate.envers.test.entityNames.oneToManyAudited.Car" />--> +<!-- </bag>--> +<!-- --> +<!-- </class> --> + +</hibernate-mapping> \ No newline at end of file Added: core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/oneToManyNotAudited/mappings.hbm.xml =================================================================== --- core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/oneToManyNotAudited/mappings.hbm.xml (rev 0) +++ core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/oneToManyNotAudited/mappings.hbm.xml 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,34 @@ +<?xml version="1.0"?> +<!DOCTYPE hibernate-mapping PUBLIC "-//Hibernate/Hibernate Mapping DTD 3.0//EN" "http://hibernate.sourceforge.net/hibernate-mapping-3.0.dtd"> +<hibernate-mapping> + + <class name="org.hibernate.envers.test.entityNames.oneToManyNotAudited.Person" entity-name="Personaje"> + + <id name="id" column="ID_PERSON" type="long" > + <generator class="native" /> + </id> + + <property name="name" type="string" length="255" + column="NAME" not-null="true" /> + + <property name="age" type="int" column="AGE"/> + + </class> + + <class name="org.hibernate.envers.test.entityNames.oneToManyNotAudited.Car" > + + <id name="id" column="ID_CAR" type="long"> + <generator class="native" /> + </id> + + <property name="number" type="int" column="int" not-null="true" /> + + <bag name="owners" cascade="save-update"> + <key column="ID_GARAGE" not-null="true" update="false" /> + <one-to-many entity-name="Personaje"/> + </bag> + + </class> + + +</hibernate-mapping> \ No newline at end of file Added: core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/singleAssociatedAudited/mappings.hbm.xml =================================================================== --- core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/singleAssociatedAudited/mappings.hbm.xml (rev 0) +++ core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/singleAssociatedAudited/mappings.hbm.xml 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,30 @@ +<?xml version="1.0"?> +<!DOCTYPE hibernate-mapping PUBLIC "-//Hibernate/Hibernate Mapping DTD 3.0//EN" "http://hibernate.sourceforge.net/hibernate-mapping-3.0.dtd"> +<hibernate-mapping> + + <class name="org.hibernate.envers.test.entityNames.singleAssociatedAudited.Person" entity-name="Personaje"> + + <id name="id" column="ID_person" type="long" > + <generator class="native" /> + </id> + + <property name="name" type="string" length="255" + column="NAME" not-null="true" /> + + <property name="age" type="int" column="AGE"/> + + </class> + + <class name="org.hibernate.envers.test.entityNames.singleAssociatedAudited.Car" > + + <id name="id" column="ID_BED" type="long"> + <generator class="native" /> + </id> + + <property name="number" type="int" column="int" not-null="true" /> + + <many-to-one name="owner" entity-name="Personaje"/> + + </class> + +</hibernate-mapping> \ No newline at end of file Added: core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/singleAssociatedNotAudited/mappings.hbm.xml =================================================================== --- core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/singleAssociatedNotAudited/mappings.hbm.xml (rev 0) +++ core/branches/Branch_3_5/envers/src/test/resources/mappings/entityNames/singleAssociatedNotAudited/mappings.hbm.xml 2010-07-06 06:31:40 UTC (rev 19902) @@ -0,0 +1,30 @@ +<?xml version="1.0"?> +<!DOCTYPE hibernate-mapping PUBLIC "-//Hibernate/Hibernate Mapping DTD 3.0//EN" "http://hibernate.sourceforge.net/hibernate-mapping-3.0.dtd"> +<hibernate-mapping> + + <class name="org.hibernate.envers.test.entityNames.singleAssociatedNotAudited.Person" entity-name="Personaje"> + + <id name="id" column="ID_person" type="long" > + <generator class="native" /> + </id> + + <property name="name" type="string" length="255" + column="NAME" not-null="true" /> + + <property name="age" type="int" column="AGE"/> + + </class> + + <class name="org.hibernate.envers.test.entityNames.singleAssociatedNotAudited.Car" > + + <id name="id" column="ID_BED" type="long"> + <generator class="native" /> + </id> + + <property name="number" type="int" column="int" not-null="true" /> + + <many-to-one name="owner" entity-name="Personaje"/> + + </class> + +</hibernate-mapping> \ No newline at end of file Modified: core/branches/Branch_3_5/envers/src/test/resources/testng.xml =================================================================== --- core/branches/Branch_3_5/envers/src/test/resources/testng.xml 2010-07-05 17:36:04 UTC (rev 19901) +++ core/branches/Branch_3_5/envers/src/test/resources/testng.xml 2010-07-06 06:31:40 UTC (rev 19902) @@ -71,6 +71,13 @@ <package name="org.hibernate.envers.test.integration.secondary.ids" /> <package name="org.hibernate.envers.test.integration.serialization" /> <package name="org.hibernate.envers.test.integration.superclass" /> + + <package name="org.hibernate.envers.test.entityNames.auditedEntity" /> + <package name="org.hibernate.envers.test.entityNames.manyToManyAudited" /> + <package name="org.hibernate.envers.test.entityNames.oneToManyAudited" /> + <package name="org.hibernate.envers.test.entityNames.oneToManyNotAudited" /> + <package name="org.hibernate.envers.test.entityNames.singleAssociatedAudited" /> + <package name="org.hibernate.envers.test.entityNames.singleAssociatedNotAudited" /> </packages> </test> </suite>