3scale/porta

class TablesForBeast < ActiveRecord::Migration
Method `up` has 72 lines of code (exceeds 25 allowed). Consider refactoring.
TablesForBeast#self.up has approx 66 statements
  def self.up 
TablesForBeast#self.up has the variable name 't'
   create_table "forums", :force => true do |t|
TablesForBeast#self.up calls 't.integer "site_id"' 3 times
    t.integer "site_id"
TablesForBeast#self.up calls 't.string  "name"' 2 times
    t.string  "name"
    t.string  "description"
TablesForBeast#self.up calls 't.integer "topics_count",     :default => 0' 2 times
    t.integer "topics_count",     :default => 0
TablesForBeast#self.up calls 't.integer "posts_count",      :default => 0' 3 times
    t.integer "posts_count",      :default => 0
    t.integer "position",         :default => 0
    t.text    "description_html"
    t.string  "state",            :default => "public"
TablesForBeast#self.up calls 't.string  "permalink"' 2 times
    t.string  "permalink"
  end
 
  add_index "forums", ["position", "site_id"], :name => "index_forums_on_position_and_site_id"
  add_index "forums", ["site_id", "permalink"], :name => "index_forums_on_site_id_and_permalink"
 
  create_table "moderatorships", :force => true do |t|
TablesForBeast#self.up calls 't.integer  "forum_id"' 3 times
    t.integer  "forum_id"
TablesForBeast#self.up calls 't.integer  "user_id"' 4 times
    t.integer  "user_id"
TablesForBeast#self.up calls 't.datetime "created_at"' 5 times
    t.datetime "created_at"
TablesForBeast#self.up calls 't.datetime "updated_at"' 5 times
    t.datetime "updated_at"
  end
 
Identical blocks of code found in 2 locations. Consider refactoring.
  create_table "monitorships", :force => true do |t|
    t.integer  "user_id"
TablesForBeast#self.up calls 't.integer  "topic_id"' 2 times
    t.integer  "topic_id"
    t.datetime "created_at"
    t.datetime "updated_at"
    t.boolean  "active",     :default => true
  end
 
Similar blocks of code found in 5 locations. Consider refactoring.
  create_table "posts", :force => true do |t|
    t.integer  "user_id"
    t.integer  "topic_id"
    t.text     "body"
    t.datetime "created_at"
    t.datetime "updated_at"
    t.integer  "forum_id"
    t.text     "body_html"
    t.integer  "site_id"
  end
 
  add_index "posts", ["created_at", "forum_id"], :name => "index_posts_on_forum_id"
  add_index "posts", ["created_at", "user_id"], :name => "index_posts_on_user_id"
  add_index "posts", ["created_at", "topic_id"], :name => "index_posts_on_topic_id"
 
  create_table "sites", :force => true do |t|
    t.string   "name"
    t.string   "host"
    t.datetime "created_at"
    t.datetime "updated_at"
    t.integer  "topics_count", :default => 0
    t.integer  "users_count",  :default => 0
    t.integer  "posts_count",  :default => 0
  end
 
  create_table "topics", :force => true do |t|
    t.integer  "forum_id"
    t.integer  "user_id"
    t.string   "title"
    t.datetime "created_at"
    t.datetime "updated_at"
    t.integer  "hits",            :default => 0
    t.integer  "sticky",          :default => 0
    t.integer  "posts_count",     :default => 0
    t.boolean  "locked",          :default => false
    t.integer  "last_post_id"
    t.datetime "last_updated_at"
    t.integer  "last_user_id"
    t.integer  "site_id"
    t.string   "permalink"
  end
 
  add_index "topics", ["sticky", "last_updated_at", "forum_id"], :name => "index_topics_on_sticky_and_last_updated_at"
  add_index "topics", ["last_updated_at", "forum_id"], :name => "index_topics_on_forum_id_and_last_updated_at"
  add_index "topics", ["forum_id", "permalink"], :name => "index_topics_on_forum_id_and_permalink"
 
 
  add_column :users, :site_id, :integer
  add_column :users, :posts_count, :integer, :default => 0
  add_column :users, :last_seen_at, :datetime
  
 #   t.string   "login"
#    t.string   "email"
#    t.string   "crypted_password",          :limit => 40
 #   t.string   "salt",                      :limit => 40
 #   t.datetime "created_at"
#    t.datetime "updated_at"
#   t.string   "remember_token"
#    t.datetime "remember_token_expires_at"
#    t.string   "activation_code",           :limit => 40
#    t.datetime "activated_at"
#    t.string   "state",                                   :default => "passive"
#    t.datetime "deleted_at"
#   t.boolean  "admin",                                   :default => false
#    t.integer  "site_id"
#    t.datetime "last_login_at"
#    t.text     "bio_html"
#    t.string   "openid_url"
#    t.datetime "last_seen_at"
#   t.string   "website"
#    t.integer  "posts_count",                             :default => 0
#    t.string   "bio"
#    t.string   "display_name"
#    t.string   "permalink"
 
  add_index "users", ["last_seen_at"], :name => "index_users_on_last_seen_at"
  add_index "users", ["site_id", "posts_count"], :name => "index_site_users_on_posts_count"
#  add_index "users", ["site_id", "permalink"], :name => "index_site_users_on_permalink"
   
  end
 
TablesForBeast#self.down has approx 9 statements
  def self.down
    
    drop_table :forums
    drop_table :moderatorships
    drop_table :monitorships
    drop_table :posts
    drop_table :sites
    drop_table :topics
    
    remove_column :users, :site_id
    remove_column :users, :posts_count
    remove_column :users, :last_seen_at
    
  end
end