silverstripe / silverstripe-environmentcheck

code/DevCheckController.php

<?php

class DevCheckController extends Controller {

As per PSR2, the opening brace for this class should be on a new line.

	/**
	 * @var array
	 */
	public static $allowed_actions = array(
		'index'
	);

	/**
	 * Permission code to check for access to this controller.
	 *
	 * @var string
	 */
	private static $permission = 'ADMIN';

	/**
	 * @param SS_HTTPRequest $request
	 *
	 * @return EnvironmentChecker
	 *
	 * @throws SS_HTTPResponse_Exception
	 */
	function index($request) {

It is generally recommended to explicitly declare the visibility for methods.

		$suite = 'check';

		if ($name = $request->param('Suite')) {
			$suite = $name;
		}

		$checker = new EnvironmentChecker($suite, 'Environment status');
		$checker->init($this->config()->permission);

		return $checker;
	}
}