event_tracker 0.0.4 → 0.1.0

data/lib/event_tracker.rb

@@ -14,49 +14,62 @@ module EventTracker
   end
 
   module ActionControllerExtension
-    def append_event_tracking_tags
-      mixpanel_key = Rails.application.config.event_tracker.mixpanel_key
-      kissmetrics_key = Rails.application.config.event_tracker.kissmetrics_key
-      return unless mixpanel_key || kissmetrics_key
+    def mixpanel_tracker
+      @mixpanel_tracker ||= begin
+        mixpanel_key = Rails.application.config.event_tracker.mixpanel_key
+        EventTracker::Mixpanel.new(mixpanel_key) if mixpanel_key
+      end
+    end
 
-      body = response.body
-      insert_at = body.index('</head')
-      if insert_at
-        trackers = []
-        a = [ %q{<script type="text/javascript">} ]
+    def kissmetrics_tracker
+      @kissmetrics_tracker ||= begin
+        kissmetrics_key = Rails.application.config.event_tracker.kissmetrics_key
+        EventTracker::Kissmetrics.new(kissmetrics_key) if kissmetrics_key
+      end
+    end
 
-        if mixpanel_key
-          trackers << EventTracker::Mixpanel
-          distinct_id = respond_to?(:mixpanel_distinct_id) && mixpanel_distinct_id
-          name_tag = respond_to?(:mixpanel_name_tag) && mixpanel_name_tag
-          a << EventTracker::Mixpanel.init(mixpanel_key)
-          a << EventTracker::Mixpanel.identify(distinct_id) if distinct_id
-          a << EventTracker::Mixpanel.name_tag(name_tag) if name_tag
-        end
+    def event_trackers
+      @event_trackers ||= begin
+        trackers = []
+        trackers << mixpanel_tracker if mixpanel_tracker
+        trackers << kissmetrics_tracker if kissmetrics_tracker
+        trackers
+      end
+    end
 
-        if kissmetrics_key
-          trackers << EventTracker::Kissmetrics
-          identity = respond_to?(:kissmetrics_identity) && kissmetrics_identity
-          a << EventTracker::Kissmetrics.init(kissmetrics_key)
-          a << EventTracker::Kissmetrics.identify(identity) if identity
-        end
+    def append_event_tracking_tags
+      return if event_trackers.empty?
 
-        registered_properties = session.delete(:registered_properties)
-        event_tracker_queue = session.delete(:event_tracker_queue)
+      body = response.body
+      head_insert_at = body.index('</head')
+      return unless head_insert_at
+      body.insert head_insert_at, view_context.javascript_tag(event_trackers.map {|t| t.init }.join("\n"))
+      body_insert_at = body.index('</body')
+      return unless body_insert_at
+      a = []
+      if distinct_id = respond_to?(:mixpanel_distinct_id) && mixpanel_distinct_id
+        a << mixpanel_tracker.identify(distinct_id)
+      end
+      if name_tag = respond_to?(:mixpanel_name_tag) && mixpanel_name_tag
+        a << mixpanel_tracker.name_tag(name_tag)
+      end
+      if identity = respond_to?(:kissmetrics_identity) && kissmetrics_identity
+        a << kissmetrics_tracker.identify(identity)
+      end
+      registered_properties = session.delete(:registered_properties)
+      event_tracker_queue = session.delete(:event_tracker_queue)
 
-        trackers.each do |tracker| 
-          a << tracker.register(registered_properties) if registered_properties.present?
+      event_trackers.each do |tracker|
+        a << tracker.register(registered_properties) if registered_properties.present?
 
-          if event_tracker_queue.present?
-            event_tracker_queue.each do |event_name, properties| 
-              a << tracker.track(event_name, properties)
-            end
+        if event_tracker_queue.present?
+          event_tracker_queue.each do |event_name, properties|
+            a << tracker.track(event_name, properties)
           end
         end
-        a << %q{</script>}
-        body.insert insert_at, a.join("\n")
-        response.body = body
       end
+      body.insert body_insert_at, view_context.javascript_tag(a.join("\n"))
+      response.body = body
     end
 
   end

data/lib/event_tracker/kissmetrics.rb

@@ -1,8 +1,12 @@
-module EventTracker::Kissmetrics
-  def self.init(key)
+class EventTracker::Kissmetrics
+  def initialize(key)
+    @key = key
+  end
+
+  def init
     <<-EOD
       var _kmq = _kmq || [];
-      var _kmk = _kmk || '#{key}';
+      var _kmk = _kmk || '#{@key}';
       function _kms(u){
         setTimeout(function(){
           var d = document, f = d.getElementsByTagName('script')[0],
@@ -16,16 +20,16 @@ module EventTracker::Kissmetrics
     EOD
   end
 
-  def self.register(registered_properties)
+  def register(registered_properties)
     %Q{_kmq.push(['set', #{registered_properties.to_json}]);}
   end
 
-  def self.track(event_name, properties)
+  def track(event_name, properties)
     p = properties.empty? ? "" : ", #{properties.to_json}"
     %Q{_kmq.push(['record', '#{event_name}'#{p}]);}
   end
 
-  def self.identify(identity)
+  def identify(identity)
     %Q{_kmq.push(['identify', '#{identity}']);}
   end
 end

data/lib/event_tracker/mixpanel.rb

@@ -1,5 +1,9 @@
-module EventTracker::Mixpanel
-  def self.init(mixpanel_key)
+class EventTracker::Mixpanel
+  def initialize(key)
+    @key = key
+  end
+
+  def init
     s = <<-EOD
       (function(c,a){window.mixpanel=a;var b,d,h,e;b=c.createElement("script");
       b.type="text/javascript";b.async=!0;b.src=("https:"===c.location.protocol?"https:":"http:")+
@@ -11,24 +15,24 @@ module EventTracker::Mixpanel
       'track_forms','register','register_once','unregister','identify','name_tag',
       'set_config','people.identify','people.set','people.increment'];for(e=0;e<h.length;e++)d(g,h[e]);
       a._i.push([b,c,f])};a.__SV=1.1;})(document,window.mixpanel||[]);
-      mixpanel.init("#{mixpanel_key}");
+      mixpanel.init("#{@key}");
     EOD
   end
 
-  def self.register(registered_properties)
+  def register(registered_properties)
     %Q{mixpanel.register(#{registered_properties.to_json});}
   end
 
-  def self.track(event_name, properties)
+  def track(event_name, properties)
     p = properties.empty? ? "" : ", #{properties.to_json}"
     %Q{mixpanel.track("#{event_name}"#{p});}
   end
 
-  def self.name_tag(name_tag)
+  def name_tag(name_tag)
     %Q{mixpanel.name_tag("#{name_tag}");}
   end
 
-  def self.identify(distinct_id)
+  def identify(distinct_id)
     %Q{mixpanel.identify("#{distinct_id}");}
   end
 end

data/lib/event_tracker/version.rb

@@ -1,3 +1,3 @@
 module EventTracker
-  VERSION = "0.0.4"
+  VERSION = "0.1.0"
 end

data/spec/event_tracker_spec.rb

@@ -1,6 +1,7 @@
 require "spec_helper"
 
 shared_examples_for "init" do
+  subject { page.find("head script").native.content }
   it { should include('mixpanel.init("YOUR_TOKEN")') }
   it { should include(%q{var _kmk = _kmk || 'KISSMETRICS_KEY'}) }
 end
@@ -25,7 +26,7 @@ shared_examples_for "with event" do
 end
 
 feature 'basic integration' do
-  subject { page.find("script").native.content }
+  subject { page.find("body script").native.content }
 
   class BasicController < ApplicationController
     after_filter :append_event_tracking_tags

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: event_tracker
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.1.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-11-30 00:00:00.000000000 Z
+date: 2013-01-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -78,7 +78,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -308392474469118674
+      hash: 2100786682895310824
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -87,7 +87,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -308392474469118674
+      hash: 2100786682895310824
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.24