byjg / restserver

src/HandleOutput/JsonHandler.php

<?php

namespace ByJG\RestServer\HandleOutput;

use ByJG\RestServer\ServiceAbstract;
use ByJG\RestServer\Whoops\JsonResponseHandler;
use ByJG\Serializer\Formatter\JsonFormatter;

class JsonHandler implements HandleOutputInterface

This class seems to be duplicated in your project.

{

    public function writeHeader()
    {
        header('Content-Type: application/json');
        // header('Access-Control-Allow-Origin: *');

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

        // header("Access-Control-Allow-Headers: X-API-KEY, Origin, X-Requested-With, Content-Type, Accept, Access-Control-Request-Method");

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

        // header("Access-Control-Allow-Methods: GET, POST, OPTIONS, PUT, DELETE");

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

    }

    public function writeOutput(ServiceAbstract $instance)
    {
        $serialized = $instance->getResponse()->getResponseBag()->process();
        return (new JsonFormatter())->process($serialized);
    }

    public function getErrorHandler()
    {
        return new JsonResponseHandler();
    }
}