Jaxon\App\Traits\AjaxSendTrait::sendResponse()

Complexity

Conditions	4
Paths	4

Size

Total Lines	24
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function sendResponse()

Expected 2 blank lines before function; 0 found

    {
        // Check to see if headers have already been sent out, in which case we can't do our job
        if(headers_sent($sFilename, $nLineNumber))
        {
            throw new RequestException($this->xTranslator->trans('errors.output.already-sent',
                    ['location' => $sFilename . ':' . $nLineNumber]) . "\n" .
                $this->xTranslator->trans('errors.output.advice'));
        }
        if(empty($sContent = $this->xResponseManager->getOutput()))

Variable assignment found within a condition. Did you mean to do a comparison ?

        {
            return;
        }

        if($this->di()->getRequest()->getMethod() === 'GET')

It seems like di() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        {
            header("Expires: Mon, 26 Jul 1997 05:00:00 GMT");
            header("Last-Modified: " . gmdate("D, d M Y H:i:s") . " GMT");
            header("Cache-Control: no-cache, must-revalidate");
            header("Pragma: no-cache");
        }
        header('content-type: ' . $this->xResponseManager->getContentType());

        print $sContent;
    }