[SM-CVS] SF.net SVN: squirrelmail: [13103] branches/SM-1_4-STABLE/squirrelmail/ include/options/display.php

[<pdo...@us...>]

Revision: 13103
          http://squirrelmail.svn.sourceforge.net/squirrelmail/?rev=13103&view=rev
Author:   pdontthink
Date:     2008-05-07 09:28:41 -0700 (Wed, 07 May 2008)

Log Message:
-----------
Cannot assume we are the first form on the page.  f is a terrible form name, but changing that could have consequences

Modified Paths:
--------------
    branches/SM-1_4-STABLE/squirrelmail/include/options/display.php

Modified: branches/SM-1_4-STABLE/squirrelmail/include/options/display.php
===================================================================
--- branches/SM-1_4-STABLE/squirrelmail/include/options/display.php	2008-05-07 16:19:09 UTC (rev 13102)
+++ branches/SM-1_4-STABLE/squirrelmail/include/options/display.php	2008-05-07 16:28:41 UTC (rev 13103)
@@ -118,7 +118,8 @@
     );
 
     if ($optmode != 'submit')
-       $onLoadScript = 'document.forms[0].new_js_autodetect_results.value = \'' . SMPREF_JS_ON . '\'';
+       // FIXME: our form is named "f" - why? let's use something better
+       $onLoadScript = 'document.f.new_js_autodetect_results.value = \'' . SMPREF_JS_ON . '\'';
     else
        $onLoadScript = '';
 


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.