library\storage\storage\AbstractStorage

Complexity

Total Complexity

6

Size/Duplication

Total Lines	62
Duplicated Lines	35.48 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
B	validateFilename()	22	30	4
A	save()	0	4	1

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

namespace library\storage\storage;


use library\storage\Repository;

abstract class AbstractStorage
{
	/**
	 * @var Repository
	 */
	protected $repository;

	public function __construct($repository)
	{
		$this->repository = $repository;
	}

	/**
	 * Converts filename to lowercase, remove non-ascii chars
	 * And adds "-copy" if the file already exists
	 *
	 * @param $filename
	 * @param $path
	 *
	 * @return string
	 */
	protected function validateFilename($filename, $path)
	{
		$fileParts = explode('.', $filename);
		if (count($fileParts) > 1) {

This code seems to be duplicated across your project.

			$extension = end($fileParts);
			array_pop($fileParts);
			$fileNameWithoutExtension = implode('-', $fileParts);
			$fileNameWithoutExtension = slugify($fileNameWithoutExtension);
			$filename = $fileNameWithoutExtension . '.' . $extension;
		} else {
			$filename = slugify($filename);
		}

		if (file_exists($path . '/' . $filename)) {

This code seems to be duplicated across your project.

			$fileParts = explode('.', $filename);
			if (count($fileParts) > 1) {
				$extension = end($fileParts);
				array_pop($fileParts);
				$fileNameWithoutExtension = implode('-', $fileParts);
				$fileNameWithoutExtension .= '-copy';
				$filename = $fileNameWithoutExtension . '.' . $extension;
			} else {
				$filename .= '-copy';
			}

			return $this->validateFilename($filename, $path);
		}

		return $filename;
	}

	/**
	 * Save changes made to the repository
	 *
	 * @throws \Exception
	 */
	protected function save()
	{
		$this->repository->save();
	}
}