Two bugs found in Unity Rider Plugin

Hi. 

I've recently found two bugs when using Rider and Unity I've figured I'd let you know.

The first one is connected to UnityEngine.Assert() method. When this method throws errors in Unity from a prefab asset the default behavior (when Rider plugin is not enabled) is that double-clicking this error message in Console will open the Prefab Editor. This doesn't happen when the Rider plugin is enabled which makes it hard to locate the assets that are throwing the assertion error.

---

The second one is connected to using

PlayerSettings.SetScriptingDefineSymbolsForGroup() 

If you change the ScriptingDefineSymbols from a script they will not refresh in Rider, and RIder will be showing the #if macros as disabled when in fact they are not. If you edit the defined symbols from the default Project Settings window they are refreshed correctly.

For this bug, I've found a workaround. After changing the define symbols via the method above you also need to invoke this code

var riderScriptEditor = new SerializedObject(RiderScriptEditorData.instance);

riderScriptEditor.Update();
riderScriptEditor.FindProperty("HasChanges").boolValue = true;
riderScriptEditor.ApplyModifiedProperties();

CodeEditor.CurrentEditor.SyncAll();

But this code relies on modifying the HasChanges property that's set as internal so I don't think this is the desired behavior.