From 7d4936b577ee522f285df2ac85497bab63babfcb Mon Sep 17 00:00:00 2001
From: palomahnhp <hernandezp@madrid.es>
Date: Sun, 13 Mar 2016 18:29:55 +0100
Subject: [PATCH] track user verification and add statistics of verified users
 who did not vote proposals

---
 app/assets/javascripts/application.js         |   2 +
 app/assets/javascripts/tracks.js.coffee       |  28 ++++
 app/controllers/admin/stats_controller.rb     |   4 +
 app/helpers/tracks_helper.rb                  |  12 ++
 app/models/concerns/verification.rb           |  13 +-
 app/views/admin/stats/show.html.erb           |   7 +
 app/views/layouts/_tracking_data.html.erb     |   1 +
 app/views/layouts/application.html.erb        |   1 +
 app/views/verification/letter/edit.html.erb   |   1 +
 app/views/verification/letter/new.html.erb    |   1 +
 app/views/verification/letter/show.html.erb   |   1 +
 app/views/verification/residence/new.html.erb |   1 +
 app/views/verification/sms/edit.html.erb      |   1 +
 app/views/verification/sms/new.html.erb       |   1 +
 config/locales/admin.en.yml                   |   1 +
 config/locales/admin.es.yml                   |   1 +
 db/dev_seeds.rb                               |   2 +-
 spec/features/tracks_spec.rb                  | 134 ++++++++++++++++++
 18 files changed, 210 insertions(+), 2 deletions(-)
 create mode 100644 app/assets/javascripts/tracks.js.coffee
 create mode 100644 app/helpers/tracks_helper.rb
 create mode 100644 app/views/layouts/_tracking_data.html.erb
 create mode 100644 spec/features/tracks_spec.rb

diff --git a/app/assets/javascripts/application.js b/app/assets/javascripts/application.js
index aa27109d9..7bc4b5274 100644
--- a/app/assets/javascripts/application.js
+++ b/app/assets/javascripts/application.js
@@ -41,6 +41,7 @@
 //= require registration_form
 //= require suggest
 //= require forms
+//= require tracks
 
 var initialize_modules = function() {
   App.Comments.initialize();
@@ -57,6 +58,7 @@ var initialize_modules = function() {
   App.RegistrationForm.initialize();
   App.Suggest.initialize();
   App.Forms.initialize();
+  App.Tracks.initialize();  
 };
 
 $(function(){
diff --git a/app/assets/javascripts/tracks.js.coffee b/app/assets/javascripts/tracks.js.coffee
new file mode 100644
index 000000000..1b6cfe82e
--- /dev/null
+++ b/app/assets/javascripts/tracks.js.coffee
@@ -0,0 +1,28 @@
+App.Tracks =
+  
+  tracking_enabled: -> 
+    _paq?
+
+  set_custom_var: (id, name, value, scope) -> 
+    _paq.push(['setCustomVariable', id, name, value, scope])
+    _paq.push(['trackPageView'])
+
+  track_event: ($this) -> 
+    category = $this.data('track-event-category')
+    action   = $this.data('track-event-action')
+    _paq.push(['trackEvent', category, action])
+
+  initialize: ->
+    if App.Tracks.tracking_enabled()
+      $('[data-track-usertype]').each ->
+        $this = $(this)
+        usertype = $this.data('track-usertype')
+        App.Tracks.set_custom_var(1, "usertype", usertype, "visit")      
+
+      $('[data-track-event-category]').each ->  
+        $this = $(this)
+        App.Tracks.track_event($this)
+
+      $('[data-track-click]').on 'click', ->
+        $this = $(this)
+        App.Tracks.track_event($this)
diff --git a/app/controllers/admin/stats_controller.rb b/app/controllers/admin/stats_controller.rb
index ec5c2d19d..f724a054c 100644
--- a/app/controllers/admin/stats_controller.rb
+++ b/app/controllers/admin/stats_controller.rb
@@ -18,6 +18,10 @@ class Admin::StatsController < Admin::BaseController
     @verified_users = User.with_hidden.level_two_or_three_verified.count
     @unverified_users = User.with_hidden.unverified.count
     @users = User.with_hidden.count
+
+    @user_ids_who_voted_proposals = 
+     ActsAsVotable::Vote.where(votable_type: 'Proposal').pluck(:voter_id).uniq.count 
+    @user_ids_who_didnt_vote_proposals = @verified_users - @user_ids_who_voted_proposals
   end
 
 end
diff --git a/app/helpers/tracks_helper.rb b/app/helpers/tracks_helper.rb
new file mode 100644
index 000000000..53c110d49
--- /dev/null
+++ b/app/helpers/tracks_helper.rb
@@ -0,0 +1,12 @@
+module TracksHelper
+  def track_event(data={})
+    track_data = ""
+    prefix = " data-track-event-"
+    data.each do |key, value|
+      track_data = track_data + prefix + key.to_s + '=' + value + " "
+    end
+    content_for :track_event do
+      track_data
+    end
+  end
+end
\ No newline at end of file
diff --git a/app/models/concerns/verification.rb b/app/models/concerns/verification.rb
index 3520e5b31..c876ef5ed 100644
--- a/app/models/concerns/verification.rb
+++ b/app/models/concerns/verification.rb
@@ -53,7 +53,18 @@ module Verification
   def no_phone_available?
     !verification_sms_sent?
   end
-
+  
+  def user_type
+    case
+    when level_three_verified?
+      :level_3_user
+    when level_two_verified?
+      :level_2_user
+    else
+      :level_1_user
+    end  
+  end
+  
   def sms_code_not_confirmed?
     !sms_verified?
   end
diff --git a/app/views/admin/stats/show.html.erb b/app/views/admin/stats/show.html.erb
index 84abd7be9..93b6be821 100644
--- a/app/views/admin/stats/show.html.erb
+++ b/app/views/admin/stats/show.html.erb
@@ -76,6 +76,13 @@
             <%= t "admin.stats.show.summary.user_level_three" %><br>
             <span class="number"><%= number_with_delimiter(@user_level_three) %></span>
           </p>
+
+          <p>
+            <%= t "admin.stats.show.summary.verified_users_who_didnt_vote_proposals" %><br>
+            <span class="number">
+              <%=number_with_delimiter(@user_ids_who_didnt_vote_proposals)%>
+            </span>
+          </p>
         </div>
       </div>
 
diff --git a/app/views/layouts/_tracking_data.html.erb b/app/views/layouts/_tracking_data.html.erb
new file mode 100644
index 000000000..a0be1bafa
--- /dev/null
+++ b/app/views/layouts/_tracking_data.html.erb
@@ -0,0 +1 @@
+<meta name="tracking_data" data-track-usertype="<%=current_user ? current_user.user_type : :anonymous%>" <%= yield(:track_event) %>/>
diff --git a/app/views/layouts/application.html.erb b/app/views/layouts/application.html.erb
index ff41ea1ea..e7c5fdf56 100644
--- a/app/views/layouts/application.html.erb
+++ b/app/views/layouts/application.html.erb
@@ -4,6 +4,7 @@
     <meta charset="utf-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
     <meta http-equiv="X-UA-Compatible" content="IE=edge" />
+    <%=render "layouts/tracking_data"%>
     <title><%= content_for?(:title) ? yield(:title) : setting['org_name'] %></title>
     <%= stylesheet_link_tag    "application" %>
     <%= javascript_include_tag "application", 'data-turbolinks-track' => true %>
diff --git a/app/views/verification/letter/edit.html.erb b/app/views/verification/letter/edit.html.erb
index c792e219d..5ca194d33 100644
--- a/app/views/verification/letter/edit.html.erb
+++ b/app/views/verification/letter/edit.html.erb
@@ -1,4 +1,5 @@
 <div class="row verify">
+  <%track_event(category: "verification", action: "edit_letter" )%>
   <div class="small-12 medium-9 large-6 small-centered column">
     <div class="text-center">
       <h1>
diff --git a/app/views/verification/letter/new.html.erb b/app/views/verification/letter/new.html.erb
index 130fb106c..9bd6a2ddd 100644
--- a/app/views/verification/letter/new.html.erb
+++ b/app/views/verification/letter/new.html.erb
@@ -1,4 +1,5 @@
 <div class="verification account row">
+  <%track_event(category: "verification", action: "success_sms" )%>
   <div class="small-12 column">
 
     <div class="text-center">
diff --git a/app/views/verification/letter/show.html.erb b/app/views/verification/letter/show.html.erb
index a21cfb4ed..3d1e81d4d 100644
--- a/app/views/verification/letter/show.html.erb
+++ b/app/views/verification/letter/show.html.erb
@@ -1,4 +1,5 @@
 <div class="row">
+  <%track_event(category: "verification", action: "start_letter" )%>
   <div class="small-12 column">
 
     <%= link_to account_path, class: "back" do %>
diff --git a/app/views/verification/residence/new.html.erb b/app/views/verification/residence/new.html.erb
index 3ad074277..382709676 100644
--- a/app/views/verification/residence/new.html.erb
+++ b/app/views/verification/residence/new.html.erb
@@ -1,4 +1,5 @@
 <div class="verification account row">
+  <%track_event(category: "verification", action: "start_census" )%>
   <div class="small-12 column">
 
     <div class="text-center">
diff --git a/app/views/verification/sms/edit.html.erb b/app/views/verification/sms/edit.html.erb
index 94fed7fd4..dd154c695 100644
--- a/app/views/verification/sms/edit.html.erb
+++ b/app/views/verification/sms/edit.html.erb
@@ -1,4 +1,5 @@
 <div class="verification account row">
+  <%track_event(category: "verification", action: "start_sms" )%>
   <div class="small-12 column">
     <div class="text-center">
 
diff --git a/app/views/verification/sms/new.html.erb b/app/views/verification/sms/new.html.erb
index 2e304ddda..e3f9fa891 100644
--- a/app/views/verification/sms/new.html.erb
+++ b/app/views/verification/sms/new.html.erb
@@ -1,4 +1,5 @@
 <div class="verification account row">
+  <%track_event(category: "verification", action: "success_census" )%>
   <div class="small-12 column">
     <div class="text-center">
 
diff --git a/config/locales/admin.en.yml b/config/locales/admin.en.yml
index 4ecf9cbda..1d028df46 100755
--- a/config/locales/admin.en.yml
+++ b/config/locales/admin.en.yml
@@ -198,6 +198,7 @@ en:
           user_level_two: Level two users
           users: Total users
           verified_users: Verified users
+          verified_users_who_didnt_vote_proposals: Verified users who didn't votes proposals
           visits: Visits
           votes: Total votes
         visits_title: Visits
diff --git a/config/locales/admin.es.yml b/config/locales/admin.es.yml
index bc882fab2..766cb5b51 100644
--- a/config/locales/admin.es.yml
+++ b/config/locales/admin.es.yml
@@ -198,6 +198,7 @@ es:
           user_level_two: Usuarios de nivel dos
           users: Usuarios
           verified_users: Usuarios verificados
+          verified_users_who_didnt_vote_proposals: Usuarios verificados que no han votado propuestas
           visits: Visitas
           votes: Votos
         visits_title: Visitas
diff --git a/db/dev_seeds.rb b/db/dev_seeds.rb
index 14af135da..74d4a4381 100644
--- a/db/dev_seeds.rb
+++ b/db/dev_seeds.rb
@@ -23,7 +23,7 @@ Setting.create(key: 'feature.spending_proposals', value: "true")
 Setting.create(key: 'feature.twitter_login', value: "true")
 Setting.create(key: 'feature.facebook_login', value: "true")
 Setting.create(key: 'feature.google_login', value: "true")
-
+Setting.create(key: 'per_page_code', value: "")
 Setting.create(key: 'comments_body_max_length', value: '1000')
 
 puts "Creating Geozones"
diff --git a/spec/features/tracks_spec.rb b/spec/features/tracks_spec.rb
new file mode 100644
index 000000000..059d2f679
--- /dev/null
+++ b/spec/features/tracks_spec.rb
@@ -0,0 +1,134 @@
+require 'rails_helper'
+
+feature 'Tracking' do
+
+  context 'Custom variable' do
+    
+     scenario 'Usertype anonymous' do
+      visit proposals_path
+      
+      expect(page.html).to include "anonymous"    
+    end
+
+    scenario 'Usertype level_1_user' do  
+      create(:geozone)
+      user = create(:user)
+      login_as(user)
+
+      visit proposals_path
+      
+      expect(page.html).to include "level_1_user"
+    end
+
+    scenario 'Usertype level_2_user' do  
+      create(:geozone)
+      user = create(:user)
+      login_as(user)
+
+      visit account_path
+      click_link 'Verify my account'
+
+      verify_residence
+
+      fill_in 'sms_phone', with: "611111111"
+      click_button 'Send'
+
+      user = user.reload
+      fill_in 'sms_confirmation_code', with: user.sms_confirmation_code
+      click_button 'Send'
+      
+      expect(page.html).to include "level_2_user"
+    end 
+  end
+
+  context 'Tracking events' do 
+    scenario 'Verification: start census' do
+      user = create(:user)
+      login_as(user)
+
+      visit account_path
+      click_link 'Verify my account'
+
+      expect(page.html).to include "data-track-event-category=verification"
+      expect(page.html).to include "data-track-event-action=start_census"
+    end
+
+    scenario 'Verification: success census' do
+      create(:geozone)
+      user = create(:user)
+      login_as(user)
+
+      visit account_path
+      click_link 'Verify my account'
+
+      verify_residence
+
+      fill_in 'sms_phone', with: "611111111"
+      click_button 'Send'
+
+      expect(page.html).to include "data-track-event-category=verification"
+      expect(page.html).to include "data-track-event-action=start_sms"   
+    end
+
+    scenario 'Verification: start sms' do
+      create(:geozone)
+      user = create(:user)
+      login_as(user)
+
+      visit account_path
+      click_link 'Verify my account'
+
+      verify_residence
+
+      fill_in 'sms_phone', with: "611111111"
+      click_button 'Send' 
+
+      expect(page.html).to include "data-track-event-category=verification"
+      expect(page.html).to include "data-track-event-action=start_sms"   
+    end
+  
+    scenario 'Verification: success sms' do
+      create(:geozone)
+      user = create(:user)
+      login_as(user)
+
+      visit account_path
+      click_link 'Verify my account'
+
+      verify_residence 
+
+      fill_in 'sms_phone', with: "611111111"
+      click_button 'Send'
+
+      user = user.reload
+      fill_in 'sms_confirmation_code', with: user.sms_confirmation_code
+      click_button 'Send'
+
+      expect(page.html).to include "data-track-event-category=verification"
+      expect(page.html).to include "data-track-event-action=success_sms"
+    end
+
+    scenario 'Verification: letter' do
+      create(:geozone)
+      user = create(:user)
+      login_as(user)
+
+      visit account_path
+      click_link 'Verify my account'
+
+      verify_residence
+
+      fill_in 'sms_phone', with: "611111111"
+      click_button 'Send'
+
+      user = user.reload
+      fill_in 'sms_confirmation_code', with: user.sms_confirmation_code
+      click_button 'Send'
+
+      click_link "Send me a letter with the code"
+
+      expect(page.html).to include "data-track-event-category=verification"
+      expect(page.html).to include "data-track-event-action=start_letter"    
+    end
+  end
+end