ankitjain28may / openchat

src/Login.php

<?php
/**
 * Login Class Doc Comment
 *
 * PHP version 5
 *
 * @category PHP
 * @package  OpenChat
 * @author   Ankit Jain <ankitjain28may77@gmail.com>
 * @license  The MIT License (MIT)
 * @link     https://github.com/ankitjain28may/openchat
 */
namespace ChatApp;

require_once dirname(__DIR__).'/vendor/autoload.php';
use ChatApp\Session;
use mysqli;
use Dotenv\Dotenv;
$dotenv = new Dotenv(dirname(__DIR__));
$dotenv->load();

/**
 * To Login the User
 *
 * @category PHP
 * @package  OpenChat
 * @author   Ankit Jain <ankitjain28may77@gmail.com>
 * @license  The MIT License (MIT)
 * @link     https://github.com/ankitjain28may/openchat
 */
class Login
{
    /*
    |--------------------------------------------------------------------------
    | Login Class
    |--------------------------------------------------------------------------
    |
    | To Login the User.
    |
    */

    protected $flag;
    protected $error;
    protected $connect;

    /**
     * Create a new class instance.
     *
     * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct()

This method seems to be duplicated in your project.

    {
        $this->flag = 0;
        $this->connect = new mysqli(
            getenv('DB_HOST'),
            getenv('DB_USER'),
            getenv('DB_PASSWORD'),
            getenv('DB_NAME')
        );
        $this->error = array();
    }

    /**
     * To Authenticate User Credentials
     *
     * @param array $data To store User Credentials
     *
     * @return string
     */
    public function authLogin($data)
    {
        $data = $this->emptyValue($data);

        $login = $data["login"];
        $password = $data["passLogin"];

        if (preg_match("/^.+[@]{1}.+$/", $login)) {
            if (filter_var($login, FILTER_VALIDATE_EMAIL) == false) {
                $this->onError("login", " *Enter correct Email address");
            }
        }

        if ($this->flag == 0) {
            $password = md5($password);
            $query = "SELECT * FROM login WHERE email = '$login' or
                        username = '$login'";
            if ($result = $this->connect->query($query)) {
                if ($result->num_rows > 0) {
                    $row = $result->fetch_assoc();
                    $loginID = $row['login_id'];
                    $query = "SELECT id FROM register WHERE id = '$loginID' and
                                password = '$password'";

                    if ($result = $this->connect->query($query)) {
                        if ($result->num_rows > 0) {
                            Session::put('start', $loginID);
                            return json_encode(
                                [
                                "location" => getenv('APP_URL')."/views/account.php"
                                ]
                            );
                        }
                        $this->onError("passLogin", " *Invalid password");
                        return json_encode($this->error);
                    }
                    return json_encode(
                        ["Error" => "You are not registered, ".$this->connect->error]
                    );
                }
                $this->onError("login", " *Invalid username or email");
                return json_encode($this->error);
            }
            return json_encode(
                ["Error" => "You are not registered, ".$this->connect->error]
            );
        } else {
            return json_encode($this->error);
        }
    }

    /**
     * For Adding Error statements
     *
     * @param string $key   To store Key
     * @param string $value To store Key
     *
     * @return void
     */
    public function onError($key, $value)

This method seems to be duplicated in your project.

    {
        $this->flag = 1;
        $this->error = array_merge(
            $this->error,
            [["key" => $key, "value" => $value]]
        );
    }

    /**
     * For Traversing data to check for error
     *
     * @param array $data To store Data
     *
     * @return array
     */
    public function emptyValue($data)
    {
        $errorCode = array(
            "login" => " *Enter the login field",
            "passLogin" => " *Enter the password"
        );

        foreach ($data as $key => $value) {

This code seems to be duplicated across your project.

            $data[$key] = trim($data[$key]);
            $value = trim($value);
            if (empty($value)) {
                $this->onError($key, $errorCode[$key]);
            }
        }
        return $data;
    }

}