From a2b7fca705b6561892eb6f1a064703ea845be444 Mon Sep 17 00:00:00 2001
From: hickert <hickert@594d385d-05f5-0310-b6e9-bd551577e9d8>
Date: Fri, 11 Aug 2006 06:04:31 +0000
Subject: [PATCH] Updated hotPlugDialog

git-svn-id: https://oss.gonicus.de/repositories/gosa/trunk@4475 594d385d-05f5-0310-b6e9-bd551577e9d8
---
 .../environment/class_hotplugDialog.inc       | 31 +++++++++++++------
 .../personal/environment/hotplugDialogNew.tpl | 14 ++++-----
 2 files changed, 29 insertions(+), 16 deletions(-)

diff --git a/plugins/personal/environment/class_hotplugDialog.inc b/plugins/personal/environment/class_hotplugDialog.inc
index dbb082141..dc61ecf84 100644
--- a/plugins/personal/environment/class_hotplugDialog.inc
+++ b/plugins/personal/environment/class_hotplugDialog.inc
@@ -92,14 +92,18 @@ class hotplugDialog extends plugin
   function save_object()
   {
     if(isset($_POST['dialogissubmitted'])){
+      plugin::save_object();
       foreach($this->attributes as $s_attr){
         if(isset($_POST[$s_attr])){
-          $this->$s_attr = stripslashes($_POST[$s_attr]);
+          if(in_array($s_attr,array("HOT_produkt","HOT_vendor"))){
+            $this->$s_attr = strtolower(stripslashes($_POST[$s_attr]));
+          }else{
+            $this->$s_attr = stripslashes($_POST[$s_attr]);
+          }
         }else{
           $this->$s_attr = false;
         }
       }
-      plugin::save_object();
     }
   }
 
@@ -111,20 +115,22 @@ class hotplugDialog extends plugin
     $message= plugin::check();
 
     if(!$this->use_existing){
-      if((empty($this->HOT_name))||(preg_match("/[^a-z0-9]/i",$this->HOT_name))){
+
+      if(empty($this->HOT_name)||(preg_match("/[^a-z0-9]/i",$this->HOT_name))){
         $message[]=_("Please specify a valid name. Only 0-9 a-Z is allowed.");
       }
-      if((empty($this->HOT_description))||(preg_match("/[^a-z0-9!\"?.,;:-_\(\) ]/i",$this->HOT_description))){
+      if(preg_match("/[^a-z0-9!\"?.,;:-_\(\) ]/i",$this->HOT_description)){
         $message[]=_("Invalid character in description. Please specify a valid description.");
       }
-      if((empty($this->HOT_id))||(preg_match("/[\|]/i",$this->HOT_id))){
+
+      if(preg_match("/[\|]/i",$this->HOT_id)){
         $message[]=_("Please specify a valid id.");
       }
-      if((empty($this->HOT_vendor))||(preg_match("/[\|]/i",$this->HOT_vendor))){
-        $message[]=_("Please specify a valid id.");
+      if(!$this->is_2byteHex($this->HOT_vendor)){
+        $message[]=_("Please specify a valid vendor id. (2 byte hex like '0xFFFF')");
       }
-      if((empty($this->HOT_produkt))||(preg_match("/[\|]/i",$this->HOT_produkt))){
-        $message[]=_("Please specify a valid id.");
+      if(!$this->is_2byteHex($this->HOT_produkt)){
+        $message[]=_("Please specify a valid product id. (2 byte hex like '0xFFFF')");
       }
 
     $ldap = $this->config->get_ldap_link();
@@ -142,6 +148,13 @@ class hotplugDialog extends plugin
     return ($message);
   }
 
+  function is_2byteHex($str)
+  {
+    return empty($str) || preg_match("/^(0x|x|)[a-f0-9][a-f0-9][a-f0-9][a-f0-9]/i",$str);
+  }
+  
+
+
 
   /* Save to LDAP */
   function save()
diff --git a/plugins/personal/environment/hotplugDialogNew.tpl b/plugins/personal/environment/hotplugDialogNew.tpl
index 5a6fb3386..927395d01 100644
--- a/plugins/personal/environment/hotplugDialogNew.tpl
+++ b/plugins/personal/environment/hotplugDialogNew.tpl
@@ -14,7 +14,7 @@
 			</td>
 		</tr>
 		<tr>
-			<td><LABEL for="HOT_description">{t}Description{/t}</LABEL>{$must}
+			<td><LABEL for="HOT_description">{t}Description{/t}</LABEL>
 			</td>
 			<td>
 				<input type="text" size=40 value="{$HOT_description}" name="HOT_description" id="HOT_description">
@@ -25,22 +25,22 @@
    <td style="vertical-align:top">
 	<table summary="">
 		<tr>
-			<td><LABEL for="HOT_vendor">{t}Vendor ID{/t}</LABEL>{$must}
+			<td><LABEL for="HOT_id">{t}Serial number{/t}&nbsp;{t}(iSerial){/t}</LABEL>
 			</td>
 			<td>
-				<input type="text" value="{$HOT_vendor}" name="HOT_vendor" id="HOT_vendor">
+				<input type="text" value="{$HOT_id}" name="HOT_id" id="HOT_id">
 			</td>
+			<td colspan="2">&nbsp;</td>
 		</tr>
 		<tr>
-			<td><LABEL for="HOT_id">{t}Device ID{/t}</LABEL>{$must}
+			<td><LABEL for="HOT_vendor">{t}Vendor-ID{/t}&nbsp;{t}(idVendor){/t}</LABEL>
 			</td>
 			<td>
-				<input type="text" value="{$HOT_id}" name="HOT_id" id="HOT_id">
+				<input type="text" value="{$HOT_vendor}" name="HOT_vendor" id="HOT_vendor">
 			</td>
-			<td colspan="2">&nbsp;</td>
 		</tr>
 		<tr>
-			<td><LABEL for="HOT_produkt">{t}Product ID{/t}</LABEL>{$must}
+			<td><LABEL for="HOT_produkt">{t}Product-ID{/t}&nbsp;{t}(idProduct){/t}</LABEL>
 			</td>
 			<td>
 				<input type="text" value="{$HOT_produkt}" name="HOT_produkt" id="HOT_produkt">
-- 
2.30.2