HHH-19348 Change scope of private methods in IdentifierLoadAccessImpl

DavideD · DavideD · commit cdd2f6c63baf · 2025-04-15T14:02:15.000+02:00
For Hibernate Reactive Change scopes of some private methods, and small refactoring to avoid duplication in Hibernate Reactive. Related to hibernate/hibernate-reactive#2181
diff --git a/hibernate-core/src/main/java/org/hibernate/loader/internal/IdentifierLoadAccessImpl.java b/hibernate-core/src/main/java/org/hibernate/loader/internal/IdentifierLoadAccessImpl.java
@@ -129,7 +129,8 @@ protected T doGetReference(Object id) {
 				concreteType.getEntityName(), lockOptions, isReadOnly( session ) );
 	}
 
-	private Boolean isReadOnly(SessionImplementor session) {
+	// Hibernate Reactive might need to call this
+	protected Boolean isReadOnly(SessionImplementor session) {
 		return readOnly != null
 				? readOnly
 				: session.getLoadQueryInfluencers().getReadOnly();

Original file line number	Diff line number	Diff line change
`@@ -129,7 +129,8 @@ protected T doGetReference(Object id) {`
`129`	`129`	`concreteType.getEntityName(), lockOptions, isReadOnly( session ) );`
`130`	`130`	`}`
`131`	`131`
`132`		`- private Boolean isReadOnly(SessionImplementor session) {`
	`132`	`+ // Hibernate Reactive might need to call this`
	`133`	`+ protected Boolean isReadOnly(SessionImplementor session) {`
`133`	`134`	`return readOnly != null`
`134`	`135`	`? readOnly`
`135`	`136`	`: session.getLoadQueryInfluencers().getReadOnly();`