ankitjain28may / openchat

tests/TestReply.php

Unused Use Statements -2 removed lines

@@ -3,10 +3,8 @@
 namespace ChatApp\Tests;
 use PHPUnit_Framework_TestCase;
 use ChatApp\Register;
-use ChatApp\Search;
 use ChatApp\Reply;
 use ChatApp\Session;
-
 use Dotenv\Dotenv;
 $dotenv = new Dotenv(dirname(__DIR__));
 $dotenv->load();

src/Reply.php

Braces +22 added lines, -17 removed lines

@@ -31,17 +31,21 @@ 
     public function replyTo($msg)
     {
         $msg = json_decode($msg);   //decode json value
-        if(Session::get('start') != null && !empty($msg))  //checks for session login and the value send
+        if(Session::get('start') != null && !empty($msg)) {
+            //checks for session login and the value send
         {
             $userId = Session::get('start');
+        }
             $identifier = $msg->name;
 
             $receiverID = $identifier;  //stores id of the person whom message is to be sent
 
-            if($identifier > $userId)    // geneate specific unique code to store messages
+            if($identifier > $userId) {
+                // geneate specific unique code to store messages
                 $identifier = $userId . ":" . $identifier;
-            else
-                $identifier = $identifier . ":" . $userId;
+            } else {
+                            $identifier = $identifier . ":" . $userId;
+            }
 
             $reply = addslashes(trim($msg->reply[0])); // stores the message sent by the user.
 
@@ -55,27 +59,31 @@ 
                 $query = "SELECT * from login where login_id = '$receiverID'";
 
                 $result = $this->connect->query($query);
-                if($result->num_rows > 0)     // if true
+                if($result->num_rows > 0) {
+                    // if true
                 {
                     //check whether he is sending message for thr first time or he has sent messages before
                     $query = "SELECT * from total_message where identifier = '$identifier'";
+                }
                     $result = $this->connect->query($query);
-                    if($result->num_rows>0)               // if he has sent messages before
+                    if($result->num_rows>0) {
+                        // if he has sent messages before
                     {
                         // Update Total_Message Table
                         $query = "UPDATE total_message SET total_messages = total_messages+1, time = '$time', unread = 1, id = '$time_id' WHERE identifier = '$identifier'";
+                    }
                         return $this->updateMessages($query, $identifier, $reply, $userId, $time);
 
-                    }
-                    else    // if he sends message for the first time
+                    } else    // if he sends message for the first time
                     {
                         $length = strlen($userId);
-                        if(substr($identifier, 0, $length) == $userId) // generate specific unique code
+                        if(substr($identifier, 0, $length) == $userId) {
+                            // generate specific unique code
                         {
                             $user2 = substr($identifier, $length+1);
-                            $user1 = $userId;
                         }
-                        else
+                            $user1 = $userId;
+                        } else
                         {
                             $user2 = $userId;
                             $length = strlen($identifier) - $length-1;
@@ -85,14 +93,12 @@ 
                         $query = "INSERT into total_message values('$identifier', 1, '$user1', '$user2', 1, '$time', '$time_id')";
                         return $this->updateMessages($query, $identifier, $reply, $userId, $time);
                     }
-                }
-                else // if he is unauthorized echo message is failed
+                } else // if he is unauthorized echo message is failed
                 {
                     return "Invalid Authentication";
                 }
             }
-        }
-        else
+        } else
         {
             return "Failed";
         }
@@ -107,8 +113,7 @@ 
             if($this->connect->query($query))
             {
                 return "Messages is sent";    // if query is executed return true
-            }
-            else
+            } else
             {
                 return "Message is failed";
             }