diff options
Diffstat (limited to 'jpa/plugins/org.eclipse.jpt.core/src/org/eclipse/jpt/core/internal/platform/JavaSingleRelationshipMappingContext.java')
-rw-r--r-- | jpa/plugins/org.eclipse.jpt.core/src/org/eclipse/jpt/core/internal/platform/JavaSingleRelationshipMappingContext.java | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/jpa/plugins/org.eclipse.jpt.core/src/org/eclipse/jpt/core/internal/platform/JavaSingleRelationshipMappingContext.java b/jpa/plugins/org.eclipse.jpt.core/src/org/eclipse/jpt/core/internal/platform/JavaSingleRelationshipMappingContext.java index ba852c1a90..a9526299ee 100644 --- a/jpa/plugins/org.eclipse.jpt.core/src/org/eclipse/jpt/core/internal/platform/JavaSingleRelationshipMappingContext.java +++ b/jpa/plugins/org.eclipse.jpt.core/src/org/eclipse/jpt/core/internal/platform/JavaSingleRelationshipMappingContext.java @@ -88,11 +88,17 @@ public abstract class JavaSingleRelationshipMappingContext public void addToMessages(List<IMessage> messages) { super.addToMessages(messages); - if (entityOwned()) { + //bug 192287 - do not want joinColumn validation errors on the non-owning side + //of a bidirectional relationship. This is a low risk fix for RC3, but a better + //solution would be to not have the default joinColumns on the non-owning side. + //This would fix another bug that we show default joinColumns in this situation. + if (entityOwned() && isOwningSide()) { addJoinColumnMessages(messages); } } + protected abstract boolean isOwningSide(); + protected void addJoinColumnMessages(List<IMessage> messages) { ISingleRelationshipMapping mapping = getMapping(); ITypeMapping typeMapping = mapping.typeMapping(); |