Drone\Exception\Storage::store()

Complexity

Conditions	12
Paths	41

Size

Total Lines	67
Code Lines	32

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	24
CRAP Score	14.25

Importance

Changes	5
Bugs	1	Features	0

<?php
    public function store(\Exception $exception)
    {
        # simple way to generate a unique id
        $id = time() . uniqid();

        $data = [];

        if (file_exists($this->outputFile)) {
            $string = file_get_contents($this->outputFile);

            if (!empty($string)) {
                $data   = json_decode($string, true);

                # json_encode can return TRUE, FALSE or NULL (http://php.net/manual/en/function.json-decode.php)
                if (is_null($data) || $data === false) {
                    $this->error(Errno::JSON_DECODE_ERROR, $this->outputFile);

The method error() does not exist on Drone\Exception\Storage. Since you implemented __call, consider adding a @method annotation.

                    return false;
                }
            }
        } else {
            $directory = strstr($this->outputFile, basename($this->outputFile), true);

            if (!file_exists($directory)) {
                $this->error(Errno::FILE_NOT_FOUND, $directory);

                return false;
            }
        }

        $data[$id] = [
            "message" => $exception->getMessage(),
            "object"  => serialize($exception),
        ];

        if (!function_exists('mb_detect_encoding')) {
            throw new \RuntimeException("mbstring library is not installed!");
        }

        /*
         * Encodes to UTF8 all messages. It ensures JSON encoding.
         */
        if (!mb_detect_encoding($data[$id]["message"], 'UTF-8', true)) {
            $data[$id]["message"] = utf8_encode($data[$id]["message"]);
        }

        if (!mb_detect_encoding($data[$id]["object"], 'UTF-8', true)) {
            $data[$id]["object"] = utf8_decode($data[$id]["object"]);
        }

        if (($encoded_data = json_encode($data)) === false) {
            $this->error(Errno::JSON_ENCODE_ERROR, $this->outputFile);

            return false;
        }

        $hd = @fopen($this->outputFile, "w+");

        if (!$hd || !@fwrite($hd, $encoded_data)) {

$hd is of type false|resource, thus it always evaluated to false.

            $this->error(Errno::FILE_PERMISSION_DENIED, $this->outputFile);

            return false;
        }

        @fclose($hd);

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

        return $id;
    }