<?xml version="1.0" encoding="UTF-8"?> 
<?xml-stylesheet href="https://dev.horde.org/themes/horde//default/feed-rss.xsl" type="text/xsl"?> 
<rss version="2.0" xmlns:atom="http://www.w3.org/2005/Atom"> 
 <channel> 
  <title>login.php IE7 minor bug if no loginparams set</title> 
  <pubDate>Fri, 10 Apr 2026 14:49:33 +0000</pubDate> 
  <link>https://bugs.horde.org/ticket/7892</link> 
  <atom:link rel="self" type="application/rss+xml" title="login.php IE7 minor bug if no loginparams set" href="https://bugs.horde.org/ticket/7892/rss" /> 
  <description>login.php IE7 minor bug if no loginparams set</description> 
 
   
   
  <item> 
   <title>We use the followign version:

 $Horde: gollem/login.php,v 1</title> 
   <description>We use the followign version:

 $Horde: gollem/login.php,v 1.126 2009-01-06 17:50:05 jan Exp $



And we recive the following malfunction:

If i do not set loginparams (jsut use an empty array) then only in IE7 the login screen is empty.



the IE stop working in the line 198 exit; - other borwsers skip this problem.



the problem originally in the line 190 where we set up:

$redirect_params[&#039;autologin&#039;] = 1;



i modify the 188-189 lines like this:

    } elseif (Util::getFormData(&#039;change_backend&#039;) &amp;&amp;

              empty($GLOBALS[&#039;gollem_backends&#039;][$backend_key][&#039;loginparams&#039;]) &amp;&amp;

              Gollem::canAutoLogin($backend_key, $autologin)

              ) {



it&#039;s works but i&#039;m not sure this solution is good or not...</description> 
   <pubDate>Mon, 26 Jan 2009 13:15:00 +0000</pubDate> 
   <link>https://bugs.horde.org/ticket/7892#t51962</link> 
  </item> 
   
  <item> 
   <title>Changes have been made in CVS for this ticket:

http://cvs.h</title> 
   <description>Changes have been made in CVS for this ticket:

http://cvs.horde.org/diff.php/gollem/docs/CHANGES?rt=horde&amp;r1=1.114.2.44&amp;r2=1.114.2.45&amp;ty=u
http://cvs.horde.org/diff.php/gollem/login.php?rt=horde&amp;r1=1.94.2.15&amp;r2=1.94.2.16&amp;ty=u</description> 
   <pubDate>Tue, 27 Jan 2009 19:17:44 +0000</pubDate> 
   <link>https://bugs.horde.org/ticket/7892#t52011</link> 
  </item> 
   
  <item> 
   <title>Changes have been made in CVS for this ticket:

http://cvs.h</title> 
   <description>Changes have been made in CVS for this ticket:

http://cvs.horde.org/diff.php/gollem/docs/CHANGES?rt=horde&amp;r1=1.165&amp;r2=1.166&amp;ty=u
http://cvs.horde.org/diff.php/gollem/login.php?rt=horde&amp;r1=1.126&amp;r2=1.127&amp;ty=u</description> 
   <pubDate>Tue, 27 Jan 2009 19:19:35 +0000</pubDate> 
   <link>https://bugs.horde.org/ticket/7892#t52012</link> 
  </item> 
   
  <item> 
   <title>Does this slightly modified patch fix things? (The empty che</title> 
   <description>Does this slightly modified patch fix things? (The empty check for loginparams is not needed since it is done in Gollem::canAutologin()).</description> 
   <pubDate>Tue, 27 Jan 2009 19:20:23 +0000</pubDate> 
   <link>https://bugs.horde.org/ticket/7892#t52013</link> 
  </item> 
   
  <item> 
   <title>Thanks a lot. it&#039;s works perfectly now!



Lawrence



&gt; Doe</title> 
   <description>Thanks a lot. it&#039;s works perfectly now!



Lawrence



&gt; Does this slightly modified patch fix things? (The empty check for 

&gt; loginparams is not needed since it is done in Gollem::canAutologin()).

</description> 
   <pubDate>Wed, 28 Jan 2009 12:42:38 +0000</pubDate> 
   <link>https://bugs.horde.org/ticket/7892#t52044</link> 
  </item> 
   
   
 
 </channel> 
</rss> 
