Merge pull request #143 from NerosoftDev/develop

Develop

Author: Codespilot

Source/Euonia.Domain/Commands/Command.cs

@@ -1,5 +1,4 @@
-using System.ComponentModel;
-using Nerosoft.Euonia.Reflection;
+using Nerosoft.Euonia.Reflection;
 
 namespace Nerosoft.Euonia.Domain;
 

Source/Euonia.Osba/Abstracts/IBusinessObject.cs

@@ -45,6 +45,17 @@ public interface IBusinessObject : IUseBusinessContext, INotifyPropertyChanged,
 	/// <returns>The value of the specified property, or null if the property has not been set.</returns>
 	object ReadProperty(IPropertyInfo propertyInfo);
 
+	/// <summary>
+	/// Retrieves the value of the property specified by its name.
+	/// </summary>
+	/// <remarks>
+	///	Use this method to access the current value of a property identified by its name.
+	/// Ensure that the property name provided corresponds to a valid property that can be read. This method does not set or modify the property value.
+	/// </remarks>
+	/// <param name="propertyName">The name of the property to read. Must represent a readable property.</param>
+	/// <returns>The value of the specified property, or null if the property has not been set.</returns>
+	object ReadProperty(string propertyName);
+	
 	/// <summary>
 	/// Reads the value of the specified property and returns it as the requested type.
 	/// </summary>
@@ -58,6 +69,18 @@ public interface IBusinessObject : IUseBusinessContext, INotifyPropertyChanged,
 	/// null.</param>
 	/// <returns>The value of the specified property, cast to the type specified by <typeparamref name="TValue"/>.</returns>
 	TValue ReadProperty<TValue>(PropertyInfo<TValue> propertyInfo);
+	
+	/// <summary>
+	/// Reads the value of the specified property by its name and returns it as the requested type.
+	/// </summary>
+	/// <remarks>
+	///	Ensure that the property name provided corresponds to a valid property that can be read and that the value can be cast to <typeparamref name="TValue"/>.
+	/// An exception may be thrown if the property is not readable or if the value cannot be cast to the specified type.
+	/// </remarks>
+	/// <param name="propertyName">The name of the property to read. Must represent a readable property.</param>
+	/// <typeparam name="TValue">The type of the property value to be read.</typeparam>
+	/// <returns>The value of the specified property, cast to the type specified by <typeparamref name="TValue"/>.</returns>
+	TValue ReadProperty<TValue>(string propertyName);
 
 	/// <summary>
 	/// Loads the specified property with a new value, updating the property's value according to its metadata information.

Source/Euonia.Osba/Core/BusinessObject.cs

@@ -351,7 +351,7 @@ protected void PropertyHasChanged(string propertyName)
 	protected internal sealed class BypassRuleChecksObject : IDisposable
 	{
 		private BusinessObject _target;
-		private static readonly object _lock = new();
+		private static readonly Lock _lock = new();
 
 		private BypassRuleChecksObject(BusinessObject target)
 		{
@@ -416,12 +416,14 @@ private void DeRef()
 			lock (_lock)
 			{
 				_refCount -= 1;
-				if (_refCount == 0)
+				if (_refCount != 0)
 				{
-					_target.IsBypassingRuleChecks = false;
-					_target.InternalBypassRuleChecks = null;
-					_target = null;
+					return;
 				}
+
+				_target.IsBypassingRuleChecks = false;
+				_target.InternalBypassRuleChecks = null;
+				_target = null;
 			}
 		}
 
@@ -508,6 +510,52 @@ public virtual object ReadProperty(IPropertyInfo propertyInfo)
 		return result;
 	}
 
+	/// <summary>
+	/// Gets a property's value by property name.
+	/// </summary>
+	/// <param name="propertyName"></param>
+	/// <returns></returns>
+	/// <exception cref="InvalidOperationException"></exception>
+	public virtual object ReadProperty(string propertyName)
+	{
+		var propertyInfo = FieldManager.GetRegisteredProperty(propertyName);
+		if (propertyInfo == null)
+		{
+			throw new InvalidOperationException($"Property {propertyName} is not registered.");
+		}
+
+		return ReadProperty(propertyInfo);
+	}
+
+	/// <summary>
+	/// Reads the value of the specified property by its name and returns it as the requested type.
+	/// </summary>
+	/// <param name="propertyName">The name of the property to read. Must represent a readable property.</param>
+	/// <typeparam name="TValue">The type of the property value to be read.</typeparam>
+	/// <returns>The value of the specified property, cast to the type specified by <typeparamref name="TValue"/>.</returns>
+	/// <exception cref="InvalidOperationException">
+	///	Thrown if the property name provided does not correspond to a valid property that can be read, or if the value cannot be cast to the specified type.
+	/// </exception>
+	public virtual TValue ReadProperty<TValue>(string propertyName)
+	{
+		var propertyInfo = FieldManager.GetRegisteredProperty(propertyName);
+		
+		if (propertyInfo == null)
+		{
+			throw new InvalidOperationException($"Property {propertyName} is not registered.");
+		}
+
+		if (propertyInfo is not PropertyInfo<TValue> property)
+		{
+			throw new InvalidOperationException("The property type does not match the expected type.");
+		}
+
+		{
+		}
+
+		return ReadProperty(property);
+	}
+
 	#endregion
 
 	#region Load Properties

Source/Euonia.Osba/Core/ObservableList.cs

@@ -1,11 +1,6 @@
-using System;
-using System.Collections.Generic;
-using System.Collections.ObjectModel;
+using System.Collections.ObjectModel;
 using System.Collections.Specialized;
 using System.ComponentModel;
-using System.Linq;
-using System.Text;
-using System.Threading.Tasks;
 
 namespace Nerosoft.Euonia.Osba;
 
@@ -21,8 +16,8 @@ namespace Nerosoft.Euonia.Osba;
 /// can suppress change notifications during batch updates to improve performance and avoid unnecessary UI refreshes. It
 /// also propagates property and busy state changes from child items, enabling more granular change tracking.
 /// </remarks>
-/// <typeparam name="T">The type of elements contained in the observable list.</typeparam>
-public class ObservableList<T> : ObservableCollection<T>, INotifyBusy
+/// <typeparam name="TItem">The type of elements contained in the observable list.</typeparam>
+public class ObservableList<TItem> : ObservableCollection<TItem>, INotifyBusy
 {
 	private EventHandler<ObjectChangedEventArgs> _childChanged = null;
 
@@ -50,7 +45,8 @@ public event EventHandler<ObjectChangedEventArgs> ChildChanged
 	public bool RaiseListChangedEvents { get; set; } = true;
 
 	#region BusyChanged
-	private BusyChangedEventHandler _busyChanged = null;
+
+	private BusyChangedEventHandler _busyChanged;
 
 	/// <summary>
 	/// Event indicating that the busy status of the
@@ -96,6 +92,7 @@ protected void OnBusyChanged(string propertyName, bool busy)
 	/// <remarks>This property reflects the state of the IsBusy property, providing a convenient way to check if the
 	/// instance is currently engaged in operations.</remarks>
 	public virtual bool IsSelfBusy => IsBusy;
+
 	#endregion
 
 	/// <summary>
@@ -117,7 +114,7 @@ protected override void RemoveItem(int index)
 	/// after insertion. This allows the collection to respond to events raised by the item.</remarks>
 	/// <param name="index">The zero-based index at which the item should be inserted into the collection.</param>
 	/// <param name="item">The item to insert and attach event hooks to.</param>
-	protected override void InsertItem(int index, T item)
+	protected override void InsertItem(int index, TItem item)
 	{
 		base.InsertItem(index, item);
 		AddEventHooks(item);
@@ -142,7 +139,7 @@ protected override void OnCollectionChanged(NotifyCollectionChangedEventArgs e)
 	/// Removes event hooks from an item.
 	/// </summary>
 	/// <param name="item"></param>
-	protected virtual void RemoveEventHooks(T item)
+	protected virtual void RemoveEventHooks(TItem item)
 	{
 		if (item == null)
 		{
@@ -167,7 +164,7 @@ protected virtual void RemoveEventHooks(T item)
 	/// PropertyChanged event if the item implements INotifyPropertyChanged. These subscriptions enable the system to
 	/// respond to changes in the item's state or properties.</remarks>
 	/// <param name="item">The item to which event handlers are added. This parameter must not be null; if null, no handlers are attached.</param>
-	protected virtual void AddEventHooks(T item)
+	protected virtual void AddEventHooks(TItem item)
 	{
 		if (item == null)
 		{
@@ -203,6 +200,7 @@ protected virtual void OnChildChanged(ObjectChangedEventArgs args)
 	}
 
 	#region Event Subscriptions
+
 	private void OnItemBusyChanged(object sender, BusyChangedEventArgs e)
 	{
 		OnBusyChanged(e);
@@ -212,13 +210,14 @@ private void OnItemPropertyChanged(object sender, PropertyChangedEventArgs e)
 	{
 		RaiseChildChanged(sender, e, null);
 	}
+
 	#endregion
 
 	/// <summary>
 	/// Use this object to suppress ListChangedEvents for an entire code block.
 	/// May be nested in multiple levels for the same object.
 	/// </summary>
-	public IDisposable SuppressListChangedEvents => new SuppressListChangedEventsClass<T>(this);
+	public IDisposable SuppressListChangedEvents => new SuppressListChangedEventsClass<TItem>(this);
 
 	/// <summary>
 	/// <![CDATA[Provides a mechanism to temporarily suppress change notifications for an ObservableList<T> instance.]]>
@@ -230,13 +229,13 @@ private void OnItemPropertyChanged(object sender, PropertyChangedEventArgs e)
 	/// improving performance and avoiding unnecessary updates to data-bound controls during batch operations.
 	/// ]]>
 	/// </remarks>
-	/// <typeparam name="D">The type of elements contained in the observable list.</typeparam>
-	private class SuppressListChangedEventsClass<D> : IDisposable
+	/// <typeparam name="TList">The type of elements contained in the observable list.</typeparam>
+	private class SuppressListChangedEventsClass<TList> : IDisposable
 	{
-		private readonly ObservableList<D> _listObject;
+		private readonly ObservableList<TList> _listObject;
 		private readonly bool _initialRaiseListChangedEvents;
 
-		public SuppressListChangedEventsClass(ObservableList<D> listObject)
+		public SuppressListChangedEventsClass(ObservableList<TList> listObject)
 		{
 			_listObject = listObject;
 			_initialRaiseListChangedEvents = listObject.RaiseListChangedEvents;

Source/Euonia.Osba/Factory/ObjectReflector.cs

@@ -194,8 +194,9 @@ private static MethodInfo FindMatchedMethod<TTarget>(Type attributeType, object[
 	public static MethodInfo FindMatchedMethod<TTarget>(Type attributeType, IReadOnlyList<Type> parameterTypes)
 	{
 		var methods = typeof(TTarget).GetRuntimeMethods()
-		                             .Where(t => t.GetCustomAttribute(attributeType) != null);
-		if (methods == null || !methods.Any())
+		                             .Where(t => t.GetCustomAttribute(attributeType) != null)
+		                             .ToList();
+		if (methods is not { Count: > 0 })
 		{
 			throw new MissingMethodException($"Missing method with attribute '{attributeType.Name}' on {typeof(TTarget).FullName}");
 		}

Source/Euonia.Osba/Reflection/FieldDataManager.cs

@@ -103,6 +103,24 @@ public IFieldData GetFieldData(IPropertyInfo property)
 		}
 	}
 
+	/// <summary>
+	/// Gets the field data for a property with specified name.
+	/// </summary>
+	/// <param name="propertyName"></param>
+	/// <returns></returns>
+	/// <exception cref="InvalidOperationException"></exception>
+	public IFieldData GetFieldData(string propertyName)
+	{
+		try
+		{
+			return _fieldData.GetValueOrDefault(propertyName);
+		}
+		catch (IndexOutOfRangeException ex)
+		{
+			throw new InvalidOperationException(RESOURCE_PROPERTY_NAME_NOT_REGISTERED, ex);
+		}
+	}
+
 	private IFieldData GetOrCreateFieldData(IPropertyInfo property)
 	{
 		try

Source/Euonia.Validation/Core/Validator.cs

@@ -14,7 +14,7 @@ public class Validator
     public static void Validate<T>(T item)
         where T : class
     {
-		ArgumentAssert.ThrowIfNull(item, nameof(item));
+		ArgumentAssert.ThrowIfNull(item);
 
         if (item is IValidatableObject @object)
         {
@@ -42,7 +42,7 @@ public static void Validate<T>(T item)
 	public static async Task ValidateAsync<T>(T item)
 		where T : class
 	{
-		ArgumentAssert.ThrowIfNull(item, nameof(item));
+		ArgumentAssert.ThrowIfNull(item);
 
 		if (item is IValidatableObject @object)
 		{

project.props

@@ -1,6 +1,6 @@
 <Project>
     <PropertyGroup>
-        <Version>10.4.1</Version>
+        <Version>10.4.2</Version>
         <Authors>damon</Authors>
         <Company>Nerosoft Ltd.</Company>
         <Product>Euonia</Product>