Fixed customroles. Exiled 9.0.0-beta.1 release (#2769)

* Fixed Pawn.cs being null if try to use it by Player::Get().Cast<Pawn>(). Adding a method for load a constructor without any param or constructor. Removing abstract from RoleBehaviour.cs for allow the creating of an instance. Added custom role example under Exiled.Example for testing purpose.

* Fixed custom roles.

* Added test role (testing purposes)

* Fixed customroles. Release 9.0.0-beta.1

Author: Monaldcry7788

EXILED.props

@@ -15,7 +15,7 @@
 
     <PropertyGroup>
         <!-- This is the global version and is used for all projects that don't have a version -->
-        <Version Condition="$(Version) == ''">9.0.0-alpha.19</Version>
+        <Version Condition="$(Version) == ''">9.0.0-beta.1</Version>
         <!-- Enables public beta warning via the PUBLIC_BETA constant -->
         <PublicBeta>false</PublicBeta>
 

Exiled.API/Features/Core/EActor.cs

@@ -481,7 +481,6 @@ protected virtual void PostInitialize()
         /// </summary>
         protected virtual void OnBeginPlay()
         {
-            SubscribeEvents();
         }
 
         /// <summary>

Exiled.API/Features/Core/EObject.cs

@@ -918,4 +918,4 @@ protected virtual void OnDestroyed()
         {
         }
     }
-}
+}

Exiled.API/Features/Core/GameEntity.cs

@@ -163,7 +163,6 @@ public T AddComponent<T>(string name = "")
         public EActor AddComponent(Type type, string name = "")
         {
             EActor component = EObject.CreateDefaultSubobject(type, GameObject).Cast<EActor>();
-            component.Base = GameObject;
 
             if (!component)
                 return null;

Exiled.API/Features/Core/Generic/EBehaviour.cs

@@ -25,6 +25,13 @@ namespace Exiled.API.Features.Core.Generic
     public abstract class EBehaviour<T> : EActor
         where T : GameEntity
     {
+        /// <summary>
+        /// Initializes a new instance of the <see cref="EBehaviour{T}"/> class.
+        /// </summary>
+        protected EBehaviour()
+        {
+        }
+
         /// <summary>
         /// Gets or sets the owner of the <see cref="EBehaviour{T}"/>.
         /// </summary>
@@ -78,6 +85,12 @@ protected override void PostInitialize()
             }
         }
 
+        /// <inheritdoc />
+        protected override void OnBeginPlay()
+        {
+            base.OnBeginPlay();
+        }
+
         /// <inheritdoc/>
         protected override void Tick()
         {

Exiled.CustomModules/API/Features/CustomModule.cs

@@ -318,9 +318,7 @@ public static void LoadAll()
         /// <param name="shouldBeEnabled">Determines whether the loaded modules should be enabled after loading.</param>
         public static void Load(Assembly assembly = null, bool shouldBeEnabled = false)
         {
-            Log.InfoWithContext($"Using defined assembly? {assembly is null}");
             assembly ??= Assembly.GetCallingAssembly();
-            Log.InfoWithContext($"{assembly.GetName().Name}");
 
             UUModuleType FindClosestModuleType(Type t, IEnumerable<FieldInfo> source)
             {
@@ -331,7 +329,6 @@ UUModuleType FindClosestModuleType(Type t, IEnumerable<FieldInfo> source)
 
             Type runtime_ModuleType = assembly.GetTypes().FirstOrDefault(t => !t.IsAbstract && typeof(UUModuleType).IsAssignableFrom(t)) ?? typeof(UUModuleType);
             IEnumerable<FieldInfo> moduleTypeValuesInfo = runtime_ModuleType.GetFields(BindingFlags.Static | BindingFlags.Public).Where(f => f.GetValue(null) is UUModuleType);
-            Log.InfoWithContext($"DB1");
 
             foreach (Type type in assembly.GetTypes())
             {

Exiled.CustomModules/API/Features/CustomRoles/RoleBehaviour.cs

@@ -355,9 +355,9 @@ protected override void PostInitialize()
         protected override void OnBeginPlay()
         {
             base.OnBeginPlay();
-
             if (!Owner)
             {
+                Log.WarnWithContext("Owner is null");
                 Destroy();
                 return;
             }
@@ -369,6 +369,8 @@ protected override void OnBeginPlay()
                 Owner.ChangeAppearance(FakeAppearance, false);
 
             PermanentEffects?.ForEach(x => Owner.SyncEffect(x));
+
+            SubscribeEvents();
         }
 
         /// <inheritdoc/>
@@ -450,9 +452,6 @@ protected override void SubscribeEvents()
         {
             base.SubscribeEvents();
 
-            EscapingEventDispatcher.Bind(this, OnEscaping);
-            EscapedEventDispatcher.Bind(this, OnEscaped);
-
             Exiled.Events.Handlers.Player.ChangingItem += ChangingItemBehaviour;
             Exiled.Events.Handlers.Player.Destroying += DestroyOnLeave;
             Exiled.Events.Handlers.Player.ChangingRole += DestroyOnChangingRole;
@@ -474,6 +473,8 @@ protected override void SubscribeEvents()
             Exiled.Events.Handlers.Player.Handcuffing += HandcuffingBehavior;
             Exiled.Events.Handlers.Map.PlacingBlood += PlacingBloodBehavior;
             Exiled.Events.Handlers.Player.ChangingNickname += OnInternalChangingNickname;
+            EscapingEventDispatcher.Bind(this, OnEscaping);
+            EscapedEventDispatcher.Bind(this, OnEscaped);
         }
 
         /// <inheritdoc/>
@@ -759,21 +760,8 @@ protected virtual void PlacingBloodBehavior(PlacingBloodEventArgs ev)
         /// <inheritdoc cref="Exiled.Events.Handlers.Player.OnEnteringPocketDimension(EnteringPocketDimensionEventArgs)"/>
         protected virtual void DroppingItemBehavior(DroppingItemEventArgs ev)
         {
-            if (ev.Item is null)
-            {
-                Log.Error("Item is null");
-            }
-
-            Log.InfoWithContext($"{ev.Player} is trying to drop {ev.Item.Type}");
-
-            Log.WarnWithContext(Settings.CanDropItems);
             if (!Check(ev.Player) || Settings.CanDropItems)
-            {
-                Log.InfoWithContext($"{ev.Player} is not {CustomRole.Name} or can drop items as {CustomRole.Name}");
                 return;
-            }
-
-            Log.InfoWithContext($"{ev.Player} cannot drop items");
 
             ev.IsAllowed = false;
         }