diff options
author | Endi S. Dewata <edewata@redhat.com> | 2012-02-08 11:38:39 -0600 |
---|---|---|
committer | Endi S. Dewata <edewata@redhat.com> | 2012-02-09 13:21:12 -0600 |
commit | eba3a341e607b542afabdedf0f6508e7c0a5e96c (patch) | |
tree | 03f682095646e0c2c1e898cd721e064b4d88782b /install | |
parent | bba4ccb3a01125ebc9f074f624f106905bbb4fed (diff) | |
download | freeipa.git-eba3a341e607b542afabdedf0f6508e7c0a5e96c.tar.gz freeipa.git-eba3a341e607b542afabdedf0f6508e7c0a5e96c.tar.xz freeipa.git-eba3a341e607b542afabdedf0f6508e7c0a5e96c.zip |
Fixed ipa.js for sessions.
The patch fixes a problem in error_handler_login() when it gets
an error other than 401.
The login_url is not needed for fixtures because it does not need
authentication.
The patch also fixes jslint warnings and formatting issues.
Diffstat (limited to 'install')
-rw-r--r-- | install/ui/ipa.js | 17 |
1 files changed, 9 insertions, 8 deletions
diff --git a/install/ui/ipa.js b/install/ui/ipa.js index a424fe95..82e89204 100644 --- a/install/ui/ipa.js +++ b/install/ui/ipa.js @@ -59,10 +59,11 @@ var IPA = function() { // if current path matches live server path, use live data if (that.url && window.location.pathname.substring(0, that.url.length) === that.url) { that.json_url = params.url || '/ipa/json'; - that.login_url = params.url || '/ipa/login'; // FIXME, what about the other case below? + that.login_url = params.url || '/ipa/login'; } else { // otherwise use fixtures that.json_path = params.url || "test/data"; + // that.login_url is not needed for fixtures } $.ajaxSetup(that.ajax_options); @@ -300,8 +301,8 @@ IPA.get_credentials = function() { var request = { url: IPA.login_url, - async: false, - type: "GET", + async: false, + type: "GET", success: success_handler, error: error_handler }; @@ -309,7 +310,7 @@ IPA.get_credentials = function() { $.ajax(request); return status; -} +}; /** * Call an IPA command over JSON-RPC. @@ -421,13 +422,13 @@ IPA.command = function(spec) { var login_status = IPA.get_credentials(); if (login_status === 200) { - that.request.error = error_handler + that.request.error = error_handler; $.ajax(that.request); - } else { - // error_handler() calls IPA.hide_activity_icon() - error_handler.call(this, xhr, text_status, error_thrown); + return; } } + // error_handler() calls IPA.hide_activity_icon() + error_handler.call(this, xhr, text_status, error_thrown); } /* |