hightop 0.1.3 → 0.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 8123384a25f9687e070f4ee86677c1afc5060a1f
-  data.tar.gz: 3b3776446331302a639859ba54d12cfa77b450aa
+SHA256:
+  metadata.gz: 340c609cb491de09f5c69f80b3e237126c081a8167565c78d6f28cac692cc34b
+  data.tar.gz: 0aa4d15e3ac6503a73f664602d0d07f1a0e956202f93411ec51d663dcca9b2ff
 SHA512:
-  metadata.gz: 51c28c43483583167675608a90698f5a3e5b2b7bcf362278ab89b63a56c065d7adad51993e111005201b70e2e63ae5f7570db5e5eb1ce39cec1df085f32952ad
-  data.tar.gz: 191b537904fcb730a4e313714f9168b475e8017a791131e60c9173ea4ec5c7a5e817e73324af5388c0161a106eb336e20da3b4a8e6074f2ed5bcc81db0d2dd8c
+  metadata.gz: 31bcc5dcc655886f5dad2ebe27f33a7f56702b0503883bac779a944b502dc9bf1f27694c1eb83e4d36ca908fbb1bbe5e9b09706b5536f1ba53616e555204c579
+  data.tar.gz: 1e4d59093c22c49d3166368d8b23386ca7366dd9fc1b31e813e9df81bdd5d3b3e7d87ece4d36bf6b94deef69d2cd093927c0d9eeb62582fd357feed0c381564d

data/CHANGELOG.md

@@ -1,33 +1,54 @@
-## 0.1.3
+## 0.2.3 (2020-06-18)
+
+- Dropped support for Rails 4.2 and Ruby 2.3
+- Fixed deprecation warning in Ruby 2.7
+
+## 0.2.2 (2019-08-12)
+
+- Added support for Mongoid
+
+## 0.2.1 (2019-08-04)
+
+- Added support for arrays and hashes
+
+## 0.2.0 (2017-03-19)
+
+- Use keyword arguments
+
+## 0.1.4 (2016-02-04)
+
+- Added `distinct` option to replace `uniq`
+
+## 0.1.3 (2015-06-18)
 
 - Fixed `min` option with `uniq`
 
-## 0.1.2
+## 0.1.2 (2014-11-05)
 
 - Added `min` option
 
-## 0.1.1
+## 0.1.1 (2014-07-02)
 
 - Added `uniq` option
 - Fixed `Model.limit(n).top`
 
-## 0.1.0
+## 0.1.0 (2014-06-11)
 
 - No changes, just bump
 
-## 0.0.4
+## 0.0.4 (2014-06-11)
 
 - Added support for multiple groups
 - Added `nil` option
 
-## 0.0.3
+## 0.0.3 (2014-06-11)
 
 - Fixed escaping
 
-## 0.0.2
+## 0.0.2 (2014-05-29)
 
 - Added `limit` parameter
 
-## 0.0.1
+## 0.0.1 (2014-05-11)
 
 - First release

data/LICENSE.txt

@@ -1,4 +1,4 @@
-Copyright (c) 2014 Andrew Kane
+Copyright (c) 2014-2020 Andrew Kane
 
 MIT License
 

data/README.md

@@ -2,19 +2,28 @@
 
 A nice shortcut for group count queries
 
-[![Build Status](https://travis-ci.org/ankane/hightop.svg)](https://travis-ci.org/ankane/hightop)
-
 ```ruby
 Visit.top(:browser)
+# {
+#   "Chrome"  => 63,
+#   "Safari"  => 50,
+#   "Firefox" => 34
+# }
 ```
 
-instead of
+Works with Active Record, Mongoid, arrays and hashes
+
+[![Build Status](https://travis-ci.org/ankane/hightop.svg?branch=master)](https://travis-ci.org/ankane/hightop)
+
+## Installation
+
+Add this line to your application’s Gemfile:
 
 ```ruby
-Visit.group(:browser).where("browser IS NOT NULL").order("count_all DESC, browser").count
+gem 'hightop'
 ```
 
-Be sure to [sanitize user input](http://rails-sqli.org/) like you must with `group`.
+## Options
 
 Limit the results
 
@@ -37,13 +46,13 @@ Visit.top([:city, :browser])
 And expressions
 
 ```ruby
-Visit.top("LOWER(referring_domain)")
+Visit.top(Arel.sql("LOWER(referring_domain)"))
 ```
 
 And distinct
 
 ```ruby
-Visit.top(:city, uniq: :user_id)
+Visit.top(:city, distinct: :user_id)
 ```
 
 And min count
@@ -52,18 +61,49 @@ And min count
 Visit.top(:city, min: 10)
 ```
 
-## Installation
+## User Input
 
-Add this line to your application’s Gemfile:
+If passing user input as the column, be sure to sanitize it first [like you must](https://rails-sqli.org/) with `group`.
 
 ```ruby
-gem 'hightop'
+column = params[:column]
+
+# check against permitted columns
+raise "Unpermitted column" unless ["column_a", "column_b"].include?(column)
+
+User.top(column)
 ```
 
-And then execute:
+## Arrays and Hashes
 
-```sh
-bundle
+Arrays
+
+```ruby
+["up", "up", "down"].top
+```
+
+Hashes
+
+```ruby
+{a: "up", b: "up", c: "down"}.top { |k, v| v }
+```
+
+Limit the results
+
+```ruby
+["up", "up", "down"].top(1)
+```
+
+Include nil values
+
+```ruby
+[nil, nil, "down"].top(nil: true)
+```
+
+Min count
+
+```ruby
+["up", "up", "down"].top(min: 2)
 ```
 
 ## History
@@ -78,3 +118,12 @@ Everyone is encouraged to help improve this project. Here are a few ways you can
 - Fix bugs and [submit pull requests](https://github.com/ankane/hightop/pulls)
 - Write, clarify, or fix documentation
 - Suggest or add new features
+
+To get started with development and testing:
+
+```sh
+git clone https://github.com/ankane/hightop.git
+cd hightop
+bundle install
+bundle exec rake test
+```

data/lib/hightop.rb

@@ -1,35 +1,16 @@
-require "hightop/version"
-require "active_record"
-
-module Hightop
-  def top(column, limit = nil, options = {})
-    if limit.is_a?(Hash)
-      options = limit
-      limit = nil
-    end
-
-    order_str = column.is_a?(Array) ? column.map(&:to_s).join(", ") : column
-    relation = group(column).order("count_#{options[:uniq] || 'all'} DESC, #{order_str}")
-    if limit
-      relation = relation.limit(limit)
-    end
+# dependencies
+require "active_support"
 
-    unless options[:nil]
-      (column.is_a?(Array) ? column : [column]).each do |c|
-        relation = relation.where("#{c} IS NOT NULL")
-      end
-    end
-
-    if options[:min]
-      relation = relation.having("COUNT(#{options[:uniq] ? "DISTINCT #{options[:uniq]}" : '*'}) >= #{options[:min].to_i}")
-    end
+# modules
+require "hightop/enumerable"
+require "hightop/version"
 
-    if options[:uniq]
-      relation.uniq.count(options[:uniq])
-    else
-      relation.count
-    end
-  end
+ActiveSupport.on_load(:active_record) do
+  require "hightop/kicks"
+  extend Hightop::Kicks
 end
 
-ActiveRecord::Base.send :extend, Hightop
+ActiveSupport.on_load(:mongoid) do
+  require "hightop/mongoid"
+  Mongoid::Document::ClassMethods.include(Hightop::Mongoid)
+end

data/lib/hightop/enumerable.rb

@@ -0,0 +1,24 @@
+module Enumerable
+  def top(*args, **options, &block)
+    if block || !(respond_to?(:scoping) || respond_to?(:with_scope))
+      # TODO raise error if too many arguments
+      limit = args[0]
+      min = options[:min]
+
+      counts = Hash.new(0)
+      map(&block).each do |v|
+        counts[v] += 1
+      end
+      counts.delete(nil) unless options[:nil]
+      counts.select! { |_, v| v >= min } if min
+
+      arr = counts.sort_by { |_, v| -v }
+      arr = arr[0...limit] if limit
+      Hash[arr]
+    elsif respond_to?(:scoping)
+      scoping { @klass.send(:top, *args, **options, &block) }
+    else
+      with_scope(self) { klass.send(:top, *args, **options, &block) }
+    end
+  end
+end

data/lib/hightop/kicks.rb

@@ -0,0 +1,29 @@
+module Hightop
+  module Kicks
+    def top(column, limit = nil, distinct: nil, uniq: nil, min: nil, nil: nil)
+      distinct ||= uniq
+      order_str = column.is_a?(Array) ? column.map(&:to_s).join(", ") : column
+      relation = group(column).order(["count_#{distinct || 'all'} DESC", order_str])
+      if limit
+        relation = relation.limit(limit)
+      end
+
+      # terribly named option
+      unless binding.local_variable_get(:nil)
+        (column.is_a?(Array) ? column : [column]).each do |c|
+          relation = relation.where("#{c} IS NOT NULL")
+        end
+      end
+
+      if min
+        relation = relation.having("COUNT(#{distinct ? "DISTINCT #{distinct}" : '*'}) >= #{min.to_i}")
+      end
+
+      if distinct
+        relation.distinct.count(distinct)
+      else
+        relation.count
+      end
+    end
+  end
+end

data/lib/hightop/mongoid.rb

@@ -0,0 +1,52 @@
+module Hightop
+  module Mongoid
+    # super helpful article
+    # https://maximomussini.com/posts/mongoid-aggregation-dsl/
+    def top(column, limit = nil, distinct: nil, uniq: nil, min: nil, nil: nil)
+      distinct ||= uniq
+
+      relation = all
+
+      # terribly named option
+      unless binding.local_variable_get(:nil)
+        (column.is_a?(Array) ? column : [column]).each do |c|
+          relation = relation.and(c.ne => nil)
+        end
+      end
+
+      ids = {}
+      Array(column).each_with_index do |c, i|
+        ids["c#{i}"] = "$#{c}"
+      end
+
+      if distinct
+        # group with distinct column first, then group without it
+        # https://stackoverflow.com/questions/24761266/select-group-by-count-and-distinct-count-in-same-mongodb-query/24770233#24770233
+        distinct_ids = ids.merge("c#{ids.size}" => "$#{distinct}")
+        relation = relation.group(_id: distinct_ids, count: {"$sum" => 1})
+        ids.each_key do |k|
+          ids[k] = "$_id.#{k}"
+        end
+      end
+
+      relation = relation.group(_id: ids, count: {"$sum" => 1})
+
+      if min
+        relation.pipeline.push("$match" => {"count" => {"$gte" => min}})
+      end
+
+      relation = relation.desc(:count)
+      if limit
+        relation = relation.limit(limit)
+      end
+
+      result = {}
+      collection.aggregate(relation.pipeline).each do |doc|
+        key = doc["_id"].values
+        key = key[0] if key.size == 1
+        result[key] = doc["count"]
+      end
+      result
+    end
+  end
+end

data/lib/hightop/version.rb

@@ -1,3 +1,3 @@
 module Hightop
-  VERSION = "0.1.3"
+  VERSION = "0.2.3"
 end

metadata

@@ -1,43 +1,43 @@
 --- !ruby/object:Gem::Specification
 name: hightop
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.2.3
 platform: ruby
 authors:
 - Andrew Kane
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-06-18 00:00:00.000000000 Z
+date: 2020-06-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: activerecord
+  name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '5'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '5'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.6'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.6'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -58,14 +58,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '5'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '5'
 - !ruby/object:Gem::Dependency
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement
@@ -80,25 +80,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: A nice shortcut for group count queries
-email:
-- andrew@chartkick.com
+description: 
+email: andrew@chartkick.com
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- ".gitignore"
-- ".travis.yml"
 - CHANGELOG.md
-- Gemfile
 - LICENSE.txt
 - README.md
-- Rakefile
-- hightop.gemspec
 - lib/hightop.rb
+- lib/hightop/enumerable.rb
+- lib/hightop/kicks.rb
+- lib/hightop/mongoid.rb
 - lib/hightop/version.rb
-- test/hightop_test.rb
-- test/test_helper.rb
 homepage: https://github.com/ankane/hightop
 licenses:
 - MIT
@@ -111,19 +106,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.4'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.4.5
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: A nice shortcut for group count queries
-test_files:
-- test/hightop_test.rb
-- test/test_helper.rb
-has_rdoc: 
+test_files: []

data/.gitignore

@@ -1,22 +0,0 @@
-*.gem
-*.rbc
-.bundle
-.config
-.yardoc
-Gemfile.lock
-InstalledFiles
-_yardoc
-coverage
-doc/
-lib/bundler/man
-pkg
-rdoc
-spec/reports
-test/tmp
-test/version_tmp
-tmp
-*.bundle
-*.so
-*.o
-*.a
-mkmf.log

data/.travis.yml

@@ -1,10 +0,0 @@
-language: ruby
-rvm:
-  - 2.1
-gemfile:
-  - Gemfile
-script: bundle exec rake test
-notifications:
-  email:
-    on_success: never
-    on_failure: change

data/Gemfile

@@ -1,4 +0,0 @@
-source "https://rubygems.org"
-
-# Specify your gem's dependencies in hightop.gemspec
-gemspec

data/Rakefile

@@ -1,8 +0,0 @@
-require "bundler/gem_tasks"
-require "rake/testtask"
-
-task default: :test
-Rake::TestTask.new do |t|
-  t.libs << "test"
-  t.pattern = "test/**/*_test.rb"
-end

data/hightop.gemspec

@@ -1,27 +0,0 @@
-# coding: utf-8
-lib = File.expand_path("../lib", __FILE__)
-$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require "hightop/version"
-
-Gem::Specification.new do |spec|
-  spec.name          = "hightop"
-  spec.version       = Hightop::VERSION
-  spec.authors       = ["Andrew Kane"]
-  spec.email         = ["andrew@chartkick.com"]
-  spec.summary       = "A nice shortcut for group count queries"
-  spec.description   = "A nice shortcut for group count queries"
-  spec.homepage      = "https://github.com/ankane/hightop"
-  spec.license       = "MIT"
-
-  spec.files         = `git ls-files -z`.split("\x0")
-  spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
-  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
-  spec.require_paths = ["lib"]
-
-  spec.add_dependency "activerecord"
-
-  spec.add_development_dependency "bundler", "~> 1.6"
-  spec.add_development_dependency "rake"
-  spec.add_development_dependency "minitest"
-  spec.add_development_dependency "sqlite3"
-end

data/test/hightop_test.rb

@@ -1,90 +0,0 @@
-require_relative "test_helper"
-
-class TestHightop < Minitest::Test
-  def setup
-    Visit.delete_all
-  end
-
-  def test_top
-    create_city("San Francisco", 3)
-    create_city("Chicago", 2)
-    expected = {
-      "San Francisco" => 3,
-      "Chicago" => 2
-    }
-    assert_equal expected, Visit.top(:city)
-  end
-
-  def test_limit
-    create_city("San Francisco", 3)
-    create_city("Chicago", 2)
-    create_city("Boston", 1)
-    expected = {
-      "San Francisco" => 3,
-      "Chicago" => 2
-    }
-    assert_equal expected, Visit.top(:city, 2)
-    assert_equal expected, Visit.limit(2).top(:city)
-  end
-
-  def test_nil_values
-    create_city("San Francisco", 3)
-    create_city(nil, 2)
-    expected = {
-      "San Francisco" => 3
-    }
-    assert_equal expected, Visit.top(:city)
-  end
-
-  def test_nil_option
-    create_city("San Francisco", 3)
-    create_city(nil, 2)
-    expected = {
-      "San Francisco" => 3,
-      nil => 2
-    }
-    assert_equal expected, Visit.top(:city, nil: true)
-  end
-
-  def test_multiple_groups
-    create_city("San Francisco")
-    expected = {
-      ["San Francisco", "San Francisco"] => 1
-    }
-    assert_equal expected, Visit.top([:city, :city])
-  end
-
-  def test_expressions
-    create_city("San Francisco")
-    expected = {
-      "san francisco" => 1
-    }
-    assert_equal expected, Visit.top("LOWER(city)")
-  end
-
-  def test_uniq
-    create(city: "San Francisco", user_id: 1)
-    create(city: "San Francisco", user_id: 1)
-    expected = {
-      "San Francisco" => 1
-    }
-    assert_equal expected, Visit.top(:city, uniq: :user_id)
-  end
-
-  def test_min
-    create_city("San Francisco", 3)
-    create_city("Chicago", 2)
-    expected = {
-      "San Francisco" => 3
-    }
-    assert_equal expected, Visit.top(:city, min: 3)
-  end
-
-  def create_city(city, count = 1)
-    create({city: city}, count)
-  end
-
-  def create(attributes, count = 1)
-    count.times { Visit.create!(attributes) }
-  end
-end

data/test/test_helper.rb

@@ -1,19 +0,0 @@
-require "bundler/setup"
-Bundler.require(:default)
-require "minitest/autorun"
-require "minitest/pride"
-require "logger"
-
-# for debugging
-# ActiveRecord::Base.logger = Logger.new(STDOUT)
-
-# migrations
-ActiveRecord::Base.establish_connection adapter: "sqlite3", database: ":memory:"
-
-ActiveRecord::Migration.create_table :visits do |t|
-  t.string :city
-  t.string :user_id
-end
-
-class Visit < ActiveRecord::Base
-end