Elgg / Elgg

engine/classes/Elgg/Database/AccessCollections.php

Doc Comments +1 added lines, -1 removed lines

@@ -711,7 +711,7 @@
 	 * @see get_members_of_access_collection()
 	 *
 	 * @param int $collection_id The collection ID
-	 * @return \ElggAccessCollection|false
+	 * @return string
 	 */
 	public function get($collection_id) {
 

engine/classes/Elgg/I18n/Translator.php

Doc Comments +3 added lines, -3 removed lines

@@ -78,7 +78,7 @@ 
 	/**
 	 * Get a map of all loaded translations
 	 *
-	 * @return array
+	 * @return string
 	 */
 	public function getLoadedTranslations() {
 		return $this->translations;
@@ -478,7 +478,7 @@ 
 	 *
 	 * @param string $language Language
 	 *
-	 * @return int
+	 * @return double
 	 */
 	public function getLanguageCompleteness($language) {
 
@@ -579,7 +579,7 @@ 
 	/**
 	 * Returns an array of language codes.
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	public static function getAllLanguageCodes() {
 		return [

Spacing +5 added lines, -5 removed lines

@@ -65,7 +65,7 @@ 
 	 */
 	public function __construct(Config $config) {
 		$this->config = $config;
-		$this->defaultPath = dirname(dirname(dirname(dirname(__DIR__)))) . "/languages/";
+		$this->defaultPath = dirname(dirname(dirname(dirname(__DIR__))))."/languages/";
 	}
 
 	/**
@@ -98,7 +98,7 @@ 
 	public function translate($message_key, array $args = [], $language = "") {
 		if (!is_string($message_key) || strlen($message_key) < 1) {
 			_elgg_services()->logger->warn(
-				'$message_key needs to be a string in ' . __METHOD__ . '(), ' . gettype($message_key) . ' provided'
+				'$message_key needs to be a string in '.__METHOD__.'(), '.gettype($message_key).' provided'
 			);
 			return '';
 		}
@@ -327,7 +327,7 @@ 
 	 * @return bool Success
 	 */
 	public function registerPluginTranslations($path) {
-		$languages_path = rtrim($path, "\\/") . "/languages";
+		$languages_path = rtrim($path, "\\/")."/languages";
 
 		// don't need to have translations
 		if (!is_dir($languages_path)) {
@@ -385,7 +385,7 @@ 
 			}
 
 			if (in_array($language_file, $load_language_files) || $load_all) {
-				$result = (include $path . $language_file);
+				$result = (include $path.$language_file);
 				if ($result === false) {
 					$return = false;
 					continue;
@@ -466,7 +466,7 @@ 
 			
 			$completeness = $this->getLanguageCompleteness($k);
 			if ($completeness < 100) {
-				$installed[$k] .= " (" . $completeness . "% " . $this->translate('complete') . ")";
+				$installed[$k] .= " (".$completeness."% ".$this->translate('complete').")";
 			}
 		}
 

engine/classes/Elgg/Assets/ExternalFiles.php

Spacing +1 added lines, -1 removed lines

@@ -180,7 +180,7 @@
 			return $v->loaded == true && $v->location == $location;
 		});
 		if ($items) {
-			array_walk($items, function(&$v, $k){
+			array_walk($items, function(&$v, $k) {
 				$v = $v->url;
 			});
 		}

engine/lib/actions.php

Doc Comments +2 added lines, -2 removed lines

@@ -151,7 +151,7 @@ 
  *
  * @param string $action The action being performed
  *
- * @return mixed True if valid or redirects.
+ * @return boolean True if valid or redirects.
  * @access private
  */
 function action_gatekeeper($action) {
@@ -182,7 +182,7 @@ 
 /**
  * Regenerate a new site key (32 bytes: "z" to indicate format + 186-bit key in Base64 URL).
  *
- * @return mixed The site secret hash
+ * @return string The site secret hash
  * @access private
  */
 function init_site_secret() {

engine/classes/Elgg/Database/SiteSecret.php

Spacing +1 added lines, -1 removed lines

@@ -108,7 +108,7 @@
 	 * @return SiteSecret
 	 */
 	public static function regenerate(ElggCrypto $crypto, ConfigTable $table) {
-		$key = 'z' . $crypto->getRandomString(31);
+		$key = 'z'.$crypto->getRandomString(31);
 
 		$table->set(self::CONFIG_KEY, $key);
 

engine/classes/Elgg/Application.php

Doc Comments +3 added lines, -3 removed lines

@@ -453,7 +453,7 @@ 
 	/**
 	 * Elgg's front controller. Handles basically all incoming URL requests.
 	 *
-	 * @return bool True if Elgg will handle the request, false if the server should (PHP-CLI server)
+	 * @return boolean|null True if Elgg will handle the request, false if the server should (PHP-CLI server)
 	 */
 	public static function index() {
 		$req = Request::createFromGlobals();
@@ -470,7 +470,7 @@ 
 	/**
 	 * Routes the request, booting core if not yet booted
 	 *
-	 * @return bool False if Elgg wants the PHP CLI server to handle the request
+	 * @return boolean|null False if Elgg wants the PHP CLI server to handle the request
 	 */
 	public function run() {
 		$config = $this->services->config;
@@ -781,7 +781,7 @@ 
 	 * @param int    $linenum  The line number the error was raised at
 	 * @param array  $vars     An array that points to the active symbol table where error occurred
 	 *
-	 * @return true
+	 * @return boolean
 	 * @throws \Exception
 	 * @access private
 	 */

Spacing +22 added lines, -22 removed lines

@@ -95,13 +95,13 @@ 
 		}
 
 		if ($config->elgg_config_locks) {
-			$lock = function ($name) use ($config) {
+			$lock = function($name) use ($config) {
 				$config->lock($name);
 			};
 		} else {
 			// the installer needs to build an application with defaults then update
 			// them after they're validated, so we don't want to lock them.
-			$lock = function () {
+			$lock = function() {
 			};
 		}
 
@@ -113,7 +113,7 @@ 
 		}
 
 		if ($config->dataroot) {
-			$config->dataroot = rtrim($config->dataroot, '\\/') . DIRECTORY_SEPARATOR;
+			$config->dataroot = rtrim($config->dataroot, '\\/').DIRECTORY_SEPARATOR;
 		} else {
 			if (!$config->installer_running) {
 				throw new ConfigurationException('Config value "dataroot" is required.');
@@ -122,14 +122,14 @@ 
 		$lock('dataroot');
 
 		if ($config->cacheroot) {
-			$config->cacheroot = rtrim($config->cacheroot, '\\/') . DIRECTORY_SEPARATOR;
+			$config->cacheroot = rtrim($config->cacheroot, '\\/').DIRECTORY_SEPARATOR;
 		} else {
 			$config->cacheroot = $config->dataroot;
 		}
 		$lock('cacheroot');
 
 		if ($config->wwwroot) {
-			$config->wwwroot = rtrim($config->wwwroot, '/') . '/';
+			$config->wwwroot = rtrim($config->wwwroot, '/').'/';
 		} else {
 			$config->wwwroot = $this->services->request->sniffElggUrl();
 		}
@@ -148,8 +148,8 @@ 
 		$locked_props = [
 			'site_guid' => 1,
 			'path' => Paths::project(),
-			'plugins_path' => Paths::project() . "mod/",
-			'pluginspath' => Paths::project() . "mod/",
+			'plugins_path' => Paths::project()."mod/",
+			'pluginspath' => Paths::project()."mod/",
 			'url' => $config->wwwroot,
 		];
 		foreach ($locked_props as $name => $value) {
@@ -197,7 +197,7 @@ 
 		}
 
 		$setups = [];
-		$path = Paths::elgg() . 'engine/lib';
+		$path = Paths::elgg().'engine/lib';
 
 		// include library files, capturing setup functions
 		foreach (self::getEngineLibs() as $file) {
@@ -258,7 +258,7 @@ 
 		$config = $this->services->config;
 
 		if ($this->isTestingApplication()) {
-			throw new \RuntimeException('Unit tests should not call ' . __METHOD__);
+			throw new \RuntimeException('Unit tests should not call '.__METHOD__);
 		}
 
 		if ($config->boot_complete) {
@@ -297,7 +297,7 @@ 
 			// as a custom plugin that is always loaded last and can't be disabled...
 			if (!$config->system_cache_loaded) {
 				// configure view locations for the custom plugin (not Elgg core)
-				$viewsFile = Paths::project() . 'views.php';
+				$viewsFile = Paths::project().'views.php';
 				if (is_file($viewsFile)) {
 					$viewsSpec = Includer::includeFile($viewsFile);
 					if (is_array($viewsSpec)) {
@@ -314,7 +314,7 @@ 
 			}
 
 			// This is root directory start.php
-			$root_start = Paths::project() . "start.php";
+			$root_start = Paths::project()."start.php";
 			if (is_file($root_start)) {
 				require $root_start;
 			}
@@ -364,7 +364,7 @@ 
 		if (isset(self::$public_services[$name])) {
 			return $this->services->{$name};
 		}
-		trigger_error("Undefined property: " . __CLASS__ . ":\${$name}");
+		trigger_error("Undefined property: ".__CLASS__.":\${$name}");
 	}
 
 	/**
@@ -414,7 +414,7 @@ 
 			if ($spec['request'] instanceof Request) {
 				$spec['service_provider']->setValue('request', $spec['request']);
 			} else {
-				throw new \InvalidArgumentException("Given request is not a " . Request::class);
+				throw new \InvalidArgumentException("Given request is not a ".Request::class);
 			}
 		}
 
@@ -428,7 +428,7 @@ 
 		if ($spec['handle_shutdown']) {
 			// we need to register for shutdown before Symfony registers the
 			// session_write_close() function. https://github.com/Elgg/Elgg/issues/9243
-			register_shutdown_function(function () {
+			register_shutdown_function(function() {
 				// There are cases where we may exit before this function is defined
 				if (function_exists('_elgg_shutdown_hook')) {
 					_elgg_shutdown_hook();
@@ -478,7 +478,7 @@ 
 			}
 
 			// overwrite value from settings
-			$www_root = rtrim($request->getSchemeAndHttpHost() . $request->getBaseUrl(), '/') . '/';
+			$www_root = rtrim($request->getSchemeAndHttpHost().$request->getBaseUrl(), '/').'/';
 			$config->wwwroot = $www_root;
 			$config->wwwroot_cli_server = $www_root;
 		}
@@ -556,7 +556,7 @@ 
 		ini_set('display_errors', 1);
 		$is_cli = (php_sapi_name() === 'cli');
 
-		$forward = function ($url) use ($is_cli) {
+		$forward = function($url) use ($is_cli) {
 			if ($is_cli) {
 				echo "Open $url in your browser to continue.";
 				exit;
@@ -576,14 +576,14 @@ 
 		}
 		
 		$site_url = _elgg_config()->url;
-		$site_host = parse_url($site_url, PHP_URL_HOST) . '/';
+		$site_host = parse_url($site_url, PHP_URL_HOST).'/';
 
 		// turn any full in-site URLs into absolute paths
 		$forward_url = get_input('forward', '/admin', false);
 		$forward_url = str_replace([$site_url, $site_host], '/', $forward_url);
 
 		if (strpos($forward_url, '/') !== 0) {
-			$forward_url = '/' . $forward_url;
+			$forward_url = '/'.$forward_url;
 		}
 
 		if ($is_cli || (get_input('upgrade') == 'upgrade')) {
@@ -605,7 +605,7 @@ 
 			}
 		} else {
 			$rewriteTester = new \ElggRewriteTester();
-			$url = elgg_get_site_url() . "__testing_rewrite?__testing_rewrite=1";
+			$url = elgg_get_site_url()."__testing_rewrite?__testing_rewrite=1";
 			if (!$rewriteTester->runRewriteTest($url)) {
 				// see if there is a problem accessing the site at all
 				// due to ip restrictions for example
@@ -792,9 +792,9 @@ 
 	 * @access private
 	 */
 	public function handleErrors($errno, $errmsg, $filename, $linenum, $vars) {
-		$error = date("Y-m-d H:i:s (T)") . ": \"$errmsg\" in file $filename (line $linenum)";
+		$error = date("Y-m-d H:i:s (T)").": \"$errmsg\" in file $filename (line $linenum)";
 
-		$log = function ($message, $level) {
+		$log = function($message, $level) {
 			if (self::$core_loaded) {
 				return elgg_log($message, $level);
 			}
@@ -850,7 +850,7 @@ 
 	 * @deprecated
 	 */
 	public function loadSettings() {
-		trigger_error(__METHOD__ . ' is no longer needed and will be removed.');
+		trigger_error(__METHOD__.' is no longer needed and will be removed.');
 	}
 
 	/**

engine/classes/Elgg/Application/CacheHandler.php

Doc Comments -1 removed lines

@@ -74,7 +74,6 @@
 	/**
 	 * Handle a request for a cached view
 	 *
-	 * @param array $path URL path
 	 * @return Response (unprepared)
 	 */
 	public function handleRequest(Request $request) {

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 use Elgg\Http\Request;
 use Symfony\Component\HttpFoundation\BinaryFileResponse;
 use Symfony\Component\HttpFoundation\Response;
-use Symfony\Component\HttpFoundation\StreamedResponse;
 
 /**
  * Simplecache handler

Spacing +4 added lines, -4 removed lines

@@ -116,7 +116,7 @@ 
 				return $this->send403();
 			}
 
-			$etag = '"' . md5($content) . '"';
+			$etag = '"'.md5($content).'"';
 			$this->setRevalidateHeaders($etag, $response);
 			if ($this->is304($etag)) {
 				return Response::create()->setNotModified();
@@ -131,7 +131,7 @@ 
 		}
 
 		// trust the client but check for an existing cache file
-		$filename = $config->cacheroot . "views_simplecache/$ts/$viewtype/$view";
+		$filename = $config->cacheroot."views_simplecache/$ts/$viewtype/$view";
 		if (file_exists($filename)) {
 			$this->sendCacheHeaders($etag, $response);
 			return BinaryFileResponse::create($filename, 200, $response->headers->all());
@@ -150,7 +150,7 @@ 
 
 		$lastcache = (int) $config->lastcache;
 
-		$filename = $config->cacheroot . "views_simplecache/$lastcache/$viewtype/$view";
+		$filename = $config->cacheroot."views_simplecache/$lastcache/$viewtype/$view";
 
 		if ($lastcache == $ts) {
 			$this->sendCacheHeaders($etag, $response);
@@ -213,7 +213,7 @@ 
 	 */
 	protected function isCacheableView($view) {
 		if (preg_match('~^languages/(.*)\.js$~', $view, $m)) {
-			return in_array($m[1],  _elgg_services()->translator->getAllLanguageCodes());
+			return in_array($m[1], _elgg_services()->translator->getAllLanguageCodes());
 		}
 		return _elgg_services()->views->isCacheableView($view);
 	}

engine/classes/Elgg/Database/ConfigTable.php

Doc Comments +1 added lines, -1 removed lines

@@ -129,7 +129,7 @@
 	 *
 	 * @param string $name The name of the config value
 	 *
-	 * @return mixed|null
+	 * @return string|boolean
 	 */
 	function get($name) {
 		$sql = "

engine/classes/Elgg/Database/Plugins.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 namespace Elgg\Database;
 
 use Elgg\Cache\Pool;
-use Elgg\Config;
 use Elgg\Profilable;
 use Exception;
 use Elgg\Cache\PluginSettingsCache;