Capture email opt-in IP

ee/app/controllers/ee/registrations_controller.rb

 module EE
   module RegistrationsController
     extend ActiveSupport::Concern
-    
+
     private
 
     def sign_up_params
-      params.require(:user).permit(:username, :email, :email_confirmation, :name, :password, :email_opted_in)
+      clean_params = params.require(:user).permit(:username, :email, :email_confirmation, :name, :password, :email_opted_in)
+
+      clean_params[:email_opted_in_ip] = clean_params[:email_opted_in] == '1' ? request.remote_ip : nil
+
+      clean_params
     end
   end
 end

ee/app/services/ee/users/build_service.rb

@@ -3,7 +3,7 @@ module EE
     module BuildService
 
       private
-      
+
       def signup_params
         [
           :email,
@@ -12,7 +12,8 @@ module EE
           :name,
           :password,
           :username,
-          :email_opted_in
+          :email_opted_in,
+          :email_opted_in_ip
         ]
       end
     end

spec/ee/spec/controllers/ee/registrations_controller_spec.rb

+require 'spec_helper'
+
+describe RegistrationsController do
+  describe '#create' do
+    let(:user_params) { { user: { name: 'new_user', username: 'new_username', email: 'new@user.com', password: 'Any_password', email_opted_in: email_opted_in } } }
+
+    context 'when the user opted-in' do
+      let(:email_opted_in) { '1' }
+
+      it 'sets email_opted_in_ip to an IP' do
+        post :create, user_params
+        user = User.find_by_username('new_username')
+        expect(user.email_opted_in_ip).to be_present
+      end
+    end
+
+    context 'when the user opted-out' do
+      let(:email_opted_in) { '0' }
+
+      it 'sets email_opted_in_ip to nil' do
+        post :create, user_params
+        user = User.find_by_username('new_username')
+        expect(user.email_opted_in_ip).to be_nil
+      end
+    end
+  end
+end

spec/ee/spec/features/signup_spec.rb

@@ -22,6 +22,7 @@ feature 'Signup on EE' do
 
         user = User.find_by_username!(user.username)
         expect(user.email_opted_in).to be_truthy
+        expect(user.email_opted_in_ip).to be_present
       end
     end
 
@@ -40,6 +41,7 @@ feature 'Signup on EE' do
 
         user = User.find_by_username!(user.username)
         expect(user.email_opted_in).to be_falsey
+        expect(user.email_opted_in_ip).to be_nil
       end
     end
   end
@@ -65,6 +67,7 @@ feature 'Signup on EE' do
 
       user = User.find_by_username!(user.username)
       expect(user.email_opted_in).to be_falsey
+      expect(user.email_opted_in_ip).to be_nil
     end
   end
 end