Document the process of creating docs with doxygen

Source/Csla.Blazor/State/NoOpSessionManager.cs

@@ -31,12 +31,12 @@ public Session GetCachedSession()
     /// <exception cref="NotSupportedException"></exception>
     public Session GetSession()
     { throw new NotSupportedException(); }
-    /// <summary>
-    /// Not supported
-    /// </summary>
-    /// <returns></returns>
-    /// <exception cref="NotSupportedException"></exception>
-    public void PurgeSessions(TimeSpan expiration)
+  /// <summary>
+  /// Not supported.
+  /// </summary>
+  /// <param name="expiration">Unused expiration value.</param>
+  /// <exception cref="NotSupportedException"></exception>
+  public void PurgeSessions(TimeSpan expiration)
     { throw new NotSupportedException(); }
     /// <summary>
     /// Not supported

Source/Csla.Blazor/State/StateManager.cs

@@ -6,17 +6,29 @@
 // <summary>Get and save state from Blazor pages</summary>
 //-----------------------------------------------------------------------
 
+using System;
+using System.Threading;
+using System.Threading.Tasks;
 using Csla.State;
 
 namespace Csla.Blazor.State
 {
   /// <summary>
   /// Get and save state from Blazor pages.
   /// </summary>
-  /// <param name="sessionManager">The session manager to use for state management.</param>
-  public class StateManager(ISessionManager sessionManager)
+  public class StateManager
   {
-    private readonly ISessionManager _sessionManager = sessionManager;
+    private readonly ISessionManager _sessionManager;
+
+    /// <summary>
+    /// Initializes a new instance of the <see cref="StateManager"/> class.
+    /// </summary>
+    /// <param name="sessionManager">Session manager implementation used to persist page state.</param>
+    /// <exception cref="ArgumentNullException"><paramref name="sessionManager"/> is <see langword="null"/>.</exception>
+    public StateManager(ISessionManager sessionManager)
+    {
+      _sessionManager = sessionManager ?? throw new ArgumentNullException(nameof(sessionManager));
+    }
 
     /// <summary>
     /// Get state from cache.

Source/Csla.Web/Design/CslaDesignerDataSourceView.cs

@@ -90,7 +90,7 @@ public override IEnumerable GetDesignTimeData(int minimumRows, out bool isSample
     /// </summary>
     /// <remarks>
     /// All public properties are returned except for those marked
-    /// with the <see cref="BrowsableAttribute">Browsable attribute</see>
+    /// with the <c>System.ComponentModel.BrowsableAttribute</c>
     /// as False.
     /// </remarks>
     public override IDataSourceViewSchema Schema =>

Source/Csla.Web/Design/ObjectFieldInfo.cs

@@ -66,7 +66,7 @@ public ObjectFieldInfo(PropertyDescriptor field)
     /// </summary>
     /// <remarks>
     /// Returns the optional value provided through
-    /// the <see cref="DataObjectFieldAttribute">DataObjectField</see>
+    /// the <c>System.ComponentModel.DataObjectFieldAttribute</c>
     /// attribute on the property.
     /// </remarks>
     public bool Identity { get; }
@@ -93,7 +93,7 @@ public ObjectFieldInfo(PropertyDescriptor field)
     /// </summary>
     /// <remarks>
     /// Returns the optional value provided through
-    /// the <see cref="DataObjectFieldAttribute">DataObjectField</see>
+    /// the <c>System.ComponentModel.DataObjectFieldAttribute</c>
     /// attribute on the property.
     /// </remarks>
     public int Length { get; }
@@ -111,7 +111,7 @@ public ObjectFieldInfo(PropertyDescriptor field)
     /// Returns True for reference types, and for
     /// value types wrapped in the Nullable generic.
     /// The result can also be set to True through
-    /// the <see cref="DataObjectFieldAttribute">DataObjectField</see>
+    /// the <c>System.ComponentModel.DataObjectFieldAttribute</c>
     /// attribute on the property.
     /// </remarks>
     public bool Nullable { get; }
@@ -128,7 +128,7 @@ public ObjectFieldInfo(PropertyDescriptor field)
     /// </summary>
     /// <remarks>
     /// Returns the optional value provided through
-    /// the <see cref="DataObjectFieldAttribute">DataObjectField</see>
+    /// the <c>System.ComponentModel.DataObjectFieldAttribute</c>
     /// attribute on the property.
     /// </remarks>
     public bool PrimaryKey => IsUnique;

Source/Csla.Web/Design/ObjectViewSchema.cs

@@ -55,7 +55,7 @@ public ObjectViewSchema(CslaDataSourceDesigner? site, string typeName)
     /// <remarks>All public properties on the object
     /// will be reflected in this schema list except
     /// for those properties where the 
-    /// <see cref="BrowsableAttribute">Browsable</see> attribute
+    /// <c>System.ComponentModel.BrowsableAttribute</c>
     /// is False.
     /// </remarks>
     public IDataSourceFieldSchema[] GetFields()

Source/Csla/Configuration/Fluent/DataPortalServerOptions.cs

@@ -6,6 +6,7 @@
 // <summary>Server-side data portal options.</summary>
 //-----------------------------------------------------------------------
 
+using System;
 using System.Diagnostics.CodeAnalysis;
 using Csla.Server;
 using Microsoft.Extensions.DependencyInjection;
@@ -15,14 +16,22 @@ namespace Csla.Configuration
   /// <summary>
   /// Server-side data portal options.
   /// </summary>
-  /// <param name="services">Service collection.</param>
-  /// <exception cref="ArgumentNullException"><paramref name="services"/> is <see langword="null"/>.</exception>
-  public class DataPortalServerOptions(IServiceCollection services)
+  public class DataPortalServerOptions
   {
+    /// <summary>
+    /// Initializes a new instance of the <see cref="DataPortalServerOptions"/> class.
+    /// </summary>
+    /// <param name="services">Service collection.</param>
+    /// <exception cref="ArgumentNullException"><paramref name="services"/> is <see langword="null"/>.</exception>
+    public DataPortalServerOptions(IServiceCollection services)
+    {
+      Services = services ?? throw new ArgumentNullException(nameof(services));
+    }
+
     /// <summary>
     /// Gets the service collection.
     /// </summary>
-    public IServiceCollection Services{ get; } = services ?? throw new ArgumentNullException(nameof(services));
+    public IServiceCollection Services { get; }
 
     /// <summary>
     /// Gets or sets a value containing the type of the

Source/Csla/DataPortalT.cs

@@ -52,6 +52,7 @@ public DataPortal(ApplicationContext applicationContext, IDataPortalProxy proxy,
       _dataPortalClientOptions = dataPortalOptions.DataPortalClientOptions;
     }
 
+    /// @cond INTERNAL
     private class DataPortalAsyncRequest
     {
       private ApplicationContext _applicationContext;
@@ -90,6 +91,7 @@ public DataPortalAsyncResult(T result, Exception error, object userState)
       }
     }
 
+    /// @endcond
     private Reflection.ServiceProviderMethodCaller? _serviceProviderMethodCaller;
     private Reflection.ServiceProviderMethodCaller ServiceProviderMethodCaller
     {
@@ -627,10 +629,10 @@ public T CreateChild()
     }
 
     /// <inheritdoc />
-    public T CreateChild(params object?[]? parameters)
+    public T CreateChild(params object?[]? criteria)
     {
       var portal = new Server.ChildDataPortal(_applicationContext);
-      return (T)(portal.Create(typeof(T), parameters));
+      return (T)(portal.Create(typeof(T), criteria));
     }
 
     /// <summary>
@@ -644,10 +646,10 @@ public async Task<T> CreateChildAsync()
     }
 
     /// <inheritdoc />
-    public async Task<T> CreateChildAsync(params object?[]? parameters)
+    public async Task<T> CreateChildAsync(params object?[]? criteria)
     {
       var portal = new Server.ChildDataPortal(_applicationContext);
-      return await portal.CreateAsync<T>(parameters);
+      return await portal.CreateAsync<T>(criteria);
     }
 
     /// <summary>
@@ -661,10 +663,10 @@ public T FetchChild()
     }
 
     /// <inheritdoc />
-    public T FetchChild(params object?[]? parameters)
+    public T FetchChild(params object?[]? criteria)
     {
       var portal = new Server.ChildDataPortal(_applicationContext);
-      return (T)(portal.Fetch(typeof(T), parameters));
+      return (T)(portal.Fetch(typeof(T), criteria));
     }
 
     /// <summary>
@@ -678,10 +680,10 @@ public async Task<T> FetchChildAsync()
     }
 
     /// <inheritdoc />
-    public async Task<T> FetchChildAsync(params object?[]? parameters)
+    public async Task<T> FetchChildAsync(params object?[]? criteria)
     {
       var portal = new Server.ChildDataPortal(_applicationContext);
-      return await portal.FetchAsync<T>(parameters);
+      return await portal.FetchAsync<T>(criteria);
     }
 
     /// <inheritdoc />

Source/Csla/FilteredBindingList.cs

@@ -704,7 +704,9 @@ public T this[int index]
     private IBindingList? _bindingList;
     private object? _filter;
 
-    private readonly List<ListItem> _filterIndex = [];
+    /// @cond DOXYGEN_IGNORE
+    private readonly List<ListItem> _filterIndex = new();
+    /// @endcond
 
     /// <summary>
     /// Creates a new view based on the provided IList object.

Source/Csla/IChildDataPortal.cs

@@ -63,8 +63,7 @@ public interface IChildDataPortal
     /// to use this new object.
     /// </remarks>
     /// <param name="obj">A reference to the business object to be updated.</param>
-    /// <param name="parameters"></param>
-    /// <returns>A reference to the updated business object.</returns>
+    /// <param name="parameters">Optional arguments passed to child update methods.</param>
     /// <exception cref="ArgumentNullException"><paramref name="obj"/> is <see langword="null"/>.</exception>
     void UpdateChild(object obj, params object?[]? parameters);
   }

Source/Csla/IChildDataPortalT.cs

@@ -36,10 +36,10 @@ public interface IChildDataPortal<[DynamicallyAccessedMembers(DynamicallyAccesse
     /// Called by a factory method in a business class or
     /// by the UI to update an object.
     /// </summary>
-    /// <param name="obj">Object to update.</param>
-    /// <param name="parameters">Additional, optional parameters to pass</param>
-    /// /// <exception cref="ArgumentNullException"><paramref name="obj"/> is <see langword="null"/>.</exception>
-    Task UpdateChildAsync(T obj, params object?[]? parameters);
+    /// <param name="child">Object to update.</param>
+    /// <param name="parameters">Additional, optional parameters to pass.</param>
+    /// <exception cref="ArgumentNullException"><paramref name="child"/> is <see langword="null"/>.</exception>
+    Task UpdateChildAsync(T child, params object?[]? parameters);
     /// <summary>
     /// Called by a factory method in a business class to create 
     /// a new object, which is loaded with default
@@ -65,10 +65,9 @@ public interface IChildDataPortal<[DynamicallyAccessedMembers(DynamicallyAccesse
     /// different object from the original, and all object references MUST be updated
     /// to use this new object.
     /// </remarks>
-    /// <param name="obj">A reference to the business object to be updated.</param>
-    /// <param name="parameters"></param>
-    /// <returns>A reference to the updated business object.</returns>
-    /// <exception cref="ArgumentNullException"><paramref name="obj"/> is <see langword="null"/>.</exception>
-    void UpdateChild(T obj, params object?[]? parameters);
+    /// <param name="child">A reference to the business object to be updated.</param>
+    /// <param name="parameters">Optional arguments passed to child update methods.</param>
+    /// <exception cref="ArgumentNullException"><paramref name="child"/> is <see langword="null"/>.</exception>
+    void UpdateChild(T child, params object?[]? parameters);
   }
 }

Source/Csla/LazySingleton.cs

@@ -35,7 +35,7 @@ public LazySingleton()
     /// Initializes a new instance of the <see cref="LazySingleton&lt;T&gt;"/> class.
     /// Will call the supplied delegate to create an instance of T (the value)
     /// </summary>
-    /// <param name="delegate">The @delegate.</param>
+  /// <param name="delegate">Factory delegate used to create the singleton value.</param>
     /// <exception cref="ArgumentNullException"><paramref name="delegate"/> is <see langword="null"/>.</exception>
     public LazySingleton(Func<T> @delegate)
     {

Source/Csla/Server/GenericBusinessException.cs

@@ -24,9 +24,6 @@ public class GenericBusinessException : Exception
     /// </summary>
     /// <value></value>
     /// <returns>A string that describes the immediate frames of the call stack.</returns>
-    /// <PermissionSet>
-    /// 	<IPermission class="System.Security.Permissions.FileIOPermission, mscorlib, Version=2.0.3600.0, Culture=neutral, PublicKeyToken=b77a5c561934e089" version="1" PathDiscovery="*AllFiles*"/>
-    /// </PermissionSet>
     public override string StackTrace { get; }
 
     /// <summary>
@@ -78,9 +75,6 @@ public GenericBusinessException(Exception wrappedException, Exception innerExcep
     /// <returns>
     /// A <see cref="System.String"/> that represents this instance.
     /// </returns>
-    /// <PermissionSet>
-    /// 	<IPermission class="System.Security.Permissions.FileIOPermission, mscorlib, Version=2.0.3600.0, Culture=neutral, PublicKeyToken=b77a5c561934e089" version="1" PathDiscovery="*AllFiles*"/>
-    /// </PermissionSet>
     public override string ToString()
     {
       string className;

Source/Csla/Server/ObjectFactoryLoader.cs

@@ -38,7 +38,7 @@ public ObjectFactoryLoader(ApplicationContext applicationContext)
     /// on the business object.
     /// </param>
     /// <exception cref="ArgumentException"><paramref name="factoryName"/> is <see langword="null"/>, <see cref="string.Empty"/> or only consists of white spaces.</exception>
-    /// <exception cref="InvalidOperationException"><see cref="Type"/> for <paramref name="factoryName"/> could not be loaded.</exception>
+  /// <exception cref="InvalidOperationException"><c>System.Type</c> for <paramref name="factoryName"/> could not be loaded.</exception>
     public Type GetFactoryType(string factoryName)
     {
       if (factoryName is null)
@@ -62,7 +62,7 @@ public Type GetFactoryType(string factoryName)
     /// type name parameter.
     /// </returns>
     /// <exception cref="ArgumentException"><paramref name="factoryName"/> is <see langword="null"/>, <see cref="string.Empty"/> or only consists of white spaces.</exception>
-    /// <exception cref="InvalidOperationException"><see cref="Type"/> for <paramref name="factoryName"/> could not be loaded.</exception>
+  /// <exception cref="InvalidOperationException"><c>System.Type</c> for <paramref name="factoryName"/> could not be loaded.</exception>
     public object GetFactory(string factoryName)
     {
       if (factoryName is null)

Source/Csla/Server/ServicedDataPortalReadCommitted.cs

@@ -42,7 +42,7 @@ public ServicedDataPortalReadCommitted(DataPortalBroker dataPortalBroker)
     /// but wraps that call within a COM+ transaction
     /// to provide transactional support.
     /// </remarks>
-    /// <param name="objectType">A <see cref="Type">Type</see> object
+  /// <param name="objectType">A <c>System.Type</c> object
     /// indicating the type of business object to be created.</param>
     /// <param name="criteria">A custom criteria object providing any
     /// extra information that may be required to properly create
@@ -66,7 +66,7 @@ public Task<DataPortalResult> Create(Type objectType, object criteria, DataPorta
     /// but wraps that call within a COM+ transaction
     /// to provide transactional support.
     /// </remarks>
-    /// <param name="objectType">Type of business object to retrieve.</param>
+  /// <param name="objectType">Type of business object to retrieve.</param>
     /// <param name="criteria">Object-specific criteria.</param>
     /// <param name="context">Object containing context data from client.</param>
     /// <param name="isSync">True if the client-side proxy should synchronously invoke the server.</param>
@@ -107,7 +107,7 @@ public Task<DataPortalResult> Update(object obj, DataPortalContext context, bool
     /// but wraps that call within a COM+ transaction
     /// to provide transactional support.
     /// </remarks>
-    /// <param name="objectType">Type of business object to create.</param>
+  /// <param name="objectType">Type of business object to create.</param>
     /// <param name="criteria">Object-specific criteria.</param>
     /// <param name="context">Context data from the client.</param>
     /// <param name="isSync">True if the client-side proxy should synchronously invoke the server.</param>

Source/Csla/Server/ServicedDataPortalReadUncommitted.cs

@@ -41,7 +41,7 @@ public ServicedDataPortalReadUncommitted(DataPortalBroker dataPortalBroker)
     /// but wraps that call within a COM+ transaction
     /// to provide transactional support.
     /// </remarks>
-    /// <param name="objectType">A <see cref="Type">Type</see> object
+  /// <param name="objectType">A <c>System.Type</c> object
     /// indicating the type of business object to be created.</param>
     /// <param name="criteria">A custom criteria object providing any
     /// extra information that may be required to properly create

Source/Csla/Server/ServicedDataPortalRepeatableRead.cs

@@ -41,7 +41,7 @@ public ServicedDataPortalRepeatableRead(DataPortalBroker dataPortalBroker)
     /// but wraps that call within a COM+ transaction
     /// to provide transactional support.
     /// </remarks>
-    /// <param name="objectType">A <see cref="Type">Type</see> object
+  /// <param name="objectType">A <c>System.Type</c> object
     /// indicating the type of business object to be created.</param>
     /// <param name="criteria">A custom criteria object providing any
     /// extra information that may be required to properly create

Source/Csla/Server/ServicedDataPortalSerializable.cs

@@ -41,7 +41,7 @@ public ServicedDataPortalSerializable(DataPortalBroker dataPortalBroker)
     /// but wraps that call within a COM+ transaction
     /// to provide transactional support.
     /// </remarks>
-    /// <param name="objectType">A <see cref="Type">Type</see> object
+  /// <param name="objectType">A <c>System.Type</c> object
     /// indicating the type of business object to be created.</param>
     /// <param name="criteria">A custom criteria object providing any
     /// extra information that may be required to properly create

Source/Csla/Server/TransactionalDataPortal.cs

@@ -47,7 +47,7 @@ public TransactionalDataPortal(DataPortalBroker dataPortalBroker, TransactionalA
     /// to provide transactional support via
     /// System.Transactions.
     /// </remarks>
-    /// <param name="objectType">A <see cref="Type">Type</see> object
+  /// <param name="objectType">A <c>System.Type</c> object
     /// indicating the type of business object to be created.</param>
     /// <param name="criteria">A custom criteria object providing any
     /// extra information that may be required to properly create

Source/Csla/SortedBindingList.cs

@@ -698,7 +698,9 @@ public T this[int index]
     private IBindingList? _bindingList;
     private bool _initiatedLocally;
 
-    private readonly List<ListItem> _sortIndex = [];
+    /// @cond DOXYGEN_IGNORE
+    private readonly List<ListItem> _sortIndex = new();
+    /// @endcond
 
     /// <summary>
     /// Creates a new view based on the provided IList object.