Skip to content

Commit 22b1c37

Browse files
committed
Use ab_user instead of session
1 parent 3d99582 commit 22b1c37

2 files changed

Lines changed: 2 additions & 4 deletions

File tree

lib/split/analytics.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,9 +31,9 @@ def tracking_code(options={})
3131
end
3232

3333
def custom_variables
34-
return nil if session[:split].nil?
34+
return nil if ab_user[:split].nil?
3535
arr = []
36-
session[:split].each_with_index do |h,i|
36+
ab_user[:split].each_with_index do |h,i|
3737
arr << "_gaq.push(['_setCustomVar', #{i+1}, '#{h[0]}', '#{h[1]}', 1]);"
3838
end
3939
arr.reverse[0..4].reverse.join("\n")

spec/analytics_spec.rb

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,6 @@
2121
first_alt = ab_test('link_colour', 'red', 'blue')
2222
second_alt = ab_test('link_text', 'Join', 'Signup')
2323

24-
session[:split].should eql({'link_colour' => first_alt,'link_text' => second_alt})
25-
2624
tracking_code = tracking_code(:account => 'UA-12345-6')
2725
tracking_code.should eql(" <script type=\"text/javascript\">\n var _gaq = _gaq || [];\n _gaq.push(['_setAccount', 'UA-12345-6']);\n _gaq.push(['_setCustomVar', 1, 'link_colour', '#{first_alt}', 1]);\n_gaq.push(['_setCustomVar', 2, 'link_text', '#{second_alt}', 1]);\n\n (function() {\n var ga = document.createElement('script'); ga.type = 'text/javascript'; ga.async = true;\n ga.src = ('https:' == document.location.protocol ? 'https://ssl.google-analytics.com/ga.js' : 'http://www.google-analytics.com/ga.js');\n var s = document.getElementsByTagName('script')[0]; s.parentNode.insertBefore(ga, s);\n })();\n </script>\n")
2826
end

0 commit comments

Comments
 (0)