ByTIC\MediaLibrary\PathGenerator\AbstractPathGenerator

Complexity

Total Complexity

6

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Importance

Changes	2
Bugs	1	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getBasePathForMediaConversion()	0	3	1
A	getBasePathForMediaOriginal()	0	9	2
A	getFolderNameForMedia()	0	8	2
A	getBasePathForMedia()	0	6	1

<?php

namespace ByTIC\MediaLibrary\PathGenerator;

use ByTIC\MediaLibrary\Media\Media;

/**
 * Class AbstractPathGenerator.
 */
abstract class AbstractPathGenerator
{
    /**
     * @param Media $media
     *
     * @return string
     */
    public static function getBasePathForMediaOriginal($media)
    {
        $basePath = self::getBasePathForMedia($media);
        $originalPath = $media->getCollection()->getOriginalPath();
        if (!empty($originalPath)) {
            $basePath .= DIRECTORY_SEPARATOR.$media->getCollection()->getOriginalPath();
        }

        return $basePath;
    }

    /**
     * @param Media $media
     *
     * @return string
     */
    public static function getBasePathForMedia($media)
    {
        return '/'.$media->getCollection()->getName()
            .'/'.static::getFolderNameForMedia($media)
            .'/'.$media->getModel()->getPrimaryKey()

Are you sure $media->getModel()->getPrimaryKey() of type array|mixed can be used in concatenation?

            .'/';
    }

    /**
     * @param Media $media
     *
     * @return string
     */
    public static function getFolderNameForMedia($media)
    {
        $model = $media->getModel();
        if (method_exists($model, 'getFolderNameForMedia')) {
            return $model->getFolderNameForMedia();

The expression return $model->getFolderNameForMedia() returns the type Nip\Records\AbstractModels\Record|Nip\Records\Collections\Collection which is incompatible with the documented return type string.

        }

        return $media->getModel()->getManager()->getTable();
    }

    /**
     * @param Media  $media
     * @param string $conversionName
     *
     * @return string
     */
    public static function getBasePathForMediaConversion($media, $conversionName)
    {
        return self::getBasePathForMedia($media).DIRECTORY_SEPARATOR.$conversionName;
    }
}