add 'name includes .' ember_assert to get/set

[grosser]

Ive been bitten by this so many times, that I really would like to be notified if I get or set something that has a '.', its such a subtle error to do and would make it so much easier to spot.

get('record.foo') -> Boom

Other solution:
always getPath when doing get + introduce getSimple for the edge-cases where you might not want it

[lukemelia]

I like the idea. This would be a definite win for a debug build. Would need to be benchmarked to see how it impacted performance for the normal ember_assert dev build.

[wagenet]

@grosser The reason why we don't do this is the same reason why we have get and getPath separate. get is a very hot code path and we don't want to do anything to slow it down, even checking for the presence of a period. If we used ember_assert it would only be in development mode, but even then, I think the slow down would likely be noticeable enough that we wouldn't even want it there.

[wagenet]

We've finally benchmarked this and discovered that the difference isn't all that significant. It seems likely that we'll merge getPath into get.