dblint 0.1.1 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5882547f7ba7be4f29dfbd2e4fc685168e656036
-  data.tar.gz: 1171ff0b3c9bdd3f8fb3d94d48fdff3ff041fb35
+  metadata.gz: 52f5eacd08e69f907b7bcac11162a2ef2d270321
+  data.tar.gz: 1de50cef76aceccdaf9a7f4b5bc32839b90e3d8a
 SHA512:
-  metadata.gz: 72f796d6ddf84cf2ccb305e6e182b4172b0ced77455ed1377d563a65fe851ee731b6538386b3ad25b97b4e7467a45e993b321cc7bbf25bb36a31439ecabd8845
-  data.tar.gz: 4734c644bb32a0f13a61cce0e2235351516ab241c3e403f65fe035f2e2f571c80b7da91a676281d317aca882948aa944dfa43a1fca74dbca29fe0c76a413e12b
+  metadata.gz: 2d61a6924fd91595b8f456332108c4ffdef1b20e4fb55b73ae79306f4fdff20db778cae8aab81d5bff7e8db394d50f393e30262aafe1628e7be371a5a5dcfeab
+  data.tar.gz: cd397ed2568760db4aafa1b66a2eb5f89d757691d0009147a272c74e511c0a9831b926a111cdc8bc30a599434a95b2d322ddae68cc0e4906f1faf110f04213b8

data/.rubocop.yml

@@ -3,6 +3,7 @@ require: rubocop-rspec
 AllCops:
   Exclude:
     - spec/**/*
+    - vendor/**/*
 
 Documentation:
   Enabled: false

data/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 0.2.0    2016-07-29
+
+* Rails 5 support [#2](https://github.com/lfittl/dblint/pull/2) [@ksykulev](https://github.com/ksykulev)
+
+
 ## 0.1.1    2016-04-27
 
 * Fix handling of vendored gems [#1](https://github.com/lfittl/dblint/pull/1) [@ksykulev](https://github.com/ksykulev)

data/lib/dblint/checks/long_held_lock.rb

@@ -61,7 +61,7 @@ module Dblint
         @existing_ids = {}
 
         ActiveRecord::Base.connection.tables.each do |table|
-          next if table == 'schema_migrations'
+          next if %w(schema_migrations ar_internal_metadata).include?(table)
           @existing_ids[table] = ActiveRecord::Base.connection.select_values("SELECT id FROM #{table}", 'DBLINT').map(&:to_i)
         end
       end

data/lib/dblint/rails_integration.rb

@@ -27,9 +27,18 @@ module Dblint
 
       CHECKS.each do |check_klass|
         check = check_instance_for_connection(payload[:connection_id], check_klass)
-        check.statement_finished(name, id, payload)
+        check.statement_finished(name, id, payload_from_version(ActiveRecord::VERSION::MAJOR, payload))
       end
     end
+
+    private
+
+    def payload_from_version(version, payload)
+      return payload if version == 4
+
+      compatible_binds = payload[:binds].map { |bind| [bind, bind.value] }
+      payload.merge(binds: compatible_binds)
+    end
   end
 
   ActiveSupport::Notifications.subscribe('sql.active_record', RailsIntegration.new)

data/lib/dblint/version.rb

@@ -1,3 +1,3 @@
 module Dblint
-  VERSION = '0.1.1'.freeze
+  VERSION = '0.2.0'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: dblint
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.2.0
 platform: ruby
 authors:
 - Lukas Fittl
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-04-27 00:00:00.000000000 Z
+date: 2016-07-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -179,6 +179,7 @@ files:
 - CHANGELOG.md
 - CODE_OF_CONDUCT.md
 - Gemfile
+- Gemfile.lock
 - LICENSE.txt
 - README.md
 - Rakefile