minor changes and readme update

Author: GieziJo

Editor/SOVariant.cs

@@ -216,7 +216,6 @@ private Tuple<string, T, List<string>, List<string>> ExtractData(string data)
                     case (1):
                         Debug.Log($"UserData not overwritten.");
                         _SOVariantProperlyLoaded = false;
-                        Selection.activeObject = null;
                         return null;
                     case (2):
                         return ExtractData(ReadUpdatedMetaFile(data));

Editor/SOVariantAttributeProcessor.cs

@@ -58,8 +58,11 @@ public override void ProcessMemberProperties(List<InspectorPropertyInfo> propert
                 {
                     Selection.selectionChanged -= OnSelectionChanged;
                     _selectionChangedFlag = false;
-
                     AssemblyReloadEvents.beforeAssemblyReload -= OnBeforeAssemblyReloads;
+                    
+                    Selection.activeObject = null;
+                    
+                    return;
                 }
 
                 BoxGroupAttribute bxa = new BoxGroupAttribute("Scriptable Object Variant", true, false, 2);

Editor/UpgradeSOVariantHelper.cs

@@ -22,6 +22,8 @@ public static void UpgradeSOVariantUserData()
 
                 var extractedData = ExtractData(importer.userData);
                 var _parent = extractedData.Item1;
+                if(_parent == null)
+                    continue;
                 var _overridden = extractedData.Item3 ?? new List<string>();
                 var _children = extractedData.Item4 ?? new List<string>();
 

README.md

@@ -131,13 +131,4 @@ if (AssetDatabase.GetAssetPath(parent) == AssetDatabase.GetAssetPath(target))
  ```
 It would be alot better to directly filter the possible candidates when selecting in the object, but adding the `AssetSelector` attribute with a filter, or building a custom `ValueDropdown` both did not work, not sure why.
 
-### [Data serialization (parent and overriden fields)](https://github.com/GieziJo/ScriptableObjectVariant/issues/4)
-As mentioned above, the serialized data is kept in `userData`, but is set with `_import.userData = *mySerializedDataString*`. This would override any other data that would come to this field from other scripts, might be an issue.
-
-### [Saving Data](https://github.com/GieziJo/ScriptableObjectVariant/issues/5)
-Saving data to the `.meta` file occurs when the asset is deselected (`Selection.selectionChanged += OnSelectionChanged;`). It would be better to tie this to the serialization and deserialization of the data itself, but unity does not seem to expose the process as a delegate (not sure?), so I haven't found a way to tap into this routine.
-At least checking when the editor recompiles should be possible.
-
-If the asset is not deselected in the editor before the editor reloads, the override changes are not saved.
-
 </details>