Skip to content

Commit bd84bb6

Browse files
committed
Fix MetadataTest failures, the behaviour introduced by HHH-4702 has been reverted, SingularAttribute#getBindableJavaType() and SingularAttribute#getType().getJavaType() return Integer.class also for int attributes
1 parent 70f8498 commit bd84bb6

31 files changed

+532
-535
lines changed

hibernate-core/src/main/java/org/hibernate/metamodel/internal/MetadataContext.java

Lines changed: 13 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -385,17 +385,20 @@ private void applyIdMetadata(PersistentClass persistentClass, IdentifiableDomain
385385
// Handle the actual id-attributes
386386
final Component cidValue = (Component) persistentClass.getIdentifier();
387387
final Iterator<Property> cidPropertyItr = cidValue.getPropertyIterator();
388-
final Set<SingularPersistentAttribute<?,?>> idAttributes = new HashSet<>( cidValue.getPropertySpan() );
389388

390-
while ( cidPropertyItr.hasNext() ) {
391-
final Property cidSubProperty = cidPropertyItr.next();
392-
393-
final SingularPersistentAttribute<?, Object> cidSubAttr = attributeFactory.buildIdAttribute(
394-
identifiableType,
395-
cidSubProperty
396-
);
397-
398-
idAttributes.add( cidSubAttr );
389+
AbstractIdentifiableType idType = (AbstractIdentifiableType) entityTypes.get( cidValue.getOwner().getMappedClass() );
390+
Set idAttributes = idType.getIdClassAttributesSafely();
391+
if ( idAttributes == null ) {
392+
idAttributes = new HashSet<>( cidValue.getPropertySpan() );
393+
while ( cidPropertyItr.hasNext() ) {
394+
final Property cidSubProperty = cidPropertyItr.next();
395+
final SingularPersistentAttribute<?, Object> cidSubAttr = attributeFactory.buildIdAttribute(
396+
idType,
397+
cidSubProperty
398+
);
399+
400+
idAttributes.add( cidSubAttr );
401+
}
399402
}
400403

401404
( ( AttributeContainer) identifiableType ).getInFlightAccess().applyNonAggregatedIdAttributes( idAttributes );

hibernate-core/src/test/java/org/hibernate/jpa/test/criteria/selectcase/GroupBySelectCaseTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
import org.hibernate.dialect.DerbyDialect;
2121
import org.hibernate.dialect.PostgreSQL95Dialect;
2222
import org.hibernate.jpa.test.BaseEntityManagerFunctionalTestCase;
23-
import org.hibernate.jpa.test.metadata.Person_;
23+
import org.hibernate.orm.test.jpa.metadata.Person_;
2424

2525
import org.hibernate.testing.RequiresDialect;
2626
import org.hibernate.testing.SkipForDialect;

hibernate-core/src/test/java/org/hibernate/jpa/test/metadata/MetadataTest.java

Lines changed: 0 additions & 447 deletions
This file was deleted.

hibernate-core/src/test/java/org/hibernate/jpa/test/metadata/Address.java renamed to hibernate-core/src/test/java/org/hibernate/orm/test/jpa/metadata/Address.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
* License: GNU Lesser General Public License (LGPL), version 2.1 or later.
55
* See the lgpl.txt file in the root directory or <http://www.gnu.org/licenses/lgpl-2.1.html>.
66
*/
7-
package org.hibernate.jpa.test.metadata;
7+
package org.hibernate.orm.test.jpa.metadata;
88
import javax.persistence.Basic;
99
import javax.persistence.Embeddable;
1010

hibernate-core/src/test/java/org/hibernate/jpa/test/metadata/Animal.java renamed to hibernate-core/src/test/java/org/hibernate/orm/test/jpa/metadata/Animal.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
* License: GNU Lesser General Public License (LGPL), version 2.1 or later.
55
* See the lgpl.txt file in the root directory or <http://www.gnu.org/licenses/lgpl-2.1.html>.
66
*/
7-
package org.hibernate.jpa.test.metadata;
7+
package org.hibernate.orm.test.jpa.metadata;
88
import javax.persistence.GeneratedValue;
99
import javax.persistence.Id;
1010
import javax.persistence.MappedSuperclass;

hibernate-core/src/test/java/org/hibernate/jpa/test/metadata/Being.java renamed to hibernate-core/src/test/java/org/hibernate/orm/test/jpa/metadata/Being.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
* License: GNU Lesser General Public License (LGPL), version 2.1 or later.
55
* See the lgpl.txt file in the root directory or <http://www.gnu.org/licenses/lgpl-2.1.html>.
66
*/
7-
package org.hibernate.jpa.test.metadata;
7+
package org.hibernate.orm.test.jpa.metadata;
88
import javax.persistence.MappedSuperclass;
99

1010
/**

hibernate-core/src/test/java/org/hibernate/jpa/test/metadata/Cat.java renamed to hibernate-core/src/test/java/org/hibernate/orm/test/jpa/metadata/Cat.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
* License: GNU Lesser General Public License (LGPL), version 2.1 or later.
55
* See the lgpl.txt file in the root directory or <http://www.gnu.org/licenses/lgpl-2.1.html>.
66
*/
7-
package org.hibernate.jpa.test.metadata;
7+
package org.hibernate.orm.test.jpa.metadata;
88
import javax.persistence.Entity;
99

1010
/**

hibernate-core/src/test/java/org/hibernate/jpa/test/metadata/Cattish.java renamed to hibernate-core/src/test/java/org/hibernate/orm/test/jpa/metadata/Cattish.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
* License: GNU Lesser General Public License (LGPL), version 2.1 or later.
55
* See the lgpl.txt file in the root directory or <http://www.gnu.org/licenses/lgpl-2.1.html>.
66
*/
7-
package org.hibernate.jpa.test.metadata;
7+
package org.hibernate.orm.test.jpa.metadata;
88
import javax.persistence.MappedSuperclass;
99

1010
/**

hibernate-core/src/test/java/org/hibernate/jpa/test/metadata/Child.java renamed to hibernate-core/src/test/java/org/hibernate/orm/test/jpa/metadata/Child.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
* License: GNU Lesser General Public License (LGPL), version 2.1 or later.
55
* See the lgpl.txt file in the root directory or <http://www.gnu.org/licenses/lgpl-2.1.html>.
66
*/
7-
package org.hibernate.jpa.test.metadata;
7+
package org.hibernate.orm.test.jpa.metadata;
88
import javax.persistence.Entity;
99
import javax.persistence.GeneratedValue;
1010
import javax.persistence.Id;

hibernate-core/src/test/java/org/hibernate/jpa/test/metadata/DeskWithRawType.java renamed to hibernate-core/src/test/java/org/hibernate/orm/test/jpa/metadata/DeskWithRawType.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
* License: GNU Lesser General Public License (LGPL), version 2.1 or later.
55
* See the lgpl.txt file in the root directory or <http://www.gnu.org/licenses/lgpl-2.1.html>.
66
*/
7-
package org.hibernate.jpa.test.metadata;
7+
package org.hibernate.orm.test.jpa.metadata;
88
import java.util.Collection;
99
import javax.persistence.Basic;
1010
import javax.persistence.CascadeType;

0 commit comments

Comments
 (0)