also_migrate 0.1.0 → 0.1.1

data/{MIT-LICENSE → LICENSE}

@@ -1,4 +1,4 @@
-Copyright (c) 2009 Winton Welsh
+Copyright (c) 2010 Winton Welsh
 
 Permission is hereby granted, free of charge, to any person obtaining a copy of 
 this software and associated documentation files (the "Software"), to deal in 
@@ -15,4 +15,4 @@ IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS
 FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR 
 COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER 
 IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN 
-CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

data/{README.markdown → README.md}

@@ -1,7 +1,7 @@
 AlsoMigrate
 ===========
 
-Migrate multiple tables with similar schema.
+Migrate multiple tables with similar schema at once.
 
 Requirements
 ------------
@@ -21,8 +21,8 @@ end
 
 Options:
 
-* <code>:ignore</code> Ignore migrations that apply to certain columns (defaults to none)
-* <code>:indexes</code> Only index certain columns (defaults to all)
+* <code>ignore</code> Ignore migrations that apply to certain columns (defaults to none)
+* <code>indexes</code> Only index certain columns (defaults to all)
 
 That's it!
 ----------

data/lib/also_migrate.rb

@@ -1,5 +1,9 @@
-require File.expand_path("#{File.dirname(__FILE__)}/../require")
-Require.lib!
+$:.unshift File.dirname(__FILE__) + '/also_migrate'
+
+require 'version'
+require 'base'
+require 'migration'
+require 'migrator'
 
 ActiveRecord::Base.send(:include, AlsoMigrate::Base)
 ActiveRecord::Migrator.send(:include, AlsoMigrate::Migrator)

data/lib/also_migrate/migration.rb

@@ -34,6 +34,7 @@ module AlsoMigrate
           Object.subclasses_of(ActiveRecord::Base).each do |klass|
             if klass.respond_to?(:also_migrate_config)
               next unless klass.table_name == table_name
+              next if klass.also_migrate_config.nil?
               klass.also_migrate_config.each do |config|
                 options = config[:options]
                 tables = config[:tables]
@@ -52,7 +53,11 @@ module AlsoMigrate
                   elsif connection.table_exists?(table)
                     args[0] = table
                     args[1] = table if method == :rename_table
-                    connection.send(method, *args, &block)
+                    begin
+                      connection.send(method, *args, &block)
+                    rescue Exception => e
+                      puts "(also_migrate warning) #{e.message}"
+                    end
                   end
                 end
               end

data/lib/also_migrate/migrator.rb

@@ -19,6 +19,9 @@ module AlsoMigrate
             AlsoMigrate.create_tables(klass)
           end
         end
+      rescue Exception => e
+        puts "AlsoMigrate error: #{e.message}"
+        puts e.backtrace.join("\n")
       ensure
         migrate_without_also_migrate
       end
@@ -32,6 +35,7 @@ module AlsoMigrate
         
           def create_tables(klass)
             config = klass.also_migrate_config
+            return unless config
             old_table = klass.table_name
             config.each do |config|
               options = config[:options]
@@ -47,60 +51,27 @@ module AlsoMigrate
                         WHERE Name = '#{old_table}'
                       SQL
                     end
-                  connection.execute(<<-SQL)
-                    CREATE TABLE #{new_table} #{engine}
-                    AS SELECT #{columns.join(',')}
-                    FROM #{old_table}
-                    WHERE false;
-                  SQL
                   indexes = options[:indexes]
-                  indexes ||= indexed_columns(old_table)
-                  indexes.each do |column|
-                    connection.add_index(new_table, column)
+                  if indexes
+                    connection.execute(<<-SQL)
+                      CREATE TABLE #{new_table} #{engine}
+                      AS SELECT #{columns.join(',')}
+                      FROM #{old_table}
+                      WHERE false;
+                    SQL
+                    indexes.each do |column|
+                      connection.add_index(new_table, column)
+                    end
+                  else
+                    connection.execute(<<-SQL)
+                      CREATE TABLE #{new_table}
+                      LIKE #{old_table};
+                    SQL
                   end
                 end
               end
             end
           end
-
-          def indexed_columns(table_name)
-            # MySQL
-            if connection.class.to_s.include?('Mysql')
-              index_query = "SHOW INDEX FROM #{table_name}"
-              connection.select_all(index_query).collect do |r|
-                r["Column_name"]
-              end
-            # PostgreSQL
-            # http://stackoverflow.com/questions/2204058/show-which-columns-an-index-is-on-in-postgresql/2213199
-            elsif connection.class.to_s.include?('PostgreSQL')
-              index_query = <<-SQL
-                select
-                  t.relname as table_name,
-                  i.relname as index_name,
-                  a.attname as column_name
-                from
-                  pg_class t,
-                  pg_class i,
-                  pg_index ix,
-                  pg_attribute a
-                where
-                  t.oid = ix.indrelid
-                  and i.oid = ix.indexrelid
-                  and a.attrelid = t.oid
-                  and a.attnum = ANY(ix.indkey)
-                  and t.relkind = 'r'
-                  and t.relname = '#{table_name}'
-                order by
-                  t.relname,
-                  i.relname
-              SQL
-              connection.select_all(index_query).collect do |r|
-                r["column_name"]
-              end
-            else
-              raise 'AlsoMigrate does not support this database adapter'
-            end
-          end
         end
       end
     end

data/lib/also_migrate/version.rb

@@ -0,0 +1,3 @@
+module AlsoMigrate
+  VERSION = "0.1.1" unless defined?(::AlsoMigrate::VERSION)
+end

metadata

@@ -1,7 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: also_migrate
 version: !ruby/object:Gem::Version 
-  version: 0.1.0
+  hash: 25
+  prerelease: false
+  segments: 
+  - 0
+  - 1
+  - 1
+  version: 0.1.1
 platform: ruby
 authors: 
 - Winton Welsh
@@ -9,48 +15,60 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-04-03 00:00:00 -07:00
+date: 2010-07-19 00:00:00 -07:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
-  name: require
-  type: :runtime
-  version_requirement: 
-  version_requirements: !ruby/object:Gem::Requirement 
+  type: :development
+  prerelease: false
+  name: bundler
+  version_requirements: &id001 !ruby/object:Gem::Requirement 
+    none: false
     requirements: 
     - - "="
       - !ruby/object:Gem::Version 
-        version: 0.2.6
-    version: 
-description: 
-email: mail@wintoni.us
+        hash: 62196361
+        segments: 
+        - 1
+        - 0
+        - 0
+        - beta
+        - 5
+        version: 1.0.0.beta.5
+  requirement: *id001
+- !ruby/object:Gem::Dependency 
+  type: :development
+  prerelease: false
+  name: rspec
+  version_requirements: &id002 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - "="
+      - !ruby/object:Gem::Version 
+        hash: 27
+        segments: 
+        - 1
+        - 3
+        - 0
+        version: 1.3.0
+  requirement: *id002
+description: Migrate multiple tables with similar schema at once
+email: 
+- mail@wintoni.us
 executables: []
 
 extensions: []
 
-extra_rdoc_files: 
-- README.markdown
+extra_rdoc_files: []
+
 files: 
-- init.rb
 - lib/also_migrate/base.rb
 - lib/also_migrate/migration.rb
 - lib/also_migrate/migrator.rb
+- lib/also_migrate/version.rb
 - lib/also_migrate.rb
-- log/development.log
-- MIT-LICENSE
-- rails/init.rb
-- Rakefile
-- README.markdown
-- require.rb
-- spec/also_migrate_spec.rb
-- spec/config/database.yml.example
-- spec/db/migrate/001_create_articles.rb
-- spec/db/migrate/002_add_permalink.rb
-- spec/db/migrate/003_remove_ignored.rb
-- spec/fixtures/article.rb
-- spec/log/test.log
-- spec/Rakefile
-- spec/spec_helper.rb
+- LICENSE
+- README.md
 has_rdoc: true
 homepage: http://github.com/winton/also_migrate
 licenses: []
@@ -61,23 +79,29 @@ rdoc_options: []
 require_paths: 
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement 
+  none: false
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
+      hash: 3
+      segments: 
+      - 0
       version: "0"
-  version: 
 required_rubygems_version: !ruby/object:Gem::Requirement 
+  none: false
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
+      hash: 3
+      segments: 
+      - 0
       version: "0"
-  version: 
 requirements: []
 
 rubyforge_project: 
-rubygems_version: 1.3.5
+rubygems_version: 1.3.7
 signing_key: 
 specification_version: 3
-summary: Migrate multiple tables with similar schema
+summary: Migrate multiple tables with similar schema at once
 test_files: []
 

data/Rakefile

@@ -1,2 +0,0 @@
-require "#{File.dirname(__FILE__)}/require"
-Require.rakefile!

data/init.rb

@@ -1 +0,0 @@
-require File.dirname(__FILE__) + "/rails/init"

data/log/development.log

@@ -1,6 +0,0 @@
-  SQL (0.2ms)   SET SQL_AUTO_IS_NULL=0
-  User Load (59.6ms)   SELECT * FROM `users` ORDER BY users.id DESC LIMIT 1
-  User Columns (21.6ms)   SHOW FIELDS FROM `users`
-  SQL (0.3ms)   SET SQL_AUTO_IS_NULL=0
-  SQL (0.3ms)   SHOW TABLES
-  User Columns (13.0ms)   SHOW FIELDS FROM `users`

data/rails/init.rb

@@ -1,2 +0,0 @@
-require File.expand_path("#{File.dirname(__FILE__)}/../require")
-Require.rails_init!

data/require.rb

@@ -1,52 +0,0 @@
-require 'rubygems'
-gem 'require'
-require 'require'
-
-Require do
-  gem(:active_wrapper, '=0.2.3') { require 'active_wrapper' }
-  gem :require, '=0.2.6'
-  gem(:rake, '=0.8.7') { require 'rake' }
-  gem :rspec, '=1.3.0'
-  
-  gemspec do
-    author 'Winton Welsh'
-    dependencies do
-      gem :require
-    end
-    email 'mail@wintoni.us'
-    name 'also_migrate'
-    homepage "http://github.com/winton/#{name}"
-    summary "Migrate multiple tables with similar schema"
-    version '0.1.0'
-  end
-  
-  bin { require 'lib/also_migrate' }
-  
-  lib do
-    require 'lib/also_migrate/base'
-    require 'lib/also_migrate/migration'
-    require 'lib/also_migrate/migrator'
-  end
-  
-  rails_init { require 'lib/also_migrate' }
-  
-  rakefile do
-    gem(:active_wrapper)
-    gem(:rake) { require 'rake/gempackagetask' }
-    gem(:rspec) { require 'spec/rake/spectask' }
-    require 'require/tasks'
-  end
-  
-  spec_helper do
-    gem(:active_wrapper)
-    require 'require/spec_helper'
-    require 'rails/init'
-    require 'pp'
-    require 'spec/fixtures/article'
-  end
-  
-  spec_rakefile do
-    gem(:rake)
-    gem(:active_wrapper) { require 'active_wrapper/tasks' }
-  end
-end

data/spec/Rakefile

@@ -1,10 +0,0 @@
-require File.expand_path("#{File.dirname(__FILE__)}/../require")
-Require.spec_rakefile!
-
-begin
-  ActiveWrapper::Tasks.setup(
-    :base => File.dirname(__FILE__),
-    :env => 'test'
-  )
-rescue Exception
-end

data/spec/also_migrate_spec.rb

@@ -1,53 +0,0 @@
-require "spec_helper"
-
-describe AlsoMigrate do
-  
-  describe 'fixture config' do
-    
-    before(:each) do
-      $db.migrate(1)
-      $db.migrate(0)
-      $db.migrate(1)
-    end
-    
-    it 'should migrate both tables up' do
-      migrate_with_state(2)
-      (@new_article_columns - @old_article_columns).should == [ 'permalink' ]
-      (@new_archive_columns - @old_archive_columns).should == [ 'permalink' ]
-    end
-    
-    it 'should migrate both tables down' do
-      $db.migrate(2)
-      migrate_with_state(1)
-      (@old_article_columns - @new_article_columns).should == [ 'permalink' ]
-      (@old_archive_columns - @new_archive_columns).should == [ 'permalink' ]
-    end
-    
-    it "should ignore the body column column" do
-      (columns('articles') - columns('article_archives')).should == [ 'body' ]
-      connection.remove_column(:articles, :body)
-      (columns('articles') - columns('article_archives')).should == []
-    end
-    
-    it "should only add an index for id" do
-      ActiveRecord::Migrator::AlsoMigrate.indexed_columns('articles').should == [ 'id', 'read' ]
-      ActiveRecord::Migrator::AlsoMigrate.indexed_columns('article_archives').should == [ 'id' ]
-    end
-  end
-  
-  describe 'no index config' do
-    
-    before(:each) do
-      Article.also_migrate_config = nil
-      Article.also_migrate :article_archives
-      $db.migrate(1)
-      $db.migrate(0)
-      $db.migrate(1)
-    end
-  
-    it "should add all indexes" do
-      ActiveRecord::Migrator::AlsoMigrate.indexed_columns('articles').should == [ 'id', 'read' ]
-      ActiveRecord::Migrator::AlsoMigrate.indexed_columns('article_archives').should == [ 'id', 'read' ]
-    end
-  end
-end

data/spec/config/database.yml.example

@@ -1,6 +0,0 @@
-test:
-  adapter: mysql
-  database: also_migrate
-  username: root
-  password: 
-  host: localhost