Pleets / dronephp

src/Exception/StorableTrait.php

<?php
/**
 * DronePHP (http://www.dronephp.com)
 *
 * @link      http://github.com/Pleets/DronePHP
 * @copyright Copyright (c) 2016-2018 Pleets. (http://www.pleets.org)
 * @license   http://www.dronephp.com/license
 * @author    Darío Rivera <fermius.us@gmail.com>
 */

namespace Drone\Exception;

/**
 * StorableTrait Trait
 *
 * This is a helper tait that provides essential methods to store Exceptions.
 * All Exceptions that extends from this, will be stored with store() method.
 */
trait StorableTrait
{
    use \Drone\Error\ErrorTrait;

    /**
     * Local file when exceptions will be stored
     *
     * @var string
     */
    protected $outputFile;

    /**
     * Returns the outputFile attribute
     *
     * @return string
     */
    public function getOutputFile()
    {
        return $this->outputFile;
    }

    /**
     * Sets outputFile attribute
     *
     * @param string $value
     *
     * @return null
     */
    public function setOutputFile($value)
    {
        $this->outputFile = $value;
    }

    /**
     * Stores the exception
     *
     * By default exceptions are stored in a JSON file << $this->outputFile >>
     *
     * @return string|boolean
     */
    public function store()
    {
        $storage = new Storage($this->outputFile);

        $st = $storage->store($this);

$this of type Drone\Exception\StorableTrait is incompatible with the type Exception expected by parameter $exception of Drone\Exception\Storage::store().

        if (!$st) {

The condition $st is always false.

            $_errors = $st->getErrors();

            foreach ($_errors as $errno => $error) {
                $this->error($errno, $error);

The method error() does not exist on Exception. It seems like you code against a sub-type of Exception such as Drone\Exception\Exception.

            }
        }

        return $st;
    }
}