silverstripe / silverstripe-environmentcheck

src/Checks/CacheHeadersCheck.php

Spacing +4 added lines, -4 removed lines

@@ -123,7 +123,7 @@ 
         $goodTypes = [ValidationResult::TYPE_GOOD, ValidationResult::TYPE_INFO];
         $good = array_filter(
             $this->result->getMessages(),
-            function ($val, $key) use ($goodTypes) {
+            function($val, $key) use ($goodTypes) {
                 if (in_array($val['messageType'], $goodTypes)) {
                     return true;
                 }
@@ -132,14 +132,14 @@ 
             ARRAY_FILTER_USE_BOTH
         );
         if (!empty($good)) {
-            $ret .= "GOOD: " . implode('; ', array_column($good, 'message')) . " ";
+            $ret .= "GOOD: ".implode('; ', array_column($good, 'message'))." ";
         }
 
         // Filter bad messages
         $badTypes = [ValidationResult::TYPE_ERROR, ValidationResult::TYPE_WARNING];
         $bad = array_filter(
             $this->result->getMessages(),
-            function ($val, $key) use ($badTypes) {
+            function($val, $key) use ($badTypes) {
                 if (in_array($val['messageType'], $badTypes)) {
                     return true;
                 }
@@ -148,7 +148,7 @@ 
             ARRAY_FILTER_USE_BOTH
         );
         if (!empty($bad)) {
-            $ret .= "BAD: " . implode('; ', array_column($bad, 'message'));
+            $ret .= "BAD: ".implode('; ', array_column($bad, 'message'));
         }
         return $ret;
     }

Indentation +184 added lines, -184 removed lines

@@ -20,188 +20,188 @@
  */
 class CacheHeadersCheck implements EnvironmentCheck
 {
-    use Fetcher;
-
-    /**
-     * Settings that must be included in the Cache-Control header
-     *
-     * @var array
-     */
-    protected $mustInclude = [];
-
-    /**
-     * Settings that must be excluded in the Cache-Control header
-     *
-     * @var array
-     */
-    protected $mustExclude = [];
-
-    /**
-     * Result to keep track of status and messages for all checks, reuses
-     * ValidationResult for convenience.
-     *
-     * @var ValidationResult
-     */
-    protected $result;
-
-    /**
-     * Set up with URL, arrays of header settings to check.
-     *
-     * @param string $url
-     * @param array $mustInclude Settings that must be included in Cache-Control
-     * @param array $mustExclude Settings that must be excluded in Cache-Control
-     */
-    public function __construct($url = '', $mustInclude = [], $mustExclude = [])
-    {
-        $this->setURL($url);
-        $this->mustInclude = $mustInclude;
-        $this->mustExclude = $mustExclude;
-    }
-
-    /**
-     * Check that correct caching headers are present.
-     *
-     * @return void
-     */
-    public function check()
-    {
-        // Using a validation result to capture messages
-        $this->result = new ValidationResult();
-
-        $response = $this->client->get($this->getURL());
-        $fullURL = $this->getURL();
-        if ($response === null) {
-            return [
-                EnvironmentCheck::ERROR,
-                "Cache headers check request failed for $fullURL",
-            ];
-        }
-
-        //Check that Etag exists
-        $this->checkEtag($response);
-
-        // Check Cache-Control settings
-        $this->checkCacheControl($response);
-
-        if ($this->result->isValid()) {
-            return [
-                EnvironmentCheck::OK,
-                $this->getMessage(),
-            ];
-        } else {
-            // @todo Ability to return a warning
-            return [
-                EnvironmentCheck::ERROR,
-                $this->getMessage(),
-            ];
-        }
-    }
-
-    /**
-     * Collate messages from ValidationResult so that it is clear which parts
-     * of the check passed and which failed.
-     *
-     * @return string
-     */
-    private function getMessage()
-    {
-        $ret = '';
-        // Filter good messages
-        $goodTypes = [ValidationResult::TYPE_GOOD, ValidationResult::TYPE_INFO];
-        $good = array_filter(
-            $this->result->getMessages(),
-            function ($val, $key) use ($goodTypes) {
-                if (in_array($val['messageType'], $goodTypes)) {
-                    return true;
-                }
-                return false;
-            },
-            ARRAY_FILTER_USE_BOTH
-        );
-        if (!empty($good)) {
-            $ret .= "GOOD: " . implode('; ', array_column($good, 'message')) . " ";
-        }
-
-        // Filter bad messages
-        $badTypes = [ValidationResult::TYPE_ERROR, ValidationResult::TYPE_WARNING];
-        $bad = array_filter(
-            $this->result->getMessages(),
-            function ($val, $key) use ($badTypes) {
-                if (in_array($val['messageType'], $badTypes)) {
-                    return true;
-                }
-                return false;
-            },
-            ARRAY_FILTER_USE_BOTH
-        );
-        if (!empty($bad)) {
-            $ret .= "BAD: " . implode('; ', array_column($bad, 'message'));
-        }
-        return $ret;
-    }
-
-    /**
-     * Check that ETag header exists
-     *
-     * @param ResponseInterface $response
-     * @return void
-     */
-    private function checkEtag(ResponseInterface $response)
-    {
-        $eTag = $response->getHeaderLine('ETag');
-        $fullURL = Controller::join_links(Director::absoluteBaseURL(), $this->url);
-
-        if ($eTag) {
-            $this->result->addMessage(
-                "$fullURL includes an Etag header in response",
-                ValidationResult::TYPE_GOOD
-            );
-            return;
-        }
-        $this->result->addError(
-            "$fullURL is missing an Etag header",
-            ValidationResult::TYPE_WARNING
-        );
-    }
-
-    /**
-     * Check that the correct header settings are either included or excluded.
-     *
-     * @param ResponseInterface $response
-     * @return void
-     */
-    private function checkCacheControl(ResponseInterface $response)
-    {
-        $cacheControl = $response->getHeaderLine('Cache-Control');
-        $vals = array_map('trim', explode(',', $cacheControl));
-        $fullURL = Controller::join_links(Director::absoluteBaseURL(), $this->url);
-
-        // All entries from must contain should be present
-        if ($this->mustInclude == array_intersect($this->mustInclude, $vals)) {
-            $matched = implode(",", $this->mustInclude);
-            $this->result->addMessage(
-                "$fullURL includes all settings: {$matched}",
-                ValidationResult::TYPE_GOOD
-            );
-        } else {
-            $missing = implode(",", array_diff($this->mustInclude, $vals));
-            $this->result->addError(
-                "$fullURL is excluding some settings: {$missing}"
-            );
-        }
-
-        // All entries from must exclude should not be present
-        if (empty(array_intersect($this->mustExclude, $vals))) {
-            $missing = implode(",", $this->mustExclude);
-            $this->result->addMessage(
-                "$fullURL excludes all settings: {$missing}",
-                ValidationResult::TYPE_GOOD
-            );
-        } else {
-            $matched = implode(",", array_intersect($this->mustExclude, $vals));
-            $this->result->addError(
-                "$fullURL is including some settings: {$matched}"
-            );
-        }
-    }
+	use Fetcher;
+
+	/**
+	 * Settings that must be included in the Cache-Control header
+	 *
+	 * @var array
+	 */
+	protected $mustInclude = [];
+
+	/**
+	 * Settings that must be excluded in the Cache-Control header
+	 *
+	 * @var array
+	 */
+	protected $mustExclude = [];
+
+	/**
+	 * Result to keep track of status and messages for all checks, reuses
+	 * ValidationResult for convenience.
+	 *
+	 * @var ValidationResult
+	 */
+	protected $result;
+
+	/**
+	 * Set up with URL, arrays of header settings to check.
+	 *
+	 * @param string $url
+	 * @param array $mustInclude Settings that must be included in Cache-Control
+	 * @param array $mustExclude Settings that must be excluded in Cache-Control
+	 */
+	public function __construct($url = '', $mustInclude = [], $mustExclude = [])
+	{
+		$this->setURL($url);
+		$this->mustInclude = $mustInclude;
+		$this->mustExclude = $mustExclude;
+	}
+
+	/**
+	 * Check that correct caching headers are present.
+	 *
+	 * @return void
+	 */
+	public function check()
+	{
+		// Using a validation result to capture messages
+		$this->result = new ValidationResult();
+
+		$response = $this->client->get($this->getURL());
+		$fullURL = $this->getURL();
+		if ($response === null) {
+			return [
+				EnvironmentCheck::ERROR,
+				"Cache headers check request failed for $fullURL",
+			];
+		}
+
+		//Check that Etag exists
+		$this->checkEtag($response);
+
+		// Check Cache-Control settings
+		$this->checkCacheControl($response);
+
+		if ($this->result->isValid()) {
+			return [
+				EnvironmentCheck::OK,
+				$this->getMessage(),
+			];
+		} else {
+			// @todo Ability to return a warning
+			return [
+				EnvironmentCheck::ERROR,
+				$this->getMessage(),
+			];
+		}
+	}
+
+	/**
+	 * Collate messages from ValidationResult so that it is clear which parts
+	 * of the check passed and which failed.
+	 *
+	 * @return string
+	 */
+	private function getMessage()
+	{
+		$ret = '';
+		// Filter good messages
+		$goodTypes = [ValidationResult::TYPE_GOOD, ValidationResult::TYPE_INFO];
+		$good = array_filter(
+			$this->result->getMessages(),
+			function ($val, $key) use ($goodTypes) {
+				if (in_array($val['messageType'], $goodTypes)) {
+					return true;
+				}
+				return false;
+			},
+			ARRAY_FILTER_USE_BOTH
+		);
+		if (!empty($good)) {
+			$ret .= "GOOD: " . implode('; ', array_column($good, 'message')) . " ";
+		}
+
+		// Filter bad messages
+		$badTypes = [ValidationResult::TYPE_ERROR, ValidationResult::TYPE_WARNING];
+		$bad = array_filter(
+			$this->result->getMessages(),
+			function ($val, $key) use ($badTypes) {
+				if (in_array($val['messageType'], $badTypes)) {
+					return true;
+				}
+				return false;
+			},
+			ARRAY_FILTER_USE_BOTH
+		);
+		if (!empty($bad)) {
+			$ret .= "BAD: " . implode('; ', array_column($bad, 'message'));
+		}
+		return $ret;
+	}
+
+	/**
+	 * Check that ETag header exists
+	 *
+	 * @param ResponseInterface $response
+	 * @return void
+	 */
+	private function checkEtag(ResponseInterface $response)
+	{
+		$eTag = $response->getHeaderLine('ETag');
+		$fullURL = Controller::join_links(Director::absoluteBaseURL(), $this->url);
+
+		if ($eTag) {
+			$this->result->addMessage(
+				"$fullURL includes an Etag header in response",
+				ValidationResult::TYPE_GOOD
+			);
+			return;
+		}
+		$this->result->addError(
+			"$fullURL is missing an Etag header",
+			ValidationResult::TYPE_WARNING
+		);
+	}
+
+	/**
+	 * Check that the correct header settings are either included or excluded.
+	 *
+	 * @param ResponseInterface $response
+	 * @return void
+	 */
+	private function checkCacheControl(ResponseInterface $response)
+	{
+		$cacheControl = $response->getHeaderLine('Cache-Control');
+		$vals = array_map('trim', explode(',', $cacheControl));
+		$fullURL = Controller::join_links(Director::absoluteBaseURL(), $this->url);
+
+		// All entries from must contain should be present
+		if ($this->mustInclude == array_intersect($this->mustInclude, $vals)) {
+			$matched = implode(",", $this->mustInclude);
+			$this->result->addMessage(
+				"$fullURL includes all settings: {$matched}",
+				ValidationResult::TYPE_GOOD
+			);
+		} else {
+			$missing = implode(",", array_diff($this->mustInclude, $vals));
+			$this->result->addError(
+				"$fullURL is excluding some settings: {$missing}"
+			);
+		}
+
+		// All entries from must exclude should not be present
+		if (empty(array_intersect($this->mustExclude, $vals))) {
+			$missing = implode(",", $this->mustExclude);
+			$this->result->addMessage(
+				"$fullURL excludes all settings: {$missing}",
+				ValidationResult::TYPE_GOOD
+			);
+		} else {
+			$matched = implode(",", array_intersect($this->mustExclude, $vals));
+			$this->result->addError(
+				"$fullURL is including some settings: {$matched}"
+			);
+		}
+	}
 }

src/Checks/SessionCheck.php

Spacing +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
         if ($cookie) {
             return [
                 EnvironmentCheck::ERROR,
-                "Sessions are being set for {$fullURL} : Set-Cookie => " . $cookie,
+                "Sessions are being set for {$fullURL} : Set-Cookie => ".$cookie,
             ];
         }
         return [

Indentation +49 added lines, -49 removed lines

@@ -14,58 +14,58 @@
  */
 class SessionCheck implements EnvironmentCheck
 {
-    use Fetcher;
+	use Fetcher;
 
-    /**
-     * Set up check with URL
-     *
-     * @param string $url The route, excluding the domain
-     * @inheritdoc
-     */
-    public function __construct($url = '')
-    {
-        $this->setURL($url);
-    }
+	/**
+	 * Set up check with URL
+	 *
+	 * @param string $url The route, excluding the domain
+	 * @inheritdoc
+	 */
+	public function __construct($url = '')
+	{
+		$this->setURL($url);
+	}
 
-    /**
-     * Check that the response for URL does not create a session
-     *
-     * @return array
-     */
-    public function check()
-    {
-        $response = $this->client->get($this->getURL());
-        $cookie = $this->getCookie($response);
-        $fullURL = $this->getURL();
+	/**
+	 * Check that the response for URL does not create a session
+	 *
+	 * @return array
+	 */
+	public function check()
+	{
+		$response = $this->client->get($this->getURL());
+		$cookie = $this->getCookie($response);
+		$fullURL = $this->getURL();
 
-        if ($cookie) {
-            return [
-                EnvironmentCheck::ERROR,
-                "Sessions are being set for {$fullURL} : Set-Cookie => " . $cookie,
-            ];
-        }
-        return [
-            EnvironmentCheck::OK,
-            "Sessions are not being created for {$fullURL} 

tests/Checks/CacheHeadersCheckTest.php

Indentation +72 added lines, -72 removed lines

@@ -15,87 +15,87 @@
  */
 class CacheHeadersCheckTest extends SapphireTest
 {
-    /**
-     * Test that directives that must be included, are.
-     *
-     * @return void
-     */
-    public function testMustInclude()
-    {
-        // Create a mock and queue responses
-        $mock = new MockHandler([
-            new Response(200, ['Cache-Control' => 'must-revalidate', 'ETag' => '123']),
-            new Response(200, ['Cache-Control' =>'no-cache', 'ETag' => '123']),
-            new Response(200, ['ETag' => '123']),
-            new Response(200, ['Cache-Control' => 'must-revalidate, private', 'ETag' => '123']),
-        ]);
+	/**
+	 * Test that directives that must be included, are.
+	 *
+	 * @return void
+	 */
+	public function testMustInclude()
+	{
+		// Create a mock and queue responses
+		$mock = new MockHandler([
+			new Response(200, ['Cache-Control' => 'must-revalidate', 'ETag' => '123']),
+			new Response(200, ['Cache-Control' =>'no-cache', 'ETag' => '123']),
+			new Response(200, ['ETag' => '123']),
+			new Response(200, ['Cache-Control' => 'must-revalidate, private', 'ETag' => '123']),
+		]);
 
-        $handler = HandlerStack::create($mock);
-        $client = new Client(['handler' => $handler]);
+		$handler = HandlerStack::create($mock);
+		$client = new Client(['handler' => $handler]);
 
-        $cacheHeadersCheck = new CacheHeadersCheck('/', ['must-revalidate']);
-        $cacheHeadersCheck->client = $client;
+		$cacheHeadersCheck = new CacheHeadersCheck('/', ['must-revalidate']);
+		$cacheHeadersCheck->client = $client;
 
-        // Run checks for each response above
-        $this->assertContains(EnvironmentCheck::OK, $cacheHeadersCheck->check());
-        $this->assertContains(EnvironmentCheck::ERROR, $cacheHeadersCheck->check());
-        $this->assertContains(EnvironmentCheck::ERROR, $cacheHeadersCheck->check());
-        $this->assertContains(EnvironmentCheck::OK, $cacheHeadersCheck->check());
-    }
+		// Run checks for each response above
+		$this->assertContains(EnvironmentCheck::OK, $cacheHeadersCheck->check());
+		$this->assertContains(EnvironmentCheck::ERROR, $cacheHeadersCheck->check());
+		$this->assertContains(EnvironmentCheck::ERROR, $cacheHeadersCheck->check());
+		$this->assertContains(EnvironmentCheck::OK, $cacheHeadersCheck->check());
+	}
 
-    /**
-     * Test that directives that must be excluded, are.
-     *
-     * @return void
-     */
-    public function testMustExclude()
-    {
-        // Create a mock and queue responses
-        $mock = new MockHandler([
-            new Response(200, ['Cache-Control' => 'must-revalidate', 'ETag' => '123']),
-            new Response(200, ['Cache-Control' =>'no-cache', 'ETag' => '123']),
-            new Response(200, ['ETag' => '123']),
-            new Response(200, ['Cache-Control' =>'private, no-store', ' ETag' => '123']),
-        ]);
+	/**
+	 * Test that directives that must be excluded, are.
+	 *
+	 * @return void
+	 */
+	public function testMustExclude()
+	{
+		// Create a mock and queue responses
+		$mock = new MockHandler([
+			new Response(200, ['Cache-Control' => 'must-revalidate', 'ETag' => '123']),
+			new Response(200, ['Cache-Control' =>'no-cache', 'ETag' => '123']),
+			new Response(200, ['ETag' => '123']),
+			new Response(200, ['Cache-Control' =>'private, no-store', ' ETag' => '123']),
+		]);
 
-        $handler = HandlerStack::create($mock);
-        $client = new Client(['handler' => $handler]);
+		$handler = HandlerStack::create($mock);
+		$client = new Client(['handler' => $handler]);
 
-        $cacheHeadersCheck = new CacheHeadersCheck('/', [], ["no-store", "no-cache", "private"]);
-        $cacheHeadersCheck->client = $client;
+		$cacheHeadersCheck = new CacheHeadersCheck('/', [], ["no-store", "no-cache", "private"]);
+		$cacheHeadersCheck->client = $client;
 
-        // Run checks for each response above
-        $this->assertContains(EnvironmentCheck::OK, $cacheHeadersCheck->check());
-        $this->assertContains(EnvironmentCheck::ERROR, $cacheHeadersCheck->check());
-        $this->assertContains(EnvironmentCheck::OK, $cacheHeadersCheck->check());
-        $this->assertContains(EnvironmentCheck::ERROR, $cacheHeadersCheck->check());
-    }
+		// Run checks for each response above
+		$this->assertContains(EnvironmentCheck::OK, $cacheHeadersCheck->check());
+		$this->assertContains(EnvironmentCheck::ERROR, $cacheHeadersCheck->check());
+		$this->assertContains(EnvironmentCheck::OK, $cacheHeadersCheck->check());
+		$this->assertContains(EnvironmentCheck::ERROR, $cacheHeadersCheck->check());
+	}
 
-    /**
-     * Test that Etag header must exist in response.
-     *
-     * @return void
-     */
-    public function testEtag()
-    {
-        // Create a mock and queue responses
-        $mock = new MockHandler([
-            new Response(200, ['Cache-Control' => 'must-revalidate', 'ETag' => '123']),
-            new Response(200, ['Cache-Control' =>'no-cache']),
-            new Response(200, ['ETag' => '123']),
-            new Response(200, []),
-        ]);
+	/**
+	 * Test that Etag header must exist in response.
+	 *
+	 * @return void
+	 */
+	public function testEtag()
+	{
+		// Create a mock and queue responses
+		$mock = new MockHandler([
+			new Response(200, ['Cache-Control' => 'must-revalidate', 'ETag' => '123']),
+			new Response(200, ['Cache-Control' =>'no-cache']),
+			new Response(200, ['ETag' => '123']),
+			new Response(200, []),
+		]);
 
-        $handler = HandlerStack::create($mock);
-        $client = new Client(['handler' => $handler]);
+		$handler = HandlerStack::create($mock);
+		$client = new Client(['handler' => $handler]);
 
-        $cacheHeadersCheck = new CacheHeadersCheck('/');
-        $cacheHeadersCheck->client = $client;
+		$cacheHeadersCheck = new CacheHeadersCheck('/');
+		$cacheHeadersCheck->client = $client;
 
-        // Run checks for each response above
-        $this->assertContains(EnvironmentCheck::OK, $cacheHeadersCheck->check());
-        $this->assertContains(EnvironmentCheck::ERROR, $cacheHeadersCheck->check());
-        $this->assertContains(EnvironmentCheck::OK, $cacheHeadersCheck->check());
-        $this->assertContains(EnvironmentCheck::ERROR, $cacheHeadersCheck->check());
-    }
+		// Run checks for each response above
+		$this->assertContains(EnvironmentCheck::OK, $cacheHeadersCheck->check());
+		$this->assertContains(EnvironmentCheck::ERROR, $cacheHeadersCheck->check());
+		$this->assertContains(EnvironmentCheck::OK, $cacheHeadersCheck->check());
+		$this->assertContains(EnvironmentCheck::ERROR, $cacheHeadersCheck->check());
+	}
 }

tests/Checks/SessionCheckTest.php

Indentation +50 added lines, -50 removed lines

@@ -15,62 +15,62 @@
  */
 class SessionCheckTest extends SapphireTest
 {
-    /**
-     * @var SilverStripe\EnvironmentCheck\Checks\SessionCheck
-     */
-    public $sessionCheck = null;
+	/**
+	 * @var SilverStripe\EnvironmentCheck\Checks\SessionCheck
+	 */
+	public $sessionCheck = null;
 
-    /**
-     * Create a session check for use by tests.
-     *
-     * @return void
-     */
-    protected function setUp()
-    {
-        parent::setUp();
-        $this->sessionCheck = new SessionCheck('/');
-    }
+	/**
+	 * Create a session check for use by tests.
+	 *
+	 * @return void
+	 */
+	protected function setUp()
+	{
+		parent::setUp();
+		$this->sessionCheck = new SessionCheck('/');
+	}
 
-    /**
-     * Env check reports error when session cookies are being set.
-     *
-     * @return void
-     */
-    public function testSessionSet()
-    {
-        // Create a mock and queue two responses.
-        $mock = new MockHandler([
-            new Response(200, ['Set-Cookie' => 'PHPSESSID:foo']),
-            new Response(200, ['Set-Cookie' => 'SECSESSID:bar'])
-        ]);
+	/**
+	 * Env check reports error when session cookies are being set.
+	 *
+	 * @return void
+	 */
+	public function testSessionSet()
+	{
+		// Create a mock and queue two responses.
+		$mock = new MockHandler([
+			new Response(200, ['Set-Cookie' => 'PHPSESSID:foo']),
+			new Response(200, ['Set-Cookie' => 'SECSESSID:bar'])
+		]);
 
-        $handler = HandlerStack::create($mock);
-        $client = new Client(['handler' => $handler]);
-        $this->sessionCheck->client = $client;
+		$handler = HandlerStack::create($mock);
+		$client = new Client(['handler' => $handler]);
+		$this->sessionCheck->client = $client;
 
-        // Check for PHPSESSID
-        $this->assertContains(EnvironmentCheck::ERROR, $this->sessionCheck->check());
+		// Check for PHPSESSID
+		$this->assertContains(EnvironmentCheck::ERROR, $this->sessionCheck->check());
 
-        // Check for SECSESSID
-        $this->assertContains(EnvironmentCheck::ERROR, $this->sessionCheck->check());
-    }
+		// Check for SECSESSID
+		$this->assertContains(EnvironmentCheck::ERROR, $this->sessionCheck->check());
+	}
 
-    /**
-     * Env check responds OK when no session cookies are set in response.
-     *
-     * @return void
-     */
-    public function testSessionNotSet()
-    {
-        // Create a mock and queue two responses.
-        $mock = new MockHandler([
-            new Response(200)
-        ]);
+	/**
+	 * Env check responds OK when no session cookies are set in response.
+	 *
+	 * @return void
+	 */
+	public function testSessionNotSet()
+	{
+		// Create a mock and queue two responses.
+		$mock = new MockHandler([
+			new Response(200)
+		]);
 
-        $handler = HandlerStack::create($mock);
-        $client = new Client(['handler' => $handler]);
-        $this->sessionCheck->client = $client;
+		$handler = HandlerStack::create($mock);
+		$client = new Client(['handler' => $handler]);
+		$this->sessionCheck->client = $client;
 
-        $this->assertContains(EnvironmentCheck::OK, $this->sessionCheck->check());
-    }
+		$this->assertContains(EnvironmentCheck::OK, $this->sessionCheck->check());
+	}
 }

src/Services/ClientFactory.php

Indentation +29 added lines, -29 removed lines

@@ -14,36 +14,36 @@
  */
 class ClientFactory implements Factory
 {
-    use Configurable;
+	use Configurable;
 
-    /**
-     * Default config for Guzzle client.
-     *
-     * @var array
-     */
-    private static $default_config = [];
+	/**
+	 * Default config for Guzzle client.
+	 *
+	 * @var array
+	 */
+	private static $default_config = [];
 
-    /**
-     * Wrapper to create a Guzzle client.
-     *
-     * {@inheritdoc}
-     */
-    public function create($service, array $params = [])
-    {
-        return new GuzzleClient($this->getConfig($params));
-    }
+	/**
+	 * Wrapper to create a Guzzle client.
+	 *
+	 * {@inheritdoc}
+	 */
+	public function create($service, array $params = [])
+	{
+		return new GuzzleClient($this->getConfig($params));
+	}
 
-    /**
-     * Merge config provided from yaml with default config
-     *
-     * @param array $overrides
-     * @return array
-     */
-    public function getConfig(array $overrides)
-    {
-        return array_merge(
-            $this->config()->get('default_config'),
-            $overrides
-        );
-    }
+	/**
+	 * Merge config provided from yaml with default config
+	 *
+	 * @param array $overrides
+	 * @return array
+	 */
+	public function getConfig(array $overrides)
+	{
+		return array_merge(
+			$this->config()->get('default_config'),
+			$overrides
+		);
+	}
 }

src/Traits/Fetcher.php

Indentation +34 added lines, -34 removed lines

@@ -11,41 +11,41 @@
  */
 trait Fetcher
 {
-    /**
-     * Client for making requests, set vi Injector.
-     *
-     * @see SilverStripe\EnvironmentCheck\Services
-     *
-     * @var GuzzleHttp\Client
-     */
-    public $client = null;
+	/**
+	 * Client for making requests, set vi Injector.
+	 *
+	 * @see SilverStripe\EnvironmentCheck\Services
+	 *
+	 * @var GuzzleHttp\Client
+	 */
+	public $client = null;
 
-    /**
-     * Absolute URL for requests.
-     *
-     * @var string
-     */
-    protected $url;
+	/**
+	 * Absolute URL for requests.
+	 *
+	 * @var string
+	 */
+	protected $url;
 
-    /**
-     * Set URL for requests.
-     *
-     * @param string $url Relative URL
-     * @return self
-     */
-    public function setURL($url)
-    {
-        $this->url = Director::absoluteURL($url);
-        return $this;
-    }
+	/**
+	 * Set URL for requests.
+	 *
+	 * @param string $url Relative URL
+	 * @return self
+	 */
+	public function setURL($url)
+	{
+		$this->url = Director::absoluteURL($url);
+		return $this;
+	}
 
-    /**
-     * Getter for URL
-     *
-     * @return string
-     */
-    public function getURL()
-    {
-        return $this->url;
-    }
+	/**
+	 * Getter for URL
+	 *
+	 * @return string
+	 */
+	public function getURL()
+	{
+		return $this->url;
+	}
 }