summaryrefslogtreecommitdiff
path: root/users
diff options
context:
space:
mode:
Diffstat (limited to 'users')
-rw-r--r--users/app/assets/javascripts/users.js.coffee22
-rw-r--r--users/app/controllers/controller_extension/authentication.rb1
-rw-r--r--users/lib/warden/strategies/secure_remote_password.rb4
3 files changed, 21 insertions, 6 deletions
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