Jackal\ImageMerge\Metadata\Parser\IPTCParser

Complexity

Total Complexity

11

Size/Duplication

Total Lines	118
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	1

9 Methods

Rating	Name	Duplication	Size	Complexity
A	getCreationDateTime()	0	9	2
A	__construct()	0	8	2
A	getKeywords()	0	3	1
A	getCreator()	0	3	1
A	getCategory()	0	3	1
A	getDescription()	0	3	1
A	getCopyrights()	0	3	1
A	isUTF8()	0	3	1
A	toArray()	0	10	1

<?php

namespace Jackal\ImageMerge\Metadata\Parser;

use DateTime;
use Exception;
use Jackal\ImageMerge\Model\File\FileObjectInterface;

/**
 * Class IPTCParser
 * @package Jackal\ImageMerge\Metadata\Parser
 */
class IPTCParser extends AbstractParser
{
    const TITLE = '2#005';
    const URGENCY = '2#010';
    const CATEGORY = '2#015';
    const SUB_CATEGORY = '2#020';
    const SPECIAL_INSTRUCTION = '2#040';
    const CREATION_DATE = '2#055';
    const CREATION_TIME = '2#060';
    const DIGITAL_CREATION_DATE = '2#062';
    const DIGITAL_CREATION_TIME = '2#063';
    const BY_LINE = '2#080';
    const BY_LINE_TITLE = '2#085';
    const CITY = '2#090';
    const LOCATION = '2#092';
    const STATE = '2#095';
    const COUNTRY_CODE = '2#100';
    const COUNTRY_NAME = '2#101';
    const OTR = '2#103';
    const HEADLINE = '2#105';
    const CREDIT = '2#110';
    const SOURCE = '2#115';
    const COPYRIGHT = '2#116';
    const CONTACT = '2#118';
    const CAPTION = '2#120';
    const CAPTION_WRITER = '2#122';
    const CHARSET = '1#090';
    const KEYWORDS = '2#025';

    /**
     * IPTCParser constructor.
     * @param FileObjectInterface $file
     */
    public function __construct(FileObjectInterface $file)
    {
        @iptcembed('', $file->getPathname(), 0);

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

        $info = null;
        getimagesize($file->getPathname(), $info);

        if (isset($info['APP13'])) {
            $this->data = iptcparse($info['APP13']);

It seems like iptcparse($info['APP13']) can also be of type false. However, the property $data is declared as type array. Maybe add an additional type check?

        }
    }

    /**
     * @return array|null|string
     */
    public function getCategory()
    {
        return $this->getValue(self::CATEGORY);
    }

    /**
     * @return DateTime|null
     * @throws Exception
     */
    public function getCreationDateTime()
    {
        if ($this->getSingleValue(self::CREATION_DATE)) {
            $dt = trim($this->getSingleValue(self::CREATION_DATE) . ' ' . $this->getSingleValue(self::CREATION_TIME));

            return new DateTime($dt);
        }

        return null;
    }

    /**
     * @return array|null|string
     */
    public function getKeywords()
    {
        return $this->getValue(self::KEYWORDS);
    }

    /**
     * @return bool
     */
    public function isUTF8()
    {
        return $this->getSingleValue(self::CHARSET) == "\x1B%G";
    }

    /**
     * @return null|string
     */
    public function getCreator()
    {
        return $this->getSingleValue(self::CAPTION_WRITER);
    }

    /**
     * @return null|string
     */
    public function getDescription()
    {
        return $this->getSingleValue(self::CAPTION);
    }

    public function getCopyrights()
    {
        return $this->getSingleValue(self::COPYRIGHT);
    }

    /**
     * @return array
     * @throws Exception
     */
    public function toArray()
    {
        return [
            'category' => $this->getCategory(),
            'created_at' => $this->getCreationDateTime(),
            'keywords' => $this->getKeywords(),
            'utf8' => $this->isUTF8(),
            'created_by' => $this->getCreator(),
            'description' => $this->getDescription(),
            'copyrights' => $this->getCopyrights(),
        ];
    }
}