Code cleanup

git-svn-id: https://oss.gonicus.de/repositories/gosa/trunk@950 594d385d-05f5-0310-b6e9-bd551577e9d8

diff --git a/include/class_plugin.inc b/include/class_plugin.inc
index 1f89ad0839fdb199f58114229dfed756d758af38..0240722a706074e227e6a1246ff073dbfd95afb5 100644 (file)
--- a/include/class_plugin.inc
+++ b/include/class_plugin.inc
@@ -243,19 +243,17 @@ class plugin
           $this->is_modified= TRUE;
         }
     
-      /* Okay, how can I explain this fix ... 
-       * In firefox, disabled option fields aren't selectable ... but in IE you can select these fileds. 
-       * So IE posts these 'unselectable' option, with value = chr(194) 
-       * chr(194) seems to be the &nbsp; in between the ...option>&nbsp;</option.. because there is no value=".." specified in these option fields  
-       * This &nbsp; was added for W3c compliance, but now causes these ... ldap errors ... 
-       * So we set these Fields to ""; a normal empty string, and we can check these values in plugin::check() again ...
-       */
-      if(isset($data[0])){
-        if($data[0] == chr(194)) {
+        /* Okay, how can I explain this fix ... 
+         * In firefox, disabled option fields aren't selectable ... but in IE you can select these fileds. 
+         * So IE posts these 'unselectable' option, with value = chr(194) 
+         * chr(194) seems to be the &nbsp; in between the ...option>&nbsp;</option.. because there is no value=".." specified in these option fields  
+         * This &nbsp; was added for W3c compliance, but now causes these ... ldap errors ... 
+         * So we set these Fields to ""; a normal empty string, and we can check these values in plugin::check() again ...
+         */
+        if(isset($data[0]) && $data[0] == chr(194)) {
           $data = "";  
         }
-      }
-      $this->$val= $data;
+        $this->$val= $data;
       }
     }
   }