alphaz\Common\FTP

Complexity

Total Complexity

40

Size/Duplication

Total Lines	305
Duplicated Lines	0 %

Importance

Changes

0

18 Methods

Rating	Name	Duplication	Size	Complexity
A	put()	0	8	3
A	rename()	0	3	2
A	get()	0	3	2
A	chmod()	0	3	2
A	pasv()	0	3	2
A	rmdir()	0	3	2
A	fileExists()	0	3	2
A	delete()	0	3	2
A	chdir()	0	3	2
A	mkdirs()	0	19	5
A	ftpFiles()	0	3	2
A	pwd()	0	3	2
A	getConnection()	0	3	1
A	isConnected()	0	3	2
A	mkdir()	0	3	2
A	dirExists()	0	3	2
A	__construct()	0	15	3
A	disconnect()	0	4	2

<?php

/**
 * This file is part of the alphaz Framework.
 *
 * @author Muhammad Umer Farooq (Malik) <mumerfarooqlablnet01@gmail.com>
 *
 * @link https://github.com/alphazframework/framework
 *
 * For the full copyright and license information, please view the LICENSE
 *  file that was distributed with this source code.
 * @since 1.0.0
 *
 * @license MIT
 */

namespace alphaz\Common;

class FTP
{
    /**
     * Connection.
     *
     * @since 1.0.0
     *
     * @var resource
     */
    private $connection;

    /**
     * connectionString.
     *
     * @since 1.0.0
     *
     * @var resource
     */
    private $connectionString;

    /**
     * Instantiate the FTP object.
     *
     * @param (string) $host    server host
     * @param (string) $user    username
     * @param (string) $pass    password
     * @param (string) $secured ftp or sftp
     *
     * @since 1.0.0
     */
    public function __construct($host, $user, $pass, $secured)
    {
        if ($secured === false) {

The condition $secured === false is always false.

            $conn = ftp_connect($host);
            $login_result = ftp_login($conn, $user, $pass);

The assignment to $login_result is dead and can be removed.

            $this->connection = $conn;
            $this->connectionString = 'ftp://'.$user.':'.$pass.'@'.$host;
        } elseif ($secured === true) {

The condition $secured === true is always false.

            $conn = ftp_ssl_connect($host);
            $login_result = ftp_login($conn, $user, $pass);
            $this->connection = $conn;
            $this->connectionString = 'sftp://'.$user.':'.$pass.'@'.$host;
        } else {
            $this->connection = null;
            $this->connectionString = null;
        }
    }

    /**
     * get the connection.
     *
     * @since 1.0.0
     *
     * @return resource
     */
    public function getConnection()
    {
        return $this->connection;
    }

    /**
     * check whether the ftp is connected.
     *
     * @since 1.0.0
     *
     * @return bool
     */
    public function isConnected()
    {
        return (is_resource($this->getConnection())) ? true : false;
    }

    /**
     * get the list of files.
     *
     * @param (string) $dir directory
     *
     * @since 1.0.0
     *
     * @return mixed
     */
    public function ftpFiles($dir)
    {
        return ($this->isConnected() === true) ? ftp_nlist($this->getConnection(), $dir) : false;
    }

    /**
     * get the current working directory.
     *
     * @since 1.0.0
     *
     * @return mixed
     */
    public function pwd()
    {
        return ($this->isConnected() === true) ? ftp_pwd($this->getConnection()) : false;
    }

    /**
     * Change directories.
     *
     * @param (string) $dir directory
     * @param (string) $new naw name
     *
     * @since 1.0.0
     *
     * @return bool
     */
    public function chdir($dir)
    {
        return ($this->isConnected() === true) ? ftp_chdir($this->getConnection(), $dir) : false;
    }

    /**
     * Make directory.
     *
     * @param (string) $dir directory name
     *
     * @since 1.0.0
     *
     * @return bool
     */
    public function mkdir($dir)
    {
        return ($this->isConnected() === true) ? ftp_mkdir($this->getConnection(), $dir) : false;

The expression return $this->isConnected() === true ? ftp_mkdir($this->getConnection(), $dir) : false also could return the type string which is incompatible with the documented return type boolean.

    }

    /**
     * Make nested sub-directories.
     *
     * @param (array) $dirs
     *
     * @since 1.0.0
     *
     * @return object
     */
    public function mkdirs($dirs)
    {
        if (substr($dirs, 0, 1) == '/') {

$dirs of type array is incompatible with the type string expected by parameter $string of substr().

            $dirs = substr($dirs, 1);
        }
        if (substr($dirs, -1) == '/') {
            $dirs = substr($dirs, 0, -1);
        }
        $dirs = explode('/', $dirs);

It seems like $dirs can also be of type array; however, parameter $string of explode() does only seem to accept string, maybe add an additional type check?

        $curDir = $this->connectionString;
        foreach ($dirs as $dir) {
            $curDir .= '/'.$dir;
            if (!is_dir($curDir)) {
                $this->mkdir($dir);
            }
            $this->chdir($dir);
        }

        return $this;
    }

    /**
     * Remove directory.
     *
     * @param (string) $dir directory
     *
     * @since 1.0.0
     *
     * @return bool
     */
    public function rmdir($dir)
    {
        return ($this->isConnected() === true) ? ftp_rmdir($this->getConnection(), $dir) : false;
    }

    /**
     * Check if file exists.
     *
     * @param (string) $file
     *
     * @since 1.0.0
     *
     * @return bool
     */
    public function fileExists($file)
    {
        return ($this->isConnected() === true) ? ftp_rmdir($this->connectionString.$file) : false;

$this->connectionString . $file of type string is incompatible with the type resource expected by parameter $ftp of ftp_rmdir().

Are you sure $this->connectionString of type resource can be used in concatenation?

The call to ftp_rmdir() has too few arguments starting with directory.

    }

    /**
     * Check is the dir is exists.
     *
     * @param (string) $dir directory
     *
     * @since 1.0.0
     *
     * @return bool
     */
    public function dirExists($dir)
    {
        return ($this->isConnected() === true) ? ftp_rmdir($this->connectionString.$dir) : false;

The call to ftp_rmdir() has too few arguments starting with directory.

$this->connectionString . $dir of type string is incompatible with the type resource expected by parameter $ftp of ftp_rmdir().

Are you sure $this->connectionString of type resource can be used in concatenation?

    }

    /**
     * Get the file.
     *
     * @param (mixed) $local  local
     * @param (mixed) $remote remote
     * @param (mixed) $mode   mode
     *
     * @since 1.0.0
     *
     * @return bool
     */
    public function get($local, $remote, $mode = FTP_BINARY)
    {
        return ($this->isConnected() === true) ? ftp_get($this->getConnection(), $local, $remote, $mode) : false;
    }

    /**
     * Rename file.
     *
     * @param (string) $old old
     * @param (string) $new naw name
     *
     * @since 1.0.0
     *
     * @return bool
     */
    public function rename($old, $new)
    {
        return ($this->isConnected() === true) ? ftp_rename($this->getConnection(), $old, $new) : false;
    }

    /**
     * Change permission.
     *
     * @param (string) $file file
     * @param (mixed)  $mode mode
     *
     * @since 1.0.0
     *
     * @return bool
     */
    public function chmod($file, $mode)
    {
        return ($this->isConnected() === true) ? ftp_chmod($this->getConnection(), $mode, $file) : false;

The expression return $this->isConnected() === true ? ftp_chmod($this->getConnection(), $mode, $file) : false also could return the type integer which is incompatible with the documented return type boolean.

    }

    /**
     * Delete the files.
     *
     * @param (string) $file file you want to delete
     *
     * @since 1.0.0
     *
     * @return bool
     */
    public function delete($file)
    {
        return ($this->isConnected() === true) ? ftp_delete($this->getConnection(), $file) : false;
    }

    /**
     * Switch the passive mod.
     *
     * @param (bool) $flag flag
     *
     * @since 1.0.0
     *
     * @return bool
     */
    public function pasv($flag = true)
    {
        return ($this->isConnected() === true) ? ftp_pasv($this->getConnection(), $flag) : false;
    }

    /**
     * Close the FTP connection.
     *
     * @since 1.0.0
     *
     * @return void
     */
    public function disconnect()
    {
        if ($this->isConnected()) {
            ftp_close($this->connection);
        }
    }

    /**
     * Upload the files.
     *
     * @param (array)  $files number of files you want to uplaod
     * @param (string) $root  Server root directory or sub
     *
     * @since 1.0.0
     *
     * @return mixed
     */
    public function put($files, $root = 'public_html')
    {
        if ($this->isConnected() === true) {
            foreach ($files as $key => $value) {
                ftp_put($this->getConnection(), $root.'/'.$value, $value, FTP_ASCII);
            }
        } else {
            return false;
        }
    }
}