phreddrickk
Member
Code:
> js toJson({ x: null })
Unexpected error, debug log printed.
> js toJson({ x: undefined })
Unexpected error, debug log printed.
Bug toJson fails for nullish values
- Thread starter phreddrickk
- Start date
IMO, debug logs should not be generated by a script unless there really is a bug in Mafia.
That said, my inclination would be to make any function call from JS to ASH that passes an unrepresentable value to throw a JS exception, rather than passing null to some ASH function (whether user-defined or built-in) that probably can't handle it.
That said, my inclination would be to make any function call from JS to ASH that passes an unrepresentable value to throw a JS exception, rather than passing null to some ASH function (whether user-defined or built-in) that probably can't handle it.
I forgot that I had a branch that tinkered with this, but I've now created https://github.com/kolmafia/kolmafia/pull/2381.