NH-4003 - renaming then converting to properties some methods, to be squashed or dropped.

Author: fredericDelaporte

src/NHibernate/Impl/AbstractSessionImpl.cs

@@ -46,8 +46,8 @@ protected internal AbstractSessionImpl(ISessionFactoryImplementor factory, ISess
 		{
 			_factory = factory;
 			Timestamp = factory.Settings.CacheProvider.NextTimestamp();
-			_flushMode = options.GetInitialSessionFlushMode();
-			Interceptor = options.GetInterceptor() ?? EmptyInterceptor.Instance;
+			_flushMode = options.InitialSessionFlushMode;
+			Interceptor = options.SessionInterceptor ?? EmptyInterceptor.Instance;
 		}
 
 		#region ISessionImplementor Members

src/NHibernate/Impl/ISessionCreationOptions.cs

@@ -8,17 +8,17 @@ namespace NHibernate.Impl
 	/// <seealso cref="ISessionBuilder"/>
 	public interface ISessionCreationOptions
 	{
-		// NH note: it is tempting to convert them to properties, but then their names conflict
-		// with ISessionBuilder, which is implemented along this interface.
-		FlushMode GetInitialSessionFlushMode();
+		// NH note: naming "adjusted" for converting Java methods to properties while avoiding conflicts with
+		// ISessionBuilder.
+		FlushMode InitialSessionFlushMode { get; }
 
-		bool ShouldAutoClose();
+		bool ShouldAutoClose { get; }
 
-		DbConnection GetConnection();
+		DbConnection UserSuppliedConnection { get; }
 
-		IInterceptor GetInterceptor();
+		IInterceptor SessionInterceptor { get; }
 
 		// Todo: port PhysicalConnectionHandlingMode
-		ConnectionReleaseMode GetConnectionReleaseMode();
+		ConnectionReleaseMode SessionConnectionReleaseMode { get; }
 	}
 }

src/NHibernate/Impl/ISharedSessionCreationOptions.cs

@@ -10,8 +10,8 @@ namespace NHibernate.Impl
 	public interface ISharedSessionCreationOptions : ISessionCreationOptions
 	{
 		// NH note: not converting them to properties for staying in sync with ISessionCreationOptions.
-		bool IsTransactionCoordinatorShared();
+		bool IsTransactionCoordinatorShared { get; }
 		// NH different implementation: need to port Hibernate transaction management.
-		ConnectionManager GetConnectionManager();
+		ConnectionManager ConnectionManager { get; }
 	}
 }

src/NHibernate/Impl/SessionFactoryImpl.cs

@@ -1309,20 +1309,18 @@ protected void SetSelf(T self)
 
 			// ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 			// SessionCreationOptions
-			// NH note: it is tempting to convert them to properties, but then their names conflict
-			// with SessionBuilder interface.
 
-			public virtual FlushMode GetInitialSessionFlushMode() => _flushMode;
+			public virtual FlushMode InitialSessionFlushMode => _flushMode;
 
-			public virtual bool ShouldAutoClose() => _autoClose;
+			public virtual bool ShouldAutoClose => _autoClose;
 
-			public DbConnection GetConnection() => _connection;
+			public DbConnection UserSuppliedConnection => _connection;
 
 			// NH different implementation: Hibernate here ignore EmptyInterceptor.Instance too, resulting
 			// in the "NoInterceptor" being unable to override a session factory interceptor.
-			public virtual IInterceptor GetInterceptor() => _interceptor ?? _sessionFactory.Interceptor;
+			public virtual IInterceptor SessionInterceptor => _interceptor ?? _sessionFactory.Interceptor;
 
-			public virtual ConnectionReleaseMode GetConnectionReleaseMode() => _connectionReleaseMode;
+			public virtual ConnectionReleaseMode SessionConnectionReleaseMode => _connectionReleaseMode;
 
 			// ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 			// SessionBuilder
@@ -1400,15 +1398,15 @@ public IStatelessSessionBuilder Connection(DbConnection connection)
 				return this;
 			}
 
-			public FlushMode GetInitialSessionFlushMode() => FlushMode.Always;
+			public FlushMode InitialSessionFlushMode => FlushMode.Always;
 
-			public bool ShouldAutoClose() => false;
+			public bool ShouldAutoClose => false;
 
-			public DbConnection GetConnection() => _connection;
+			public DbConnection UserSuppliedConnection => _connection;
 
-			public IInterceptor GetInterceptor() => EmptyInterceptor.Instance;
+			public IInterceptor SessionInterceptor => EmptyInterceptor.Instance;
 
-			public ConnectionReleaseMode GetConnectionReleaseMode() => ConnectionReleaseMode.AfterTransaction;
+			public ConnectionReleaseMode SessionConnectionReleaseMode => ConnectionReleaseMode.AfterTransaction;
 		}
 	}
 }

src/NHibernate/Impl/SessionImpl.cs

@@ -189,22 +189,22 @@ internal SessionImpl(SessionFactoryImpl factory, ISessionCreationOptions options
 				actionQueue = new ActionQueue(this);
 				persistenceContext = new StatefulPersistenceContext(this);
 
-				autoCloseSessionEnabled = options.ShouldAutoClose();
+				autoCloseSessionEnabled = options.ShouldAutoClose;
 
 				listeners = factory.EventListeners;
-				connectionReleaseMode = options.GetConnectionReleaseMode();
+				connectionReleaseMode = options.SessionConnectionReleaseMode;
 
-				if (options is ISharedSessionCreationOptions sharedOptions && sharedOptions.IsTransactionCoordinatorShared())
+				if (options is ISharedSessionCreationOptions sharedOptions && sharedOptions.IsTransactionCoordinatorShared)
 				{
 					// NH specific implementation: need to port Hibernate transaction management.
 					_transactionCoordinatorShared = true;
-					if (options.GetConnection() != null)
+					if (options.UserSuppliedConnection != null)
 						throw new SessionException("Cannot simultaneously share transaction context and specify connection");
-					connectionManager = sharedOptions.GetConnectionManager();
+					connectionManager = sharedOptions.ConnectionManager;
 				}
 				else
 				{
-					connectionManager = new ConnectionManager(this, options.GetConnection(), connectionReleaseMode, Interceptor);
+					connectionManager = new ConnectionManager(this, options.UserSuppliedConnection, connectionReleaseMode, Interceptor);
 				}
 
 				if (factory.Statistics.IsStatisticsEnabled)
@@ -2552,10 +2552,10 @@ public override ISharedSessionBuilder Connection(DbConnection connection)
 			// SharedSessionCreationOptions
 
 			// NH note: not converting them to properties for staying in sync with ISessionCreationOptions.
-			public virtual bool IsTransactionCoordinatorShared() => _shareTransactionContext;
+			public virtual bool IsTransactionCoordinatorShared => _shareTransactionContext;
 
 			// NH different implementation: need to port Hibernate transaction management.
-			public ConnectionManager GetConnectionManager() => _shareTransactionContext ? _session.ConnectionManager : null;
+			public ConnectionManager ConnectionManager => _shareTransactionContext ? _session.ConnectionManager : null;
 		}
 	}
 }

src/NHibernate/Impl/StatelessSessionImpl.cs

@@ -38,7 +38,7 @@ internal StatelessSessionImpl(SessionFactoryImpl factory, ISessionCreationOption
 			using (new SessionIdLoggingContext(SessionId))
 			{
 				temporaryPersistenceContext = new StatefulPersistenceContext(this);
-				connectionManager = new ConnectionManager(this, options.GetConnection(), ConnectionReleaseMode.AfterTransaction,
+				connectionManager = new ConnectionManager(this, options.UserSuppliedConnection, ConnectionReleaseMode.AfterTransaction,
 					EmptyInterceptor.Instance);
 
 				if (log.IsDebugEnabled)