Merge pull request #335 from Unity-Technologies/Uni-39667-ValidationContextMenus

Uni-39667-ValidationContextMenus-Added validation methods for contextual menus

Author: MarcShakour

Assets/FbxExporters/Editor/FbxExporter.cs

@@ -2973,6 +2973,24 @@ static void OnClipContextClick(MenuCommand command)
                 }
             }
 
+            /// <summary>
+            /// Validate the menu item defined by the function OnClipContextClick.
+            /// </summary>
+            [MenuItem(TimelineClipMenuItemName, true, 31)]
+            static bool ValidateOnClipContextClick()
+            {
+                Object[] selectedObjects = Selection.objects;
+
+                foreach (Object editorClipSelected in selectedObjects)
+                {
+                    if (editorClipSelected.GetType().Name.Contains("EditorClip"))
+                    {         
+                        return true;
+                    }
+                }
+                return false;
+            }
+
             public static bool ExportSingleEditorClip(Object editorClipSelected)
             {
                 if (editorClipSelected.GetType().Name.Contains("EditorClip"))
@@ -3052,6 +3070,31 @@ public static void OnPlayableDirectorGameObjectContextClick(MenuCommand command)
                 }
             }
 
+            /// <summary>
+            /// Validate the menu item defined by the function OnPlayableDirectorGameObjectContextClick.
+            /// </summary>
+            [MenuItem(ClipMenuItemName, true, 31)]
+            public static bool ValidateClipContextClick()
+            {
+                Object[] selection = Selection.objects;
+
+                if (selection == null || selection.Length == 0)
+                {
+                    return false;
+                }
+
+                foreach (Object obj in selection)
+                {
+                    GameObject gameObj = obj as GameObject;
+                    if (gameObj != null && gameObj.GetComponent<PlayableDirector>() != null)
+                    {
+                        return true;
+                    }
+                }
+
+                return false;
+            }
+
             public static void ExportAllTimelineClips(GameObject objectWithPlayableDirector, string folderPath)
             {
                 PlayableDirector pd = objectWithPlayableDirector.GetComponent<PlayableDirector>();
@@ -3078,31 +3121,6 @@ public static void ExportAllTimelineClips(GameObject objectWithPlayableDirector,
                     }
                 }
             }
-            
-            /// <summary>
-            /// Validate the menu item defined by the function above.
-            /// </summary>
-            [MenuItem(ClipMenuItemName, true, 31)]
-            public static bool ValidateClipContextClick()
-            {
-                Object[] selection = Selection.objects;
-
-                if (selection == null || selection.Length == 0)
-                {
-                    return false;
-                }
-
-                foreach (Object obj in selection)
-                {
-                    GameObject gameObj = obj as GameObject;
-                    if (gameObj !=null && gameObj.GetComponent<PlayableDirector>() != null)
-                    {
-                        return true;
-                    }
-                }
-
-                return false;
-            }
 
             /// <summary>
             /// Add a menu item "Export Model..." to a GameObject's context menu.
@@ -3119,7 +3137,7 @@ static void OnContextItem (MenuCommand command)
             }
 
             /// <summary>
-            /// Validate the menu item defined by the function above.
+            /// Validate the menu item defined by the function OnContextItem.
             /// </summary>
             [MenuItem (MenuItemName, true, 30)]
             public static bool OnValidateMenuItem ()
@@ -3142,7 +3160,7 @@ static void ModelOnlyOnContextItem (MenuCommand command)
             }
 
             /// <summary>
-            /// Validate the menu item defined by the function above.
+            /// Validate the menu item defined by the function ModelOnlyOnContextItem.
             /// </summary>
             [MenuItem (ModelOnlyMenuItemName, true, 30)]
             public static bool ModelOnlyOnValidateMenuItem ()
@@ -3172,7 +3190,23 @@ static void OnAnimOnlyContextItem (MenuCommand command)
             [MenuItem (AnimOnlyMenuItemName, true, 30)]
             public static bool OnValidateAnimOnlyMenuItem ()
             {
-                return true;
+                Object[] selection = Selection.objects;
+
+                if (selection == null || selection.Length == 0)
+                {
+                    return false;
+                }
+
+                foreach (Object obj in selection)
+                {
+                    GameObject gameObj = obj as GameObject;
+                    if (gameObj != null && (gameObj.GetComponent<Animation>() != null || gameObj.GetComponent<Animator>() != null))
+                    {
+                        return true;
+                    }
+                }
+
+                return false;
             }
 
             public static void DisplayNoSelectionDialog()