Merge pull request #1980 from zz9pzza/issue_4211

issue_4211 Record ip addresses on new works

app/controllers/works_controller.rb

@@ -246,6 +246,7 @@ def new
     @series = current_user.series.uniq
     @unposted = current_user.unposted_work
 
+    @work.ip_address = request.remote_ip
     # for clarity, add the collection and recipient
     if params[:assignment_id] && (@challenge_assignment = ChallengeAssignment.find(params[:assignment_id])) && @challenge_assignment.offering_user == current_user
       @work.challenge_assignments << @challenge_assignment
@@ -280,6 +281,7 @@ def create
     @series = current_user.series.uniq
     @collection = Collection.find_by_name(params[:work][:collection_names])
 
+    @work.ip_address = request.remote_ip
     if params[:edit_button]
       render :new
     elsif params[:cancel_button]

app/views/works/_meta.html.erb

@@ -56,11 +56,23 @@
       </dd>
     <% end %>
 
-    <dt class="stats"><%= ts("Stats") %>:</dt>
+    <dt class="stats"><%= ts("Stats:") %></dt>
     <dd class="stats">
 <% end %><!-- end of cache -->
 
       <%= work_meta_list(@work, @chapter) %>
-    </dd>
+      </dd>
+      <% if logged_in_as_admin? %>
+        <dt class="ip">
+          <%= ts('IP Address:') %>
+        </dt>
+        <dd class="ip">
+          <% if @work.ip_address.nil? %>
+             <%= ts('No address recorded') %>
+          <% else %>
+             <%= @work.ip_address %>
+          <% end %>
+        </dd>
+      <% end %>
   </dl>
 </div>

db/migrate/20150106211421_add_ip_address_to_works.rb

@@ -0,0 +1,11 @@
+class AddIpAddressToWorks < ActiveRecord::Migration
+  def self.up
+    add_column :works, :ip_address, :string
+    add_index "works", "ip_address"
+  end
+
+  def self.down
+    remove_index "works", "ip_address"
+    remove_column :works, :ip_address
+  end
+end