Naneau\Obfuscator\Obfuscator\Event\FileError

Complexity

Total Complexity

2

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes	1
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getErrorMessage()	0	4	1

<?php
/**
 * The file that handles parsing error events
 *
 * @package         Obfuscator
 * @subpackage      Obfuscator
 */

namespace Naneau\Obfuscator\Obfuscator\Event;

use Symfony\Component\EventDispatcher\Event;
use Naneau\Obfuscator\Obfuscator\Event\File;

/**
 * FileError
 *
 * The file being obfuscated that causes an error
 *
 * @category        Naneau
 * @package         Obfuscator
 * @subpackage      Obfuscator
 */
class FileError extends File
{
    /**
     * The error message from Exception
     * @var string
     **/
    private $errorMessage;

    /**
     * Constructor
     *
     * @param string $file
     * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     **/
    public function __construct($file, $errorMessage)
    {
        parent::setFile($file);

It seems like you call parent on a different method (setFile() instead of __construct()). Are you sure this is correct? If so, you might want to change this to $this->setFile().

        $this->errorMessage = $errorMessage;
    }

    /**
     * Get the error message
     *
     * @return string
     */
    public function getErrorMessage()
    {
        return $this->errorMessage;
    }
}