support different email for signin

barkeep_server.rb

@@ -173,7 +173,7 @@ def ensure_required_params(*required_params)
 
   before do
     # When running in read-only demo mode, if the user is not logged in, treat them as a demo user.
-    self.current_user ||= User.find(:email => session[:email])
+    self.current_user ||= User.find(:login => session[:login])
     if current_user.nil? && (defined?(ENABLE_READONLY_DEMO_MODE) && ENABLE_READONLY_DEMO_MODE)
       self.current_user = User.first(:permission => "demo")
       current_user.rack_session = session
@@ -245,11 +245,11 @@ def ensure_required_params(*required_params)
           halt 401, "Your email #{email} is not authorized to login to Barkeep."
         end
       end
-      session[:email] = email
-      unless User.find(:email => email)
+      session[:login] = email
+      unless User.find(:login => email)
         # If there are no admin users yet, make the first user to log in the first admin.
         permission = User.find(:permission => "admin").nil? ? "admin" : "normal"
-        User.new(:email => email, :name => email, :permission => permission).save
+        User.new(:login => email, :email => email, :name => email, :permission => permission).save
       end
       redirect session[:login_started_url] || "/"
     end
@@ -264,7 +264,10 @@ def ensure_required_params(*required_params)
 
   put "/settings/:preference" do
     preference = params[:preference]
-    if preference == "displayname"
+    if preference == "email"
+      current_user.email = params[:value]
+      current_user.save
+    elsif preference == "displayname"
       current_user.name = params[:value]
       current_user.save
     elsif ["line_length", "default_to_side_by_side"].include? preference

migrations/20130521220244_add_login_column_to_users.rb

@@ -0,0 +1,20 @@
+require "bundler/setup"
+require "pathological"
+require "migrations/migration_helper.rb"
+
+Sequel.migration do
+  up do
+    alter_table(:users) {
+	add_column :login, String
+        add_index :login, :unique => true, :name => "login_is_unique_per_user"
+    }
+    run 'UPDATE users SET login=email'
+  end
+
+  down do
+    alter_table(:users) {
+      drop_index :login, :name => "login_is_unique_per_user"
+      drop_column :login
+    }
+  end
+end

public/coffee/user_settings.coffee

@@ -1,6 +1,7 @@
 window.Settings =
   init: ->
     $("#editDisplayName").on "click", (e) => @makeTextEditable(e)
+    $("#editEmail").on "click", (e) => @makeTextEditable(e)
     $("#lineLengthSlider").slider
       value: parseInt($("#lineLengthSlider").attr("data-value"))
       min: 80

views/user_settings.erb

@@ -13,7 +13,15 @@
           </span>
         </td>
       </tr>
-      <tr><td>Email</td><td><%= current_user.email %></td></tr>
+      <tr><td>Login</td><td><%= current_user.login %></td></tr>
+      <tr>
+        <td>Email</td>
+        <td>
+          <span id="editEmail" class="editable west" rel="tipsy" title="Edit">
+            <%= current_user.email %>
+          </span>
+        </td>
+      </tr>
       <tr>
         <td>Gravatar</td>
         <td>