h4kuna\Upload\Driver\Ftp

Complexity

Total Complexity

6

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	createURI()	0	4	1
A	isFileExists()	0	4	1
A	save()	0	10	2
A	remove()	0	7	1

<?php

namespace h4kuna\Upload\Driver;

use h4kuna\Upload,
	Nette\Http;

class Ftp implements Upload\IDriver
{
	/** @var string */
	private $hostUrl;

	/** @var \Ftp */
	private $ftp;

	public function __construct($hostUrl, \Ftp $ftp)
	{
		$this->hostUrl = $hostUrl;
		$this->ftp = $ftp;
	}

	public function createURI($relativePath)
	{
		return $this->hostUrl . '/' . Upload\Utils::makeRelativePath($relativePath);
	}

	public function isFileExists($relativePath)
	{
		return $this->ftp->fileExists(Upload\Utils::makeRelativePath($relativePath));
	}

	public function save(Http\FileUpload $fileUpload, $relativePath)
	{
		$path = Upload\Utils::makeRelativePath($relativePath);
		$dir = dirname($relativePath);
		if ($dir !== '.') {
			$this->ftp->mkDirRecursive($dir);
		}
		$this->ftp->put($path, $fileUpload->getTemporaryFile(), FTP_BINARY);
		@unlink($fileUpload->getTemporaryFile());

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

	}

	public function remove($relativePath)
	{
		$path = Upload\Utils::makeRelativePath($relativePath);
		$this->ftp->delete($path);

		return !$this->isFileExists($path);
	}
}