I'm personally rather with Tim on that. Let's not bring the joy of `[object Object]` to Squeak. :D I'm personally more a fan of EAFP than LBYL, but silent invalidation of data just makes debugging harder IMO. I even wonder whether `JsonObject` should reject any non-string keys during construction and manipulation (e.g., `add:`) in the first place. However, the latter idea might impact compatibility (but seriously, the name `JsonObject` conveys a pretty clear idea what types of keys might be allowed there.)
just like `Association>> jsonWriteOn:` does (there is an argument for removing this method though, because unlike all other methods it does not by itself produce valid JSON)
Oh yes indeed. I placed an #isThisEverCalled in this method in my image and if we do not identify any surprising usages, I support removing this method (or deprecating it at least).
squeak-dev@lists.squeakfoundation.org