active_record_schema 0.2.5 → 0.3.2

data/README.md

@@ -47,6 +47,7 @@ and the following migration will be generated
 ``` rb    
 class InitPostsSchema < ActiveRecord::Migration
   def change
+    create_table :posts 
     add_column :posts, :title, :string
     add_column :posts, :body, :text
 
@@ -56,6 +57,8 @@ class InitPostsSchema < ActiveRecord::Migration
 end
 ```
 
+**NOTE:** `create_table :posts` is added only if `posts` table does not exist yet
+
 Generating a migration for new columns is the same, lets add a new field to `Post` (eg. `pubdate`):
 
 ``` rb    
@@ -82,6 +85,8 @@ class AddPubdateToPosts < ActiveRecord::Migration
 end
 ```
 
+**NOTE:** No migration will be generated in case all changes are up-to-date 
+
 ## Has and Belongs To Many (HBTM) associations
 
 Lets try to add a HBTM association to our `Post` model

data/VERSION

@@ -1 +1 @@
-0.2.5
+0.3.2

data/active_record_schema.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = "active_record_schema"
-  s.version = "0.2.5"
+  s.version = "0.3.2"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["mcasimir"]

data/lib/active_record_schema/schema_diff.rb

@@ -7,6 +7,10 @@ module  ActiveRecordSchema
       self.respond_to?(diff_method) ? self.send(diff_method) : []
     end
     
+    def nothing_to_do?
+      table_exists? && diff(:fields, :add).empty? && diff(:indexes, :add).empty? && diff(:joins, :add).empty? 
+    end
+    
     def table_exists?
       _table_exists?
     end
@@ -16,14 +20,13 @@ module  ActiveRecordSchema
     end
 
     def _diff_indexes_add
-      model.schema.indexes.values.delete_if {|index| _index_names.include?(index.name.to_s) }
+      model.schema.indexes.values.delete_if {|index| _index_exists?(index.name) }
     end
 
     def _diff_joins_add
       model.schema.joins.values.delete_if {|join| _table_names.include?(join.table.to_s) }
     end
     
-
     def _connection
       ActiveRecord::Base.connection
     end
@@ -36,10 +39,6 @@ module  ActiveRecordSchema
       _table_exists? ? _connection.columns(_table).map(&:name) : []
     end
     
-    def _index_names
-      _table_exists? ? _connection.indexes(_table).map(&:name) : []
-    end
-    
     def _table_names
       _connection.tables
     end
@@ -47,6 +46,10 @@ module  ActiveRecordSchema
     def _table_exists?
       _connection.table_exists?(_table)
     end
+    
+    def _index_exists?(columns)
+      _connection.index_exists?(_table, columns)      
+    end
 
   end
 end

data/lib/generators/active_record_schema/migration/migration_generator.rb

@@ -15,8 +15,11 @@ module ActiveRecordSchema
            ActiveRecordSchema.autoload_paths.each do |p|
              load(p)
            end
-           
-           migration_template "migration_from_model.rb.erb", "db/migrate/#{file_name}.rb"
+           if model.schema.nothing_to_do?
+             say "Nothing to do for '#{options[:from]}'"
+           else
+             migration_template "migration_from_model.rb.erb", "db/migrate/#{file_name}.rb"          
+           end
          else
            migration_template "migration.rb", "db/migrate/#{file_name}.rb"
          end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: active_record_schema
 version: !ruby/object:Gem::Version
-  version: 0.2.5
+  version: 0.3.2
   prerelease: 
 platform: ruby
 authors:
@@ -89,7 +89,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -3601192281659392675
+      hash: -1716665284013531094
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: