Author: aadamchik
Date: Sun Nov 11 11:54:33 2012
New Revision: 1407947
URL: http://svn.apache.org/viewvc?rev=1407947&view=rev
Log:
minot cleanup... this class is a total mess
Modified:
cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/main/java/org/apache/cayenne/util/EntityMergeSupport.java
Modified:
cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/main/java/org/apache/cayenne/util/EntityMergeSupport.java
URL:
http://svn.apache.org/viewvc/cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/main/java/org/apache/cayenne/util/EntityMergeSupport.java?rev=1407947&r1=1407946&r2=1407947&view=diff
==============================================================================
---
cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/main/java/org/apache/cayenne/util/EntityMergeSupport.java
(original)
+++
cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/main/java/org/apache/cayenne/util/EntityMergeSupport.java
Sun Nov 11 11:54:33 2012
@@ -208,21 +208,17 @@ public class EntityMergeSupport {
.getDbEntity());
for (DbAttribute dba : objEntity.getDbEntity().getAttributes()) {
- // already there
+
+ if (dba.getName() == null) {
+ continue;
+ }
+
if (objEntity.getAttributeForDbAttribute(dba) != null) {
continue;
}
- // check if adding it makes sense at all
- if (!removeMeaningfulPKs) {
- if (dba.getName() == null) {
- continue;
- }
- }
- else {
- if (dba.getName() == null || dba.isPrimaryKey()) {
- continue;
- }
+ if (removeMeaningfulPKs && dba.isPrimaryKey()) {
+ continue;
}
// check FK's