TreeHouse\Feeder\Resource\Transformer\RemoveByteOrderMarkTransformer

Complexity

Total Complexity

8

Size/Duplication

Total Lines	92
Duplicated Lines	45.65 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

75.68%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	10	1
B	transform()	31	31	3
A	needsTransforming()	11	11	1
A	rename()	0	6	2
A	getBomRegex()	0	4	1

<?php

namespace TreeHouse\Feeder\Resource\Transformer;

use TreeHouse\Feeder\Resource\FileResource;
use TreeHouse\Feeder\Resource\ResourceCollection;
use TreeHouse\Feeder\Resource\ResourceInterface;
use TreeHouse\Feeder\Transport\FileTransport;

/**
 * Strips the BOM from the beginning of the resource.
 */
class RemoveByteOrderMarkTransformer implements ResourceTransformerInterface
{
    /**
     * @var array
     */
    protected $boms;

    /**
     * Constructor.
     */
    public function __construct()
    {
        $this->boms = [
            '\x00\x00\xFE\xFF', // UTF-32 (BE)

38% of this comment could be valid code. Did you maybe forget this after debugging?

            '\xFF\xFE\x00\x00', // UTF-32 (LE)

38% of this comment could be valid code. Did you maybe forget this after debugging?

            '\xFE\xFF',         // UTF-16 (BE)

38% of this comment could be valid code. Did you maybe forget this after debugging?

            '\xFF\xFE',         // UTF-16 (LE)

38% of this comment could be valid code. Did you maybe forget this after debugging?

            '\xEF\xBB\xBF',     // UTF-8
        ];
    }

    /**
     * @inheritdoc
     */
    public function transform(ResourceInterface $resource, ResourceCollection $collection)

This method seems to be duplicated in your project.

    {
        $file = $resource->getFile()->getPathname();

        // the file could be big, so just read the
        $tmpFile = tempnam(sys_get_temp_dir(), $file);
        $old = fopen($file, 'r');
        $new = fopen($tmpFile, 'w');

        // write the beginning with the BOM stripped
        fwrite($new, preg_replace($this->getBomRegex(), '', fread($old, 16)));

        // now copy the rest of the file
        while (!feof($old)) {
            fwrite($new, fread($old, 8192));
        }

        fclose($old);
        fclose($new);

        // atomic write
        $this->rename($tmpFile, $file);

        $transport = FileTransport::create($file);

        if ($resource->getTransport()) {
            $transport->setDestinationDir($resource->getTransport()->getDestinationDir());
        }

        return new FileResource($transport);
    }

    /**
     * @inheritdoc
     */
    public function needsTransforming(ResourceInterface $resource)

This method seems to be duplicated in your project.

    {
        $file = $resource->getFile()->getPathname();
        $handle = fopen($file, 'r');

        $result = (bool) preg_match($this->getBomRegex(), fread($handle, 16));

        fclose($handle);

        return $result;
    }

    /**
     * @param string $old
     * @param string $new
     *
     * @throws \RuntimeException
     */
    protected function rename($old, $new)
    {
        if (!rename($old, $new)) {
            throw new \RuntimeException(sprintf('Could not rename %s to %s', $old, $new));
        }
    }

    /**
     * @return string
     */
    protected function getBomRegex()
    {
        return sprintf('/^(%s)/', implode('|', $this->boms));
    }
}