ChangesIn40834::removeResponseHeaders() - Code Metrics - Inspection of "Daily Inspection: [touch:10738]{t:20}resolved a we..." - eventespresso/event-espresso-core - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — BUG-10738-inconsistency-in-ses... (cda363)

unknown

created 2017-09-22 21:14 UTC

ChangesIn40834::removeResponseHeaders() A

↳ Parent: ChangesIn40834

Complexity

Conditions	3
Paths	2

Size

Total Lines	9
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	3
eloc	5
nc	2
nop	3
dl	0
loc	9
rs	9.6666
c	0
b	0
f	0

<?php namespace EventEspresso\core\libraries\rest_api\changes;

use EventEspresso\core\libraries\rest_api\controllers\Base;

/* 
 * The checkin and checkout endpoints were added in 4.8.34, 
 * where we just added a response headers
 */



class ChangesIn40834 extends ChangesInBase
{

    /**
     * Adds hooks so requests to 4.8.29 don't have the checkin endpoints
     */
    public function setHooks()
    {
        //set a hook to remove the checkout/checkout endpoints if the request
        //is for lower than 4.8.33
        add_filter(
            'FHEE__EventEspresso\core\libraries\rest_api\controllers\Base___get_response_headers',
            array($this, 'removeResponseHeaders'),
            10,
            3
        );
    }



    /**
     * Removes the checkin and checkout endpoints from the index for requests
     * to api versions lowers than 4.8.33
     *
     * @param array  $response_headers
     * @param Base   $controller
     * @param string $requested_version
     * @return array like $routes_on_this_version
     */
    public function removeResponseHeaders($response_headers, $controller, $requested_version)
    {
        if ($controller instanceof Base
            && $this->appliesToVersion($requested_version)
        ) {
            return array();
        }
        return $response_headers;
    }
}


1			<?php namespace EventEspresso\core\libraries\rest_api\changes;
2
3			use EventEspresso\core\libraries\rest_api\controllers\Base;
4
5			/*
6			* The checkin and checkout endpoints were added in 4.8.34,
7			* where we just added a response headers
8			*/
9
10
11
12			class ChangesIn40834 extends ChangesInBase
13			{
14
15			/**
16			* Adds hooks so requests to 4.8.29 don't have the checkin endpoints
17			*/
18			public function setHooks()
19			{
20			//set a hook to remove the checkout/checkout endpoints if the request
21			//is for lower than 4.8.33
22			add_filter(
23			'FHEE__EventEspresso\core\libraries\rest_api\controllers\Base___get_response_headers',
24			array($this, 'removeResponseHeaders'),
25			10,
26			3
27			);
28			}
29
30
31
32			/**
33			* Removes the checkin and checkout endpoints from the index for requests
34			* to api versions lowers than 4.8.33
35			*
36			* @param array $response_headers
37			* @param Base $controller
38			* @param string $requested_version
39			* @return array like $routes_on_this_version
40			*/
41			public function removeResponseHeaders($response_headers, $controller, $requested_version)
42			{
43			if ($controller instanceof Base
44			&& $this->appliesToVersion($requested_version)
45			) {
46			return array();
47			}
48			return $response_headers;
49			}
50			}
51

eventespresso / event-espresso-core

Branch — BUG-10738-inconsistency-in-ses... (cda363)

ChangesIn40834::removeResponseHeaders() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like