From: wolffc <wolffc@594d385d-05f5-0310-b6e9-bd551577e9d8>
Date: Thu, 16 Sep 2010 14:38:38 +0000 (+0000)
Subject: Check for invalid names first before trimming name, catching trailing
X-Git-Url: https://git.tokkee.org/?p=gosa.git;a=commitdiff_plain;h=6e2d4178af239bb07199e73204239b8ab924934a

Check for invalid names first before trimming name, catching trailing
spaces at the end or beginning


git-svn-id: https://oss.gonicus.de/repositories/gosa/branches/2.6-lhm@19721 594d385d-05f5-0310-b6e9-bd551577e9d8
---

diff --git a/trunk/gosa-plugins/fai/admin/fai/class_faiScriptEntry.inc b/trunk/gosa-plugins/fai/admin/fai/class_faiScriptEntry.inc
index db8366b1b..7600ac934 100644
--- a/trunk/gosa-plugins/fai/admin/fai/class_faiScriptEntry.inc
+++ b/trunk/gosa-plugins/fai/admin/fai/class_faiScriptEntry.inc
@@ -137,14 +137,13 @@ class faiScriptEntry extends plugin
       $message[] = msgPool::duplicated(_("Name"));
     }
 
-    $c = trim($this->cn);
-    if($c == ""){
-      $message[] = msgPool::required(_("Name"));
-    }
-    if(preg_match("/[^a-z0-9_\-]/i",$c)){
-      $message[] = msgPool::invalid(_("Name"),$c,"/[a-z0-9_\-]/i");
+    if(preg_match("/[^a-z0-9_\-]/i",$this->cn)){
+      $message[] = msgPool::invalid(_("Name"),$this->cn,"/[a-z0-9_\-]/i");
     }
 
+    if(trim($this->cn)  == ""){
+      $message[] = msgPool::required(_("Name"));
+    }
     $s = trim($this->FAIscript);
     if($s == ""){
       $message[]= msgPool::required(_("Script"));