Yeah, sorry, maybe I wasn't clear in my previous comment.

What I wanted to say is that we should get rid of this check all together. If someone passes `None` for `data` argument, this means it's not using the method incorrectly and the method should throw.

Imo, a better and correct solution would be to fix methods which are calling `_to_node` and `_to_location` to never call this method with `None` as an argument.