schema_plus_foreign_keys 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 18b2164c375e6447f51ff241d12deaa5748736e4
-  data.tar.gz: 0c46ee8eb9792365ae22ecc13ecf305afde81a39
+  metadata.gz: b2ffa8e1095bb42bec9a1caf9c6a90048964a771
+  data.tar.gz: d184941dfb77d130e6420bad3018466e984104c5
 SHA512:
-  metadata.gz: b930008367e71b0c7b53771be9974d61e37c5ed8ba2c3a6ec049a41185299656c8126c1f097155b0776d9210c7bf295ea8a5a7e4014df52bbc9a954e87a8de64
-  data.tar.gz: bc1f9db77881d39cc24f418458711b486cabed9a7c3b1d479d1af9eb3ac8fe4ab1129cac8c0efe66938b437a0626e6e1b25ae63cb75637b6bdada084a89d6f7c
+  metadata.gz: dcac94e7194fda6a55479f573be6250191f2f1d47ed5e7c6e29c6f4c3772348fe297b8b07349a2c7abaa3c58fb294c1a6e1e232c65dbfbbd84079c71e53173d2
+  data.tar.gz: ec5104854ab13660508d486d661216e0d56bbeda754c65108eeb3b0d37be8a0ebcd46af8f216b5e68e5949dd7243eb96fb5d0db827746aa231db5af48e2d7f73

data/README.md

@@ -149,6 +149,7 @@ SchemaPlus::ForeignKeys is tested on:
 
 ## History
 
+* 0.1.2 - Handle very long names
 * 0.1.1 - Cleanup; use (new) core Migration::RenameTable stack rather than monkey patching.
 * 0.1.0 - Initial release, brought over from schema_plus 1.x via 2.0.0.pre*
 

data/lib/schema_plus/foreign_keys/active_record/connection_adapters/foreign_key_definition.rb

@@ -1,3 +1,4 @@
+require 'openssl'
 require 'active_record/connection_adapters/abstract/schema_definitions'
 
 module SchemaPlus::ForeignKeys
@@ -113,7 +114,9 @@ module SchemaPlus::ForeignKeys
         end
 
         def self.default_name(from_table, column)
-          "fk_#{fixup_schema_name(from_table)}_#{Array.wrap(column).join('_and_')}"
+          name = "fk_#{fixup_schema_name(from_table)}_#{Array.wrap(column).join('_and_')}"
+          name = name.slice(0, 27) + "_" + OpenSSL::Digest::MD5.new.hexdigest(name) if name.length > 60
+          name
         end
 
         def self.fixup_schema_name(table_name)

data/lib/schema_plus/foreign_keys/version.rb

@@ -1,5 +1,5 @@
 module SchemaPlus
   module ForeignKeys
-    VERSION = "0.1.1"
+    VERSION = "0.1.2"
   end
 end

data/spec/migration_spec.rb

@@ -106,6 +106,17 @@ describe ActiveRecord::Migration do
       expect(@model).to reference(:users, :id).with_name("wugga")
     end
 
+    it "handles very long names" do
+      table = ("ta"*15)
+      column = ("co"*15 + "_id")
+      expect {
+        ActiveRecord::Migration.create_table table do |t|
+          t.integer column, references: :members
+        end
+      }.not_to raise_error
+      expect(ActiveRecord::Base.connection.foreign_keys(table).first.column).to eq(column)
+    end
+
     it "should allow multiple foreign keys to be made" do
       recreate_table(@model) do |t|
         t.integer :user_id, :references => :users

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: schema_plus_foreign_keys
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - ronen barzel