Yet another bug on windows. Added a unit test.

It was another issue of wrong slashes.

Author: Benoit Hudson

Assets/FbxExporters/Editor/ConvertToModel.cs

@@ -130,7 +130,7 @@ public static GameObject[] CreateInstantiatedModelPrefab (GameObject [] unityGam
                     // Replace w Model asset. LoadMainAssetAtPath wants a path
                     // relative to the project, not relative to the assets
                     // folder.
-                    Object unityMainAsset = AssetDatabase.LoadMainAssetAtPath(Path.Combine("Assets", relativePath));
+                    Object unityMainAsset = AssetDatabase.LoadMainAssetAtPath("Assets/" + relativePath);
 
                     if (unityMainAsset != null) {
                         Object unityObj = PrefabUtility.InstantiatePrefab (unityMainAsset);

Assets/FbxExporters/Editor/FbxExportSettings.cs

@@ -61,9 +61,7 @@ public override void OnInspectorGUI() {
                 // Unless the user canceled, make sure they chose something in the Assets folder.
                 if (!string.IsNullOrEmpty (fullPath)) {
                     var relativePath = ExportSettings.ConvertToAssetRelativePath(fullPath);
-                    if (string.IsNullOrEmpty(relativePath)
-                            || relativePath == ".."
-                            || relativePath.StartsWith(".." + Path.DirectorySeparatorChar)) {
+                    if (string.IsNullOrEmpty(relativePath)) {
                         Debug.LogWarning ("Please select a location in the Assets folder");
                     } else {
                         ExportSettings.SetRelativeSavePath(relativePath);
@@ -153,10 +151,21 @@ public static void SetRelativeSavePath(string newPath) {
         /// get from GetRelativeSavePath.
         ///
         /// This uses '/' as the path separator.
+        ///
+        /// If 'requireSubdirectory' is the default on, return empty-string if the full
+        /// path is not in a subdirectory of assets.
         /// </summary>
-        public static string ConvertToAssetRelativePath(string fullPathInAssets)
+        public static string ConvertToAssetRelativePath(string fullPathInAssets, bool requireSubdirectory = true)
         {
-            return GetRelativePath(Application.dataPath, fullPathInAssets);
+            var relativePath = GetRelativePath(Application.dataPath, fullPathInAssets);
+            if (requireSubdirectory && relativePath.StartsWith("..")) {
+                if (relativePath.Length == 2 || relativePath[2] == '/') {
+                    // The relative path has us pop out to another directory,
+                    // so return an empty string as requested.
+                    return "";
+                }
+            }
+            return relativePath;
         }
 
         /// <summary>

Assets/FbxExporters/Editor/UnitTests/FbxExportSettingsTest.cs

@@ -153,6 +153,13 @@ public void TestGetRelativePath()
             to = Path.Combine(Application.dataPath, "foo");
             relative = ExportSettings.ConvertToAssetRelativePath(to);
             Assert.AreEqual("foo", relative);
+
+            relative = ExportSettings.ConvertToAssetRelativePath("/path/to/somewhere/else");
+            Assert.AreEqual("", relative);
+
+            relative = ExportSettings.ConvertToAssetRelativePath("/path/to/somewhere/else", requireSubdirectory: false);
+            Assert.IsTrue(relative.StartsWith("../"));
+            Assert.IsFalse(relative.Contains("\\"));
         }
 
         [Test]