Merge pull request #20 from cblanc/drop_rcon

Drop RCON (danger!)

README.md

@@ -27,7 +27,7 @@ Features:
 - Map database
 - Votable polls
 - Twitter feed
-- Server database and RCON interface
+- Server database
 - Log file parsing (partially complete)
 
 ## Contributors

app/models/log.rb

@@ -21,8 +21,6 @@ class Log < ActiveRecord::Base
   attr_accessor :text
 
   DOMAIN_LOG = 1
-  DOMAIN_RCON_COMMAND = 2
-  DOMAIN_RCON_RESPONSE = 3
   DOMAIN_INFO = 4
 
   TEAM_MARINES = 1
@@ -52,10 +50,6 @@ class Log < ActiveRecord::Base
   belongs_to :actor, :class_name => "Rounder"
   belongs_to :target, :class_name => "Rounder"
 
-  #	def domains
-  #		return {DOMAIN_LOG => "HL Log", DOMAIN_RCON_COMMAND => "Rcon Command Log", DOMAIN_RCON_RESPONSE => "Rcon Response Log", DOMAIN_INFO => "Action Log"}
-  #	end
-
   def since
     (created_at - round.start).to_i
   end

app/models/server.rb

@@ -8,7 +8,6 @@
 #  dns             :string(255)
 #  ip              :string(255)
 #  port            :string(255)
-#  rcon            :string(255)
 #  password        :string(255)
 #  irc             :string(255)
 #  user_id         :integer
@@ -44,7 +43,7 @@ class Server < ActiveRecord::Base
   attr_protected :id, :user_id, :updated_at, :created_at, :map, :players, :maxplayers, :ping, :version
 
   validates_length_of [:name, :dns,], :in => 1..30
-  validates_length_of [:rcon, :password, :irc], :maximum => 30, :allow_blank => true
+  validates_length_of [:password, :irc], :maximum => 30, :allow_blank => true
   validates_length_of :description, :maximum => 255, :allow_blank => true
   validates_format_of :ip, :with => /\A[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\z/
   validates_format_of :port, :with => /\A[0-9]{1,5}\z/
@@ -81,7 +80,6 @@ class Server < ActiveRecord::Base
   non_versioned_columns << 'dns'
   non_versioned_columns << 'ip'
   non_versioned_columns << 'port'
-  non_versioned_columns << 'rcon'
   non_versioned_columns << 'password'
   non_versioned_columns << 'irc'
   non_versioned_columns << 'user_id'

app/views/servers/_form.html.erb

@@ -16,10 +16,6 @@
       </div>      
     </div>
   </div>
-  <div class="fields horizontal">
-    <%= f.label :rcon %>
-    <%= f.text_field :rcon %>
-  </div>
   <div class="fields horizontal">
     <%= f.label :password %>
     <%= f.text_field :password %>

db/migrate/20150518162749_remove_rcon_from_servers.rb

@@ -0,0 +1,9 @@
+class RemoveRconFromServers < ActiveRecord::Migration
+  def up
+    remove_column :servers, :rcon
+  end
+
+  def down
+    add_column :servers, :rcon, :string
+  end
+end

db/schema.rb

@@ -11,7 +11,7 @@
 #
 # It's strongly recommended to check this file into your version control system.
 
-ActiveRecord::Schema.define(:version => 20141010193221) do
+ActiveRecord::Schema.define(:version => 20150518162749) do
 
   create_table "admin_requests", :force => true do |t|
     t.string   "addr"
@@ -722,7 +722,6 @@ ActiveRecord::Schema.define(:version => 20141010193221) do
     t.string   "dns"
     t.string   "ip"
     t.string   "port"
-    t.string   "rcon"
     t.string   "password"
     t.string   "irc"
     t.integer  "user_id"

spec/models/server_spec.rb

@@ -8,7 +8,6 @@
 #  dns             :string(255)
 #  ip              :string(255)
 #  port            :string(255)
-#  rcon            :string(255)
 #  password        :string(255)
 #  irc             :string(255)
 #  user_id         :integer

spec/support/features/server_helpers.rb

@@ -4,7 +4,6 @@ module Features
       dns = 'ServerDns.com'
       ip = '192.168.1.1'
       port = '8000'
-      rcon = 'whatsrcon'
       password = 'secret'
       name = 'MyNsServer'
       description = 'My NS Server'