HHH-6230 Fixing wrong usage of equals. A string was compared against an enum

This commit is contained in:
Hardy Ferentschik 2011-05-17 12:19:44 +02:00
parent 755e6fcd25
commit fc942278cc
1 changed files with 7 additions and 4 deletions

View File

@ -69,7 +69,9 @@ public class HbmPluralAttributeBindingState extends AbstractHbmAttributeBindingS
collection.isOptimisticLock() collection.isOptimisticLock()
); );
this.collection = collection; this.collection = collection;
this.collectionPersisterClass = MappingHelper.classForName( collection.getPersister(), getDefaults().getServiceRegistry() ); this.collectionPersisterClass = MappingHelper.classForName(
collection.getPersister(), getDefaults().getServiceRegistry()
);
this.cascade = MappingHelper.getStringValue( collection.getCascade(), mappingDefaults.getDefaultCascade() ); this.cascade = MappingHelper.getStringValue( collection.getCascade(), mappingDefaults.getDefaultCascade() );
//Attribute typeNode = collectionElement.attribute( "collection-type" ); //Attribute typeNode = collectionElement.attribute( "collection-type" );
@ -93,7 +95,7 @@ public class HbmPluralAttributeBindingState extends AbstractHbmAttributeBindingS
public FetchMode getFetchMode() { public FetchMode getFetchMode() {
FetchMode fetchMode; FetchMode fetchMode;
if ( collection.getFetch() != null ) { if ( collection.getFetch() != null ) {
fetchMode = "join".equals( collection.getFetch() ) ? FetchMode.JOIN : FetchMode.SELECT; fetchMode = "join".equals( collection.getFetch().value() ) ? FetchMode.JOIN : FetchMode.SELECT;
} }
else { else {
String jfNodeValue = ( collection.getOuterJoin().value() == null ? "auto" : collection.getOuterJoin() String jfNodeValue = ( collection.getOuterJoin().value() == null ? "auto" : collection.getOuterJoin()
@ -117,7 +119,7 @@ public class HbmPluralAttributeBindingState extends AbstractHbmAttributeBindingS
} }
public boolean isExtraLazy() { public boolean isExtraLazy() {
return ( "extra".equals( collection.getLazy() ) ); return ( "extra".equals( collection.getLazy().value() ) );
} }
public String getElementTypeName() { public String getElementTypeName() {
@ -128,6 +130,7 @@ public class HbmPluralAttributeBindingState extends AbstractHbmAttributeBindingS
public String getElementNodeName() { public String getElementNodeName() {
return collection.getElement().getNode(); return collection.getElement().getNode();
} }
public boolean isInverse() { public boolean isInverse() {
return collection.isInverse(); return collection.isInverse();
} }
@ -137,7 +140,7 @@ public class HbmPluralAttributeBindingState extends AbstractHbmAttributeBindingS
} }
public boolean isSubselectLoadable() { public boolean isSubselectLoadable() {
return "subselect".equals( collection.getFetch() ); return "subselect".equals( collection.getFetch().value() );
} }
public String getCacheConcurrencyStrategy() { public String getCacheConcurrencyStrategy() {