NotificationChannels\Flock\FlockAttachmentViewImage

Complexity

Total Complexity

13

Size/Duplication

Total Lines	95
Duplicated Lines	40 %

Coupling/Cohesion

Components	3
Dependencies	1

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
B	original()	19	19	6
B	thumbnail()	19	19	6
A	filename()	0	6	1

<?php

namespace NotificationChannels\Flock;

use NotificationChannels\Flock\Exceptions\CouldNotSendNotification;

class FlockAttachmentViewImage
{
    /**
     * Image object.
     *
     * @var array
     */
    public $original;

    /**
     * Thumbnail object.
     *
     * @var array
     */
    public $thumbnail;

    /**
     * Filename for the attached image.
     *
     * @var string
     */
    public $filename;

    /**
     * Attach origin al image to attachment.
     *
     * @param string $src
     * @param int|null $height
     * @param int|null $width
     *
     * @return $this
     */
    public function original($src, $height, $width)

This method seems to be duplicated in your project.

    {
        if (! filter_var($src, FILTER_VALIDATE_URL)) {
            throw CouldNotSendNotification::flockAttachmentViewImageException('Source of image in attachment is missing or invalid.');
        }
        $this->original['src'] = $src;

        if (! filter_var($height, FILTER_VALIDATE_INT) or $height <= 0) {

Using logical operators such as or instead of || is generally not recommended.

            throw CouldNotSendNotification::flockAttachmentViewImageException('Height of image in attachment is missing or invalid.');
        }
        $this->original['height'] = $height;

        if (! filter_var($width, FILTER_VALIDATE_INT) or $width <= 0) {

Using logical operators such as or instead of || is generally not recommended.

            throw CouldNotSendNotification::flockAttachmentViewImageException('Width of image in attachment is missing or invalid.');
        }
        $this->original['width'] = $width;

        return $this;
    }

    /**
     * Attach thumbnail to attachment.
     *
     * @param string $src
     * @param int|null $height
     * @param int|null $width
     *
     * @return $this
     */
    public function thumbnail($src, $height, $width)

This method seems to be duplicated in your project.

    {
        if (! filter_var($src, FILTER_VALIDATE_URL)) {
            throw CouldNotSendNotification::flockAttachmentViewImageException('Source of image in attachment is missing or invalid.');
        }
        $this->thumbnail['src'] = $src;

        if (! filter_var($height, FILTER_VALIDATE_INT) or $height <= 0) {

Using logical operators such as or instead of || is generally not recommended.

            throw CouldNotSendNotification::flockAttachmentViewImageException('Height of image in attachment is missing or invalid.');
        }
        $this->thumbnail['height'] = $height;

        if (! filter_var($width, FILTER_VALIDATE_INT) or $width <= 0) {

Using logical operators such as or instead of || is generally not recommended.

            throw CouldNotSendNotification::flockAttachmentViewImageException('Width of image in attachment is missing or invalid.');
        }
        $this->thumbnail['width'] = $width;

        return $this;
    }

    /**
     * Add file name to attached image.
     *
     * @param string $filename
     *
     * @return $this
     */
    public function filename($filename)
    {
        $this->filename = $filename;

        return $this;
    }
}