Still getting complaints about this outputting plain text to the browser

instead of JS on the component... Going to pull it for now... and investigate
the possible cause for the screw ups after 1.4.0 is released.

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@4676 7612ce4b-ef26-0410-bec9-ea0150e637f0

diff --git a/functions/options.php b/functions/options.php
index 4d1a89048b303166dbc13b8c0ab04b7f4b9dca1b..85598f107123c76774dbf2d57ec776bed42b529f 100644 (file)
--- a/functions/options.php
+++ b/functions/options.php
@@ -275,33 +275,14 @@ class SquirrelOption {
 
     function createWidget_Integer() {
 
-        global $javascript_on;
+        return $this->createWidget_String();
 
-        // add onChange javascript handler to a regular string widget
-        // which will strip out all non-numeric chars
-        if ($javascript_on)
-           return preg_replace('/>/', ' onChange="a=this.value; b=\'\'; '
-                    . 'for (i=0;i<a.length;i++) { if (a.charAt(i)>=\'0\' '
-                    . '&& a.charAt(i)<=\'9\') b += a.charAt(i); } '
-                    . 'this.value=b;">', $this->createWidget_String());
-        else
-           return $this->createWidget_String();
     }
 
     function createWidget_Float() {
         
-        global $javascript_on;
+        return $this->createWidget_String();
 
-        // add onChange javascript handler to a regular string widget
-        // which will strip out all non-numeric (period also OK) chars 
-        if ($javascript_on)
-           return preg_replace('/>/', ' onChange="a=this.value; b=\'\'; '
-                    . 'for (i=0;i<a.length;i++) { if ((a.charAt(i)>=\'0\' '
-                    . '&& a.charAt(i)<=\'9\') || a.charAt(i)==\'.\') '
-                    . 'b += a.charAt(i); } this.value=b;">'
-                , $this->createWidget_String());
-        else
-           return $this->createWidget_String();
     }
 
     function createWidget_Boolean() {