Merge pull request #6 from Zanark/php_forum

Zanark · web-flow · commit 521b6ce5b789 · 2018-03-18T20:25:36.000+05:30
Added check for special characters in username &amp; set min password length = 8
diff --git a/forum/forum_functions.php b/forum/forum_functions.php
@@ -0,0 +1,8 @@
+<?php
+
+    function checkInvalidCharacters($text) {
+        
+        return(bool) preg_match("/[^\w-.]/" , $text );
+    }
+
+?>
diff --git a/forum/register_parse.php b/forum/register_parse.php
@@ -1,25 +1,50 @@
 <?php
 
     include("connect.php");
+    include("forum_functions.php");
 
     $username = $_POST['username'];
     $password = $_POST['password'];
 
+    $minpl = 8;
+
    /* echo "Registeration Details  : ";
     echo "Username : " . $username . "<br/><br/> Password : " . $password; */
-    
-    $sql = "INSERT INTO forum.users(username , password)
-            VALUES ('$username' , '$password');
-    ";
-
-    $res = mysqli_query($link , $sql);
 
-    if($res){
-        echo "<b>Successfully Registered</b> as :   " . $username . "<br/><br/>";
+    if ( $username == "" || $password == ""){
+        echo "Please fill in all the details <br/> Please check if you forgot to enter data in any of the fields <br/>";
     }
+
     else {
-        echo "<b>Registration Failed</b><br/><br/>";
-        echo "Error is =>   " . mysql_error();
+        if( checkInvalidCharacters($username) )
+            echo "Username can only contain <br/><br/><ul><li> A to Z </li><li> a to z </li><li> 0 to 9 </li><li> . and _ </li></ul>";
+
+        else {
+            if (strlen($password) >= $minpl) {
+
+                    $sql = "INSERT INTO forum.users(username , password)
+                            VALUES ('$username' , '$password');
+                            ";
+
+                    $res = mysqli_query($link , $sql);
+
+                    if($res){
+                        echo "<b>Successfully Registered</b> as :   " . $username . "<br/><br/>";
+                    }
+    
+                    else {
+                        echo "<b>Registration Failed</b><br/><br/>";
+                        echo "Error is =>   " . mysql_error();
+                    }
+
+                
+            }
+
+            else {
+                echo "Password is too short. <br/> <b>Minimum length is 8 characters</b>";
+            }
+        }
     }
+    
 
 ?>

-Original file line number
+Diff line change
@@ @@ -0,0 +1,8 @@ @@
 +<?php
++
 +    function checkInvalidCharacters($text) {
++
 +        return(bool) preg_match("/[^\w-.]/" , $text );
 +    }
++
 +?>