banana_split 0.0.1 → 0.0.2

data/lib/banana_split/controller.rb

@@ -1,21 +1,25 @@
 module BananaSplit
   module Controller
-    def self.included(base)
-      base.class_eval do
-        helper_method :current_user_id
-      end
+    extend ActiveSupport::Concern
+
+    included do
+      helper_method :session_id
     end
-    
-    def ab_goal(test_name, value = 1)
-      test_run = AbTestRun.last :conditions => { :user_id => current_user_id }
-      if test_run && test_run.value == 0
-        test_run.update_attribute(:value, value)
+
+    module InstanceMethods
+
+      def ab_goal(test_name, value = 1)
+        test = AbTest.find_by_name(test_name)
+        test_run = AbTestRun.find_by_session_id_and_ab_test_id(session_id, test.id)
+        if test_run && test_run.value == 0
+          test_run.update_attribute(:value, value)
+        end
+      end
+
+      def session_id
+        request.session_options[:id]
       end
     end
-    
-    def current_user_id
-      current_user.id
-    end
-    
+
   end
 end

data/lib/banana_split/version.rb

@@ -2,7 +2,7 @@ module BananaSplit
   module VERSION
     MAJOR = 0
     MINOR = 0
-    TINY  = 1
+    TINY  = 2
 
     STRING = [MAJOR, MINOR, TINY].join('.')
   end

data/lib/banana_split/view_helpers.rb

@@ -1,20 +1,26 @@
 module BananaSplit
   module ViewHelpers
-    
+
     def ab_test(test_name, designs)
       test = AbTest.find_or_create_by_name(:name => test_name.to_s)
-      
-      design_name = designs.keys.sample
-      design_render_options = designs[design_name]
-      design = AbTestDesign.find_or_create_by_ab_test_id_and_name :ab_test_id => test.id, :name => design_name.to_s
-      
-      test_run = test.runs.create({
-        :ab_test_design_id => design.id, 
-        :user_id => current_user_id
-      })
-      
+
+      test_run = AbTestRun.find_by_session_id_and_ab_test_id(session_id, test.id)
+
+      design = if test_run
+        test_run.ab_test_design
+      else
+        design_name = designs.keys.sample
+        AbTestDesign.find_or_create_by_ab_test_id_and_name(:ab_test_id => test.id, :name => design_name)
+      end
+
+      design_render_options = designs[design.name.to_sym]
+
+      test.runs.create({
+        :ab_test_design_id => design.id,
+        :session_id => session_id
+      }) unless test_run
+
       design_render_options ? render(design_render_options) : nil
     end
-    
   end
 end

data/lib/generators/templates/migrations/migration.rb

@@ -10,7 +10,7 @@ class CreateBananaSplitTables < ActiveRecord::Migration
     end
     
     create_table(:ab_test_runs) do |t|
-      t.references :user
+      t.string :session_id
       t.references :ab_test
       t.references :ab_test_design
       t.integer :value, :null => false, :default => 0

data/lib/generators/templates/models/ab_test_run.rb

@@ -1,3 +1,4 @@
 class AbTestRun < ActiveRecord::Base
   belongs_to :ab_test
+  belongs_to :ab_test_design
 end

metadata

@@ -1,12 +1,8 @@
 --- !ruby/object:Gem::Specification 
 name: banana_split
 version: !ruby/object:Gem::Version 
-  prerelease: false
-  segments: 
-  - 0
-  - 0
-  - 1
-  version: 0.0.1
+  prerelease: 
+  version: 0.0.2
 platform: ruby
 authors: 
 - Dennis Plougman Buus
@@ -15,10 +11,20 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-06-28 00:00:00 +02:00
+date: 2011-02-25 00:00:00 +01:00
 default_executable: 
-dependencies: []
-
+dependencies: 
+- !ruby/object:Gem::Dependency 
+  name: rails
+  prerelease: false
+  requirement: &id001 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ~>
+      - !ruby/object:Gem::Version 
+        version: "3.0"
+  type: :runtime
+  version_requirements: *id001
 description: Description not written yet.
 email: rasmusrnielsen@gmail.com
 executables: []
@@ -53,21 +59,17 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      segments: 
-      - 0
       version: "0"
 required_rubygems_version: !ruby/object:Gem::Requirement 
   none: false
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      segments: 
-      - 0
       version: "0"
 requirements: []
 
 rubyforge_project: 
-rubygems_version: 1.3.7
+rubygems_version: 1.5.0
 signing_key: 
 specification_version: 3
 summary: AB testing for Rails