merb-admin 0.5.6 → 0.5.7

data/README.rdoc

@@ -2,11 +2,11 @@
 ==== MerbAdmin is a Merb plugin that provides an easy-to-use interface for managing your data.
 It currently offers the features listed here[http://sferik.tadalist.com/lists/1352791/public].
 
-The status of the current build can be seen here[http://runcoderun.com/sferik/merb-admin].
-== Installtion
+The status of the latest build is available here[http://runcoderun.com/sferik/merb-admin].
+== Installation
   $ gem install merb-admin -s http://gemcutter.org
 In your app, add the following dependency to <tt>config/dependencies.rb</tt>:
-  dependency "merb-admin", "0.5.6"
+  dependency "merb-admin", "0.5.7"
 Add the following route to <tt>config/router.rb</tt>:
   add_slice(:merb_admin, :path_prefix => "admin")
 Then, run the following rake task:
@@ -21,10 +21,22 @@ If you're feeling crafty, you can set a couple configuration options in <tt>conf
 Start the server:
   $ merb
 You should now be able to administer your site at http://localhost:4000/admin.
-== Contact
-Please report any problems you encounter to mailto:sferik@gmail.com or {@sferik}[http://twitter.com/sferik] on Twitter.
 == WARNING
 MerbAdmin does not implement any authorization scheme. Make sure to apply authorization logic before deploying to production!
+== Contributing
+In the spirit of {free software}[http://www.fsf.org/licensing/essays/free-sw.html], people of all abilities are encouraged to help improve MerbAdmin.
+
+There are many ways to contribute:
+* by reporting bugs
+* by suggesting new features
+* by writing documentation
+* by writing specifications
+* by writing code (no patch is too small: fix typos in comments or inconsistent whitespace)
+* financially[http://pledgie.com/campaigns/5428]
+
+All contributors will be added to the credits below and will receive the respect and gratitude of the author.
+== Contact
+If you are interested in contributing to MerbAdmin, please contact Erik Michaels-Ober via email[mailto:sferik@gmail.com], GitHub[http://github.com/sferik/], or Twitter[http://twitter.com/sferik].
 == Credits
 Many thanks to:
 * {Wilson Miner}[http://www.wilsonminer.com] for contributing the stylesheets and javascripts from Django[http://www.djangoproject.com]

data/Rakefile

@@ -9,7 +9,7 @@ AUTHOR = "Erik Michaels-Ober"
 EMAIL = "sferik@gmail.com"
 HOMEPAGE = "http://github.com/sferik/merb-admin"
 SUMMARY = "MerbAdmin is a Merb plugin that provides an easy-to-use interface for managing your data."
-GEM_VERSION = "0.5.6"
+GEM_VERSION = "0.5.7"
 
 spec = Gem::Specification.new do |s|
   s.rubyforge_project = "merb"

data/lib/abstract_model.rb

@@ -1,6 +1,4 @@
 require 'generic_support'
-require 'activerecord_support'
-require 'datamapper_support'
 
 module MerbAdmin
   class AbstractModel
@@ -11,7 +9,7 @@ module MerbAdmin
       case Merb.orm
       when :activerecord
         Dir.glob(Merb.dir_for(:model) / Merb.glob_for(:model)).each do |filename|
-          # FIXME: This heuristic for finding ActiveRecord models is too strict
+          # FIXME: This heuristic for finding ActiveRecord models could be too strict
           File.read(filename).scan(/^class ([\w\d_\-:]+) < ActiveRecord::Base$/).flatten.each do |m|
             model = lookup(m.to_s.to_sym)
             @models << new(model) if model
@@ -56,8 +54,10 @@ module MerbAdmin
       self.extend(GenericSupport)
       case Merb.orm
       when :activerecord
+        require 'activerecord_support'
         self.extend(ActiverecordSupport)
       when :datamapper
+        require 'datamapper_support'
         self.extend(DatamapperSupport)
       else
         raise "MerbAdmin does not support the #{Merb.orm} ORM"

data/lib/merb-admin.rb

@@ -23,7 +23,7 @@ if defined?(Merb::Plugins)
 
     # Slice metadata
     self.description = "MerbAdmin is a Merb plugin that provides an easy-to-use interface for managing your data."
-    self.version = "0.5.6"
+    self.version = "0.5.7"
     self.author = "Erik Michaels-Ober"
 
     # Stub classes loaded hook - runs before LoadClasses BootLoader

data/lib/merb-admin/slicetasks.rb

@@ -16,41 +16,44 @@ namespace :slices do
     # task :migrate do
     # end
 
+    desc "Copies sample models, copies and runs sample migrations, and loads sample data into your app"
+    task :activerecord => ["activerecord:copy_sample_models", "activerecord:copy_sample_migrations", "activerecord:migrate", "load_sample_data"]
     namespace :activerecord do
-      desc "Loads sample ActiveRecord models and data"
-      task :load_sample => ["activerecord:load_sample:models", "activerecord:load_sample:data"]
-      namespace :load_sample do
-        desc "Loads sample ActiveRecord models"
-        task :models do
-          copy_models(:activerecord)
-          copy_migrations(:activerecord)
-        end
-
-        desc "Loads sample ActiveRecord data"
-        task :data do
-          Rake::Task["db:migrate"].reenable
-          Rake::Task["db:migrate"].invoke
-          load_data
-        end
+      desc "Copies sample models into your app"
+      task :copy_sample_models do
+        copy_models(:activerecord)
+      end
+
+      desc "Copies sample migrations into your app"
+      task :copy_sample_migrations do
+        copy_migrations(:activerecord)
+      end
+
+      desc "Migrate the database to the latest version"
+      task :migrate do
+        Rake::Task["db:migrate"].reenable
+        Rake::Task["db:migrate"].invoke
       end
     end
 
+    desc "Copies sample models, runs sample migrations, and loads sample data into your app"
+    task :datamapper => ["datamapper:copy_sample_models", "datamapper:migrate", "load_sample_data"]
     namespace :datamapper do
-      desc "Loads sample DataMapper models and data"
-      task :load_sample => ["datamapper:load_sample:models", "datamapper:load_sample:data"]
-      namespace :load_sample do
-        desc "Loads sample DataMapper models"
-        task :models do
-          copy_models(:datamapper)
-        end
-
-        desc "Loads sample DataMapper data"
-        task :data do
-          Rake::Task["db:automigrate"].reenable
-          Rake::Task["db:automigrate"].invoke
-          load_data
-        end
+      desc "Copies sample models into your app"
+      task :copy_sample_models do
+        copy_models(:datamapper)
       end
+
+      desc "Perform non destructive automigration"
+      task :migrate do
+        Rake::Task["db:automigrate"].reenable
+        Rake::Task["db:automigrate"].invoke
+      end
+    end
+
+    desc "Loads sample data into your app"
+    task :load_sample_data do
+      load_data
     end
 
   end
@@ -79,6 +82,7 @@ def load_data
       team = MerbAdmin::AbstractModel.new("Team").create(:name => mlb_team.name, :logo_url => mlb_team.logo_url, :manager => mlb_team.manager, :ballpark => mlb_team.ballpark, :mascot => mlb_team.mascot, :founded => mlb_team.founded, :wins => mlb_team.wins, :losses => mlb_team.losses, :win_percentage => ("%.3f" % (mlb_team.wins.to_f / (mlb_team.wins + mlb_team.losses))).to_f, :division => division, :league => league)
     end
     mlb_team.players.each do |player|
+      next if player.number.nil?
       MerbAdmin::AbstractModel.new("Player").create(:name => player.name, :number => player.number, :position => player.position, :team => team)
     end
   end
@@ -102,7 +106,7 @@ def copy_migrations(orm = nil)
   orm ||= set_orm
   puts "Copying sample #{orm} migrations into host application - resolves any collisions"
   seen, copied, duplicated = [], [], []
-  Dir.glob(File.dirname(__FILE__) / ".." / ".." / "schema" / "migrations" / "*.rb").each do |source_filename|
+  Dir.glob(File.dirname(__FILE__) / ".." / ".." / "spec" / "migrations" / orm.to_s.downcase / "*.rb").each do |source_filename|
     destination_filename = Merb.root / "schema" / "migrations" / File.basename(source_filename)
     next if seen.include?(source_filename)
     mirror_file(source_filename, destination_filename, copied, duplicated)

data/spec/migrations/activerecord/001_create_divisions_migration.rb

@@ -0,0 +1,13 @@
+class CreateDivisionsMigration < ActiveRecord::Migration
+  def self.up
+    create_table(:divisions) do |t|
+      t.timestamps
+      t.integer(:league_id)
+      t.string(:name, :limit => 50, :null => false)
+    end
+  end
+
+  def self.down
+    drop_table(:divisions)
+  end
+end

data/spec/migrations/activerecord/002_create_drafts_migration.rb

@@ -0,0 +1,19 @@
+class CreateDraftsMigration < ActiveRecord::Migration
+  def self.up
+    create_table(:drafts) do |t|
+      t.timestamps
+      t.integer(:player_id)
+      t.integer(:team_id)
+      t.date(:date)
+      t.integer(:round)
+      t.integer(:pick)
+      t.integer(:overall)
+      t.string(:college, :limit => 100)
+      t.text(:notes)
+    end
+  end
+
+  def self.down
+    drop_table(:drafts)
+  end
+end

data/spec/migrations/activerecord/003_create_leagues_migration.rb

@@ -0,0 +1,12 @@
+class CreateLeaguesMigration < ActiveRecord::Migration
+  def self.up
+    create_table(:leagues) do |t|
+      t.timestamps
+      t.string(:name, :limit => 50, :null => false)
+    end
+  end
+
+  def self.down
+    drop_table(:leagues)
+  end
+end

data/spec/migrations/activerecord/004_create_players_migration.rb

@@ -0,0 +1,20 @@
+class CreatePlayersMigration < ActiveRecord::Migration
+  def self.up
+    create_table(:players) do |t|
+      t.timestamps
+      t.datetime(:deleted_at)
+      t.integer(:team_id)
+      t.string(:name, :limit => 100, :null => false)
+      t.string(:position, :limit => 50)
+      t.integer(:number, :null => false)
+      t.boolean(:retired, :default => false)
+      t.boolean(:injured, :default => false)
+      t.date(:born_on)
+      t.text(:notes)
+    end
+  end
+
+  def self.down
+    drop_table(:players)
+  end
+end

data/spec/migrations/activerecord/005_create_teams_migration.rb

@@ -0,0 +1,22 @@
+class CreateTeamsMigration < ActiveRecord::Migration
+  def self.up
+    create_table(:teams) do |t|
+      t.timestamps
+      t.integer(:league_id)
+      t.integer(:division_id)
+      t.string(:name, :limit => 50, :null => false)
+      t.string(:logo_url, :limit => 255)
+      t.string(:manager, :limit => 100, :null => false)
+      t.string(:ballpark, :limit => 100)
+      t.string(:mascot, :limit => 100)
+      t.integer(:founded)
+      t.integer(:wins)
+      t.integer(:losses)
+      t.float(:win_percentage)
+    end
+  end
+
+  def self.down
+    drop_table(:teams)
+  end
+end

data/spec/models/activerecord/division.rb

@@ -1,7 +1,7 @@
 class Division < ActiveRecord::Base
-  validates_numericality_of :league_id, :only_integer => true
-  validates_presence_of :name
+  validates_numericality_of(:league_id, :only_integer => true)
+  validates_presence_of(:name)
 
-  belongs_to :league
-  has_many :teams
+  belongs_to(:league)
+  has_many(:teams)
 end

data/spec/models/activerecord/draft.rb

@@ -1,11 +1,11 @@
 class Draft < ActiveRecord::Base
-  validates_numericality_of :player_id, :only_integer => true
-  validates_numericality_of :team_id, :only_integer => true
-  validates_presence_of :date
-  validates_numericality_of :round, :only_integer => true
-  validates_numericality_of :pick, :only_integer => true
-  validates_numericality_of :overall, :only_integer => true
+  validates_numericality_of(:player_id, :only_integer => true)
+  validates_numericality_of(:team_id, :only_integer => true)
+  validates_presence_of(:date)
+  validates_numericality_of(:round, :only_integer => true)
+  validates_numericality_of(:pick, :only_integer => true)
+  validates_numericality_of(:overall, :only_integer => true)
 
-  belongs_to :team
-  belongs_to :player
+  belongs_to(:team)
+  belongs_to(:player)
 end

data/spec/models/activerecord/league.rb

@@ -1,6 +1,6 @@
 class League < ActiveRecord::Base
-  validates_presence_of :name
+  validates_presence_of(:name)
 
-  has_many :divisions
-  has_many :teams
+  has_many(:divisions)
+  has_many(:teams)
 end

data/spec/models/activerecord/player.rb

@@ -1,8 +1,8 @@
 class Player < ActiveRecord::Base
-  validates_presence_of :name
-  validates_numericality_of :number, :only_integer => true
-  validates_uniqueness_of :number, :scope => :team_id, :message => "There is already a player with that number on this team"
+  validates_presence_of(:name)
+  validates_numericality_of(:number, :only_integer => true)
+  validates_uniqueness_of(:number, :scope => :team_id, :message => "There is already a player with that number on this team")
 
-  belongs_to :team
-  has_one :draft
+  belongs_to(:team)
+  has_one(:draft)
 end

data/spec/models/activerecord/team.rb

@@ -1,14 +1,14 @@
 class Team < ActiveRecord::Base
-  validates_numericality_of :league_id, :only_integer => true
-  validates_numericality_of :division_id, :only_integer => true
-  validates_presence_of :name
-  validates_presence_of :manager
-  validates_numericality_of :founded, :only_integer => true
-  validates_numericality_of :wins, :only_integer => true
-  validates_numericality_of :losses, :only_integer => true
-  validates_numericality_of :win_percentage
+  validates_numericality_of(:league_id, :only_integer => true)
+  validates_numericality_of(:division_id, :only_integer => true)
+  validates_presence_of(:name)
+  validates_presence_of(:manager)
+  validates_numericality_of(:founded, :only_integer => true)
+  validates_numericality_of(:wins, :only_integer => true)
+  validates_numericality_of(:losses, :only_integer => true)
+  validates_numericality_of(:win_percentage)
 
-  belongs_to :league
-  belongs_to :division
-  has_many :players
+  belongs_to(:league)
+  belongs_to(:division)
+  has_many(:players)
 end

data/spec/models/datamapper/division.rb

@@ -1,12 +1,12 @@
 class Division
   include DataMapper::Resource
 
-  property :id, Serial
-  property :created_at, DateTime
-  property :updated_at, DateTime
-  property :league_id, Integer, :nullable => false, :index => true
-  property :name, String, :nullable => false, :index => true
+  property(:id, Serial)
+  property(:created_at, DateTime)
+  property(:updated_at, DateTime)
+  property(:league_id, Integer, :nullable => false, :index => true)
+  property(:name, String, :nullable => false, :index => true)
 
-  belongs_to :league
-  has n, :teams
+  belongs_to(:league)
+  has(n, :teams)
 end

data/spec/models/datamapper/draft.rb

@@ -1,18 +1,18 @@
 class Draft
   include DataMapper::Resource
 
-  property :id, Serial
-  property :created_at, DateTime
-  property :updated_at, DateTime
-  property :player_id, Integer, :nullable => false, :index => true
-  property :team_id, Integer, :nullable => false, :index => true
-  property :date, Date, :nullable => false
-  property :round, Integer, :nullable => false
-  property :pick, Integer, :nullable => false
-  property :overall, Integer, :nullable => false
-  property :college, String, :length => 100, :index => true
-  property :notes, Text
+  property(:id, Serial)
+  property(:created_at, DateTime)
+  property(:updated_at, DateTime)
+  property(:player_id, Integer, :nullable => false, :index => true)
+  property(:team_id, Integer, :nullable => false, :index => true)
+  property(:date, Date, :nullable => false)
+  property(:round, Integer, :nullable => false)
+  property(:pick, Integer, :nullable => false)
+  property(:overall, Integer, :nullable => false)
+  property(:college, String, :length => 100, :index => true)
+  property(:notes, Text)
 
-  belongs_to :team
-  belongs_to :player
+  belongs_to(:team)
+  belongs_to(:player)
 end

data/spec/models/datamapper/league.rb

@@ -1,11 +1,11 @@
 class League
   include DataMapper::Resource
 
-  property :id, Serial
-  property :created_at, DateTime
-  property :updated_at, DateTime
-  property :name, String, :nullable => false, :index => true
+  property(:id, Serial)
+  property(:created_at, DateTime)
+  property(:updated_at, DateTime)
+  property(:name, String, :nullable => false, :index => true)
 
-  has n, :divisions
-  has n, :teams
+  has(n, :divisions)
+  has(n, :teams)
 end

data/spec/models/datamapper/player.rb

@@ -1,21 +1,21 @@
 class Player
   include DataMapper::Resource
 
-  property :id, Serial
-  property :created_at, DateTime
-  property :updated_at, DateTime
-  property :deleted_at, ParanoidDateTime
-  property :team_id, Integer, :index => true
-  property :name, String, :length => 100, :nullable => false, :index => true
-  property :position, String, :index => true
-  property :number, Integer, :nullable => false
-  property :retired, Boolean, :default => false
-  property :injured, Boolean, :default => false
-  property :born_on, Date
-  property :notes, Text
+  property(:id, Serial)
+  property(:created_at, DateTime)
+  property(:updated_at, DateTime)
+  property(:deleted_at, ParanoidDateTime)
+  property(:team_id, Integer, :index => true)
+  property(:name, String, :length => 100, :nullable => false, :index => true)
+  property(:position, String, :index => true)
+  property(:number, Integer, :nullable => false)
+  property(:retired, Boolean, :default => false)
+  property(:injured, Boolean, :default => false)
+  property(:born_on, Date)
+  property(:notes, Text)
 
-  validates_is_unique :number, :scope => :team_id, :message => "There is already a player with that number on this team"
+  validates_is_unique(:number, :scope => :team_id, :message => "There is already a player with that number on this team")
 
-  belongs_to :team
-  has 1, :draft
+  belongs_to(:team)
+  has(1, :draft)
 end

data/spec/models/datamapper/team.rb

@@ -1,22 +1,22 @@
 class Team
   include DataMapper::Resource
 
-  property :id, Serial
-  property :created_at, DateTime
-  property :updated_at, DateTime
-  property :league_id, Integer, :nullable => false, :index => true
-  property :division_id, Integer, :nullable => false, :index => true
-  property :name, String, :nullable => false, :index => true
-  property :logo_url, String, :length => 255
-  property :manager, String, :length => 100, :nullable => false, :index => true
-  property :ballpark, String, :length => 100, :index => true
-  property :mascot, String, :length => 100, :index => true
-  property :founded, Integer, :nullable => false
-  property :wins, Integer, :nullable => false
-  property :losses, Integer, :nullable => false
-  property :win_percentage, Float, :nullable => false, :precision => 4, :scale => 3
+  property(:id, Serial)
+  property(:created_at, DateTime)
+  property(:updated_at, DateTime)
+  property(:league_id, Integer, :nullable => false, :index => true)
+  property(:division_id, Integer, :nullable => false, :index => true)
+  property(:name, String, :nullable => false, :index => true)
+  property(:logo_url, String, :length => 255)
+  property(:manager, String, :length => 100, :nullable => false, :index => true)
+  property(:ballpark, String, :length => 100, :index => true)
+  property(:mascot, String, :length => 100, :index => true)
+  property(:founded, Integer, :nullable => false)
+  property(:wins, Integer, :nullable => false)
+  property(:losses, Integer, :nullable => false)
+  property(:win_percentage, Float, :nullable => false, :precision => 4, :scale => 3)
 
-  belongs_to :league
-  belongs_to :division
-  has n, :players
+  belongs_to(:league)
+  belongs_to(:division)
+  has(n, :players)
 end

data/spec/spec_helper.rb

@@ -44,23 +44,20 @@ module Merb
         when :activerecord
           require 'activerecord'
           require_models(orm)
-
           unless ActiveRecord::Base.connected?
             ActiveRecord::Base.establish_connection(:adapter => 'sqlite3', :database => ':memory:')
             ActiveRecord::Migration.verbose = false
-            ActiveRecord::Migrator.run(:up, "schema/migrations/", 1)
-            ActiveRecord::Migrator.run(:up, "schema/migrations/", 2)
-            ActiveRecord::Migrator.run(:up, "schema/migrations/", 3)
-            ActiveRecord::Migrator.run(:up, "schema/migrations/", 4)
-            ActiveRecord::Migrator.run(:up, "schema/migrations/", 5)
+            ActiveRecord::Migrator.run(:up, File.join(File.dirname(__FILE__), "migrations", "activerecord"), 1)
+            ActiveRecord::Migrator.run(:up, File.join(File.dirname(__FILE__), "migrations", "activerecord"), 2)
+            ActiveRecord::Migrator.run(:up, File.join(File.dirname(__FILE__), "migrations", "activerecord"), 3)
+            ActiveRecord::Migrator.run(:up, File.join(File.dirname(__FILE__), "migrations", "activerecord"), 4)
+            ActiveRecord::Migrator.run(:up, File.join(File.dirname(__FILE__), "migrations", "activerecord"), 5)
           end
         when :datamapper
           require 'dm-core'
-          require 'dm-types'
           require 'dm-aggregates'
           require 'dm-validations'
           require_models(orm)
-
           unless DataMapper::Repository.adapters.key?(:default)
             DataMapper.setup(:default, 'sqlite3::memory:')
             DataMapper.auto_migrate!

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: merb-admin
 version: !ruby/object:Gem::Version 
-  version: 0.5.6
+  version: 0.5.7
 platform: ruby
 authors: 
 - Erik Michaels-Ober
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-10-18 00:00:00 -07:00
+date: 2009-10-26 00:00:00 -07:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -152,12 +152,12 @@ files:
 - public/stylesheets/patch-iewin.css
 - public/stylesheets/rtl.css
 - public/stylesheets/widgets.css
-- schema/migrations/001_create_divisions_migration.rb
-- schema/migrations/002_create_drafts_migration.rb
-- schema/migrations/003_create_leagues_migration.rb
-- schema/migrations/004_create_players_migration.rb
-- schema/migrations/005_create_teams_migration.rb
 - spec/controllers/main_spec.rb
+- spec/migrations/activerecord/001_create_divisions_migration.rb
+- spec/migrations/activerecord/002_create_drafts_migration.rb
+- spec/migrations/activerecord/003_create_leagues_migration.rb
+- spec/migrations/activerecord/004_create_players_migration.rb
+- spec/migrations/activerecord/005_create_teams_migration.rb
 - spec/models/activerecord/division.rb
 - spec/models/activerecord/draft.rb
 - spec/models/activerecord/league.rb

data/schema/migrations/001_create_divisions_migration.rb

@@ -1,15 +0,0 @@
-class CreateDivisionsMigration < ActiveRecord::Migration
-  def self.up
-    create_table :divisions do |t|
-      t.timestamps
-      t.integer :league_id
-      t.string :name, :limit => 50, :null => false
-    end
-    add_index :divisions, :league_id
-    add_index :divisions, :name
-  end
-
-  def self.down
-    drop_table :divisions
-  end
-end

data/schema/migrations/002_create_drafts_migration.rb

@@ -1,22 +0,0 @@
-class CreateDraftsMigration < ActiveRecord::Migration
-  def self.up
-    create_table :drafts do |t|
-      t.timestamps
-      t.integer :player_id
-      t.integer :team_id
-      t.date :date
-      t.integer :round
-      t.integer :pick
-      t.integer :overall
-      t.string :college, :limit => 100
-      t.text :notes
-    end
-    add_index :drafts, :player_id
-    add_index :drafts, :team_id
-    add_index :drafts, :college
-  end
-
-  def self.down
-    drop_table :drafts
-  end
-end

data/schema/migrations/003_create_leagues_migration.rb

@@ -1,13 +0,0 @@
-class CreateLeaguesMigration < ActiveRecord::Migration
-  def self.up
-    create_table :leagues do |t|
-      t.timestamps
-      t.string :name, :limit => 50, :null => false
-    end
-    add_index :leagues, :name
-  end
-
-  def self.down
-    drop_table :leagues
-  end
-end

data/schema/migrations/004_create_players_migration.rb

@@ -1,23 +0,0 @@
-class CreatePlayersMigration < ActiveRecord::Migration
-  def self.up
-    create_table :players do |t|
-      t.timestamps
-      t.datetime :deleted_at
-      t.integer :team_id
-      t.string :name, :limit => 100, :null => false
-      t.string :position, :limit => 50
-      t.integer :number, :null => false
-      t.boolean :retired, :default => false
-      t.boolean :injured, :default => false
-      t.date :born_on
-      t.text :notes
-    end
-    add_index :players, :team_id
-    add_index :players, :name
-    add_index :players, :position
-  end
-
-  def self.down
-    drop_table :players
-  end
-end

data/schema/migrations/005_create_teams_migration.rb

@@ -1,28 +0,0 @@
-class CreateTeamsMigration < ActiveRecord::Migration
-  def self.up
-    create_table :teams do |t|
-      t.timestamps
-      t.integer :league_id
-      t.integer :division_id
-      t.string :name, :limit => 50, :null => false
-      t.string :logo_url, :limit => 255
-      t.string :manager, :limit => 100, :null => false
-      t.string :ballpark, :limit => 100
-      t.string :mascot, :limit => 100
-      t.integer :founded
-      t.integer :wins
-      t.integer :losses
-      t.float :win_percentage
-    end
-    add_index :teams, :division_id
-    add_index :teams, :league_id
-    add_index :teams, :name
-    add_index :teams, :manager
-    add_index :teams, :ballpark
-    add_index :teams, :mascot
-  end
-
-  def self.down
-    drop_table :teams
-  end
-end