com.elestrago.unity.entitas-redux 3.3.3 → 3.4.1

package/CHANGELOG.md

@@ -2,6 +2,22 @@
 
 ---
 
+## [3.4.1](https://gitlab.com/elestrago-pkg/entitas-redux/-/tags/3.4.1)
+
+### Fixed
+
+- Possible command class extension generations in Rider IDE
+
+---
+
+## [3.4.0](https://gitlab.com/elestrago-pkg/entitas-redux/-/tags/3.4.0)
+
+### Removed
+
+- Unify Add and Replace methods for components
+
+---
+
 ## [3.3.3](https://gitlab.com/elestrago-pkg/entitas-redux/-/tags/3.3.3)
 
 ### Fixed

package/Core/Context/Context.cs

@@ -64,8 +64,6 @@ namespace JCMG.EntitasRedux
 
 		private EntityRepo<TEntity> _entityRepo;
 
-		private TEntity[] _entitiesCache;
-
 		/// <summary>
 		/// The preferred way to create a context is to use the generated methods
 		/// from the code generator, e.g. var context = new GameContext();
@@ -213,8 +211,6 @@ namespace JCMG.EntitasRedux
 
 		private void OnDestroyEntity(IEntity entity)
 		{
-			_entitiesCache = null;
-
 			using (_onEntityDestroyed.Invokable(out var listeners))
 			{
 				foreach (var listener in listeners)
@@ -328,7 +324,6 @@ namespace JCMG.EntitasRedux
 			}
 
 			entity.Retain(this);
-			_entitiesCache = null;
 
 			entity.OnComponentAdded += _cachedEntityChanged;
 			entity.OnComponentRemoved += _cachedEntityChanged;

package/Core/Context/EntityRepo.cs

@@ -1,6 +1,5 @@
 using System;
 using System.Buffers;
-using System.Runtime.CompilerServices;
 using EntitasRedux.Core.Libs;
 using EntitasRedux.Core.Libs.Collections;
 
@@ -33,7 +32,6 @@ namespace JCMG.EntitasRedux
 
 	public static class EntityRepoExtensions
 	{
-		[MethodImpl(MethodImplOptions.AggressiveInlining)]
 		public static bool Alloc<TEntity>(this ref EntityRepo<TEntity> repo, out TEntity entity)
 			where TEntity : class, IEntity
 		{
@@ -56,7 +54,6 @@ namespace JCMG.EntitasRedux
 			return true;
 		}
 
-		[MethodImpl(MethodImplOptions.AggressiveInlining)]
 		public static bool Has<TEntity>(this ref EntityRepo<TEntity> repo, GenId id)
 			where TEntity : class, IEntity
 		{
@@ -67,7 +64,6 @@ namespace JCMG.EntitasRedux
 			return entity.Id.CreationIndex == id.CreationIndex;
 		}
 
-		[MethodImpl(MethodImplOptions.AggressiveInlining)]
 		public static bool Release<TEntity>(this ref EntityRepo<TEntity> repo, GenId id)
 			where TEntity : class, IEntity
 		{
@@ -83,7 +79,6 @@ namespace JCMG.EntitasRedux
 			return true;
 		}
 
-		[MethodImpl(MethodImplOptions.AggressiveInlining)]
 		public static void ReleaseAll<TEntity>(this ref EntityRepo<TEntity> repo)
 			where TEntity : class, IEntity
 		{
@@ -97,7 +92,6 @@ namespace JCMG.EntitasRedux
 			}
 		}
 
-		[MethodImpl(MethodImplOptions.AggressiveInlining)]
 		public static TEntity GetById<TEntity>(this ref EntityRepo<TEntity> repo, GenId id)
 			where TEntity : class, IEntity
 		{
@@ -108,7 +102,6 @@ namespace JCMG.EntitasRedux
 			return entity;
 		}
 
-		[MethodImpl(MethodImplOptions.AggressiveInlining)]
 		public static void GroupHandleEntitySilently<TEntity>(
 			this ref EntityRepo<TEntity> repo,
 			IGroup<TEntity> collection
@@ -124,7 +117,6 @@ namespace JCMG.EntitasRedux
 			}
 		}
 
-		[MethodImpl(MethodImplOptions.AggressiveInlining)]
 		public static ArrayDisposable<TEntity> GetEntities<TEntity>(
 			this ref EntityRepo<TEntity> repo,
 			out Span<TEntity> entities
@@ -146,14 +138,12 @@ namespace JCMG.EntitasRedux
 			return new ArrayDisposable<TEntity>(array);
 		}
 
-		[MethodImpl(MethodImplOptions.AggressiveInlining)]
 		public static void ResetCreationIndex<TEntity>(this ref EntityRepo<TEntity> repo)
 			where TEntity : class, IEntity
 		{
 			repo.CreationIndex = 0;
 		}
 
-		[MethodImpl(MethodImplOptions.AggressiveInlining)]
 		public static TEntity[] ToArray<TEntity>(this ref EntityRepo<TEntity> repo)
 			where TEntity : class, IEntity
 		{
@@ -171,7 +161,6 @@ namespace JCMG.EntitasRedux
 			return entities;
 		}
 
-		[MethodImpl(MethodImplOptions.AggressiveInlining)]
 		public static ReadOnlySpan<TEntity> AsReadOnlySpan<TEntity>(this ref EntityRepo<TEntity> repo)
 			where TEntity : class, IEntity
 			=> new(repo.Pool, 0, repo.Length);

package/Core/Entity/Entity.cs

@@ -73,54 +73,6 @@ namespace JCMG.EntitasRedux
 			return new ContextInfo("No Context", componentNames, null);
 		}
 
-		private void ReplaceComponentInternal(int index, IComponent replacement)
-		{
-			// TODO VD PERFORMANCE
-			// _toStringCache = null;
-
-			var previousComponent = _components[index];
-			if (replacement != previousComponent)
-			{
-				_components[index] = replacement;
-				_componentsCache = null;
-				if (replacement != null)
-				{
-					using (_onComponentReplaced.Invokable(out var listeners))
-					{
-						foreach (var handler in listeners)
-							handler(this, index, previousComponent, replacement);
-					}
-				}
-				else
-				{
-					_componentIndicesCache = null;
-
-					// TODO VD PERFORMANCE
-					_toStringCache = null;
-
-					using (_onComponentRemoved.Invokable(out var listeners))
-					{
-						foreach (var handler in listeners)
-							handler(this, index, previousComponent);
-					}
-				}
-
-				// ReSharper disable once SuspiciousTypeConversion.Global
-				if (previousComponent is IDisposable disposable)
-					disposable.Dispose();
-
-				GetComponentPool(index).Push(previousComponent);
-			}
-			else
-			{
-				using (_onComponentReplaced.Invokable(out var listeners))
-				{
-					foreach (var handler in listeners)
-						handler(this, index, previousComponent, replacement);
-				}
-			}
-		}
-
 		/// <summary>
 		/// Returns a cached string to describe the entity
 		/// with the following format:
@@ -256,6 +208,7 @@ namespace JCMG.EntitasRedux
 		private readonly Event<EntityEvent> _onDestroyEntity = new();
 
 		private GenId _id;
+
 		public GenId Id => _id;
 
 		/// <summary>
@@ -318,52 +271,6 @@ namespace JCMG.EntitasRedux
 			_isEnabled = true;
 		}
 
-		/// <summary>
-		/// Adds a component at the specified index.
-		/// You can only have one component at an index.
-		/// Each component type must have its own constant index.
-		/// The preferred way is to use the
-		/// generated methods from the code generator.
-		/// </summary>
-		/// <param name="index"></param>
-		/// <param name="component"></param>
-		public void AddComponent(int index, IComponent component)
-		{
-			if (!_isEnabled)
-			{
-				throw new EntityIsNotEnabledException(
-					"Cannot add component '" +
-					_contextInfo.componentNames[index] +
-					"' to " +
-					this +
-					"!");
-			}
-
-			if (HasComponent(index))
-			{
-				throw new EntityAlreadyHasComponentException(
-					index,
-					"Cannot add component '" +
-					_contextInfo.componentNames[index] +
-					"' to " +
-					this +
-					"!",
-					"You should check if an entity already has the component " +
-					"before adding it or use entity.ReplaceComponent().");
-			}
-
-			_components[index] = component;
-			_componentsCache = null;
-			_componentIndicesCache = null;
-			_toStringCache = null;
-
-			using (_onComponentAdded.Invokable(out var listeners))
-			{
-				foreach (var handler in listeners)
-					handler(this, index, component);
-			}
-		}
-
 		/// <summary>
 		/// Removes a component at the specified index.
 		/// You can only remove a component at an index if it exists.
@@ -400,14 +307,14 @@ namespace JCMG.EntitasRedux
 		}
 
 		/// <summary>
-		/// Replaces an existing component at the specified index
+		/// Add or replace an existing component at the specified index
 		/// or adds it if it doesn't exist yet.
 		/// The preferred way is to use the
 		/// generated methods from the code generator.
 		/// </summary>
 		/// <param name="index"></param>
 		/// <param name="component"></param>
-		public void ReplaceComponent(int index, IComponent component)
+		public void AddComponent(int index, IComponent component)
 		{
 			if (!_isEnabled)
 			{
@@ -425,7 +332,69 @@ namespace JCMG.EntitasRedux
 			}
 			else if (component != null)
 			{
-				AddComponent(index, component);
+				AddComponentInternal(index, component);
+			}
+		}
+
+		private void AddComponentInternal(int index, IComponent component)
+		{
+			_components[index] = component;
+			_componentsCache = null;
+			_componentIndicesCache = null;
+			_toStringCache = null;
+
+			using (_onComponentAdded.Invokable(out var listeners))
+			{
+				foreach (var handler in listeners)
+					handler(this, index, component);
+			}
+		}
+
+		private void ReplaceComponentInternal(int index, IComponent replacement)
+		{
+			// TODO VD PERFORMANCE
+			// _toStringCache = null;
+
+			var previousComponent = _components[index];
+			if (replacement != previousComponent)
+			{
+				_components[index] = replacement;
+				_componentsCache = null;
+				if (replacement != null)
+				{
+					using (_onComponentReplaced.Invokable(out var listeners))
+					{
+						foreach (var handler in listeners)
+							handler(this, index, previousComponent, replacement);
+					}
+				}
+				else
+				{
+					_componentIndicesCache = null;
+
+					// TODO VD PERFORMANCE
+					_toStringCache = null;
+
+					using (_onComponentRemoved.Invokable(out var listeners))
+					{
+						foreach (var handler in listeners)
+							handler(this, index, previousComponent);
+					}
+				}
+
+				// ReSharper disable once SuspiciousTypeConversion.Global
+				if (previousComponent is IDisposable disposable)
+					disposable.Dispose();
+
+				GetComponentPool(index).Push(previousComponent);
+			}
+			else
+			{
+				using (_onComponentReplaced.Invokable(out var listeners))
+				{
+					foreach (var handler in listeners)
+						handler(this, index, previousComponent, replacement);
+				}
 			}
 		}
 

package/Core/Entity/IEntity.cs

@@ -60,7 +60,6 @@ namespace JCMG.EntitasRedux
 
 		void AddComponent(int index, IComponent component);
 		void RemoveComponent(int index);
-		void ReplaceComponent(int index, IComponent component);
 
 		IComponent GetComponent(int index);
 		IComponent[] GetComponents();

package/Runtime/Editor/Inspector/Entity/EntityDrawer.cs

@@ -442,7 +442,7 @@ public class ${ShortType}TypeDrawer : ITypeDrawer {
 
 						if (changed)
 						{
-							entity.ReplaceComponent(index, newComponent);
+							entity.AddComponent(index, newComponent);
 						}
 						else
 						{

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "com.elestrago.unity.entitas-redux",
-  "version": "3.3.3",
+  "version": "3.4.1",
   "displayName": "JCMG Entitas Redux",
   "description": "Entitas Redux is an fast, accessible, and feature-rich ECS framework for Unity. It leverages code generation and an extensible plugin framework to make life easier for developers.",
   "category": "Unity",