Clone the Keen Map if no timestamp. Fix #14.

keenlabs / KeenClient-Java

Official Java client for the Keen IO API. Build analytics features directly into your Java apps.

https://keen.io/docs

MIT License

74 stars 43 forks source link

Clone the Keen Map if no timestamp. Fix #14. #22

Closed smurthas closed 9 years ago

smurthas commented 10 years ago

This is a simple fix for #14. If a timestamp isn't present on the keenProperties Map, then it creates a copy of the existing map and sets the timestamp on the copy only, leaving the original untouched.

Geeber commented 10 years ago

LGTM, aside from a couple purely cosmetic thoughts.

smurthas commented 9 years ago

Ok, I made that update and also expanded the test a bit to more thoroughly cover the expectations of that function.