silverstripe / silverstripe-environmentcheck

src/Controllers/DevHealthController.php

Indentation +18 added lines, -18 removed lines

@@ -12,25 +12,25 @@
  */
 class DevHealthController extends Controller
 {
-    /**
-     * @var array
-     */
-    private static $allowed_actions = [
-        'index'
-    ];
+	/**
+	 * @var array
+	 */
+	private static $allowed_actions = [
+		'index'
+	];
 
-    /**
-     * @return EnvironmentChecker
-     *
-     * @throws HTTPResponse_Exception
-     */
-    public function index()
-    {
-        // health check does not require permission to run
+	/**
+	 * @return EnvironmentChecker
+	 *
+	 * @throws HTTPResponse_Exception
+	 */
+	public function index()
+	{
+		// health check does not require permission to run
 
-        $checker = new EnvironmentChecker('health', 'Site health');
-        $checker->setErrorCode(500);
+		$checker = new EnvironmentChecker('health', 'Site health');
+		$checker->setErrorCode(500);
 
-        return $checker;
-    }
+		return $checker;
+	}
 }

src/Controllers/DevCheckController.php

Indentation +34 added lines, -34 removed lines

@@ -12,38 +12,38 @@
  */
 class DevCheckController extends Controller
 {
-    /**
-     * @var array
-     */
-    private static $allowed_actions = [
-        'index'
-    ];
-
-    /**
-     * Permission code to check for access to this controller.
-     *
-     * @var string
-     */
-    private static $permission = 'ADMIN';
-
-    /**
-     * @param HTTPRequest $request
-     *
-     * @return EnvironmentChecker
-     *
-     * @throws HTTPResponse_Exception
-     */
-    public function index($request)
-    {
-        $suite = 'check';
-
-        if ($name = $request->param('Suite')) {
-            $suite = $name;
-        }
-
-        $checker = new EnvironmentChecker($suite, 'Environment status');
-        $checker->init($this->config()->permission);
-
-        return $checker;
-    }
+	/**
+	 * @var array
+	 */
+	private static $allowed_actions = [
+		'index'
+	];
+
+	/**
+	 * Permission code to check for access to this controller.
+	 *
+	 * @var string
+	 */
+	private static $permission = 'ADMIN';
+
+	/**
+	 * @param HTTPRequest $request
+	 *
+	 * @return EnvironmentChecker
+	 *
+	 * @throws HTTPResponse_Exception
+	 */
+	public function index($request)
+	{
+		$suite = 'check';
+
+		if ($name = $request->param('Suite')) {
+			$suite = $name;
+		}
+
+		$checker = new EnvironmentChecker($suite, 'Environment status');
+		$checker->init($this->config()->permission);
+
+		return $checker;
+	}
 }

src/EnvironmentCheckSuiteResult.php

Indentation +99 added lines, -99 removed lines

@@ -14,112 +14,112 @@
  */
 class EnvironmentCheckSuiteResult extends ViewableData
 {
-    /**
-     * @var ArrayList
-     */
-    protected $details;
+	/**
+	 * @var ArrayList
+	 */
+	protected $details;
 
-    /**
-     * @var int
-     */
-    protected $worst = 0;
+	/**
+	 * @var int
+	 */
+	protected $worst = 0;
 
-    public function __construct()
-    {
-        parent::__construct();
-        $this->details = new ArrayList();
-    }
+	public function __construct()
+	{
+		parent::__construct();
+		$this->details = new ArrayList();
+	}
 
-    /**
-     * @param int $status
-     * @param string $message
-     * @param string $checkIdentifier
-     */
-    public function addResult($status, $message, $checkIdentifier)
-    {
-        $this->details->push(new ArrayData([
-            'Check' => $checkIdentifier,
-            'Status' => $this->statusText($status),
-            'StatusCode' => $status,
-            'Message' => $message,
-        ]));
+	/**
+	 * @param int $status
+	 * @param string $message
+	 * @param string $checkIdentifier
+	 */
+	public function addResult($status, $message, $checkIdentifier)
+	{
+		$this->details->push(new ArrayData([
+			'Check' => $checkIdentifier,
+			'Status' => $this->statusText($status),
+			'StatusCode' => $status,
+			'Message' => $message,
+		]));
 
-        $this->worst = max($this->worst, $status);
-    }
+		$this->worst = max($this->worst, $status);
+	}
 
-    /**
-     * Returns true if there are no errors.
-     *
-     * @return bool
-     */
-    public function ShouldPass()
-    {
-        return $this->worst <= EnvironmentCheck::WARNING;
-    }
+	/**
+	 * Returns true if there are no errors.
+	 *
+	 * @return bool
+	 */
+	public function ShouldPass()
+	{
+		return $this->worst <= EnvironmentCheck::WARNING;
+	}
 
-    /**
-     * Returns overall (i.e. worst) status as a string.
-     *
-     * @return string
-     */
-    public function Status()
-    {
-        return $this->statusText($this->worst);
-    }
+	/**
+	 * Returns overall (i.e. worst) status as a string.
+	 *
+	 * @return string
+	 */
+	public function Status()
+	{
+		return $this->statusText($this->worst);
+	}
 
-    /**
-     * Returns detailed status information about each check.
-     *
-     * @return ArrayList
-     */
-    public function Details()
-    {
-        return $this->details;
-    }
+	/**
+	 * Returns detailed status information about each check.
+	 *
+	 * @return ArrayList
+	 */
+	public function Details()
+	{
+		return $this->details;
+	}
 
-    /**
-     * Convert the final result status and details to JSON.
-     *
-     * @return string
-     */
-    public function toJSON()
-    {
-        $result = [
-            'Status' => $this->Status(),
-            'ShouldPass' => $this->ShouldPass(),
-            'Checks' => []
-        ];
-        foreach ($this->details as $detail) {
-            $result['Checks'][] = $detail->toMap();
-        }
-        return json_encode($result);
-    }
+	/**
+	 * Convert the final result status and details to JSON.
+	 *
+	 * @return string
+	 */
+	public function toJSON()
+	{
+		$result = [
+			'Status' => $this->Status(),
+			'ShouldPass' => $this->ShouldPass(),
+			'Checks' => []
+		];
+		foreach ($this->details as $detail) {
+			$result['Checks'][] = $detail->toMap();
+		}
+		return json_encode($result);
+	}
 
-    /**
-     * Return a text version of a status code.
-     *
-     * @param  int $status
-     * @return string
-     * @throws InvalidArgumentException
-     */
-    protected function statusText($status)
-    {
-        switch ($status) {
-            case EnvironmentCheck::ERROR:
-                return 'ERROR';
-                break;
-            case EnvironmentCheck::WARNING:
-                return 'WARNING';
-                break;
-            case EnvironmentCheck::OK:
-                return 'OK';
-                break;
-            case 0:
-                return 'NO CHECKS';
-                break;
-            default:
-                throw new InvalidArgumentException("Bad environment check status '$status'");
-                break;
-        }
-    }
+	/**
+	 * Return a text version of a status code.
+	 *
+	 * @param  int $status
+	 * @return string
+	 * @throws InvalidArgumentException
+	 */
+	protected function statusText($status)
+	{
+		switch ($status) {
+			case EnvironmentCheck::ERROR:
+				return 'ERROR';
+				break;
+			case EnvironmentCheck::WARNING:
+				return 'WARNING';
+				break;
+			case EnvironmentCheck::OK:
+				return 'OK';
+				break;
+			case 0:
+				return 'NO CHECKS';
+				break;
+			default:
+				throw new InvalidArgumentException("Bad environment check status '$status'");
+				break;
+		}
+	}
 }

src/EnvironmentCheck.php

Indentation +18 added lines, -18 removed lines

@@ -16,25 +16,25 @@
  */
 interface EnvironmentCheck
 {
-    /**
-     * @var int
-     */
-    const ERROR = 3;
+	/**
+	 * @var int
+	 */
+	const ERROR = 3;
 
-    /**
-     * @var int
-     */
-    const WARNING = 2;
+	/**
+	 * @var int
+	 */
+	const WARNING = 2;
 
-    /**
-     * @var int
-     */
-    const OK = 1;
+	/**
+	 * @var int
+	 */
+	const OK = 1;
 
-    /**
-     * @return array Result with 'status' and 'message' keys.
-     *
-     * Status is EnvironmentCheck::ERROR, EnvironmentCheck::WARNING, or EnvironmentCheck::OK.
-     */
-    public function check();
+	/**
+	 * @return array Result with 'status' and 'message' keys.
+	 *
+	 * Status is EnvironmentCheck::ERROR, EnvironmentCheck::WARNING, or EnvironmentCheck::OK.
+	 */
+	public function check();
 }

src/Checks/SMTPConnectCheck.php

Indentation +55 added lines, -55 removed lines

@@ -13,67 +13,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 [
-                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 [
+				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 [
-                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 [
+				EnvironmentCheck::ERROR,
+				sprintf('Invalid mail server response: %s', $response)
+			];
+		}
 
-        return [EnvironmentCheck::OK, ''];
-    }
+		return [EnvironmentCheck::OK, ''];
+	}
 }

src/Checks/DatabaseCheck.php

Indentation +34 added lines, -34 removed lines

@@ -13,38 +13,38 @@
  */
 class DatabaseCheck implements EnvironmentCheck
 {
-    /**
-     * @var string
-     */
-    protected $checkTable;
-
-    /**
-     * By default, Member will be checked.
-     *
-     * @param string $checkTable
-     */
-    public function __construct($checkTable = 'Member')
-    {
-        $this->checkTable = $checkTable;
-    }
-
-    /**
-     * {@inheritDoc}
-     *
-     * @return array
-     */
-    public function check()
-    {
-        if (!DB::get_schema()->hasTable($this->checkTable)) {
-            return [EnvironmentCheck::ERROR, "$this->checkTable not present in the database"];
-        }
-
-        $count = DB::query("SELECT COUNT(*) FROM \"$this->checkTable\"")->value();
-
-        if ($count > 0) {
-            return [EnvironmentCheck::OK, ''];
-        }
-
-        return [EnvironmentCheck::WARNING, "$this->checkTable queried ok but has no records"];
-    }
+	/**
+	 * @var string
+	 */
+	protected $checkTable;
+
+	/**
+	 * By default, Member will be checked.
+	 *
+	 * @param string $checkTable
+	 */
+	public function __construct($checkTable = 'Member')
+	{
+		$this->checkTable = $checkTable;
+	}
+
+	/**
+	 * {@inheritDoc}
+	 *
+	 * @return array
+	 */
+	public function check()
+	{
+		if (!DB::get_schema()->hasTable($this->checkTable)) {
+			return [EnvironmentCheck::ERROR, "$this->checkTable not present in the database"];
+		}
+
+		$count = DB::query("SELECT COUNT(*) FROM \"$this->checkTable\"")->value();
+
+		if ($count > 0) {
+			return [EnvironmentCheck::OK, ''];
+		}
+
+		return [EnvironmentCheck::WARNING, "$this->checkTable queried ok but has no records"];
+	}
 }

src/Checks/HasFunctionCheck.php

Indentation +23 added lines, -23 removed lines

@@ -11,29 +11,29 @@
  */
 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 [EnvironmentCheck::OK, $this->functionName . '() exists'];
-        }
-        return [EnvironmentCheck::ERROR, $this->functionName . '() doesn\'t exist'];
-    }
+	/**
+	 * {@inheritDoc}
+	 *
+	 * @return array
+	 */
+	public function check()
+	{
+		if (function_exists($this->functionName)) {
+			return [EnvironmentCheck::OK, $this->functionName . '() exists'];
+		}
+		return [EnvironmentCheck::ERROR, $this->functionName . '() doesn\'t exist'];
+	}
 }

src/Checks/FileAccessibilityAndValidationCheck.php

Indentation +166 added lines, -166 removed lines

@@ -28,170 +28,170 @@
  */
 class FileAccessibilityAndValidationCheck implements EnvironmentCheck
 {
-    /**
-     * @var int
-     */
-    const CHECK_SINGLE = 1;
-
-    /**
-     * @var int
-     */
-    const CHECK_ALL = 2;
-
-    /**
-     * Absolute path to a file or folder, compatible with glob().
-     *
-     * @var string
-     */
-    protected $path;
-
-    /**
-     * Constant, check for a single file to match age criteria, or all of them.
-     *
-     * @var int
-     */
-    protected $fileTypeValidateFunc;
-
-    /**
-     * Constant, check for a single file to match age criteria, or all of them.
-     *
-     * @var int
-     */
-    protected $checkType;
-
-    /**
-     * @param string $path
-     * @param string $fileTypeValidateFunc
-     * @param null|int $checkType
-     */
-    public function __construct($path, $fileTypeValidateFunc = 'noVidation', $checkType = null)
-    {
-        $this->path = $path;
-        $this->fileTypeValidateFunc = ($fileTypeValidateFunc)? $fileTypeValidateFunc : 'noVidation';
-        $this->checkType = ($checkType) ? $checkType : self::CHECK_SINGLE;
-    }
-
-    /**
-     * {@inheritDoc}
-     *
-     * @return array
-     */
-    public function check()
-    {
-        $origStage = Versioned::get_reading_mode();
-        Versioned::set_reading_mode(Versioned::LIVE);
-
-        $files = $this->getFiles();
-        if ($files) {
-            $fileTypeValidateFunc = $this->fileTypeValidateFunc;
-            if (method_exists($this, $fileTypeValidateFunc)) {
-                $invalidFiles = [];
-                $validFiles = [];
-
-                foreach ($files as $file) {
-                    if ($this->$fileTypeValidateFunc($file)) {
-                        $validFiles[] = $file;
-                    } else {
-                        $invalidFiles[] = $file;
-                    }
-                }
-
-                // If at least one file was valid, count as passed
-                if ($this->checkType == self::CHECK_SINGLE && count($invalidFiles) < count($files)) {
-                    $validFileList = PHP_EOL;
-                    foreach ($validFiles as $vf) {
-                        $validFileList .= $vf . PHP_EOL;
-                    }
-                    if ($fileTypeValidateFunc == 'noVidation') {
-                        $checkReturn = [
-                            EnvironmentCheck::OK,
-                            sprintf('At least these file(s) accessible: %s', $validFileList)
-                        ];
-                    } else {
-                        $checkReturn = [
-                            EnvironmentCheck::OK,
-                            sprintf(
-                                'At least these file(s) passed file type validate function "%s": %s',
-                                $fileTypeValidateFunc,
-                                $validFileList
-                            )
-                        ];
-                    }
-                } else {
-                    if (count($invalidFiles) == 0) {
-                        $checkReturn = [EnvironmentCheck::OK, 'All files valideted'];
-                    } else {
-                        $invalidFileList = PHP_EOL;
-                        foreach ($invalidFiles as $vf) {
-                            $invalidFileList .= $vf . PHP_EOL;
-                        }
-
-                        if ($fileTypeValidateFunc == 'noVidation') {
-                            $checkReturn = [
-                                EnvironmentCheck::ERROR,
-                                sprintf('File(s) not accessible: %s', $invalidFileList)
-                            ];
-                        } else {
-                            $checkReturn = [
-                                EnvironmentCheck::ERROR,
-                                sprintf(
-                                    'File(s) not passing the file type validate function "%s": %s',
-                                    $fileTypeValidateFunc,
-                                    $invalidFileList
-                                )
-                            ];
-                        }
-                    }
-                }
-            } else {
-                $checkReturn =  array(
-                    EnvironmentCheck::ERROR,
-                    sprintf("Invalid file type validation method name passed: %s ", $fileTypeValidateFunc)
-                );
-            }
-        } else {
-            $checkReturn = array(
-                EnvironmentCheck::ERROR,
-                sprintf("No files accessible at path %s", $this->path)
-            );
-        }
-
-        Versioned::set_reading_mode($origStage);
-
-        return $checkReturn;
-    }
-
-    /**
-     * @param string $file
-     *
-     * @return bool
-     */
-    private function jsonValidate($file)
-    {
-        $json = json_decode(file_get_contents($file));
-        if (!$json) {
-            return false;
-        }
-        return true;
-    }
-
-    /**
-     * @param string $file
-     *
-     * @return bool
-     */
-    protected function noVidation($file)
-    {
-        return true;
-    }
-
-    /**
-     * Gets a list of absolute file paths.
-     *
-     * @return array
-     */
-    protected function getFiles()
-    {
-        return glob($this->path);
-    }
+	/**
+	 * @var int
+	 */
+	const CHECK_SINGLE = 1;
+
+	/**
+	 * @var int
+	 */
+	const CHECK_ALL = 2;
+
+	/**
+	 * Absolute path to a file or folder, compatible with glob().
+	 *
+	 * @var string
+	 */
+	protected $path;
+
+	/**
+	 * Constant, check for a single file to match age criteria, or all of them.
+	 *
+	 * @var int
+	 */
+	protected $fileTypeValidateFunc;
+
+	/**
+	 * Constant, check for a single file to match age criteria, or all of them.
+	 *
+	 * @var int
+	 */
+	protected $checkType;
+
+	/**
+	 * @param string $path
+	 * @param string $fileTypeValidateFunc
+	 * @param null|int $checkType
+	 */
+	public function __construct($path, $fileTypeValidateFunc = 'noVidation', $checkType = null)
+	{
+		$this->path = $path;
+		$this->fileTypeValidateFunc = ($fileTypeValidateFunc)? $fileTypeValidateFunc : 'noVidation';
+		$this->checkType = ($checkType) ? $checkType : self::CHECK_SINGLE;
+	}
+
+	/**
+	 * {@inheritDoc}
+	 *
+	 * @return array
+	 */
+	public function check()
+	{
+		$origStage = Versioned::get_reading_mode();
+		Versioned::set_reading_mode(Versioned::LIVE);
+
+		$files = $this->getFiles();
+		if ($files) {
+			$fileTypeValidateFunc = $this->fileTypeValidateFunc;
+			if (method_exists($this, $fileTypeValidateFunc)) {
+				$invalidFiles = [];
+				$validFiles = [];
+
+				foreach ($files as $file) {
+					if ($this->$fileTypeValidateFunc($file)) {
+						$validFiles[] = $file;
+					} else {
+						$invalidFiles[] = $file;
+					}
+				}
+
+				// If at least one file was valid, count as passed
+				if ($this->checkType == self::CHECK_SINGLE && count($invalidFiles) < count($files)) {
+					$validFileList = PHP_EOL;
+					foreach ($validFiles as $vf) {
+						$validFileList .= $vf . PHP_EOL;
+					}
+					if ($fileTypeValidateFunc == 'noVidation') {
+						$checkReturn = [
+							EnvironmentCheck::OK,
+							sprintf('At least these file(s) accessible: %s', $validFileList)
+						];
+					} else {
+						$checkReturn = [
+							EnvironmentCheck::OK,
+							sprintf(
+								'At least these file(s) passed file type validate function "%s": %s',
+								$fileTypeValidateFunc,
+								$validFileList
+							)
+						];
+					}
+				} else {
+					if (count($invalidFiles) == 0) {
+						$checkReturn = [EnvironmentCheck::OK, 'All files valideted'];
+					} else {
+						$invalidFileList = PHP_EOL;
+						foreach ($invalidFiles as $vf) {
+							$invalidFileList .= $vf . PHP_EOL;
+						}
+
+						if ($fileTypeValidateFunc == 'noVidation') {
+							$checkReturn = [
+								EnvironmentCheck::ERROR,
+								sprintf('File(s) not accessible: %s', $invalidFileList)
+							];
+						} else {
+							$checkReturn = [
+								EnvironmentCheck::ERROR,
+								sprintf(
+									'File(s) not passing the file type validate function "%s": %s',
+									$fileTypeValidateFunc,
+									$invalidFileList
+								)
+							];
+						}
+					}
+				}
+			} else {
+				$checkReturn =  array(
+					EnvironmentCheck::ERROR,
+					sprintf("Invalid file type validation method name passed: %s ", $fileTypeValidateFunc)
+				);
+			}
+		} else {
+			$checkReturn = array(
+				EnvironmentCheck::ERROR,
+				sprintf("No files accessible at path %s", $this->path)
+			);
+		}
+
+		Versioned::set_reading_mode($origStage);
+
+		return $checkReturn;
+	}
+
+	/**
+	 * @param string $file
+	 *
+	 * @return bool
+	 */
+	private function jsonValidate($file)
+	{
+		$json = json_decode(file_get_contents($file));
+		if (!$json) {
+			return false;
+		}
+		return true;
+	}
+
+	/**
+	 * @param string $file
+	 *
+	 * @return bool
+	 */
+	protected function noVidation($file)
+	{
+		return true;
+	}
+
+	/**
+	 * Gets a list of absolute file paths.
+	 *
+	 * @return array
+	 */
+	protected function getFiles()
+	{
+		return glob($this->path);
+	}
 }

src/Checks/URLCheck.php

Indentation +42 added lines, -42 removed lines

@@ -14,50 +14,50 @@
  */
 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;
-    }
+	/**
+	 * @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;
+	}
 
-    /**
-     * {@inheritDoc}
-     *
-     * @return array
-     * @throws HTTPResponse_Exception
-     */
-    public function check()
-    {
-        $response = Director::test($this->url);
+	/**
+	 * {@inheritDoc}
+	 *
+	 * @return array
+	 * @throws HTTPResponse_Exception
+	 */
+	public function check()
+	{
+		$response = Director::test($this->url);
 
-        if ($response->getStatusCode() != 200) {
-            return [
-                EnvironmentCheck::ERROR,
-                sprintf('Error retrieving "%s" (Code: %d)', $this->url, $response->getStatusCode())
-            ];
-        } elseif ($this->testString && (strpos($response->getBody(), $this->testString) === false)) {
-            return [
-                EnvironmentCheck::WARNING,
-                sprintf('Success retrieving "%s", but string "%s" not found', $this->url, $this->testString)
-            ];
-        }
-        return [
-            EnvironmentCheck::OK,
-            sprintf('Success retrieving "%s"', $this->url)
-        ];
-    }
+		if ($response->getStatusCode() != 200) {
+			return [
+				EnvironmentCheck::ERROR,
+				sprintf('Error retrieving "%s" (Code: %d)', $this->url, $response->getStatusCode())
+			];
+		} elseif ($this->testString && (strpos($response->getBody(), $this->testString) === false)) {
+			return [
+				EnvironmentCheck::WARNING,
+				sprintf('Success retrieving "%s", but string "%s" not found', $this->url, $this->testString)
+			];
+		}
+		return [
+			EnvironmentCheck::OK,
+			sprintf('Success retrieving "%s"', $this->url)
+		];
+	}
 }