SF2Helpers\SFTPBundle\SFTP\SFTP

Complexity

Total Complexity

10

Size/Duplication

Total Lines	95
Duplicated Lines	17.89 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	connect()	0	14	2
A	connectWithKey()	0	10	1
A	copy()	9	9	2
A	send()	8	8	2
A	getFilesList()	0	11	2
A	disconnect()	0	5	1

<?php

namespace SF2Helpers\SFTPBundle\SFTP;

class SFTP implements ConnectionInterface, ResourceTransferInterface
{
    private $connection; // connection instance

    private $sftp; // established sftp

    /**
     * @param string $host
     * @param string $username
     * @param string|null $password
     */
    public function connect($host, $username, $password = null)
    {
        $connection = ssh2_connect($host);

        if (is_null($password)) {
            ssh2_auth_agent($connection, $username);
        } else {
            ssh2_auth_password($connection, $username, $password);
        }

        $sftp = ssh2_sftp($connection);
        $this->connection = $connection;
        $this->sftp = $sftp;
    }

    /**
     * @param string $host
     * @param string $username
     * @param string  $pubkeyfile
     * @param string $privkeyfile
     * @param string|null $passphrase
     */
    public function connectWithKey($host, $username, $pubkeyfile, $privkeyfile, $passphrase = null)
    {
        $connection = ssh2_connect($host);

        ssh2_auth_pubkey_file($connection, $username, $pubkeyfile, $privkeyfile, $passphrase);

        $sftp = ssh2_sftp($connection);
        $this->connection = $connection;
        $this->sftp = $sftp;
    }

    /**
     * @param $remoteFile
     * @param $localFile
     * @throws \Exception
     */
    public function copy($remoteFile, $localFile)

This method seems to be duplicated in your project.

    {
        $sftp = "ssh2.sftp://$this->sftp";
        $data = file_get_contents($sftp . $remoteFile);
        if (!$data) {
            throw new \Exception('File can`t be loaded from server');
        }
        file_put_contents($localFile, $data);
    }

    /**
     * @param $localFile
     * @param $remoteFile
     * @throws \Exception
     */
    public function send($localFile, $remoteFile)

This method seems to be duplicated in your project.

    {
        $sftp = "ssh2.sftp://$this->sftp";
        $data = file_get_contents($localFile);
        if (!file_put_contents($sftp . $remoteFile, $data)) {
            throw new \Exception('File could not be uploaded to server');
        }
    }

    /**
     * @param $dir
     * @return array
     */
    public function getFilesList($dir)
    {
        $handle = opendir("ssh2.sftp://$this->sftp" . $dir);
        $files = array();

        while (false != ($entry = readdir($handle))) {

It seems like you are loosely comparing $entry = readdir($handle) of type string to the boolean false. If you are specifically checking for a non-empty string, consider using the more explicit !== '' instead.

            $files[] = $entry;
        }

        return $files;
    }

    public function disconnect()
    {
        ssh2_exec($this->connection, 'exit');
        unset($this->connection);
    }
}