You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by cu...@apache.org on 2009/10/02 18:43:55 UTC
svn commit: r821086 - in /openjpa/branches/1.3.x/openjpa-persistence/src:
main/java/org/apache/openjpa/persistence/
test/java/org/apache/openjpa/persistence/
test/java/org/apache/openjpa/persistence/entity/ test/resources/META-INF/
Author: curtisr7
Date: Fri Oct 2 16:43:55 2009
New Revision: 821086
URL: http://svn.apache.org/viewvc?rev=821086&view=rev
Log:
OPENJPA-859: Backing out 818927 and 818932 while investigating a regression.
Removed:
openjpa/branches/1.3.x/openjpa-persistence/src/test/java/org/apache/openjpa/persistence/TestParsing.java
openjpa/branches/1.3.x/openjpa-persistence/src/test/java/org/apache/openjpa/persistence/entity/
openjpa/branches/1.3.x/openjpa-persistence/src/test/resources/META-INF/parsing-orm.xml
Modified:
openjpa/branches/1.3.x/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/AnnotationPersistenceMetaDataParser.java
openjpa/branches/1.3.x/openjpa-persistence/src/test/resources/META-INF/persistence.xml
Modified: openjpa/branches/1.3.x/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/AnnotationPersistenceMetaDataParser.java
URL: http://svn.apache.org/viewvc/openjpa/branches/1.3.x/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/AnnotationPersistenceMetaDataParser.java?rev=821086&r1=821085&r2=821086&view=diff
==============================================================================
--- openjpa/branches/1.3.x/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/AnnotationPersistenceMetaDataParser.java (original)
+++ openjpa/branches/1.3.x/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/AnnotationPersistenceMetaDataParser.java Fri Oct 2 16:43:55 2009
@@ -464,11 +464,6 @@
* Read annotations for the current type.
*/
private ClassMetaData parseClassAnnotations() {
- // Check to see if there is cached metadata for the class that we are currently parsing. It is possible
- // that one of the annotations (Entity, Embeddable, MappedSuperclass) is in the orm.xml. We still need to look
- // at these files for other annotations and more importantly setup defaults (ie: Basic fields).
- ClassMetaData m = getRepository().getCachedMetaData(_cls);
- if(m == null) {
// check immediately whether the user is using any annotations,
// regardless of mode. this prevents adding non-entity classes to
// repository if we're ignoring these annotations in mapping mode
@@ -481,9 +476,9 @@
.isAnnotationPresentAction(_cls, MappedSuperclass.class)))
.booleanValue())
return null;
- }
+
// find / create metadata
- ClassMetaData meta = (m == null) ? getMetaData() : m;
+ ClassMetaData meta = getMetaData();
if (meta == null)
return null;
Modified: openjpa/branches/1.3.x/openjpa-persistence/src/test/resources/META-INF/persistence.xml
URL: http://svn.apache.org/viewvc/openjpa/branches/1.3.x/openjpa-persistence/src/test/resources/META-INF/persistence.xml?rev=821086&r1=821085&r2=821086&view=diff
==============================================================================
--- openjpa/branches/1.3.x/openjpa-persistence/src/test/resources/META-INF/persistence.xml (original)
+++ openjpa/branches/1.3.x/openjpa-persistence/src/test/resources/META-INF/persistence.xml Fri Oct 2 16:43:55 2009
@@ -32,8 +32,4 @@
</persistence-unit>
<persistence-unit name="encryption_plugin_default_pu" transaction-type="RESOURCE_LOCAL">
</persistence-unit>
- <persistence-unit name="test_parsing">
- <mapping-file>META-INF/parsing-orm.xml</mapping-file>
- <class>org.apache.openjpa.persistence.entity.MixedMappingLocation</class>
- </persistence-unit>
</persistence>