Upgrade to entities 0.14.0-preview.18 (#1463)

* Upgrade to entities 0.14.0-preview.18

* Fix android

* Fix TransformSync using deprecated EntityManager null

* Changelog and Upgrade guide

* Fix codegen using deprecated methods

* Remove unused usings

Author: zeroZshadow

CHANGELOG.md

@@ -5,6 +5,8 @@
 ### Breaking Changes
 
 - The minimum supported version of Unity is now 2020.1. [#1459](https://github.com/spatialos/gdk-for-unity/pull/1459)
+- Upgraded Unity Entities to 0.14.0-preview.18. [#1463](https://github.com/spatialos/gdk-for-unity/pull/1463)
+  - Projects now require the `Unity Web Request` built-in package to compile for iOS and Android.
 
 ### Added
 

UPGRADE_GUIDE.md

@@ -6,6 +6,11 @@
 
 In order to use the GDK for Unity `0.4.0`, you will need to upgrade your project to Unity 2020.1. We test against 2020.1.2f1, but any `2020.1.x` version _should_ work.
 
+### Unity Entities 0.14
+
+Due to our internal upgrade to Unity Entities `0.14.0-preview.18`, your projects are now required to include the `Unity Web Request` built-in package.
+You can find this package in the Unity editor under the `Window -> Package Manager` menu.
+
 ## From `0.3.9` to `0.3.10`
 
 ### AuthorityLossImminent support removed

workers/unity/Assets/Tests/PlaymodeTests/Correctness/TransformSynchronization/TransformInitialisationTest.cs

@@ -62,7 +62,7 @@ public IEnumerator Transform_initialises_on_enable_and_resets_on_disable()
             RequireLifecycleSystem.Update();
 
             Assert.IsFalse(transformSyncBehaviour.enabled);
-            Assert.IsNull(GetPrivateField<EntityManager>(transformSyncBehaviour, "entityManager"));
+            Assert.AreEqual(GetPrivateField<EntityManager>(transformSyncBehaviour, "entityManager"), default(EntityManager));
             Assert.IsFalse(GetPrivateField<bool>(transformSyncBehaviour, "initialised"));
         }
 

workers/unity/Packages/io.improbable.gdk.core/.codegen/Source/Generators/Core/UnityEcsViewManagerGenerator.cs

@@ -173,7 +173,7 @@ public void Init(World world)
                             {
                                 m.Line(@"
 var entity = workerSystem.GetEntity(update.EntityId);
-if (!dataFromEntity.Exists(entity))
+if (!dataFromEntity.HasComponent(entity))
 {
     return;
 }

workers/unity/Packages/io.improbable.gdk.core/Utility/PlayerLoopUtils.cs

@@ -1,10 +1,8 @@
 using System;
 using System.Collections.Generic;
 using System.Linq;
-using System.Reflection;
 using Unity.Entities;
 using UnityEngine;
-using UnityEngine.LowLevel;
 
 namespace Improbable.Gdk.Core
 {
@@ -72,38 +70,9 @@ public static void ResolveSystemGroups(World world)
                     type == typeof(PresentationSystemGroup))
                 {
                     var groupSystem = system as ComponentSystemGroup;
-                    groupSystem.SortSystemUpdateList();
+                    groupSystem.SortSystems();
                 }
             }
         }
-
-        public static void RemoveFromPlayerLoop(World world)
-        {
-            var playerLoop = PlayerLoop.GetCurrentPlayerLoop();
-
-            //Reflection to get world from PlayerLoopSystem
-            var wrapperType =
-                typeof(ScriptBehaviourUpdateOrder).Assembly.GetType(
-                    "Unity.Entities.ScriptBehaviourUpdateOrder+DummyDelegateWrapper");
-            var systemField = wrapperType.GetField("m_System", BindingFlags.NonPublic | BindingFlags.Instance);
-
-            for (var i = 0; i < playerLoop.subSystemList.Length; ++i)
-            {
-                ref var playerLoopSubSystem = ref playerLoop.subSystemList[i];
-                playerLoopSubSystem.subSystemList = playerLoopSubSystem.subSystemList.Where(s =>
-                {
-                    if (s.updateDelegate != null && s.updateDelegate.Target.GetType() == wrapperType)
-                    {
-                        var targetSystem = systemField.GetValue(s.updateDelegate.Target) as ComponentSystemBase;
-                        return targetSystem.World != world;
-                    }
-
-                    return true;
-                }).ToArray();
-            }
-
-            // Update PlayerLoop
-            ScriptBehaviourUpdateOrder.SetPlayerLoop(playerLoop);
-        }
     }
 }

workers/unity/Packages/io.improbable.gdk.core/Worker/WorkerConnector.cs

@@ -2,16 +2,12 @@
 using System.Collections;
 using System.Collections.Generic;
 using System.IO;
-using System.Linq;
 using System.Text.RegularExpressions;
 using System.Threading;
 using System.Threading.Tasks;
 using Improbable.Worker.CInterop;
-using Improbable.Worker.CInterop.Alpha;
 using Unity.Entities;
-using UnityEditor;
 using UnityEngine;
-using UnityEngine.LowLevel;
 
 namespace Improbable.Gdk.Core
 {
@@ -105,7 +101,7 @@ protected async Task Connect(IConnectionHandlerBuilder builder, ILogDispatcher l
 
                 // Update PlayerLoop
                 PlayerLoopUtils.ResolveSystemGroups(Worker.World);
-                ScriptBehaviourUpdateOrder.UpdatePlayerLoop(Worker.World, PlayerLoop.GetCurrentPlayerLoop());
+                ScriptBehaviourUpdateOrder.AddWorldToCurrentPlayerLoop(Worker.World);
             }
             catch (Exception)
             {
@@ -251,7 +247,7 @@ private void RemoveFromPlayerLoop()
             {
                 // Remove root systems from the disposing world from the PlayerLoop
                 // This only affects the loop next frame
-                PlayerLoopUtils.RemoveFromPlayerLoop(Worker.World);
+                ScriptBehaviourUpdateOrder.RemoveWorldFromCurrentPlayerLoop(Worker.World);
             }
         }
 

workers/unity/Packages/io.improbable.gdk.core/package.json

@@ -9,6 +9,6 @@
     "io.improbable.worker.sdk": "0.3.10",
     "io.improbable.gdk.tools": "0.3.10",
     "io.improbable.gdk.testutils": "0.3.10",
-    "com.unity.entities": "0.9.1-preview.15"
+    "com.unity.entities": "0.14.0-preview.18"
   }
-}
+}

workers/unity/Packages/io.improbable.gdk.debug/WorkerInspector/EntityListData.cs

@@ -41,12 +41,16 @@ public void SetNewWorld(World newWorld)
         {
             fullData.Clear();
             FilteredData.Clear();
-            query?.Dispose();
+
+            if (query != default)
+            {
+                query.Dispose();
+            }
 
             if (newWorld == null)
             {
                 world = null;
-                query = null;
+                query = default;
                 return;
             }
 
@@ -65,10 +69,10 @@ public void RefreshData()
             using (refreshDataMarker.Auto())
             {
                 fullData.Clear();
-                var spatialOSComponentType = world.EntityManager.GetArchetypeChunkComponentType<SpatialEntityId>(true);
+                var spatialOSComponentType = world.EntityManager.GetComponentTypeHandle<SpatialEntityId>(true);
                 var metadataComponentType =
-                    world.EntityManager.GetArchetypeChunkComponentType<Metadata.Component>(true);
-                var ecsEntityType = world.EntityManager.GetArchetypeChunkEntityType();
+                    world.EntityManager.GetComponentTypeHandle<Metadata.Component>(true);
+                var ecsEntityType = world.EntityManager.GetEntityTypeHandle();
 
                 using (var chunks = query.CreateArchetypeChunkArray(Allocator.TempJob))
                 {

workers/unity/Packages/io.improbable.gdk.playerlifecycle/Systems/PlayerHeartbeat/HandlePlayerHeartbeatResponseSystem.cs

@@ -44,7 +44,7 @@ protected override void OnUpdate()
             {
                 ref readonly var response = ref responses[i];
                 var spatialId = response.EntityId;
-                if (!workerSystem.TryGetEntity(spatialId, out var entity) || !heartbeats.Exists(entity))
+                if (!workerSystem.TryGetEntity(spatialId, out var entity) || !heartbeats.HasComponent(entity))
                 {
                     continue;
                 }

workers/unity/Packages/io.improbable.gdk.transformsynchronization/MonoBehaviours/TransformSynchronization.cs

@@ -79,7 +79,7 @@ private void OnDisable()
             StopAllCoroutines();
             RemoveStrategies();
 
-            entityManager = null;
+            entityManager = default;
             workerType = null;
             initialised = false;
         }