Merge remote-tracking branch 'origin/develop' into develop

Assets/Scripts/Actions/ActionManager.cs

@@ -23,12 +23,19 @@ namespace Actions {
 
         private List<ClickActionHandler> _clickActionHandlers;
         private List<NextDayActionHandler> _nextDayActionHandlers;
+        private bool _enabled;
+
+        public bool Enabled {
+            get => _enabled;
+            set => _enabled = value;
+        }
         
         private ActionManager() {
             _clickActionHandlers = new List<ClickActionHandler>();
             _nextDayActionHandlers = new List<NextDayActionHandler>();
             instantiateClickActionHandlers();
             instantiateNextDayActionHandlers();
+            Enabled = true;
         }
 
         /// <summary>
@@ -65,26 +72,28 @@ namespace Actions {
         /// <param name="gameObject">The affected gameObject</param>
         /// <param name="usableItem">the current tool</param>
         public void ClickAction(GameObject gameObject, UsableItem usableItem) {
-            foreach (ClickActionHandler actionHandler in _clickActionHandlers) {
+            if(Enabled) {
-                if(actionHandler.Matches(gameObject, usableItem)) {
+                foreach(ClickActionHandler actionHandler in _clickActionHandlers) {
-                    actionHandler.InvokeAction(gameObject);
+                    if(actionHandler.Matches(gameObject, usableItem)) {
-                    break; // Ja Herr Professor, Sie sehen richtig. Voller Stolz verwende ich ein break.
+                        actionHandler.InvokeAction(gameObject);
+                        break; // Ja Herr Professor, Sie sehen richtig. Voller Stolz verwende ich ein break.
+                    }
                 }
             }
         }
         
         /// <summary>
-        /// Used to Invoke ClickActions, all ClickActionHandlers in ClickActionHandlers list are iterated through,
+        /// Used to Invoke NextDay, all NextDayActionHandlers in NextDayActionHandlers list are iterated through,
         /// only one will be invoked per method call 
         /// </summary>
         /// <param name="gameObject">The affected gameObject</param>
-        /// <param name="usableItem">the current tool</param>
         public void NextDayAction(GameObject gameObject) {
-            Debug.Log("nextday action");
+            if(Enabled) {
-            foreach (NextDayActionHandler actionHandler in _nextDayActionHandlers) {
+                foreach(NextDayActionHandler actionHandler in _nextDayActionHandlers) {
-                if(actionHandler.Matches(gameObject)) {
+                    if(actionHandler.Matches(gameObject)) {
-                    actionHandler.InvokeAction(gameObject);
+                        actionHandler.InvokeAction(gameObject);
-                    break; // Gleich noch einmal. Und ich kann nachts immer noch zufrieden schlafen.
+                        break; // Gleich noch einmal. Und ich kann nachts immer noch zufrieden schlafen.
+                    }
                 }
             }
         }

Assets/Scripts/Actions/ClickActionHandlers.cs

@@ -100,7 +100,7 @@ namespace Assets.Scripts.Actions {
             bool rv = false;
             try {
                 _animal = gameObject.GetComponent<Animal>();
-                rv = true;
+                rv = _animal != null;
             }
             catch { }
             return rv;
@@ -277,7 +277,8 @@ namespace Assets.Scripts.Actions {
         public override bool Matches(GameObject gameObject, UsableItem usableItem) {
             bool rv = base.Matches(gameObject, usableItem);
             if(rv) {
-                rv = _animal.displayName.Equals("Cow") && usableItem.ID == ItemContainer.Instance.GetItemIdByName("Bucket");
+                    rv = _animal.displayName.Equals("Cow") &&
+                         usableItem.ID == ItemContainer.Instance.GetItemIdByName("Bucket");
             }
             return rv;
         }

Assets/Scripts/InventoryUI.cs

@@ -1,9 +1,10 @@
 using System;
 using System.Linq;
+using Actions;
 using UnityEngine;
 using UnityEngine.EventSystems;
 
-public class InventoryUI : MonoBehaviour {
+public class InventoryUI : MonoBehaviour, IPointerEnterHandler, IPointerExitHandler {
     public Transform itemsParent;
     public GameObject inventoryUI;
     private Inventory _inventory;
@@ -59,4 +60,18 @@ public class InventoryUI : MonoBehaviour {
             }
         }
     }
+
+    /**
+    * Turn off ActionManager when Pointer over UI
+    */
+    public void OnPointerEnter(PointerEventData eventData) {
+        ActionManager.Instance.Enabled = false;
+    }
+
+    /**
+    * Turn on ActionManager when Pointer over UI
+    */
+    public void OnPointerExit(PointerEventData eventData) {
+        ActionManager.Instance.Enabled = true;
+    }
 }

Assets/Scripts/Shop/ShopUI.cs

@@ -1,8 +1,12 @@
+using System.Collections.Generic;
 using System.Linq;
+using Actions;
+using Tiles;
 using UnityEngine;
+using UnityEngine.EventSystems;
 
 namespace Shop {
-    public class ShopUI : MonoBehaviour {
+    public class ShopUI : MonoBehaviour , IPointerEnterHandler, IPointerExitHandler{
         public Transform itemsParent;
         public Transform animalsParent;
         public GameObject shopUI;
@@ -81,5 +85,19 @@ namespace Shop {
                 }
             }
         }
+
+        /**
+         * Turn off ActionManager when Pointer over UI
+         */
+        public void OnPointerEnter(PointerEventData eventData) {
+            ActionManager.Instance.Enabled = false;
+        }
+
+        /**
+         * Turn on ActionManager when Pointer over UI
+         */
+        public void OnPointerExit(PointerEventData eventData) {
+            ActionManager.Instance.Enabled = true;
+        }
     }
 }

Assets/Scripts/TileController.cs

@@ -7,6 +7,20 @@ using UnityEngine.Tilemaps;
 /// Instatiates Tiles at beginning of the game
 /// </summary>
 public class TileController : MonoBehaviour {
+    #region Singleton
+
+    public static TileController instance;
+
+    private void Awake() {
+        if (instance != null) {
+            Debug.LogWarning("More than one instance of TileController found");
+        }
+
+        instance = this;
+    }
+
+    #endregion
+    
     public GameObject tile;
 
     public GameObject cameraGameObject;