View GitHub Repository openchat

Schedule Inspection
Subscribe
Test Setup Failed
Push — master ( 976b69...15356d )
by Ankit
02:31
created
Status
Category
Toggle Patches
src/Chat.php 2 patches
Unused Use Statements   -2 removed lines patch added patch discarded remove patch 
@@ -2,11 +2,9 @@
 
 block discarded – undo
2 2 
 namespace ChatApp;
3 3 
 use Ratchet\MessageComponentInterface;
4 4 
 use Ratchet\ConnectionInterface;
5 
-use ChatApp\Models\Message;
6 5 
 use ChatApp\Reply;
7 6 
 use ChatApp\Conversation;
8 7 
 use ChatApp\Receiver;
9 
-use ChatApp\SideBar;
10 8 
 use ChatApp\Search;
11 9 
 use ChatApp\Compose;
12 10
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -37,7 +37,7 @@  discard block
 
 block discarded – undo
37 37
38 38 
     public function onMessage(ConnectionInterface $from, $msg) {
39 39 
         $sessionId = $from->WebSocket->request->getCookies()['PHPSESSID'];
40 
-        if($msg == 'OpenChat initiated..!')
40 
+        if ($msg == 'OpenChat initiated..!')
41 41 
         {
42 42 
             @$initial->initial = json_decode($this->onSidebar($from->userId));
43 43 
 
@@ -98,7 +98,7 @@  discard block
 
 block discarded – undo
98 98 
                     $client->send(json_encode($result));
99 99 
                     $this->online = 1;
100 100 
                 }
101 
-                elseif($client == $from)
101 
+                elseif ($client == $from)
102 102 
                 {
103 103 
                     @$result->sidebar = json_decode($this->onSidebar($client->userId));
104 104
Please login to merge, or discard this patch.
src/Receiver.php 2 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch 
@@ -3,7 +3,6 @@
 
 block discarded – undo
3 3 
 namespace ChatApp;
4 4 
 require_once (dirname(__DIR__) . '/database.php');
5 5 
 use ChatApp\User;
6 
-use ChatApp\Time;
7 6 
 use ChatApp\Conversation;
8 7 
 /**
9 8 
 *
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -1,7 +1,7 @@  discard block
 
 block discarded – undo
1 1 
 <?php
2 2
3 3 
 namespace ChatApp;
4 
-require_once (dirname(__DIR__) . '/config/database.php');
4 
+require_once (dirname(__DIR__).'/config/database.php');
5 5 
 use ChatApp\User;
6 6 
 use ChatApp\Time;
7 7 
 use ChatApp\Conversation;
 
@@ -31,7 +31,7 @@  discard block
 
 block discarded – undo
31 31 
         $name = $this->messages['name'];
32 32 
         $this->messages = json_decode($this->conversation->ConversationLoad($msg, True));
33 33 
         $id = json_decode($msg)->username;
34 
-        for ($i=1 ; $i < count($this->messages); $i++) {
34 
+        for ($i = 1; $i < count($this->messages); $i++) {
35 35 
             $this->messages[$i]->start = $id;
36 36 
         }
37 37 
         $this->messages[0]->username = $username;
Please login to merge, or discard this patch.
login.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -1,6 +1,6 @@
 
 block discarded – undo
1 1 
 <?php
2 2 
 session_start();
3 
-if(isset($_SESSION['start']))
3 
+if (isset($_SESSION['start']))
4 4 
 {
5 5 
     header("Location: account.php");
6 6 
 }
Please login to merge, or discard this patch.
src/Time.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch 
@@ -16,14 +16,14 @@
 
 block discarded – undo
16 16 
     function TimeConversion($time)
17 17 
     {
18 18
19 
-        if(substr($time,4,11) == date("d M Y", time() + 16200))
20 
-            $time = substr($time,16,5);
21 
-        else if(substr($time,7,8) == date("M Y", time() + 16200) && substr($time, 4,2) - date("d") < 7)
22 
-            $time = substr($time,0,3);
23 
-        else if(substr($time,11,4) == date("Y", time() + 16200))
24 
-            $time = substr($time,4,6);
19 
+        if (substr($time, 4, 11) == date("d M Y", time() + 16200))
20 
+            $time = substr($time, 16, 5);
21 
+        else if (substr($time, 7, 8) == date("M Y", time() + 16200) && substr($time, 4, 2) - date("d") < 7)
22 
+            $time = substr($time, 0, 3);
23 
+        else if (substr($time, 11, 4) == date("Y", time() + 16200))
24 
+            $time = substr($time, 4, 6);
25 25 
         else
26 
-            $time = substr($time,4,11);
26 
+            $time = substr($time, 4, 11);
27 27
28 28 
         return $time;
29 29 
     }
Please login to merge, or discard this patch.
src/Models/message.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -8,7 +8,7 @@
 
 block discarded – undo
8 8
9 9 
 class Message extends Model
10 10 
 {
11 
-    protected $table='message';
11 
+    protected $table = 'message';
12 12
13 13 
     protected $fillable = ['identifier_message_number', 'message', 'sent_by', 'time'];
14 14 
 }
Please login to merge, or discard this patch.
cmd.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -7,7 +7,7 @@
 
 block discarded – undo
7 7 
 use Ratchet\WebSocket\WsServer;
8 8 
 use ChatApp\Chat;
9 9
10 
-$server =IoServer::factory(
10 
+$server = IoServer::factory(
11 11 
 	new HttpServer(
12 12 
 		new WsServer(
13 13 
 			new Chat()
Please login to merge, or discard this patch.
message.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -1,6 +1,6 @@  discard block
 
 block discarded – undo
1 1 
 <?php
2 2 
 session_start();
3 
-if(isset($_SESSION['start']) and empty($_GET['user']))
3 
+if (isset($_SESSION['start']) and empty($_GET['user']))
4 4 
 {
5 5 
 ?>
6 6 
 <!DOCTYPE html>
 
@@ -130,7 +130,7 @@  discard block
 
 block discarded – undo
130 130
131 131 
 <?php
132 132 
 }
133 
-else{
133 
+else {
134 134 
 	header('Location:http://www.localhost/openchat/login.php');
135 135 
 }
136 136 
 ?>
Please login to merge, or discard this patch.
profile_generate.php 1 patch
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch 
@@ -2,32 +2,32 @@
 
 block discarded – undo
2 2 
 session_start();
3 3 
 include 'database.php';
4 4 
 $connect = mysqli_connect(DB_HOST, DB_USER, DB_PASSWORD, DB_NAME);
5 
-$login_id=$_SESSION['start'];
6 
-if(isset($_POST['submit']))
5 
+$login_id = $_SESSION['start'];
6 
+if (isset($_POST['submit']))
7 7 
 {
8 
-	$query="SELECT * from profile where login_id='$login_id'";
9 
-	if($result=$connect->query($query))
8 
+	$query = "SELECT * from profile where login_id='$login_id'";
9 
+	if ($result = $connect->query($query))
10 10 
 	{
11 
-		if($result->num_rows>0)
11 
+		if ($result->num_rows > 0)
12 12 
 		{
13 
-			$row=$result->fetch_assoc();
13 
+			$row = $result->fetch_assoc();
14 14 
 		}
15 15 
 	}
16 
-	if(trim($_POST['status']))
17 
-		$status=$_POST['status'];
16 
+	if (trim($_POST['status']))
17 
+		$status = $_POST['status'];
18 18 
 	else
19 
-		$status=$row['status'];
20 
-	if(trim($_POST['education']))
21 
-		$edu=$_POST['education'];
19 
+		$status = $row['status'];
20 
+	if (trim($_POST['education']))
21 
+		$edu = $_POST['education'];
22 22 
 	else
23 
-		$edu=$row['education'];
24 
-	if(isset($_POST['gender']))
25 
-		$gender=$_POST['gender'];
23 
+		$edu = $row['education'];
24 
+	if (isset($_POST['gender']))
25 
+		$gender = $_POST['gender'];
26 26 
 	else
27 
-		$gender=$row['gender'];
27 
+		$gender = $row['gender'];
28 28
29 
-	$query="UPDATE profile set status='$status', education='$edu', gender='$gender' where login_id='$login_id'";
30 
-	if($result=$connect->query($query))
29 
+	$query = "UPDATE profile set status='$status', education='$edu', gender='$gender' where login_id='$login_id'";
30 
+	if ($result = $connect->query($query))
31 31 
 	{
32 32 
 		header('Location: account.php');
33 33 
 	}
Please login to merge, or discard this patch.
register.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -1,6 +1,6 @@
 
 block discarded – undo
1 1 
 <?php
2 2 
 session_start();
3 
-if(isset($_SESSION['start']))
3 
+if (isset($_SESSION['start']))
4 4 
 {
5 5 
     header("Location: account.php");
6 6 
 }
Please login to merge, or discard this patch.