search

linuxgurugamer / ScienceAlert

When is is it time for science?  
GNU General Public License v3.0
11 stars 11 forks source link

ArgumentNullException #24

Closed kfsone closed 3 years ago

kfsone commented 3 years ago

On exiting/entering a vessel, it's throwing an ANE. Probably because you explicitly try to add null keys here:

https://github.com/linuxgurugamer/ScienceAlert/blob/27a1b71f8541de7dcd8a7c9f899dbcdc84a20d8d/Source/ScienceAlert.ProfileData/ScienceAlertProfileManager.cs#L335-L336

linuxgurugamer commented 3 years ago

I would assume it needs to have a "!" in front Thank you

kfsone commented 3 years ago

Rusty on C#, and I don't have all the dependencies set up to test the change, or I'd have made a PR :)

linuxgurugamer commented 3 years ago

Not a problem, thank you for pointing it out