Assign unique identifier to each user

Author: sigmike

app/models/user.rb

@@ -19,6 +19,7 @@ class User < ActiveRecord::Base
   has_many :record_changes
 
   before_create :generate_login_token!, unless: :login_token?
+  before_create :assign_random_identifier, unless: :identifier?
 
   acts_as_commontator
   acts_as_commontable
@@ -125,4 +126,12 @@ def generate_login_token!
     end
   end
 
+  def assign_random_identifier
+    charset = '123456789ABCDEFGHJKLMNPQRSTUVWXYZabcdefghijkmnopqrstuvwxyz'.split(//)
+    loop do
+      self.identifier = (0...12).map { charset.sample }.join
+      break unless User.exists?(identifier: identifier)
+    end
+  end
+
 end

app/views/devise/registrations/edit.html.haml

@@ -3,6 +3,7 @@
     %h2
       Edit #{resource_name.to_s.humanize}
     = devise_error_messages!
+    = f.static_control :identifier
     = f.text_field :name
     = f.email_field :email
     - if devise_mapping.confirmable? && resource.pending_reconfirmation?

app/views/users/show.html.haml

@@ -1,4 +1,8 @@
 %h1= @user.name
+%p
+  %strong Identifier:
+  = @user.identifier
+
 - if @user.nickname.present?
   %p
     %strong GitHub account:

db/migrate/20140714074128_add_identifier_to_user.rb

@@ -0,0 +1,17 @@
+class AddIdentifierToUser < ActiveRecord::Migration
+  def up
+    add_column :users, :identifier, :string
+    execute("SELECT id FROM users WHERE identifier IS NULL").each do |row|
+      id = row["id"]
+      charset = '123456789ABCDEFGHJKLMNPQRSTUVWXYZabcdefghijkmnopqrstuvwxyz'.split(//)
+      identifier = (0...12).map { charset.sample }.join
+      execute "UPDATE users SET identifier='#{identifier}' WHERE id = #{id}"
+    end
+    change_column :users, :identifier, :string, null: false
+    add_index :users, :identifier, unique: true
+  end
+
+  def down
+    remove_column :users, :identifier
+  end
+end

db/schema.rb

@@ -11,7 +11,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20140706075813) do
+ActiveRecord::Schema.define(version: 20140714074128) do
 
   create_table "cold_storage_transfers", force: true do |t|
     t.integer  "project_id"
@@ -222,9 +222,11 @@
     t.string   "email"
     t.string   "nickname"
     t.boolean  "disabled",                         default: false
+    t.string   "identifier",                                       null: false
   end
 
   add_index "users", ["disabled"], name: "index_users_on_disabled"
+  add_index "users", ["identifier"], name: "index_users_on_identifier", unique: true
   add_index "users", ["reset_password_token"], name: "index_users_on_reset_password_token", unique: true
 
 end

features/step_definitions/user_identifier.rb

@@ -0,0 +1,5 @@
+Then(/^I should see the identifier of "(.*?)"$/) do |arg1|
+  identifier = User.find_by(email: arg1).identifier
+  identifier.should be_present
+  page.should have_content identifier
+end

features/step_definitions/web.rb

@@ -131,6 +131,10 @@
   visit url
 end
 
+When(/^I click on "(.*?)" in the email$/) do |arg1|
+  step "I click on the \"#{arg1}\" link in the email"
+end
+
 Then(/^the user with email "(.*?)" should have his email confirmed$/) do |arg1|
   User.find_by(email: arg1).confirmed?.should be_true
 end

features/user_identifier.feature

@@ -0,0 +1,20 @@
+Feature: Each user has an unique identifier
+  Scenario: New email user gets an unique identifier
+    When I visit the home page
+    And I click on "Sign in"
+    And I click on "Sign up"
+    And I fill "Email" with "[email protected]"
+    And I fill "Password" with "password"
+    And I fill "Password confirmation" with "password"
+    And I click on "Sign up"
+    Then I should see "confirmation link"
+
+    And an email should have been sent to "[email protected]"
+    When I click on "Confirm my account" in the email
+    Then I should see "confirmed"
+
+    And I fill "Email" with "[email protected]"
+    And I fill "Password" with "password"
+    And I click on "Sign in" in the sign in form
+    When I go to edit my profile
+    Then I should see the identifier of "[email protected]"