DaIsimsiz / planetDefs-Builder

A desktop app for Windows to help users create galaxies for the Minecraft mod "Advanced Rocketry!"
GNU General Public License v3.0
0 stars 0 forks source link

Moons/sub-planets break the app [BUG] #2

Closed PatchedInfinities closed 5 months ago

PatchedInfinities commented 5 months ago

Describe the bug Upon making a planetDefs.xml without the help of the editor and adding a moon, trying to check the moon's settings will cause a "System.ArgumentOutOfRangeException" error.

To Reproduce Steps to reproduce the behavior:

  1. Create a planetDefs.xml file with a system containing a moon, or use the default.
  2. Import the file to the editor/builder
  3. Open the planet's properties, that should contain a moon.
  4. System.ArgumentOutOfRangeException

Expected behavior Treating sub-planets/moons as either a planet or a moon.

Screenshots image

Additional context None.

DaIsimsiz commented 5 months ago

Can you send me the planetDefs.xml you used for this?

DaIsimsiz commented 5 months ago

It took some effort, but I've found and fixed the code! The root of the problem was some typos, of all things... Thank you so much for your bug report!!