Store origin IP address for every comment

I am also already adding a "spam" flag to the table, which we will use next. (ActionItem2309)

Store origin IP address for every comment
I am also already adding a "spam" flag to the table, which we will use next. (ActionItem2309)
Antonio Terceiro
1 parent 9688304f
Showing 4 changed files with 24 additions and 1 deletions Show diff stats
app/controllers/public/content_viewer_controller.rb
db/migrate/20120307200651_add_ip_address_to_comment.rb
db/schema.rb
test/functional/content_viewer_controller_test.rb
@@ -111,6 +111,7 @@ class ContentViewerController &lt; ApplicationController
   def add_comment
     @comment.author = user if logged_in?
     @comment.article = @page
+    @comment.ip_address = request.remote_ip
     if (pass_without_comment_captcha? || verify_recaptcha(:model => @comment, :message => _('Please type the words correctly'))) && @comment.save
       @page.touch
       @comment = nil # clear the comment form
@@ -0,0 +1,11 @@
+class AddIpAddressToComment < ActiveRecord::Migration
+  def self.up
+    add_column :comments, :ip_address, :string
+    add_column :comments, :spam, :boolean
+  end
+
+  def self.down
+    remove_column :comments, :ip_address
+    remove_column :comments, :boolean
+  end
+end
@@ -9,7 +9,7 @@
 #
 # It's strongly recommended to check this file into your version control system.
-ActiveRecord::Schema.define(:version => 20111004184104) do
+ActiveRecord::Schema.define(:version => 20120307200651) do
   create_table "abuse_reports", :force => true do |t|
     t.integer  "reporter_id"
@@ -204,6 +204,8 @@ ActiveRecord::Schema.define(:version =&gt; 20111004184104) do
     t.string   "email"
     t.datetime "created_at"
     t.integer  "reply_of_id"
+    t.string   "ip_address"
+    t.boolean  "spam"
   end
   create_table "contact_lists", :force => true do |t|
@@ -1370,4 +1370,13 @@ class ContentViewerControllerTest &lt; ActionController::TestCase
     post :view_page, :profile => profile.identifier, :page => ['test'], :comment => {:body => "Some comment...", :author => profile}, :confirm => 'true'
     assert_not_nil assigns(:comment)
   end
+
+  should 'store IP address for comments' do
+    page = profile.articles.create!(:name => 'myarticle', :body => 'the body of the text')
+    @request.stubs(:remote_ip).returns('33.44.55.66')
+    post :view_page, :profile => profile.identifier, :page => [ 'myarticle' ], :comment => { :title => 'title', :body => 'body', :name => "Spammer", :email => 'damn@spammer.com' }, :confirm => 'true'
+    comment = Comment.last
+    assert_equal '33.44.55.66', comment.ip_address
+  end
+
 end
@@ -0,0 +1,11 @@		@@ -0,0 +1,11 @@
	1	+class AddIpAddressToComment < ActiveRecord::Migration
	2	+ def self.up
	3	+ add_column :comments, :ip_address, :string
	4	+ add_column :comments, :spam, :boolean
	5	+ end
	6	+
	7	+ def self.down
	8	+ remove_column :comments, :ip_address
	9	+ remove_column :comments, :boolean
	10	+ end
	11	+end
	@@ -1370,4 +1370,13 @@ class ContentViewerControllerTest < ActionController::TestCase		@@ -1370,4 +1370,13 @@ class ContentViewerControllerTest < ActionController::TestCase
1370	post :view_page, :profile => profile.identifier, :page => ['test'], :comment => {:body => "Some comment...", :author => profile}, :confirm => 'true'	1370	post :view_page, :profile => profile.identifier, :page => ['test'], :comment => {:body => "Some comment...", :author => profile}, :confirm => 'true'
1371	assert_not_nil assigns(:comment)	1371	assert_not_nil assigns(:comment)
1372	end	1372	end
		1373	+
		1374	+ should 'store IP address for comments' do
		1375	+ page = profile.articles.create!(:name => 'myarticle', :body => 'the body of the text')
		1376	+ @request.stubs(:remote_ip).returns('33.44.55.66')
		1377	+ post :view_page, :profile => profile.identifier, :page => [ 'myarticle' ], :comment => { :title => 'title', :body => 'body', :name => "Spammer", :email => 'damn@spammer.com' }, :confirm => 'true'
		1378	+ comment = Comment.last
		1379	+ assert_equal '33.44.55.66', comment.ip_address
		1380	+ end
		1381	+
1373	end	1382	end