add cloning to property/options hash

[segmentio]

add cloning to property/options hash

[hschoenburg]

Hey, just a bit more detail about this issue.

Keen.io likes nested event properties like so

   "title": "Music is Peace T-shirt size L",
    "updated_at": "2013-10-01T18:15:37.758Z",
    "user": {
        "acted_auctions_count": null,
        "auction_count": 63,
        "bid_count": 32,
        "created_at": "2013-08-14T18:28:29.054Z",

But Mixpanel cannot handle nested properties and simply drops them.

SO

To handle this, you end up having to call Analytics.track twice, with different options hashes. In order for this to work you need to DEEP COPY the options hash. Cloning does not cover nested attributes, so it won't help here.

If Segment could help facilitate discussion among analytics providers regarding issues like this, that would be great. Right now, if you are using Keen and Mixpanel, this issue render segment fairly unnecessary, as you end up with two calls.

thanks for the help.

[ivolo]

@hschoenburg thanks for the feedback, we should be helping with this case! I think this manifests into two actions:

1. fix cloning in Ruby
2. flatten Mixpanel nested attributes https://github.com/segmentio/integrations/issues/8

[ivolo]

another case of cloning causing customer issues.

def test
  email = "[email protected]"
  user_id = "123213213"

  context = {
                :providers => {:All => false, :Salesforce => true},
                :salesforce => {
                  :object => "Lead",
                  :lookup => {:email => email}
                }
            }
  AnalyticsRuby.identify(:user_id => user_id, :traits => {:firstName => "ALead", :email => email}, :context => context)

  context = {
                :providers => {:Salesforce => true},
                :salesforce => {
                  :object => "Contact",
                  :operation => "update",
                  :lookup => {:email => email}
                }
            }
  AnalyticsRuby.identify(:user_id => user_id, :traits => {:firstName => "AContact", :email => email}, :context => context)
end