Index: auth_helper.rb
===================================================================
--- auth_helper.rb	(revision 647)
+++ auth_helper.rb	(revision 863)
@@ -1,11 +1,11 @@
 module AuthHelper
-
+  
   # For theme support. Idea from Typo.
   def search_paths(path)
     ["#{path}/theme/#{this_auth[:theme]}/",
     ]
   end
-
+  
   def full_template_path(template_path, extension)
     template_path_dirs = template_path.split('/')
     dir = template_path_dirs[0]
@@ -17,7 +17,7 @@
     end
     super
   end
-
+  
   # Taken from the webrick server
   module Utils
     if !const_defined? "RAND_CHARS"
@@ -32,60 +32,64 @@
       ret
     end
     module_function :random_string
-
+    
     # define a regexp for valid IPv4 host
     def valid_ipv4_host_regex
       zero_to_254 = "([0-9][0-9]?|1[0-9]{2}|2[0-4][0-9]|25[0-4])"
       one_to_254  = "([1-9][0-9]?|1[0-9]{2}|2[0-4][0-9]|25[0-4])"
-
+      
       /\A#{one_to_254}\.#{zero_to_254}\.#{zero_to_254}\.#{one_to_254}\z/
     end
     module_function :valid_ipv4_host_regex
-
+    
   end
-
+  
   # Show user informations using Ajax. Can be used on static pages / cached pages.
   def ajax_account_infos()
     "" + javascript_tag('function showloginform() {'+ 
-        remote_function(:update => "accountinfo", :url => { :controller => "auth", :action => "remotelogin" }) +
-        ' document.getElementById(\'post_login\').focus();}') + "<!-- account info --> <div id=\"accountinfo\">"+
-        link_to(auth_icon('buddy'), auth_url) + "</div>" +
-        javascript_tag("new Ajax.Updater('accountinfo', '/auth/remoteinfo', {asynchronous:true});")
+    remote_function(:update => "accountinfo", 
+    :url => { :controller => "auth", :action => "remotelogin" }, 
+    :success => "document.getElementById(\'post_login\').focus();") + "}") + "<!-- account info --> <div id=\"accountinfo\">"+
+    link_to(auth_icon('buddy'), auth_url) + "</div>" +
+    javascript_tag("new Ajax.Updater('accountinfo', '/auth/remoteinfo', {asynchronous:true});")
   end
-
+  
   # Show user information, don't use for static or cached page!
   def account_infos()
     "" + javascript_tag('function showloginform() {'+ 
-        remote_function(:update => "accountinfo", :url => { :controller => "auth", :action => "remotelogin" }) +
-    ' document.getElementById(\'post_login\').focus();}') + "<!-- account info --> <div id=\"accountinfo\">"+
+    remote_function(:update => "accountinfo", 
+    :url => { :controller => "auth", :action => "remotelogin" }, 
+    :success => "document.getElementById(\'post_login\').focus();") + "}") +
+     "<!-- account info --> <div id=\"accountinfo\">"+
     render(:partial => "auth/remoteinfo") + "</div>"
   end
-
+  
   # Javascript version to show user information. Can be used on static pages / cached pages.
   def js_account_infos()
     "" + javascript_tag('function showloginform() {'+ 
-        remote_function(:update => "accountinfo", :url => { :controller => "auth", :action => "remotelogin" }) +
-    ' document.getElementById(\'post_login\').focus();}') + "<!-- account info --> <div id=\"accountinfo\">" +
+    remote_function(:update => "accountinfo", 
+    :url => { :controller => "auth", :action => "remotelogin" }, 
+    :success => "document.getElementById(\'post_login\').focus();") + "}") + "<!-- account info --> <div id=\"accountinfo\">" +
     "<script src=\"/auth/jsinfo\" type=\"text/javascript\"></script>"+
     '<script type="text/javascript">displayAccountInfo();</script>' + "</div>"
   end
-
+  
   # store current uri in the ccokies
   # we can return to this location by calling return_location
   def store_location
     cookies[:return_to] = {:value => request.request_uri, :expires => nil }
   end
-
+  
   # Loading spinner indicator icon tag
   def spinner_tag(id = 'ident')
     image_tag('auth/spinner.gif', :id=>"#{id}_spinner", :align => 'absmiddle', :border=> 0, :style=> 'display: none;', :alt => 'loading...' )
   end
-
+  
   # auth_generator's images tags
   def auth_icon(name)
     image_tag("auth/#{name}.png", :align => 'absmiddle', :border => 0, :alt => name)
   end
-
+  
   # image tag for user
   def user_icon(user)
     if @app[:gravatar] == true
@@ -99,9 +103,9 @@
       auth_icon('buddy')
     end
   end
-
+  
   def user_logged_in?
     not @user.nil? and @user.ident == true
   end
-
+  
 end
</div></div>