Merge pull request #268 from jaglitegrann/bug/check-islistening-before-object-spawn

fix(spawning): Add check for NetworkingManager.IsListening before spa…

Author: TwoTenPvP

MLAPI/Core/NetworkedObject.cs

@@ -387,6 +387,11 @@ private void OnDestroy()
         /// <param name="destroyWithScene">Should the object be destroyd when the scene is changed</param>
         public void Spawn(Stream spawnPayload = null, bool destroyWithScene = false)
         {
+            if (!NetworkingManager.Singleton.IsListening)
+            {
+                throw new NotListeningException("NetworkingManager isn't listening, start a server, client or host before spawning objects.");
+            }
+
             if (spawnPayload != null)
                 spawnPayload.Position = 0;
 

MLAPI/Exceptions/NotListeningException.cs

@@ -0,0 +1,37 @@
+using System;
+
+namespace MLAPI.Exceptions
+{
+    /// <summary>
+    /// Exception thrown when the operation require NetworkingManager to be listening.
+    /// </summary>
+    public class NotListeningException : Exception
+    {
+        /// <summary>
+        /// Constructs a NotListeningException
+        /// </summary>
+        public NotListeningException()
+        {
+            
+        }
+
+        /// <summary>
+        /// Constructs a NotListeningException with a message
+        /// </summary>
+        /// <param name="message">The exception message</param>
+        public NotListeningException(string message) : base(message)
+        {
+            
+        }
+
+        /// <summary>
+        /// Constructs a NotListeningException with a message and a inner exception
+        /// </summary>
+        /// <param name="message">The exception message</param>
+        /// <param name="inner">The inner exception</param>
+        public NotListeningException(string message, Exception inner) : base(message, inner)
+        {
+            
+        }
+    }
+}