ga_events 1.2.0 → 2.0.0

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA256:
+  metadata.gz: 8733925b29ae71da3da081d25679727857f92ee7bcb5390c72b97a87fc3c9b53
+  data.tar.gz: 67d86fd9ea6ced7564d851b40faa957d2138a92977a5d3197a2a6f7bdcdd7068
+SHA512:
+  metadata.gz: 0ffaa24b43857c82f369e98861b82ddd99dca42bc88b97148d4a055210e0ed76b2f82961932e83e03b76264e091a7e698577b07a69cb2efef6d40b92616965c0
+  data.tar.gz: 59e2857a160797e12bfec180e10aadae8f66633e303cb9c41e739a77669ee03f2b13c22faea9f10df263e1d43f2eef172f6c5d03cbca90b7a9944597c8390a77

data/.gitignore

@@ -2,3 +2,4 @@
 .bundle
 Gemfile.lock
 pkg
+vendor

data/.rubocop.yml

@@ -0,0 +1,37 @@
+AllCops:
+  TargetRubyVersion: 2.3
+  Exclude:
+    - 'vendor/bundle/**/*'
+  DisplayCopNames: true
+Style/AsciiComments:
+  Enabled: false
+Style/Documentation:
+  Enabled: false
+Metrics/MethodLength:
+  Enabled: false
+Style/MethodDefParentheses:
+  Enabled: false
+Layout/MultilineOperationIndentation:
+  EnforcedStyle: aligned
+Layout/MultilineMethodCallIndentation:
+  EnforcedStyle: indented
+Layout/MultilineMethodDefinitionBraceLayout:
+  Enabled: true
+Layout/MultilineHashBraceLayout:
+  Enabled: true
+Style/SignalException:
+  Enabled: false
+Metrics/ClassLength:
+  Enabled: false
+Lint/AssignmentInCondition:
+  Enabled: false
+Metrics/ParameterLists:
+  Enabled: false
+Style/MultilineBlockChain:
+  Enabled: false
+Style/CommentAnnotation:
+  Enabled: false
+Style/GuardClause:
+  Enabled: false
+Metrics/BlockLength:
+  Enabled: false

data/CHANGELOG.md

@@ -15,3 +15,36 @@ The format suggested at http://keepachangelog.com/ is used.
 
 ### Fixed
 - Error returns white page
+
+## 1.2.0 - 2014-12-18
+
+### Added
+- A Test Adapter that pushes events to a `window.events` array.
+
+## 1.3.0 - 2017-04-27
+
+### Added
+- Support for turbolinks.
+
+## 1.4.0 - 2018-03-09
+
+### Fixed
+- Fixed turbolinks events. To prevent older events the be processed again after page
+  changes orchestrated by turbolinks, events are no longer processed via the
+  injected DOM node. (Because the node would be cached and the event already
+  processed.) Instead the the header-strategy is being used.
+
+## 1.4.1 - 2018-04-03
+
+### Fixed
+- Ensure category, action and label do not include any umlauts or ß chars
+- Fixed an error where events are remaining after a redirect
+
+## 2.0.0 - 2018-04-03
+
+### Removed
+- Support for rubies < 2.3
+- Support for rails < 4.2
+
+### Added
+- Compatibility for Rails 5

data/Gemfile

@@ -1,8 +1,11 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 # Specify your gem's dependencies in ga_events.gemspec
 gemspec
 
+gem 'pry'
 gem 'rake'
 gem 'rspec', '~> 3.1.0'
-gem 'pry'
+gem 'rubocop', '~> 0.79.0', require: false

data/README.md

@@ -13,8 +13,8 @@ pushes it to Google Analytics via ga.js or Google Tag Manager.
 
 ## Dependencies
 
-* Ruby >= 1.9
-* Rails 3.1 onwards
+* Ruby >= 2.3
+* Rails 4.2 onwards
 * jQuery
 
 ## Installation

data/Rakefile

@@ -1,7 +1,5 @@
-#!/usr/bin/env rake
-require 'bundler/gem_tasks'
-require 'rspec/core/rake_task'
+# frozen_string_literal: true
 
-RSpec::Core::RakeTask.new('spec')
+require 'bundler/gem_tasks'
 
 task default: :spec

data/app/assets/javascripts/ga_events.js.coffee

@@ -11,12 +11,16 @@ class GaEvents.Event
   @html_key: "ga-events"
   klass: @
 
-  # Decompose a dom-string (ruby side) into an event object.
+  # Decompose a event-string (ruby side) into an event object.
   @from_string: (string) ->
     $.map string.split("$"), (part) =>
       [category, action, label, value] = part.split "|"
       new @(category, action, label, value)
 
+  @from_dom: ->
+    dom_events = $("div[data-#{@html_key}]").data @html_key
+    @from_string dom_events if dom_events?
+
   # Events should not be send to an adapter unless the DOM has finished loading.
   @flush: ->
     if @list.length > 0 and @may_flush
@@ -28,11 +32,21 @@ class GaEvents.Event
     @klass.list.push @
     @klass.flush()
 
+  escape: (str) ->
+    return unless str
+    "#{str}".replace(/ä/g, "ae")
+            .replace(/ö/g, "oe")
+            .replace(/ü/g, "ue")
+            .replace(/Ä/g, "Ae")
+            .replace(/Ö/g, "Oe")
+            .replace(/Ü/g, "Ue")
+            .replace(/ß/g, "ss")
+
   to_hash: ->
-    # Category, action and label must be of type string.
-    action: "#{@action}"
-    category: "#{@category}"
-    label: "#{@label}"
+    # Category, action and label must be escaped and of type string.
+    action: @escape(@action)
+    category: @escape(@category)
+    label: @escape(@label)
     # Value has to be a positive integer or defaults to 1
     value: @to_positive_integer(@value)
 
@@ -46,12 +60,17 @@ class GaEvents.Event
     @may_flush = true
     @flush()
 
-    $(document).ajaxComplete (event, xhr) =>
+    process_xhr = (xhr) =>
       xhr_events = xhr.getResponseHeader @header_key
       @from_string xhr_events if xhr_events?
 
-    dom_events = $("div[data-#{@html_key}]").data @html_key
-    @from_string dom_events if dom_events?
+    $(document).ajaxComplete((_, xhr) -> process_xhr(xhr))
+    $(document).on "turbolinks:request-end", (event) ->
+      xhr = event.originalEvent.data.xhr
+      process_xhr(xhr)
+
+    @from_dom()
+
 
 class GaEvents.GoogleTagManagerAdapter
   constructor: (@event = "ga_event") ->

data/ga_events.gemspec

@@ -1,5 +1,6 @@
-# encoding: utf-8
-require File.expand_path('../lib/ga_events/version', __FILE__)
+# frozen_string_literal: true
+
+require File.expand_path('lib/ga_events/version', __dir__)
 
 Gem::Specification.new do |gem|
   gem.authors       = ['Florian Dütsch', 'Sven Winkler']
@@ -19,11 +20,11 @@ Gem::Specification.new do |gem|
   gem.homepage      = 'https://github.com/Nix-wie-weg/ga_events'
 
   gem.files         = `git ls-files`.split($OUTPUT_RECORD_SEPARATOR)
-  gem.executables   = gem.files.grep(/^bin\//).map { |f| File.basename(f) }
-  gem.test_files    = gem.files.grep(/^(test|spec|features)\//)
+  gem.executables   = gem.files.grep(%r{^bin/}).map { |f| File.basename(f) }
+  gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
   gem.name          = 'ga_events'
   gem.require_paths = ['lib']
   gem.version       = GaEvents::VERSION
 
-  gem.add_dependency 'rails', '>= 3.1'
+  gem.add_dependency 'rails', '>= 4.2'
 end

data/lib/ga_events.rb

@@ -1 +1,3 @@
-%w(middleware engine event list version).each { |f| require "ga_events/#{f}" }
+# frozen_string_literal: true
+
+%w[middleware engine event list version].each { |f| require "ga_events/#{f}" }

data/lib/ga_events/engine.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module GaEvents
   class Engine < ::Rails::Engine
     config.app_middleware.use GaEvents::Middleware

data/lib/ga_events/event.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 module GaEvents
-  class Event < Struct.new(:category, :action, :label, :value)
+  Event = Struct.new(:category, :action, :label, :value) do
     # Default values are set here, see README.md for details.
     def initialize(category = '-', action = '-', label = '-', value = 1)
       super

data/lib/ga_events/list.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # NOTE: Collecting the events is thread-safe, but will cause problems in an
 #       asynchronous/evented environment.
 

data/lib/ga_events/middleware.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rack/utils'
 
 module GaEvents
@@ -16,11 +18,11 @@ module GaEvents
 
         # Can outgrow, headers might get too big
         serialized = GaEvents::List.to_s
-        if request.xhr?
+        if xhr_or_turbolinks?(request)
           # AJAX request
           headers['X-GA-Events'] = serialized
 
-        elsif (300..399).include?(status)
+        elsif redirect?(status)
           # 30x/redirect? Then add event list to flash to survive the redirect.
           add_events_to_flash(env, serialized)
 
@@ -35,28 +37,29 @@ module GaEvents
     private
 
     def init_event_list(env)
-      # Handle events stored in flash
-      # Parts borrowed from Rails:
-      # https://github.com/rails/rails/blob/v3.2.14/actionpack/lib/action_dispatch/middleware/flash.rb
-      flash = env['rack.session'] && env['rack.session']['flash']
-
-      # Fix for Rails 4
-      flash &&= flash['flashes'] if Rails::VERSION::MAJOR > 3
+      flash = env['rack.session'] && env['rack.session']['flash'] &&
+              env['rack.session']['flash']['flashes']
 
-      GaEvents::List.init(flash && flash['ga_events'])
+      # The key has to be removed from the flash here to ensure it does not
+      # remain after the finished redirect. This copies the behaviour of the
+      # "#use" and "#sweep" methods of the rails flash middleware:
+      # https://github.com/rails/rails/blob/v3.2.14/actionpack/lib/action_dispatch/middleware/flash.rb#L220
+      GaEvents::List.init(flash&.delete('ga_events'))
     end
 
     def add_events_to_flash env, serialized_data
-      flash_hash = env[ActionDispatch::Flash::KEY]
-      flash_hash ||= ActionDispatch::Flash::FlashHash.new
-      flash_hash['ga_events'] = serialized_data
-      env[ActionDispatch::Flash::KEY] = flash_hash
+      flash = env['rack.session'] && env['rack.session']['flash'] &&
+              env['rack.session']['flash']['flashes']
+
+      return unless flash
+
+      flash['ga_events'] = serialized_data
     end
 
-    def normalize_response(r)
-      r = r.body if r.respond_to?(:body)
-      r = r.join if r.respond_to?(:join)
-      r
+    def normalize_response(response)
+      response = response.body if response.respond_to?(:body)
+      response = response.join if response.respond_to?(:join)
+      response
     end
 
     def inject_div(response, serialized_data)
@@ -71,5 +74,13 @@ module GaEvents
         headers.key?('Content-Type') &&
         headers['Content-Type'].include?('text/html')
     end
+
+    def redirect?(status)
+      (300..399).cover?(status)
+    end
+
+    def xhr_or_turbolinks?(request)
+      request.xhr? || request.env['HTTP_TURBOLINKS_REFERRER']
+    end
   end
 end

data/lib/ga_events/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module GaEvents
-  VERSION = '1.2.0'
+  VERSION = '2.0.0'
 end

data/spec/middleware_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe GaEvents::Middleware do

data/spec/spec_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'bundler/setup'
 require 'pry'
 require 'rspec'

metadata

@@ -1,33 +1,30 @@
 --- !ruby/object:Gem::Specification
 name: ga_events
 version: !ruby/object:Gem::Version
-  version: 1.2.0
-  prerelease: 
+  version: 2.0.0
 platform: ruby
 authors:
 - Florian Dütsch
 - Sven Winkler
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2014-12-18 00:00:00.000000000 Z
+date: 2020-11-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '3.1'
+        version: '4.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '3.1'
+        version: '4.2'
 description: Google Analytics' Event Tracking everywhere in your Rails app)
 email:
 - florian.duetsch@nix-wie-weg.de
@@ -36,7 +33,8 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
+- ".gitignore"
+- ".rubocop.yml"
 - CHANGELOG.md
 - Gemfile
 - LICENSE
@@ -55,27 +53,25 @@ files:
 - spec/spec_helper.rb
 homepage: https://github.com/Nix-wie-weg/ga_events
 licenses: []
-post_install_message: 
+metadata: {}
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 1.8.23
-signing_key: 
-specification_version: 3
+rubygems_version: 3.0.3
+signing_key:
+specification_version: 4
 summary: This gem allows you to annotate events everywhere in the code of your Rails
   app. A rack middleware is automatically inserted into the stack. It transports the
   event data to the client. Normal requests get a DIV injected, AJAX requests get
@@ -85,4 +81,3 @@ summary: This gem allows you to annotate events everywhere in the code of your R
 test_files:
 - spec/middleware_spec.rb
 - spec/spec_helper.rb
-has_rdoc: