Jaxon\App\Traits\AjaxSendTrait

Complexity

Total Complexity

4

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	sendResponse()	0	24	4

<?php

/**
 * AjaxSendTrait.php
 *
 * Send Jaxon ajax response.
 *
 * @package jaxon-core

Package name "jaxon-core" is not valid; consider "Jaxoncore" instead

 * @author Jared White

Content of the @author tag must be in the form "Display Name <username@example.com>"

 * @author J. Max Wilson

Content of the @author tag must be in the form "Display Name <username@example.com>"

 * @author Thierry Feuzeu

Content of the @author tag must be in the form "Display Name <username@example.com>"

 * @copyright Copyright (c) 2005-2007 by Jared White & J. Max Wilson

@copyright tag must contain a year and the name of the copyright holder

 * @copyright 2022 Thierry Feuzeu <thierry.feuzeu@gmail.com>
 * @license https://opensource.org/licenses/BSD-3-Clause BSD 3-Clause License
 * @link https://github.com/jaxon-php/jaxon-core
 */

PHP version not specified

Missing @category tag in file comment

namespace Jaxon\App\Traits;

use Jaxon\Exception\RequestException;

use function gmdate;
use function header;
use function headers_sent;

trait AjaxSendTrait

Missing doc comment for trait AjaxSendTrait

{
    /**
     * Prints the response to the output stream, thus sending the response to the browser
     *
     * @return void
     * @throws RequestException
     */
    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;
    }

Expected 2 blank lines after function; 0 found

}