ankitjain28may / openchat

src/Online.php

<?php

For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 9 and the first side effect is on line 4.

namespace ChatApp;
require_once (dirname(__DIR__) . '/config/database.php');

/**
* For checking online status
*/
class Online
{

    public static function setOnlineStatus($userId)

This method seems to be duplicated in your project.

    {
        $connect = mysqli_connect(DB_HOST, DB_USER, DB_PASSWORD, DB_NAME);
        $query = "Update login set login_status = 1 where login_id = '$userId'";
        if(!$connect->query($query));
            echo $connect->error;
        $connect->close();
    }

    public static function removeOnlineStatus($userId)

This method seems to be duplicated in your project.

    {
        $connect = mysqli_connect(DB_HOST, DB_USER, DB_PASSWORD, DB_NAME);
        $query = "Update login set login_status = 0 where login_id = '$userId'";
        if(!$connect->query($query));
            echo $connect->error;
        $connect->close();
    }
}