From a01e7686ea7c046a9cd544b618b30727f2a41b3b Mon Sep 17 00:00:00 2001
From: jessib <jessib@leap.se>
Date: Thu, 27 Jun 2013 12:31:39 -0700
Subject: Want to tweak some, but start to displaying base generic message via
 javascript.

---
 users/app/assets/javascripts/users.js.coffee       | 22 ++++++++++++++++++----
 .../controller_extension/authentication.rb         |  1 +
 .../warden/strategies/secure_remote_password.rb    |  4 ++--
 3 files changed, 21 insertions(+), 6 deletions(-)

(limited to 'users')

diff --git a/users/app/assets/javascripts/users.js.coffee b/users/app/assets/javascripts/users.js.coffee
index 955556c..b0b624c 100644
--- a/users/app/assets/javascripts/users.js.coffee
+++ b/users/app/assets/javascripts/users.js.coffee
@@ -14,13 +14,27 @@ srp.updated = ->
 
 srp.error = (message) ->
   if $.isPlainObject(message) && message.errors
-    for field, error of message.errors
-      element = $('form input[name$="['+field+']"]')
-      next unless element
-      element.trigger('element:validate:fail.ClientSideValidations', error).data('valid', false)
+    display_errors(message.errors)
   else
     alert(message)
 
+display_errors = (errors) ->
+  for field, error of errors
+    if field == 'base'
+      display_base_error(error);
+    else
+      display_field_error(field, error);
+
+display_field_error = (field, error) ->
+  element = $('form input[name$="['+field+']"]')
+  return unless element
+  element.trigger('element:validate:fail.ClientSideValidations', error).data('valid', false)
+
+display_base_error = (message) ->
+  messages = $('#messages')
+  messages.append "<div class=\"alert alert-error\"><a class=\"close\" \"data-dismiss\"=\"alert\">×</a><div class=\"flash_error\">" + message + "</div></div>"
+
+
 pollUsers = (query, process) ->
   $.get( "/users.json", query: query).done(process)
 
diff --git a/users/app/controllers/controller_extension/authentication.rb b/users/app/controllers/controller_extension/authentication.rb
index f0a6564..0dca29c 100644
--- a/users/app/controllers/controller_extension/authentication.rb
+++ b/users/app/controllers/controller_extension/authentication.rb
@@ -10,6 +10,7 @@ module ControllerExtension::Authentication
   def authentication_errors
     return unless attempted_login?
     errors = get_warden_errors
+    #default response to get_warden_errors is not an enumerable, so won't work if default is used
     errors.inject({}) do |translated,err|
       translated[err.first] = I18n.t(err.last)
       translated
diff --git a/users/lib/warden/strategies/secure_remote_password.rb b/users/lib/warden/strategies/secure_remote_password.rb
index f1b1a57..2dcd706 100644
--- a/users/lib/warden/strategies/secure_remote_password.rb
+++ b/users/lib/warden/strategies/secure_remote_password.rb
@@ -28,7 +28,7 @@ module Warden
         if client = validate
           success!(User.find_by_login(client.username))
         else
-          fail!({:login => "invalid_user_pass", :password => "invalid_user_pass"})
+          fail!(:base => "invalid_user_pass")
         end
       end
 
@@ -44,7 +44,7 @@ module Warden
           session[:handshake] = SRP::Session.new(client, params['A'].hex)
           custom! json_response(session[:handshake])
         else
-          fail!({:login => "invalid_user_pass", :password => "invalid_user_pass"})
+          fail! :base => 'invalid_user_pass'
         end
       end
 
-- 
cgit v1.2.3