silverstripe / silverstripe-environmentcheck

src/Checks/HasClassCheck.php

Indentation +23 added lines, -23 removed lines

@@ -11,29 +11,29 @@
  */
 class HasClassCheck implements EnvironmentCheck
 {
-    /**
-     * @var string
-     */
-    protected $className;
+	/**
+	 * @var string
+	 */
+	protected $className;
 
-    /**
-     * @param string $className The name of the class to look for.
-     */
-    public function __construct($className)
-    {
-        $this->className = $className;
-    }
+	/**
+	 * @param string $className The name of the class to look for.
+	 */
+	public function __construct($className)
+	{
+		$this->className = $className;
+	}
 
-    /**
-     * {@inheritDoc}
-     *
-     * @return array
-     */
-    public function check()
-    {
-        if (class_exists($this->className)) {
-            return [EnvironmentCheck::OK, 'Class ' . $this->className.' exists'];
-        }
-        return [EnvironmentCheck::ERROR, 'Class ' . $this->className.' doesn\'t exist'];
-    }
+	/**
+	 * {@inheritDoc}
+	 *
+	 * @return array
+	 */
+	public function check()
+	{
+		if (class_exists($this->className)) {
+			return [EnvironmentCheck::OK, 'Class ' . $this->className.' exists'];
+		}
+		return [EnvironmentCheck::ERROR, 'Class ' . $this->className.' doesn\'t exist'];
+	}
 }

src/Checks/FileWriteableCheck.php

Indentation +64 added lines, -64 removed lines

@@ -11,77 +11,77 @@
  */
 class FileWriteableCheck implements EnvironmentCheck
 {
-    /**
-     * @var string
-     */
-    protected $path;
+	/**
+	 * @var string
+	 */
+	protected $path;
 
-    /**
-     * @param string $path The full path. If a relative path, it will relative to the BASE_PATH.
-     */
-    public function __construct($path)
-    {
-        $this->path = $path;
-    }
+	/**
+	 * @param string $path The full path. If a relative path, it will relative to the BASE_PATH.
+	 */
+	public function __construct($path)
+	{
+		$this->path = $path;
+	}
 
-    /**
-     * {@inheritDoc}
-     *
-     * @return array
-     */
-    public function check()
-    {
-        if ($this->path[0] == '/') {
-            $filename = $this->path;
-        } else {
-            $filename = BASE_PATH . DIRECTORY_SEPARATOR . str_replace('/', DIRECTORY_SEPARATOR, $this->path);
-        }
+	/**
+	 * {@inheritDoc}
+	 *
+	 * @return array
+	 */
+	public function check()
+	{
+		if ($this->path[0] == '/') {
+			$filename = $this->path;
+		} else {
+			$filename = BASE_PATH . DIRECTORY_SEPARATOR . str_replace('/', DIRECTORY_SEPARATOR, $this->path);
+		}
 
-        if (file_exists($filename)) {
-            $isWriteable = is_writeable($filename);
-        } else {
-            $isWriteable = is_writeable(dirname($filename));
-        }
+		if (file_exists($filename)) {
+			$isWriteable = is_writeable($filename);
+		} else {
+			$isWriteable = is_writeable(dirname($filename));
+		}
 
-        if (!$isWriteable) {
-            if (function_exists('posix_getgroups')) {
-                $userID = posix_geteuid();
-                $user = posix_getpwuid($userID);
+		if (!$isWriteable) {
+			if (function_exists('posix_getgroups')) {
+				$userID = posix_geteuid();
+				$user = posix_getpwuid($userID);
 
-                $currentOwnerID = fileowner(file_exists($filename) ? $filename : dirname($filename));
-                $currentOwner = posix_getpwuid($currentOwnerID);
+				$currentOwnerID = fileowner(file_exists($filename) ? $filename : dirname($filename));
+				$currentOwner = posix_getpwuid($currentOwnerID);
 
-                $message = "User '$user[name]' needs to be able to write to this file:\n$filename\n\nThe file is "
-                    . "currently owned by '$currentOwner[name]'.  ";
+				$message = "User '$user[name]' needs to be able to write to this file:\n$filename\n\nThe file is "
+					. "currently owned by '$currentOwner[name]'.  ";
 
-                if ($user['name'] == $currentOwner['name']) {
-                    $message .= 'We recommend that you make the file writeable.';
-                } else {
-                    $groups = posix_getgroups();
-                    $groupList = [];
-                    foreach ($groups as $group) {
-                        $groupInfo = posix_getgrgid($group);
-                        if (in_array($currentOwner['name'], $groupInfo['members'])) {
-                            $groupList[] = $groupInfo['name'];
-                        }
-                    }
-                    if ($groupList) {
-                        $message .= "	We recommend that you make the file group-writeable and change the group to "
-                            . "one of these groups:\n - " . implode("\n - ", $groupList)
-                            . "\n\nFor example:\nchmod g+w $filename\nchgrp " . $groupList[0] . " $filename";
-                    } else {
-                        $message .= "  There is no user-group that contains both the web-server user and the owner "
-                            . "of this file.  Change the ownership of the file, create a new group, or temporarily "
-                            . "make the file writeable by everyone during the install process.";
-                    }
-                }
-            } else {
-                $message = "The webserver user needs to be able to write to this file:\n$filename";
-            }
+				if ($user['name'] == $currentOwner['name']) {
+					$message .= 'We recommend that you make the file writeable.';
+				} else {
+					$groups = posix_getgroups();
+					$groupList = [];
+					foreach ($groups as $group) {
+						$groupInfo = posix_getgrgid($group);
+						if (in_array($currentOwner['name'], $groupInfo['members'])) {
+							$groupList[] = $groupInfo['name'];
+						}
+					}
+					if ($groupList) {
+						$message .= "	We recommend that you make the file group-writeable and change the group to "
+							. "one of these groups:\n - " . implode("\n - ", $groupList)
+							. "\n\nFor example:\nchmod g+w $filename\nchgrp " . $groupList[0] . " $filename";
+					} else {
+						$message .= "  There is no user-group that contains both the web-server user and the owner "
+							. "of this file.  Change the ownership of the file, create a new group, or temporarily "
+							. "make the file writeable by everyone during the install process.";
+					}
+				}
+			} else {
+				$message = "The webserver user needs to be able to write to this file:\n$filename";
+			}
 
-            return [EnvironmentCheck::ERROR, $message];
-        }
+			return [EnvironmentCheck::ERROR, $message];
+		}
 
-        return [EnvironmentCheck::OK, ''];
-    }
+		return [EnvironmentCheck::OK, ''];
+	}
 }

tests/Controllers/DevHealthControllerTest.php

Indentation +17 added lines, -17 removed lines

@@ -16,27 +16,27 @@
  */
 class DevHealthControllerTest extends SapphireTest
 {
-    /**
-     * {@inheritDoc}
-     * @var array
-     */
-    protected $usesDatabase = true;
+	/**
+	 * {@inheritDoc}
+	 * @var array
+	 */
+	protected $usesDatabase = true;
 
-    public function testIndexCreatesChecker()
-    {
-        $controller = new DevHealthController();
+	public function testIndexCreatesChecker()
+	{
+		$controller = new DevHealthController();
 
-        $request = new HTTPRequest('GET', 'example.com');
+		$request = new HTTPRequest('GET', 'example.com');
 
-        // we need to fake authenticated access as BasicAuth::requireLogin doesn't like empty
-        // permission type strings, which is what health check uses.
+		// we need to fake authenticated access as BasicAuth::requireLogin doesn't like empty
+		// permission type strings, which is what health check uses.
 
-        putenv('ENVCHECK_BASICAUTH_USERNAME="foo"');
-        putenv('ENVCHECK_BASICAUTH_PASSWORD="bar"');
+		putenv('ENVCHECK_BASICAUTH_USERNAME="foo"');
+		putenv('ENVCHECK_BASICAUTH_PASSWORD="bar"');
 
-        $_SERVER['PHP_AUTH_USER'] = 'foo';
-        $_SERVER['PHP_AUTH_PW'] = 'bar';
+		$_SERVER['PHP_AUTH_USER'] = 'foo';
+		$_SERVER['PHP_AUTH_PW'] = 'bar';
 
-        $this->assertInstanceOf(EnvironmentChecker::class, $controller->index($request));
-    }
+		$this->assertInstanceOf(EnvironmentChecker::class, $controller->index($request));
+	}
 }

tests/Controllers/DevCheckControllerTest.php

Indentation +11 added lines, -11 removed lines

@@ -16,18 +16,18 @@
  */
 class DevCheckControllerTest extends SapphireTest
 {
-    /**
-     * {@inheritDoc}
-     * @var array
-     */
-    protected $usesDatabase = true;
+	/**
+	 * {@inheritDoc}
+	 * @var array
+	 */
+	protected $usesDatabase = true;
 
-    public function testIndexCreatesChecker()
-    {
-        $controller = new DevCheckController();
+	public function testIndexCreatesChecker()
+	{
+		$controller = new DevCheckController();
 
-        $request = new HTTPRequest('GET', 'example.com');
+		$request = new HTTPRequest('GET', 'example.com');
 
-        $this->assertInstanceOf(EnvironmentChecker::class, $controller->index($request));
-    }
+		$this->assertInstanceOf(EnvironmentChecker::class, $controller->index($request));
+	}
 }

tests/Checks/URLCheckTest.php

Indentation +9 added lines, -9 removed lines

@@ -15,15 +15,15 @@
  */
 class URLCheckTest extends SapphireTest
 {
-    public function testCheckReportsMissingPages()
-    {
-        $check = new URLCheck('foo', 'bar');
+	public function testCheckReportsMissingPages()
+	{
+		$check = new URLCheck('foo', 'bar');
 
-        $expected = [
-            EnvironmentCheck::ERROR,
-            'Error retrieving "foo" (Code: 404)'
-        ];
+		$expected = [
+			EnvironmentCheck::ERROR,
+			'Error retrieving "foo" (Code: 404)'
+		];
 
-        $this->assertEquals($expected, $check->check());
-    }
+		$this->assertEquals($expected, $check->check());
+	}
 }

tests/Checks/FileWritableCheckTest.php

Indentation +15 added lines, -15 removed lines

@@ -15,24 +15,24 @@
  */
 class FileWritableCheckTest extends SapphireTest
 {
-    public function testCheckReportsWritablePaths()
-    {
-        $check = new FileWriteableCheck(TEMP_FOLDER);
+	public function testCheckReportsWritablePaths()
+	{
+		$check = new FileWriteableCheck(TEMP_FOLDER);
 
-        $expected = [
-            EnvironmentCheck::OK,
-            ''
-        ];
+		$expected = [
+			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/ExternalURLCheckTest.php

Indentation +10 added lines, -10 removed lines

@@ -15,17 +15,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 = [
-            EnvironmentCheck::ERROR,
-            'Success retrieving "http://missing-site/" (Code: 404)'
-        ];
+		$expected = [
+			EnvironmentCheck::ERROR,
+			'Success retrieving "http://missing-site/" (Code: 404)'
+		];
 
-        $this->assertEquals($expected, $check->check());
-    }
+		$this->assertEquals($expected, $check->check());
+	}
 }

tests/Checks/HasClassCheckTest.php

Indentation +18 added lines, -18 removed lines

@@ -15,27 +15,27 @@
  */
 class HasClassCheckTest extends SapphireTest
 {
-    public function testCheckReportsMissingClasses()
-    {
-        $check = new HasClassCheck('foo');
+	public function testCheckReportsMissingClasses()
+	{
+		$check = new HasClassCheck('foo');
 
-        $expected = [
-            EnvironmentCheck::ERROR,
-            'Class foo doesn\'t exist'
-        ];
+		$expected = [
+			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 = [
-            EnvironmentCheck::OK,
-            'Class stdClass exists',
-        ];
+		$expected = [
+			EnvironmentCheck::OK,
+			'Class stdClass exists',
+		];
 
-        $this->assertEquals($expected, $check->check());
-    }
+		$this->assertEquals($expected, $check->check());
+	}
 }

tests/Checks/DatabaseCheckTest.php

Indentation +17 added lines, -17 removed lines

@@ -16,25 +16,25 @@
  */
 class DatabaseCheckTest extends SapphireTest
 {
-    /**
-     * {@inheritDoc}
-     * @var bool
-     */
-    protected $usesDatabase = true;
+	/**
+	 * {@inheritDoc}
+	 * @var bool
+	 */
+	protected $usesDatabase = true;
 
-    public function testCheckReportsValidConnection()
-    {
-        $member = new Member;
-        $member->FirstName = 'Bob';
-        $member->write();
+	public function testCheckReportsValidConnection()
+	{
+		$member = new Member;
+		$member->FirstName = 'Bob';
+		$member->write();
 
-        $check = new DatabaseCheck();
+		$check = new DatabaseCheck();
 
-        $expected = [
-            EnvironmentCheck::OK,
-            ''
-        ];
+		$expected = [
+			EnvironmentCheck::OK,
+			''
+		];
 
-        $this->assertEquals($expected, $check->check());
-    }
+		$this->assertEquals($expected, $check->check());
+	}
 }