Fixes #915 (booleanContains incorrect when checking 2 polygons)

[stebogit]

@rowanwins take a look at this

[stebogit]

😳 it fails!
I thought for..of was OK to replace the classic for loop; I guess it's not @DenisCarriere... 😞

Locally worked fine.

[rowanwins]

Hey @stebogit

I'm happy to take this one over, I started taking a look at this last night and found a few other things which need fixing in the booleanContains. I'll post an issue with a bit of a discussion :)

[stebogit]

👍 I worked on it last night as well, just wanted to share what I produced 😄
Feel free to discard this if you want.

[DenisCarriere]

@stebogit 👍 Nice stuff, I've been away at the cottage with horrible to no WiFi.

Yep for..of loops are not valid in ES5 (bummer 😢), maybe it might be worth converting the tests back to ES5+ (NodeJS 4 compatible) that way we can catch those types of errors quicker (just a thought).

[stebogit]

at the cottage with horrible to no WiFi.

Wouldn't be a genuine relaxing cottage with fancy hi-speed WiFi 😄

[DenisCarriere]

Wouldn't be a genuine relaxing cottage with fancy hi-speed WiFi

Waterfront properties tend to have poor Internet 😄 (the nicer the view, the slower the connection)

[rowanwins]

Closed in favour of #923