@@ -1,17 +1,17 @@ module Component::Analytics class Block < ActiveRecord::Base set_table_name "analytics_blocks" - + belongs_to :component, :class_name => "Component::Analytics::Configuration" - + before_save :compile! - + include BreezeHq::Mixins::Options option :tracking_id - + def self.template; ""; end def self.secure_template; ""; end - + def regexp if self[:regexp].blank? nil @@ -19,12 +19,12 @@ module Component::Analytics @_regexp ||= Regexp.new(self.class.normalize_regexp(self[:regexp]), Regexp::IGNORECASE) end end - + def =~(path) enabled? && (regexp.blank? || (!!(path =~ regexp) ^ exclude?)) end alias_method :match, :=~ - + def apply(rendered_page, secure = false) if tracking_id.to_s.strip.blank? rendered_page @@ -32,28 +32,28 @@ module Component::Analytics rendered_page.sub('</body>') { (secure ? secure_code : code) + $& } end end - + def code compile! if self[:code].blank? super end - + def secure_code compile! if self[:secure_code].blank? super end - + protected def compile! self.code = self.class.compile(self.class.template, self) self.secure_code = self.class.compile(self.class.secure_template, self) end - + def self.compile(template_to_compile, object) # TODO: is there a built-in way to do this? template_to_compile.gsub(/\#\{([^\}]+)\}/) { object.send $1.to_sym } end - + def self.normalize_regexp(regexp) regexp.to_s.sub /\A(\^|\\A|\/)*/, '\A/' end app/models/component/analytics/block.rb @@ -1,11 +1,11 @@ module Component::Analytics class Configuration < Component::Base has_many :blocks, :class_name => "Component::Analytics::Block", :foreign_key => "component_id" - + def after_render(rendered_content, page, controller) apply(controller.request.request_uri, rendered_content, controller.request.ssl?) end - + def apply(path, rendered_page, secure = false) blocks.inject(rendered_page) do |memo, block| (block =~ path) ? block.apply(memo, secure) : memo app/models/component/analytics/configuration.rb @@ -7,14 +7,14 @@ module Component::Analytics document.write(unescape("%3Cscript src='" + gaJsHost + "google-analytics.com/ga.js' type='text/javascript'%3E%3C/script%3E")); </script> <script type="text/javascript"> - try{ + try{ var pageTracker = _gat._getTracker("#{tracking_id}"); pageTracker._trackPageview(); - } catch(err) {} + } catch(err) {} </script> CODE end - + def apply_with_uniqueness_requirement(rendered_page, secure = false) if rendered_page.include?("google-analytics.com/ga.js") rendered_page @@ -23,7 +23,7 @@ module Component::Analytics end end alias_method_chain :apply, :uniqueness_requirement - + def self.secure_template template end app/models/component/analytics/google.rb @@ -1,7 +1,7 @@ module Component::Analytics class Quantcast < Block option :tracking_id - + def self.template <<-'CODE' <!-- Start Quantcast tag --> @@ -33,7 +33,7 @@ module Component::Analytics <!-- End Quantcast tag --> CODE end - + def apply_with_uniqueness_requirement(rendered_page, secure = false) if rendered_page.include?("quantserve.com/quant.js") rendered_page app/models/component/analytics/quantcast.rb @@ -6,55 +6,55 @@ describe Component::Analytics do @analytics = @site.components[Component::Analytics] @analytics.save end - + describe "block" do before :each do @block = Component::Analytics::Block.new end - + it "should match all pages by default" do @block.should match("/foo/bar") end - + it "should match the root page by default" do @block.should match("/") end - + it "should not match if disabled" do @block.enabled = false @block.should_not match("/foo") end - + describe "with an inclusive regexp" do before :each do @block.regexp = "^/foo.*" end - + it "should match if the regexp matches" do @block.should match("/foo/bar") end - + it "should not match if the regexp doesn't match" do @block.should_not match("/boo") end end - + describe "with an exclusive regexp" do before :each do @block.regexp = "^/foo.*" @block.exclude = true end - + it "should not match if the regexp matches" do @block.should_not match("/foo/bar") end - + it "should match if the regexp doesn't match" do @block.should match("/boo") end end end - + describe "on a page" do before :each do @html = <<-HTML @@ -79,27 +79,27 @@ describe Component::Analytics do @code = @analytics.apply "/", @html @code.should match(/_getTracker\("UA-123456-x"\);/) end - + it "should not include the code if the tracking id is blank" do @google.update_attributes :tracking_id => "" @analytics.blocks(:reload) @code = @analytics.apply "/", @html @code.should_not match(/_getTracker/) end - + it "should not include the code if it is disabled" do @google.update_attributes :enabled => false @analytics.blocks(:reload) @code = @analytics.apply "/", @html @code.should_not match(/_getTracker/) end - + describe "included multiple times" do before :each do @google2 = Component::Analytics::Google.create :component => @analytics, :tracking_id => "UA-123456-x" @analytics.blocks(:reload).size.should == 2 end - + it "should only include the code once" do @code = @analytics.apply "/", @html @code.scan('_getTracker("UA-123456-x");').size.should == 1 @@ -117,38 +117,38 @@ describe Component::Analytics do @code = @analytics.apply "/", @html @code.should match(%r{http://pixel.quantserve.com/pixel/p-c3dkwYEDZXhiY.gif}) end - + it "should apply the secure code on a secure page" do @code = @analytics.apply "/", @html, true @code.should match(%r{//secure.quantserve.com/pixel/p-c3dkwYEDZXhiY.gif}) end - + describe "included multiple times" do before :each do @qc2 = Component::Analytics::Quantcast.create :component => @analytics, :tracking_id => "p-c3dkwYEDZXhiY" @analytics.blocks(:reload).size.should == 2 end - + it "should only include the code once" do @code = @analytics.apply "/", @html @code.scan('http://pixel.quantserve.com/pixel/p-c3dkwYEDZXhiY.gif').size.should == 1 end end end - + describe "with both Google and Quantcast analytics" do before :each do @google = Component::Analytics::Google.create :component => @analytics, :tracking_id => "UA-123456-x" @qc = Component::Analytics::Quantcast.create :component => @analytics, :tracking_id => "p-c3dkwYEDZXhiY" @analytics.blocks(:reload).size.should == 2 end - + it "should insert the correct code on a page" do @code = @analytics.apply "/", @html @code.should match(/_getTracker\("UA-123456-x"\);/) @code.should match(%r{http://pixel.quantserve.com/pixel/p-c3dkwYEDZXhiY.gif}) end - + it "should apply the secure code on a secure page" do @code = @analytics.apply "/", @html, true @code.should match(/_getTracker\("UA-123456-x"\);/) spec/models/component/analytics_spec.rb 24e836043f78b3f8b867f0ce22d979eda7fec522 Matt Powell fauxparse@gmail.com http://github.com/leftclick/bhq-analytics/commit/af0d86a55a4b9a3277949d5b22a21f82e6217929 af0d86a55a4b9a3277949d5b22a21f82e6217929 2009-11-05T16:09:06-08:00 2009-11-05T16:09:06-08:00 hardcore whitespace stripping f260887ff551901690714dd1649471fe5c968f67 Matt Powell fauxparse@gmail.com