library\storage\storage\FilesStorage

Complexity

Total Complexity

12

Size/Duplication

Total Lines	96
Duplicated Lines	19.79 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getFiles()	0	6	1
B	addFile()	0	25	3
A	getFileByName()	0	11	3
A	deleteFileByName()	19	19	4
A	compareFiles()	0	4	1

<?php
/**
 * Created by jensk on 17-3-2017.
 */

namespace library\storage\storage;


class FilesStorage extends Storage
{
	/**
	 * @return array
	 */
	public function getFiles() {
		$files = $this->repository->files;
		usort($files, array($this, 'compareFiles'));

		return $files;
	}

	/**
	 * @param $postValues
	 *
	 * @throws \Exception
	 */
	public function addFile($postValues)
	{
		$destinationPath = realpath(__DIR__ . '/../../../www/files/');

		$filename = $this->validateFilename($postValues['name'], $destinationPath);
		$destination = $destinationPath . '/' . $filename;

		if ($postValues['error'] != '0') {
			throw new \Exception('Error uploading file. Error code: ' . $postValues['error']);
		}

		if (move_uploaded_file($postValues['tmp_name'], $destination)) {
			$file = new \stdClass();
			$file->file = $filename;
			$file->type = $postValues['type'];
			$file->size = $postValues['size'];

			$files = $this->repository->files;
			$files[] = $file;
			$this->repository->files = $files;
			$this->save();
		} else {
			throw new \Exception('Error moving uploaded file');
		}
	}

	/**
	 * @param $filename
	 *
	 * @return null
	 */
	public function getFileByName($filename)
	{
		$files = $this->getFiles();
		foreach ($files as $file) {
			if ($filename == $file->file) {
				return $file;
			}
		}

		return null;
	}

	/**
	 * @param $filename
	 *
	 * @throws \Exception
	 */
	public function deleteFileByName($filename)

This method seems to be duplicated in your project.

	{
		$destinationPath = realpath(__DIR__ . '/../../../www/files/');
		$destination = $destinationPath . '/' . $filename;

		if (file_exists($destination)) {
			$files = $this->getFiles();
			foreach ($files as $key => $file) {
				if ($file->file == $filename) {
					unlink($destination);
					unset($files[$key]);
				}
			}

			$files = array_values($files);
			$this->repository->files = $files;
			$this->save();
		}
	}

	/**
	 * @param $a
	 * @param $b
	 *
	 * @return int
	 */
	private function compareFiles($a, $b)
	{
		return strcmp($a->file, $b->file);
	}
}