Fix crash in SandboxMediator when currentSelectedTool is null #274

[dsaltares]

No description provided.

[azakhary]

Thanks a lot! I'll check this out first thing tomorrow morning before merging.

[azakhary]

I always wanted to put this null checks, but every time I was wandering, how can current tool be null in the first place? I don't see an obvious scenario for this to happen. But I guess that's another topic and null checks are necessary in any case.

Thanks a lot!

[dsaltares]

how can current tool be null in the first place?

I can see a few scenarios here:

• Trying to set a tool with an invalid name
• Any chance we could be passing null to the tool hotswap method?
• Calling toolHotswapBack twice in a row would definitely do it

[azakhary]

right. true.. I think twice in a row hotswap is the actual candidate, I'll look into it.