fix: Get material from Unity Object

Author: jcs090218

Assets/JCSUnity/Scripts/Effects/Tweener/JCS_ShaderTweener.cs

@@ -53,7 +53,7 @@ private void SetValue(float val)
         {
             foreach (string prop in mShaderProps)
             {
-                this.mRenderer.material.SetFloat(prop, val);
+                this.LocalMaterial.SetFloat(prop, val);
             }
         }
 
@@ -64,7 +64,7 @@ private float GetValue()
         {
             foreach (string prop in mShaderProps)
             {
-                return this.mRenderer.material.GetFloat(prop);
+                return this.LocalMaterial.GetFloat(prop);
             }
 
             return 0.0f;

Assets/JCSUnity/Scripts/JCS_UnityObject.cs

@@ -29,8 +29,6 @@ public class JCS_UnityObject : MonoBehaviour
     {
         /* Variables */
 
-        private Material mMaterial = null;
-
         [Separator("Initialize Variables (JCS_Unityobject)")]
 
         [Tooltip("Type of the Unity game object.")]
@@ -69,7 +67,6 @@ public void SetObjectType(JCS_UnityObjectType ob)
             UpdateUnityData();
         }
         public JCS_UnityObjectType GetObjectType() { return this.mObjectType; }
-        public Material GetMaterial() { return this.mMaterial; }
         public Image GetImage() { return this.mImage; }
         public Renderer GetRenderer() { return this.mRenderer; }
         public SpriteRenderer GetSpriteRenderer() { return this.mSpriteRenderer; }
@@ -131,36 +128,31 @@ public virtual void UpdateMaterial(bool force = false)
             {
                 case JCS_UnityObjectType.GAME_OBJECT:
                     {
-                        mMaterial = Instantiate(mRenderer.material);
-                        mRenderer.material = mMaterial;
+                        mRenderer.material = Instantiate(mRenderer.material);
                     }
                     break;
                 case JCS_UnityObjectType.UI:
                     {
                         if (mImage != null)
                         {
-                            mMaterial = Instantiate(mImage.material);
-                            mImage.material = mMaterial;
+                            mImage.material = Instantiate(mImage.material);
                         }
                     }
                     break;
                 case JCS_UnityObjectType.SPRITE:
                     {
-                        mMaterial = Instantiate(mSpriteRenderer.material);
-                        mSpriteRenderer.material = mMaterial;
+                        mSpriteRenderer.material = Instantiate(mSpriteRenderer.material);
                     }
                     break;
                 case JCS_UnityObjectType.TEXT:
                     {
-                        mMaterial = Instantiate(mText.material);
-                        mText.material = mMaterial;
+                        mText.material = Instantiate(mText.material);
                     }
                     break;
 #if TMP_PRO
                 case JCS_UnityObjectType.TMP:
                     {
-                        mMaterial = Instantiate(mTextMesh.material);
-                        mTextMesh.material = mMaterial;
+                        mTextMesh.material = Instantiate(mTextMesh.material);
                     }
                     break;
 #endif
@@ -237,7 +229,7 @@ public Component LocalType
         }
 
         /// <summary>
-        /// Get Current type's transform
+        /// Get Current type's transform.
         /// </summary>
         public Transform LocalTransform
         {
@@ -263,6 +255,57 @@ public Transform LocalTransform
             }
         }
 
+        /// <summary>
+        /// Return the current type's material.
+        /// </summary>
+        public Material LocalMaterial
+        {
+            get
+            {
+                switch (GetObjectType())
+                {
+                    case JCS_UnityObjectType.GAME_OBJECT:
+                        return this.mRenderer.material;
+                    case JCS_UnityObjectType.SPRITE:
+                        return this.mSpriteRenderer.material;
+                    case JCS_UnityObjectType.UI:
+                        return this.mImage.material;
+                    case JCS_UnityObjectType.TEXT:
+                        return this.mText.material;
+#if TMP_PRO
+                    case JCS_UnityObjectType.TMP:
+                        return this.mTextMesh.material;
+#endif
+                }
+
+                return null;
+            }
+
+            set
+            {
+                switch (GetObjectType())
+                {
+                    case JCS_UnityObjectType.GAME_OBJECT:
+                        this.mRenderer.material = value;
+                        break;
+                    case JCS_UnityObjectType.SPRITE:
+                        this.mSpriteRenderer.material = value;
+                        break;
+                    case JCS_UnityObjectType.UI:
+                        this.mImage.material = value;
+                        break;
+                    case JCS_UnityObjectType.TEXT:
+                        this.mText.material = value;
+                        break;
+#if TMP_PRO
+                    case JCS_UnityObjectType.TMP:
+                        this.mTextMesh.material = value;
+                        break;
+#endif
+                }
+            }
+        }
+
         /// <summary>
         /// Get Current type's position
         /// </summary>
@@ -566,21 +609,6 @@ public bool LocalEnabled
             }
         }
 
-        /// <summary>
-        /// Return the possible material.
-        /// </summary>
-        public Material LocalMaterial
-        {
-            get 
-            {
-                if (mMaterial == null)
-                    UpdateMaterial();
-
-                return this.mMaterial; 
-            }
-            set { this.mMaterial = value; }
-        }
-
         /// <summary>
         /// Return the possible material color.
         /// </summary>

docs/ScriptReference/JCS_UnityObject.md

@@ -17,13 +17,13 @@ from its component.
 | mColorProps      | Shader's color properties.                                  |
 | LocalType        | Component of this current type.                             |
 | LocalTransform   | Either regular transform or recttransform base on the type. |
+| LocalMaterial    | Local material of this gameobject.                          |
 | Position         | Position of this gameobject.                                |
 | LocalPosition    | Local position of this gameobject.                          |
 | EulerAngles      | Euler angles of this gameobject.                            |
 | LocalEulerAngles | Local euler angles of this gameobject.                      |
 | LocalScale       | Local scale of this gameobject.                             |
 | LocalEnabled     | Enabled/disable this gameobject.                            |
-| LocalMaterial    | Local material of this gameobject.                          |
 | LocalColor       | Color of this gameobject.                                   |
 | LocalAlpha       | Alpha channel of this gameobject.                           |
 | LocalRed         | Red channel of this gameobject.                             |