Panicking ByName/IndexOf/IndexByName, separate Try- methods for null/-1

[BioTomateDE]

The UndertaleData methods and extensions ByName(), IndexOf and IndexOfByName (now called IndexByName) now throw an exception when the element could not be found. The old behavior of returning null/-1 is still available via TryByName, TryIndexOf and TryIndexByName.

Caveats

This is a breaking change.

Notes

i fixed all the csx scripts (hopefully)

[github-actions]

Download the artifacts for this pull request here:

GUI:

• GUI-windows-latest-isBundled-false-isSingleFile-true
• GUI-windows-latest-isBundled-false-isSingleFile-false

CLI:

• CLI-windows-latest-isBundled-false
• CLI-ubuntu-latest-isBundled-false
• CLI-macOS-latest-isBundled-false

[colinator27]

Thinking about wrapping up the design for this, and honestly I'm super hesitant about breaking so many non-builtin scripts that rely on the existing methods...

[BioTomateDE]

@colinator27 i understand. i still think this would be useful to have in utml in some way tho. maybe the old methods keep their names and the new failable ones could be named Force- or something. or just keep the existing methods and make a bool panic = false argument. what do YOU TWO think?

[colinator27]

@colinator27 i understand. i still think this would be useful to have in utml in some way tho. maybe the old methods keep their names and the new failable ones could be named Force- or something. or just keep the existing methods and make a bool panic = false argument. what do YOU TWO think?

Yeah, that does seem reasonable. I think we would want to make new methods, but I feel like "force" isn't quite the right word... maybe something more like "require"? Not sure...