Small refactoring to simplify 6.0 migration and adjust comments

Author: bahusoid

src/NHibernate/Loader/OuterJoinableAssociation.cs

@@ -203,6 +203,7 @@ internal string GetSelectFragment(string entitySuffix, string collectionSuffix,
 			{
 				case SelectMode.Undefined:
 				case SelectMode.Fetch:
+					//TODO 6.0: Just drop obsolete parts making all 3 cases handled by the same code
 #pragma warning disable 618
 					return Joinable.SelectFragment(
 						next?.Joinable,
@@ -228,16 +229,17 @@ internal string GetSelectFragment(string entitySuffix, string collectionSuffix,
 
 				case SelectMode.FetchLazyPropertyGroup:
 					return ReflectHelper.CastOrThrow<ISupportLazyPropsJoinable>(Joinable, "fetch lazy property")
-					                    .SelectFragment(
-						                    next?.Joinable,
-						                    next?.RHSAlias,
-						                    RHSAlias,
-						                    collectionSuffix,
-						                    ShouldFetchCollectionPersister(),
-						                    new EntityLoadInfo(entitySuffix)
-						                    {
-							                    LazyProperties = EntityFetchLazyProperties
-						                    });
+										.SelectFragment(
+											next?.Joinable,
+											next?.RHSAlias,
+											RHSAlias,
+											collectionSuffix,
+											ShouldFetchCollectionPersister(),
+											new EntityLoadInfo(entitySuffix)
+											{
+												LazyProperties = EntityFetchLazyProperties, 
+												IncludeLazyProps = SelectMode == SelectMode.FetchLazyProperties,
+											});
 				case SelectMode.ChildFetch:
 					return ReflectHelper.CastOrThrow<ISupportSelectModeJoinable>(Joinable, "child fetch select mode")
 					                    .IdentifierSelectFragment(RHSAlias, entitySuffix);

src/NHibernate/SelectMode.cs

@@ -33,12 +33,12 @@ public enum SelectMode
 		JoinOnly,
 
 		/// <summary>
-		/// Skips fetching for eagerly mapped association (no-op for lazy association).
+		/// Skips fetching for fetch="join" association (no-op for lazy association).
 		/// </summary>
 		Skip,
 
 		/// <summary>
-		/// Fetch lazy property group
+		/// Fetch lazy property group (provide path to lazy property to be fetched)
 		/// Note: To fetch single property it must be mapped with unique fetch group (lazy-group)
 		/// </summary>
 		FetchLazyPropertyGroup,