- sprawdzenie dlugosci nazwy opcji

paszczus · Dec 10, 2004 · d073de8 · d073de8
1 parent 518c3ad
commit d073de8
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 9 deletions.
diff --git a/modules/configadd.php b/modules/configadd.php
@@ -37,13 +37,13 @@
 		die;
 	}
 
-	if(!eregi("^[a-z0-9_-]+$", $config['name']))
-    		$error['name'] = 'Nazwa opcji zawiera niepoprawne znaki!';
-
 	if($config['name']=='')
 		$error['name'] = 'Musisz podaæ nazwê opcji!';
-
-	if($LMS->GetConfigOptionId($config['name'], $config['section']))
+	elseif(strlen($config['name'])>64)
+		$error['name'] = 'Za d³uga nazwa opcji (max.64 znaki) !';
+	elseif(!eregi("^[a-z0-9_-]+$", $config['name']))
+    		$error['name'] = 'Nazwa opcji zawiera niepoprawne znaki!';
+	elseif($LMS->GetConfigOptionId($config['name'], $config['section']))
 		$error['name'] = 'Opcja ju¿ jest w bazie!'; 
 
 	if(!eregi("^[a-z0-9_-]+$", $config['section']) && $config['section']!='')

diff --git a/modules/configedit.php b/modules/configedit.php
@@ -48,12 +48,13 @@ function ConfigOptionExists($id)
 	foreach($cfg as $key => $val) 
 		$cfg[$key] = trim($val);
 
-	if(!eregi("^[a-z0-9_-]+$", $cfg['var']))
-    		$error['var'] = 'Nazwa opcji zawiera niepoprawne znaki!';
-
 	if($cfg['var']=='')
 		$error['var'] = 'Musisz podaæ nazwê opcji!';
-
+	elseif(strlen($cfg['var'])>64)
+		$error['var'] = 'Za d³uga nazwa opcji (max.64 znaki) !';
+	elseif(!eregi("^[a-z0-9_-]+$", $cfg['var']))
+    		$error['var'] = 'Nazwa opcji zawiera niepoprawne znaki!';
+
 	if(($cfg['var']!=$config['var'] || $cfg['section']!=$config['section'])
 		&& $LMS->GetConfigOptionId($cfg['var'], $cfg['section'])
 	)