cadmium-org / cadmium-cms

www/engine/System/Classes/Modules/Entitizer/Utils/Factory.php

Braces +6 added lines, -2 removed lines

@@ -12,9 +12,13 @@
 
 		public static function get(string $table) {
 
-			if (!isset(static::$classes[$table])) throw new Exception\General(static::$error_message);
+			if (!isset(static::$classes[$table])) {
+				throw new Exception\General(static::$error_message);
+			}
 
-			if (!isset(static::$cache[$table])) static::$cache[$table] = new static::$classes[$table];
+			if (!isset(static::$cache[$table])) {
+				static::$cache[$table] = new static::$classes[$table];
+			}
 
 			# ------------------------
 

www/engine/System/Classes/Modules/Entitizer/Entitizer.php

Braces +6 added lines, -2 removed lines

@@ -25,9 +25,13 @@
 
 		public static function get(string $table, int $id = 0) {
 
-			if (!isset(self::$classes[$table])) throw new Exception\General(self::$error_message);
+			if (!isset(self::$classes[$table])) {
+				throw new Exception\General(self::$error_message);
+			}
 
-			if (isset(self::$cache[$table][$id])) return self::$cache[$table][$id];
+			if (isset(self::$cache[$table][$id])) {
+				return self::$cache[$table][$id];
+			}
 
 			$entity = new self::$classes[$table]; $entity->init($id);
 

www/engine/System/Classes/Modules/Entitizer/Utils/Definition/Group/Indexes.php

Braces +3 added lines, -1 removed lines

@@ -10,7 +10,9 @@
 
 		public function add(string $name, string $table, string $field, string $delete = null, string $update = null) {
 
-			if ((false === $this->definition->param($name)) || isset($this->list[$name])) return;
+			if ((false === $this->definition->param($name)) || isset($this->list[$name])) {
+				return;
+			}
 
 			$this->list[$name] = new Definition\Item\Foreign($name, $table, $field, $delete, $update);
 		}

www/engine/Framework/Classes/Date/Date.php

Doc Comments +5 added lines, -5 removed lines

@@ -14,7 +14,7 @@ 
 		/**
 		 * Validate a date using a given format
 		 *
-		 * @return the formatted date or false on failure
+		 * @return string|null formatted date or false on failure
 		 */
 
 		public static function validate(string $date, string $format) {
@@ -67,7 +67,7 @@ 
 		 * Get a date from a given timestamp and format it according to a given pattern.
 		 * If the timestamp is null, a current time will be used
 		 *
-		 * @return the formatted date or false if a non-numeric value is used for timestamp
+		 * @return string formatted date or false if a non-numeric value is used for timestamp
 		 */
 
 		public static function get(string $format = DATE_FORMAT_STANDART, int $time = null) {
@@ -78,7 +78,7 @@ 
 		/**
 		 * Get a day from a given timestamp. If the timestamp is null, a current time will be used
 		 *
-		 * @return the formatted day or false if a non-numeric value is used for timestamp
+		 * @return string formatted day or false if a non-numeric value is used for timestamp
 		 */
 
 		public static function getDay(int $time = null) {
@@ -89,7 +89,7 @@ 
 		/**
 		 * Get a month from a given timestamp. If the timestamp is null, a current time will be used
 		 *
-		 * @return the formatted month or false if a non-numeric value is used for timestamp
+		 * @return string formatted month or false if a non-numeric value is used for timestamp
 		 */
 
 		public static function getMonth(int $time = null) {
@@ -100,7 +100,7 @@ 
 		/**
 		 * Get a year from a given timestamp. If the timestamp is null, a current time will be used
 		 *
-		 * @return the formatted year or false if a non-numeric value is used for timestamp
+		 * @return string formatted year or false if a non-numeric value is used for timestamp
 		 */
 
 		public static function getYear(int $time = null) {

Braces +9 added lines, -3 removed lines

@@ -19,9 +19,13 @@ 
 
 		public static function validate(string $date, string $format) {
 
-			if (false === ($datetime = date_create_from_format($format, $date))) return false;
+			if (false === ($datetime = date_create_from_format($format, $date))) {
+				return false;
+			}
 
-			if (false === ($formatted = $datetime->format($format))) return false;
+			if (false === ($formatted = $datetime->format($format))) {
+				return false;
+			}
 
 			# ------------------------
 
@@ -58,7 +62,9 @@ 
 
 		public static function validateYear(int $year) {
 
-			if (false === ($year = self::validate($year, 'Y'))) return false;
+			if (false === ($year = self::validate($year, 'Y'))) {
+				return false;
+			}
 
 			return (($year > '1900') ? (($year > self::getYear()) ? self::getYear() : $year) : '1900');
 		}

www/engine/Framework/Classes/Form/Field.php

Doc Comments +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
 		/**
 		 * Catch a POST value by the field key
 		 *
-		 * @return true on success or false on failure
+		 * @return boolean on success or false on failure
 		 */
 
 		public function post() {

Spacing +2 added lines, -2 removed lines

@@ -31,9 +31,9 @@
 
 			if (preg_match(REGEX_FORM_FIELD_KEY, $key)) {
 
-				$prefix = (('' !== $this->form->getName()) ? ($this->form->getName() . '_') : '');
+				$prefix = (('' !== $this->form->getName()) ? ($this->form->getName().'_') : '');
 
-				$this->key = $key; $this->name = ($prefix . $key); $this->id = str_replace('_', '-', $this->name);
+				$this->key = $key; $this->name = ($prefix.$key); $this->id = str_replace('_', '-', $this->name);
 			}
 
 			# Set configuration

Unused Use Statements +4 added lines, -1 removed lines

@@ -9,7 +9,10 @@
 
 namespace Form {
 
-	use Dataset, Form, Request, Tag;
+	use Dataset;
+	use Form;
+	use Request;
+	use Tag;
 
 	abstract class Field {
 

Braces +12 added lines, -4 removed lines

@@ -57,11 +57,17 @@ 
 
 			# Set appearance
 
-			if ($this->disabled)    $data['disabled']           = 'disabled';
+			if ($this->disabled) {
+				$data['disabled']           = 'disabled';
+			}
 
-			if ($this->required)    $data['data-required']      = 'required';
+			if ($this->required) {
+				$data['data-required']      = 'required';
+			}
 
-			if ($this->error)       $data['data-error']         = 'error';
+			if ($this->error) {
+				$data['data-error']         = 'error';
+			}
 
 			# ------------------------
 
@@ -76,7 +82,9 @@ 
 
 		public function post() : bool {
 
-			if ($this->posted || $this->disabled || ('' === $this->key)) return false;
+			if ($this->posted || $this->disabled || ('' === $this->key)) {
+				return false;
+			}
 
 			$this->error = (!$this->setValue(Request::post($this->name)) && $this->required);
 

www/engine/Framework/Classes/JSON/JSON.php

Doc Comments +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
 		/**
 		 * Convert a JSON value to a string
 		 *
-		 * @return the string or false on failure
+		 * @return string string or false on failure
 		 */
 
 		public static function stringify($value) {

Braces +6 added lines, -2 removed lines

@@ -41,7 +41,9 @@ 
 
 		public static function load(string $file_name) {
 
-			if (false === ($contents = Explorer::getContents($file_name))) return null;
+			if (false === ($contents = Explorer::getContents($file_name))) {
+				return null;
+			}
 
 			return self::parse($contents);
 		}
@@ -54,7 +56,9 @@ 
 
 		public static function save(string $file_name, $value) {
 
-			if (false === ($value = self::stringify($value))) return false;
+			if (false === ($value = self::stringify($value))) {
+				return false;
+			}
 
 			return Explorer::putContents($file_name, $value);
 		}

www/engine/System/Classes/Dispatcher.php

Unused Use Statements +3 added lines, -1 removed lines

@@ -2,7 +2,9 @@
 
 namespace {
 
-	use Modules\Extend, Utils\Map, Utils\Schema;
+	use Modules\Extend;
+	use Utils\Map;
+	use Utils\Schema;
 
 	class Dispatcher {
 

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 
 			if (null === ($data = Schema::get('System')->load())) {
 
-				Request::redirect(INSTALL_PATH . '/install.php');
+				Request::redirect(INSTALL_PATH.'/install.php');
 			}
 
 			# Connect to database

www/engine/System/Classes/Frames/Admin/Area/Authorized.php

Unused Use Statements +6 added lines, -1 removed lines

@@ -2,7 +2,12 @@
 
 namespace Frames\Admin\Area {
 
-	use Frames, Frames\Status, Modules, Ajax, Request, Template;
+	use Frames;
+	use Frames\Status;
+	use Modules;
+	use Ajax;
+	use Request;
+	use Template;
 
 	abstract class Authorized extends Frames\Admin\Section {
 

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
 
 			if (!Modules\Auth::check() || ((false !== Request::get('logout')) && Modules\Auth::logout())) {
 
-				Request::redirect(INSTALL_PATH . '/admin/login');
+				Request::redirect(INSTALL_PATH.'/admin/login');
 			}
 
 			# Handle request

Braces +6 added lines, -2 removed lines

@@ -19,9 +19,13 @@
 
 			# Handle request
 
-			if (Template::isBlock($result = $this->handle())) return $this->displayPage($result, STATUS_CODE_200);
+			if (Template::isBlock($result = $this->handle())) {
+				return $this->displayPage($result, STATUS_CODE_200);
+			}
 
-			if (Ajax::isResponse($result)) return Ajax::output($result);
+			if (Ajax::isResponse($result)) {
+				return Ajax::output($result);
+			}
 
 			# ------------------------
 

www/engine/System/Classes/Frames/Admin/Section.php

Unused Use Statements +11 added lines, -2 removed lines

@@ -2,8 +2,17 @@
 
 namespace Frames\Admin {
 
-	use Frames, Frames\Status, Modules\Auth, Modules\Extend, Utils\Messages, Utils\Popup, Utils\View;
-	use DB, Debug, Language, Template;
+	use Frames;
+	use Frames\Status;
+	use Modules\Auth;
+	use Modules\Extend;
+	use Utils\Messages;
+	use Utils\Popup;
+	use Utils\View;
+	use DB;
+	use Debug;
+	use Language;
+	use Template;
 
 	abstract class Section extends Frames\Section {
 

Spacing +3 added lines, -3 removed lines

@@ -23,7 +23,7 @@ 
 
 		private function getLayout(Template\Block $contents) {
 
-			$layout = View::get('Layouts/' . $this->layout);
+			$layout = View::get('Layouts/'.$this->layout);
 
 			# Set menu and user
 
@@ -82,7 +82,7 @@ 
 
 		protected function displayPage(Template\Block $contents, int $status = STATUS_CODE_200) {
 
-			$page = View::get('Main/' . $this->layout);
+			$page = View::get('Main/'.$this->layout);
 
 			# Set language
 
@@ -90,7 +90,7 @@ 
 
 			# Set title
 
-			$page->title = ((('' !== $this->title) ? (Language::get($this->title) . ' | ') : '') . CADMIUM_NAME);
+			$page->title = ((('' !== $this->title) ? (Language::get($this->title).' | ') : '').CADMIUM_NAME);
 
 			# Set layout
 

Braces +3 added lines, -1 removed lines

@@ -111,7 +111,9 @@
 
 				$ips = preg_split('/ +/', CONFIG_ADMIN_IP, -1, PREG_SPLIT_NO_EMPTY);
 
-				if (!in_array(REQUEST_CLIENT_IP, $ips, true)) return Status::error404();
+				if (!in_array(REQUEST_CLIENT_IP, $ips, true)) {
+					return Status::error404();
+				}
 			}
 
 			# ------------------------