[Bug?] sys.cull returns string sometimes
dd8kpx
dtr
{200118.0044}
sys.cull can return the following things:
a) if you specify one or more unloaded items but do not include confirm:
Confirm destruction of `0ez_21` with `Nconfirm`:`Vtrue`
b) if you specify a loaded upgrade
{`Nok`:`Vfalse`,`Nmsg`:`V"You can't destroy a loaded upgrade. Unload it then try again."`}
c) if you are not initialized
{`Nok`:`Vfalse`,`Nmsg`:`V"Your system is not yet initialized. Please use ``Fsys``V.``Linit``V to initialize your system and access sys features"`}
d) if you specify an unloaded item but include confirm
{`Nok`:`Vtrue`}
case a seems like it should also be an object for consistency with the others, and to make parsing of output easier (i.e. always check the msg property, rather than having to typecheck first), as in
{`Nok`:`Vfalse`,`Nmsg`:`V"Confirm destruction of ``0ez_21``V with ``Nconfirm`:`Vtrue"`}
- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
<user deleted>
{200202.1454}
I'm not sure if ok should be false in this case, since it's neither a failure nor a success, just a request for confirmation. I think it should have no ok property, just msg
- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -