silverstripe / silverstripe-environmentcheck

code/checks/HasFunctionCheck.php

Indentation +24 added lines, -24 removed lines

@@ -5,30 +5,30 @@
  */
 class HasFunctionCheck implements EnvironmentCheck
 {
-    /**
-     * @var string
-     */
-    protected $functionName;
+	/**
+	 * @var string
+	 */
+	protected $functionName;
 
-    /**
-     * @param string $functionName The name of the function to look for.
-     */
-    public function __construct($functionName)
-    {
-        $this->functionName = $functionName;
-    }
+	/**
+	 * @param string $functionName The name of the function to look for.
+	 */
+	public function __construct($functionName)
+	{
+		$this->functionName = $functionName;
+	}
 
-    /**
-     * @inheritdoc
-     *
-     * @return array
-     */
-    public function check()
-    {
-        if (function_exists($this->functionName)) {
-            return array(EnvironmentCheck::OK, $this->functionName.'() exists');
-        } else {
-            return array(EnvironmentCheck::ERROR, $this->functionName.'() doesn\'t exist');
-        }
-    }
+	/**
+	 * @inheritdoc
+	 *
+	 * @return array
+	 */
+	public function check()
+	{
+		if (function_exists($this->functionName)) {
+			return array(EnvironmentCheck::OK, $this->functionName.'() exists');
+		} else {
+			return array(EnvironmentCheck::ERROR, $this->functionName.'() doesn\'t exist');
+		}
+	}
 }

code/checks/SMTPConnectCheck.php

Indentation +55 added lines, -55 removed lines

@@ -7,67 +7,67 @@
  */
 class SMTPConnectCheck implements EnvironmentCheck
 {
-    /**
-     * @var string
-     */
-    protected $host;
+	/**
+	 * @var string
+	 */
+	protected $host;
 
-    /**
-     * @var int
-     */
-    protected $port;
+	/**
+	 * @var int
+	 */
+	protected $port;
 
-    /**
-     * Timeout (in seconds).
-     *
-     * @var int
-     */
-    protected $timeout;
+	/**
+	 * Timeout (in seconds).
+	 *
+	 * @var int
+	 */
+	protected $timeout;
 
-    /**
-     * @param null|string $host
-     * @param null|int $port
-     * @param int $timeout
-     */
-    public function __construct($host = null, $port = null, $timeout = 15)
-    {
-        $this->host = ($host) ? $host : ini_get('SMTP');
-        if (!$this->host) {
-            $this->host = 'localhost';
-        }
+	/**
+	 * @param null|string $host
+	 * @param null|int $port
+	 * @param int $timeout
+	 */
+	public function __construct($host = null, $port = null, $timeout = 15)
+	{
+		$this->host = ($host) ? $host : ini_get('SMTP');
+		if (!$this->host) {
+			$this->host = 'localhost';
+		}
         
-        $this->port = ($port) ? $port : ini_get('smtp_port');
-        if (!$this->port) {
-            $this->port = 25;
-        }
+		$this->port = ($port) ? $port : ini_get('smtp_port');
+		if (!$this->port) {
+			$this->port = 25;
+		}
 
-        $this->timeout = $timeout;
-    }
+		$this->timeout = $timeout;
+	}
 
-    /**
-     * @inheritdoc
-     *
-     * @return array
-     */
-    public function check()
-    {
-        $f = @fsockopen($this->host, $this->port, $errno, $errstr, $this->timeout);
-        if (!$f) {
-            return array(
-                EnvironmentCheck::ERROR,
-                sprintf("Couldn't connect to SMTP on %s:%s (Error: %s %s)", $this->host, $this->port, $errno, $errstr)
-            );
-        }
+	/**
+	 * @inheritdoc
+	 *
+	 * @return array
+	 */
+	public function check()
+	{
+		$f = @fsockopen($this->host, $this->port, $errno, $errstr, $this->timeout);
+		if (!$f) {
+			return array(
+				EnvironmentCheck::ERROR,
+				sprintf("Couldn't connect to SMTP on %s:%s (Error: %s %s)", $this->host, $this->port, $errno, $errstr)
+			);
+		}
 
-        fwrite($f, "HELO its_me\r\n");
-        $response = fread($f, 26);
-        if (substr($response, 0, 3) != '220') {
-            return array(
-                EnvironmentCheck::ERROR,
-                sprintf("Invalid mail server response: %s", $response)
-            );
-        }
+		fwrite($f, "HELO its_me\r\n");
+		$response = fread($f, 26);
+		if (substr($response, 0, 3) != '220') {
+			return array(
+				EnvironmentCheck::ERROR,
+				sprintf("Invalid mail server response: %s", $response)
+			);
+		}
 
-        return array(EnvironmentCheck::OK, '');
-    }
+		return array(EnvironmentCheck::OK, '');
+	}
 }

code/checks/SolrIndexCheck.php

Indentation +40 added lines, -40 removed lines

@@ -7,50 +7,50 @@
  */
 class SolrIndexCheck implements EnvironmentCheck
 {
-    /**
-     * @var null|string
-     */
-    protected $indexClass;
+	/**
+	 * @var null|string
+	 */
+	protected $indexClass;
 
-    /**
-     * @param string $indexClass Limit the index checks to the specified class and all its subclasses.
-     */
-    public function __construct($indexClass = null)
-    {
-        $this->indexClass = $indexClass;
-    }
+	/**
+	 * @param string $indexClass Limit the index checks to the specified class and all its subclasses.
+	 */
+	public function __construct($indexClass = null)
+	{
+		$this->indexClass = $indexClass;
+	}
 
-    /**
-     * @inheritdoc
-     *
-     * @return array
-     */
-    public function check()
-    {
-        $brokenCores = array();
+	/**
+	 * @inheritdoc
+	 *
+	 * @return array
+	 */
+	public function check()
+	{
+		$brokenCores = array();
 
-        if (!class_exists('Solr')) {
-            return array(
-                EnvironmentCheck::ERROR,
-                'Class `Solr` not found. Is the fulltextsearch module installed?'
-            );
-        }
+		if (!class_exists('Solr')) {
+			return array(
+				EnvironmentCheck::ERROR,
+				'Class `Solr` not found. Is the fulltextsearch module installed?'
+			);
+		}
 
-        $service = Solr::service();
-        foreach (Solr::get_indexes($this->indexClass) as $index) {
-            $core = $index->getIndexName();
-            if (!$service->coreIsActive($core)) {
-                $brokenCores[] = $core;
-            }
-        }
+		$service = Solr::service();
+		foreach (Solr::get_indexes($this->indexClass) as $index) {
+			$core = $index->getIndexName();
+			if (!$service->coreIsActive($core)) {
+				$brokenCores[] = $core;
+			}
+		}
 
-        if (!empty($brokenCores)) {
-            return array(
-                EnvironmentCheck::ERROR,
-                'The following indexes are unavailable: ' . implode($brokenCores, ', ')
-            );
-        }
+		if (!empty($brokenCores)) {
+			return array(
+				EnvironmentCheck::ERROR,
+				'The following indexes are unavailable: ' . implode($brokenCores, ', ')
+			);
+		}
 
-        return array(EnvironmentCheck::OK, 'Expected indexes are available.');
-    }
+		return array(EnvironmentCheck::OK, 'Expected indexes are available.');
+	}
 }

code/checks/URLCheck.php

Indentation +41 added lines, -41 removed lines

@@ -7,52 +7,52 @@
  */
 class URLCheck implements EnvironmentCheck
 {
-    /**
-     * @var string
-     */
-    protected $url;
+	/**
+	 * @var string
+	 */
+	protected $url;
 
-    /**
-     * @var string
-     */
-    protected $testString;
+	/**
+	 * @var string
+	 */
+	protected $testString;
     
-    /*
+	/*
      * @param string $url The URL to check, relative to the site (homepage is '').
      * @param string $testString An optional piece of text to search for on the homepage.
      */
-    public function __construct($url = '', $testString = '')
-    {
-        $this->url = $url;
-        $this->testString = $testString;
-    }
+	public function __construct($url = '', $testString = '')
+	{
+		$this->url = $url;
+		$this->testString = $testString;
+	}
 
-    /**
-     * @inheritdoc
-     *
-     * @return array
-     *
-     * @throws SS_HTTPResponse_Exception
-     */
-    public function check()
-    {
-        $response = Director::test($this->url);
+	/**
+	 * @inheritdoc
+	 *
+	 * @return array
+	 *
+	 * @throws SS_HTTPResponse_Exception
+	 */
+	public function check()
+	{
+		$response = Director::test($this->url);
 
-        if ($response->getStatusCode() != 200) {
-            return array(
-                EnvironmentCheck::ERROR,
-                sprintf('Error retrieving "%s" (Code: %d)', $this->url, $response->getStatusCode())
-            );
-        } elseif ($this->testString && (strpos($response->getBody(), $this->testString) === false)) {
-            return array(
-                EnvironmentCheck::WARNING,
-                sprintf('Success retrieving "%s", but string "%s" not found', $this->url, $this->testString)
-            );
-        } else {
-            return array(
-                EnvironmentCheck::OK,
-                sprintf('Success retrieving "%s"', $this->url)
-            );
-        }
-    }
+		if ($response->getStatusCode() != 200) {
+			return array(
+				EnvironmentCheck::ERROR,
+				sprintf('Error retrieving "%s" (Code: %d)', $this->url, $response->getStatusCode())
+			);
+		} elseif ($this->testString && (strpos($response->getBody(), $this->testString) === false)) {
+			return array(
+				EnvironmentCheck::WARNING,
+				sprintf('Success retrieving "%s", but string "%s" not found', $this->url, $this->testString)
+			);
+		} else {
+			return array(
+				EnvironmentCheck::OK,
+				sprintf('Success retrieving "%s"', $this->url)
+			);
+		}
+	}
 }

tests/checks/DatabaseCheckTest.php

Indentation +9 added lines, -9 removed lines

@@ -5,15 +5,15 @@
  */
 class DatabaseCheckTest extends SapphireTest
 {
-    public function testCheckReportsValidConnection()
-    {
-        $check = new DatabaseCheck();
+	public function testCheckReportsValidConnection()
+	{
+		$check = new DatabaseCheck();
 
-        $expected = array(
-            EnvironmentCheck::OK,
-            '',
-        );
+		$expected = array(
+			EnvironmentCheck::OK,
+			'',
+		);
 
-        $this->assertEquals($expected, $check->check());
-    }
+		$this->assertEquals($expected, $check->check());
+	}
 }

tests/checks/ExternalURLCheckTest.php

Indentation +10 added lines, -10 removed lines

@@ -5,17 +5,17 @@
  */
 class ExternalURLCheckTest extends SapphireTest
 {
-    public function testCheckReportsMissingPages()
-    {
-        $this->markTestSkipped('ExternalURLCheck seems faulty on some systems');
+	public function testCheckReportsMissingPages()
+	{
+		$this->markTestSkipped('ExternalURLCheck seems faulty on some systems');
 
-        $check = new ExternalURLCheck('http://missing-site/');
+		$check = new ExternalURLCheck('http://missing-site/');
 
-        $expected = array(
-            EnvironmentCheck::ERROR,
-            'Success retrieving "http://missing-site/" (Code: 404)',
-        );
+		$expected = array(
+			EnvironmentCheck::ERROR,
+			'Success retrieving "http://missing-site/" (Code: 404)',
+		);
 
-        $this->assertEquals($expected, $check->check());
-    }
+		$this->assertEquals($expected, $check->check());
+	}
 }

tests/checks/FileWritableCheckTest.php

Indentation +15 added lines, -15 removed lines

@@ -5,24 +5,24 @@
  */
 class FileWritableCheckTest extends SapphireTest
 {
-    public function testCheckReportsWritablePaths()
-    {
-        $check = new FileWriteableCheck(TEMP_FOLDER);
+	public function testCheckReportsWritablePaths()
+	{
+		$check = new FileWriteableCheck(TEMP_FOLDER);
 
-        $expected = array(
-            EnvironmentCheck::OK,
-            '',
-        );
+		$expected = array(
+			EnvironmentCheck::OK,
+			'',
+		);
 
-        $this->assertEquals($expected, $check->check());
-    }
+		$this->assertEquals($expected, $check->check());
+	}
 
-    public function testCheckReportsNonWritablePaths()
-    {
-        $check = new FileWriteableCheck('/var');
+	public function testCheckReportsNonWritablePaths()
+	{
+		$check = new FileWriteableCheck('/var');
 
-        $result = $check->check();
+		$result = $check->check();
 
-        $this->assertEquals(EnvironmentCheck::ERROR, $result[0]);
-    }
+		$this->assertEquals(EnvironmentCheck::ERROR, $result[0]);
+	}
 }

tests/checks/HasClassCheckTest.php

Indentation +18 added lines, -18 removed lines

@@ -5,27 +5,27 @@
  */
 class HasClassCheckTest extends SapphireTest
 {
-    public function testCheckReportsMissingClasses()
-    {
-        $check = new HasClassCheck('foo');
+	public function testCheckReportsMissingClasses()
+	{
+		$check = new HasClassCheck('foo');
 
-        $expected = array(
-            EnvironmentCheck::ERROR,
-            'Class foo doesn\'t exist',
-        );
+		$expected = array(
+			EnvironmentCheck::ERROR,
+			'Class foo doesn\'t exist',
+		);
 
-        $this->assertEquals($expected, $check->check());
-    }
+		$this->assertEquals($expected, $check->check());
+	}
 
-    public function testCheckReportsFoundClasses()
-    {
-        $check = new HasClassCheck('stdClass');
+	public function testCheckReportsFoundClasses()
+	{
+		$check = new HasClassCheck('stdClass');
 
-        $expected = array(
-                EnvironmentCheck::OK,
-                'Class stdClass exists',
-        );
+		$expected = array(
+				EnvironmentCheck::OK,
+				'Class stdClass exists',
+		);
 
-        $this->assertEquals($expected, $check->check());
-    }
+		$this->assertEquals($expected, $check->check());
+	}
 }

tests/checks/HasFunctionCheckTest.php

Indentation +18 added lines, -18 removed lines

@@ -5,27 +5,27 @@
  */
 class HasFunctionCheckTest extends SapphireTest
 {
-    public function testCheckReportsMissingFunctions()
-    {
-        $check = new HasFunctionCheck('foo');
+	public function testCheckReportsMissingFunctions()
+	{
+		$check = new HasFunctionCheck('foo');
 
-        $expected = array(
-            EnvironmentCheck::ERROR,
-            'foo() doesn\'t exist',
-        );
+		$expected = array(
+			EnvironmentCheck::ERROR,
+			'foo() doesn\'t exist',
+		);
 
-        $this->assertEquals($expected, $check->check());
-    }
+		$this->assertEquals($expected, $check->check());
+	}
 
-    public function testCheckReportsFoundFunctions()
-    {
-        $check = new HasFunctionCheck('class_exists');
+	public function testCheckReportsFoundFunctions()
+	{
+		$check = new HasFunctionCheck('class_exists');
 
-        $expected = array(
-            EnvironmentCheck::OK,
-            'class_exists() exists',
-        );
+		$expected = array(
+			EnvironmentCheck::OK,
+			'class_exists() exists',
+		);
 
-        $this->assertEquals($expected, $check->check());
-    }
+		$this->assertEquals($expected, $check->check());
+	}
 }