removed editor code + unnecessary netcodehooks

Author: fernando-cortez

Assets/Prefabs/Game/NetworkObjectSpawner.prefab

@@ -10,7 +10,6 @@ GameObject:
   m_Component:
   - component: {fileID: 5747206859721612921}
   - component: {fileID: -5484724568297991178}
-  - component: {fileID: 2681238421323495432}
   - component: {fileID: -6110993549598497440}
   m_Layer: 0
   m_Name: NetworkObjectSpawner
@@ -50,18 +49,6 @@ MonoBehaviour:
   AlwaysReplicateAsRoot: 0
   DontDestroyWithOwner: 0
   AutoObjectParentSync: 1
---- !u!114 &2681238421323495432
-MonoBehaviour:
-  m_ObjectHideFlags: 0
-  m_CorrespondingSourceObject: {fileID: 0}
-  m_PrefabInstance: {fileID: 0}
-  m_PrefabAsset: {fileID: 0}
-  m_GameObject: {fileID: 721719496969292127}
-  m_Enabled: 1
-  m_EditorHideFlags: 0
-  m_Script: {fileID: 11500000, guid: 6aedfcf74b3f4f248897af16490caa9d, type: 3}
-  m_Name: 
-  m_EditorClassIdentifier: 
 --- !u!114 &-6110993549598497440
 MonoBehaviour:
   m_ObjectHideFlags: 0
@@ -74,6 +61,4 @@ MonoBehaviour:
   m_Script: {fileID: 11500000, guid: 40102bedb931e0b4ab8087df94ede8b5, type: 3}
   m_Name: 
   m_EditorClassIdentifier: 
-  m_NetcodeHooks: {fileID: 2681238421323495432}
-  m_SpawnObjectDataPrefab: {fileID: -8472638279017472308, guid: 95f831895b23a9e429f2b9fe75f9747d, type: 3}
   m_SpawnObjectData: []

Packages/com.unity.multiplayer.samples.coop/Utilities/Net/NetworkObjectSpawner.cs

@@ -1,10 +1,7 @@
 using System;
 using System.Collections;
 using System.Collections.Generic;
-using System.Linq;
 using Unity.Netcode;
-using UnityEditor;
-using UnityEditor.SceneManagement;
 using UnityEngine;
 
 namespace Unity.Multiplayer.Samples.Utilities
@@ -14,36 +11,14 @@ namespace Unity.Multiplayer.Samples.Utilities
     /// marked by a special tag, collects their Transform data, destroys their prefab instance, and performs the dynamic
     /// spawning of said objects during Netcode for GameObject's (Netcode) OnNetworkSpawn() callback.
     /// </summary>
-    [RequireComponent(typeof(NetcodeHooks))]
-    public class NetworkObjectSpawner : MonoBehaviour
+    public class NetworkObjectSpawner : NetworkBehaviour
     {
-        [SerializeField]
-        NetcodeHooks m_NetcodeHooks;
-
-        [SerializeField]
-        SpawnObjectData m_SpawnObjectDataPrefab;
-
         [SerializeField]
         List<SpawnObjectData> m_SpawnObjectData;
 
-        const string k_NetworkObjectSpawnerCollectableTag = "NetworkObjectSpawnerCollectable";
-
-        void Awake()
-        {
-            m_NetcodeHooks.OnNetworkSpawnHook += OnNetworkSpawn;
-        }
-
-        void OnDestroy()
-        {
-            if (m_NetcodeHooks)
-            {
-                m_NetcodeHooks.OnNetworkSpawnHook -= OnNetworkSpawn;
-            }
-        }
-
-        void OnNetworkSpawn()
+        public override void OnNetworkSpawn()
         {
-            if (!NetworkManager.Singleton.IsServer)
+            if (!IsServer)
             {
                 enabled = false;
                 return;
@@ -75,63 +50,5 @@ void SpawnNetworkObjects()
                 Destroy(m_SpawnObjectData[i].gameObject);
             }
         }
-
-#if UNITY_EDITOR
-        public void CollectTaggedPrefabInstances()
-        {
-            var prefabStage = PrefabStageUtility.GetPrefabStage(gameObject);
-
-            var root = prefabStage.prefabContentsRoot;
-
-            var networkObjects = root.GetComponentsInChildren<NetworkObject>();
-            var taggedNetworkObjects = networkObjects.Where(obj => obj.CompareTag(k_NetworkObjectSpawnerCollectableTag));
-
-            foreach (var editorOnlyObject in taggedNetworkObjects)
-            {
-                var pathToPrefab = PrefabUtility.GetPrefabAssetPathOfNearestInstanceRoot(editorOnlyObject);
-                var original =
-                    PrefabUtility.GetCorrespondingObjectFromSourceAtPath(editorOnlyObject, pathToPrefab);
-
-                var instantiated = PrefabUtility.InstantiatePrefab(m_SpawnObjectDataPrefab.gameObject);
-                var instantiatedPrefab = instantiated as GameObject;
-
-                if (instantiatedPrefab)
-                {
-                    instantiatedPrefab.transform.SetPositionAndRotation(editorOnlyObject.transform.position,
-                        editorOnlyObject.transform.rotation);
-
-                    instantiatedPrefab.transform.localScale = editorOnlyObject.transform.lossyScale;
-                    instantiatedPrefab.transform.SetParent(root.gameObject.transform);
-
-                    var spawnedObjectData = instantiatedPrefab.GetComponent<SpawnObjectData>();
-                    spawnedObjectData.prefabReference = original.gameObject;
-                    instantiatedPrefab.name += $"({original.name})";
-
-                    m_SpawnObjectData.Add(spawnedObjectData);
-
-                    // destroy scene prefab instance
-                    DestroyImmediate(editorOnlyObject.gameObject, true);
-
-                    PrefabUtility.SaveAsPrefabAsset(root, prefabStage.assetPath, out var success);
-                }
-            }
-        }
-    }
-
-    [CustomEditor(typeof(NetworkObjectSpawner))]
-    public class NetworkObjectSpawnerEditor : Editor
-    {
-        public override void OnInspectorGUI()
-        {
-            DrawDefaultInspector();
-
-            var networkObjectSpawner = (NetworkObjectSpawner)target;
-            if (PrefabStageUtility.GetCurrentPrefabStage() &&
-                GUILayout.Button("Collect tagged prefab instances"))
-            {
-                networkObjectSpawner.CollectTaggedPrefabInstances();
-            }
-        }
     }
-#endif
 }