Completed
Pull Request — master (#7)
by Anton
04:49
created
www/engine/System/Includes/Regex.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -2,15 +2,15 @@
 block discarded – undo
2 2
 
3 3
 # Regular expressions
4 4
 
5
-define('REGEX_MAP_ITEM_PATH',                       '/^[a-zA-Z0-9_\-\+\.\,\'\*\(\)\[\]:;!$%~]+$/');
6
-define('REGEX_MAP_ITEM_HANDLER',                    '/^[A-Z][a-zA-Z0-9]*$/');
5
+define('REGEX_MAP_ITEM_PATH', '/^[a-zA-Z0-9_\-\+\.\,\'\*\(\)\[\]:;!$%~]+$/');
6
+define('REGEX_MAP_ITEM_HANDLER', '/^[A-Z][a-zA-Z0-9]*$/');
7 7
 
8
-define('REGEX_LANGUAGE_NAME',                       '/^[a-z][a-z]-[A-Z][A-Z]$/');
9
-define('REGEX_TEMPLATE_NAME',                       '/^[A-Z0-9][a-zA-Z0-9]*$/');
8
+define('REGEX_LANGUAGE_NAME', '/^[a-z][a-z]-[A-Z][A-Z]$/');
9
+define('REGEX_TEMPLATE_NAME', '/^[A-Z0-9][a-zA-Z0-9]*$/');
10 10
 
11
-define('REGEX_USER_AUTH_CODE',                      '/^[a-zA-Z0-9]{40}$/');
11
+define('REGEX_USER_AUTH_CODE', '/^[a-zA-Z0-9]{40}$/');
12 12
 
13
-define('REGEX_USER_NAME',                           '/^(?!_)(?!.*_$)(?!.*_{2,})[a-zA-Z0-9_]+$/');
14
-define('REGEX_USER_PASSWORD',                       '/^.+$/');
13
+define('REGEX_USER_NAME', '/^(?!_)(?!.*_$)(?!.*_{2,})[a-zA-Z0-9_]+$/');
14
+define('REGEX_USER_PASSWORD', '/^.+$/');
15 15
 
16
-define('REGEX_FILE_NAME',                           '/^[^\/?%*:|"<>\\\]+$/');
16
+define('REGEX_FILE_NAME', '/^[^\/?%*:|"<>\\\]+$/');
Please login to merge, or discard this patch.
www/engine/System/Includes/Constants.php 1 patch
Spacing   +41 added lines, -41 removed lines patch added patch discarded remove patch
@@ -2,78 +2,78 @@
 block discarded – undo
2 2
 
3 3
 # Cadmium
4 4
 
5
-define('CADMIUM_NAME',                              'Cadmium CMS');
6
-define('CADMIUM_HOME',                              'http://cadmium-cms.com');
5
+define('CADMIUM_NAME', 'Cadmium CMS');
6
+define('CADMIUM_HOME', 'http://cadmium-cms.com');
7 7
 
8
-define('CADMIUM_VERSION',                           '0.2.2');
9
-define('CADMIUM_COPY',                              '2016');
8
+define('CADMIUM_VERSION', '0.2.2');
9
+define('CADMIUM_COPY', '2016');
10 10
 
11 11
 # External
12 12
 
13
-define('JQUERY_VERSION',                            '2.2.0');
14
-define('SEMANTIC_UI_VERSION',                       '2.1.8');
15
-define('CKEDITOR_VERSION',                          '4.5.6');
13
+define('JQUERY_VERSION', '2.2.0');
14
+define('SEMANTIC_UI_VERSION', '2.1.8');
15
+define('CKEDITOR_VERSION', '4.5.6');
16 16
 
17 17
 # Sections
18 18
 
19
-define('SECTION_ADMIN',                             'Admin');
20
-define('SECTION_SITE',                              'Site');
19
+define('SECTION_ADMIN', 'Admin');
20
+define('SECTION_SITE', 'Site');
21 21
 
22 22
 # Entity types
23 23
 
24
-define('ENTITY_TYPE_PAGE',                          'page');
25
-define('ENTITY_TYPE_MENUITEM',                      'menuitem');
26
-define('ENTITY_TYPE_VARIABLE',                      'variable');
27
-define('ENTITY_TYPE_WIDGET',                        'widget');
28
-define('ENTITY_TYPE_USER',                          'user');
29
-define('ENTITY_TYPE_USER_SECRET',                   'user_secret');
30
-define('ENTITY_TYPE_USER_SESSION',                  'user_session');
24
+define('ENTITY_TYPE_PAGE', 'page');
25
+define('ENTITY_TYPE_MENUITEM', 'menuitem');
26
+define('ENTITY_TYPE_VARIABLE', 'variable');
27
+define('ENTITY_TYPE_WIDGET', 'widget');
28
+define('ENTITY_TYPE_USER', 'user');
29
+define('ENTITY_TYPE_USER_SECRET', 'user_secret');
30
+define('ENTITY_TYPE_USER_SESSION', 'user_session');
31 31
 
32 32
 # Filemanager types
33 33
 
34
-define('FILEMANAGER_TYPE_DIR',                      'dir');
35
-define('FILEMANAGER_TYPE_FILE',                     'file');
34
+define('FILEMANAGER_TYPE_DIR', 'dir');
35
+define('FILEMANAGER_TYPE_FILE', 'file');
36 36
 
37 37
 # Access
38 38
 
39
-define('ACCESS_PUBLIC',                             0);
40
-define('ACCESS_REGISTERED',                         1);
41
-define('ACCESS_ADMINISTRATOR',                      2);
39
+define('ACCESS_PUBLIC', 0);
40
+define('ACCESS_REGISTERED', 1);
41
+define('ACCESS_ADMINISTRATOR', 2);
42 42
 
43 43
 # Frequency
44 44
 
45
-define('FREQUENCY_ALWAYS',                          'always');
46
-define('FREQUENCY_HOURLY',                          'hourly');
47
-define('FREQUENCY_DAILY',                           'daily');
48
-define('FREQUENCY_WEEKLY',                          'weekly');
49
-define('FREQUENCY_MONTHLY',                         'monthly');
50
-define('FREQUENCY_YEARLY',                          'yearly');
51
-define('FREQUENCY_NEVER',                           'never');
45
+define('FREQUENCY_ALWAYS', 'always');
46
+define('FREQUENCY_HOURLY', 'hourly');
47
+define('FREQUENCY_DAILY', 'daily');
48
+define('FREQUENCY_WEEKLY', 'weekly');
49
+define('FREQUENCY_MONTHLY', 'monthly');
50
+define('FREQUENCY_YEARLY', 'yearly');
51
+define('FREQUENCY_NEVER', 'never');
52 52
 
53 53
 # Rank
54 54
 
55
-define('RANK_GUEST',                                0);
56
-define('RANK_USER',                                 1);
57
-define('RANK_ADMINISTRATOR',                        2);
55
+define('RANK_GUEST', 0);
56
+define('RANK_USER', 1);
57
+define('RANK_ADMINISTRATOR', 2);
58 58
 
59 59
 # Sex
60 60
 
61
-define('SEX_NOT_SELECTED',                          0);
62
-define('SEX_MALE',                                  1);
63
-define('SEX_FEMALE',                                2);
61
+define('SEX_NOT_SELECTED', 0);
62
+define('SEX_MALE', 1);
63
+define('SEX_FEMALE', 2);
64 64
 
65 65
 # Status
66 66
 
67
-define('STATUS_ONLINE',                             0);
68
-define('STATUS_MAINTENANCE',                        1);
69
-define('STATUS_UPDATE',                             2);
67
+define('STATUS_ONLINE', 0);
68
+define('STATUS_MAINTENANCE', 1);
69
+define('STATUS_UPDATE', 2);
70 70
 
71 71
 # Target
72 72
 
73
-define('TARGET_SELF',                               0);
74
-define('TARGET_BLANK',                              1);
73
+define('TARGET_SELF', 0);
74
+define('TARGET_BLANK', 1);
75 75
 
76 76
 # Visibility
77 77
 
78
-define('VISIBILITY_DRAFT',                          0);
79
-define('VISIBILITY_PUBLISHED',                      1);
78
+define('VISIBILITY_DRAFT', 0);
79
+define('VISIBILITY_PUBLISHED', 1);
Please login to merge, or discard this patch.
www/engine/System/Classes/Modules/Extend/Utils/Extension.php 1 patch
Braces   +43 added lines, -15 removed lines patch added patch discarded remove patch
@@ -32,11 +32,15 @@  discard block
 block discarded – undo
32 32
 
33 33
 				$file_name = ($dir_name . $name . '/Config.json');
34 34
 
35
-				if (false === ($config = Explorer::json($file_name))) continue;
35
+				if (false === ($config = Explorer::json($file_name))) {
36
+					continue;
37
+				}
36 38
 
37 39
 				$config = Arr::select($config, self::$data);
38 40
 
39
-				if (!(self::valid($config['name']) && ($config['name'] === $name))) continue;
41
+				if (!(self::valid($config['name']) && ($config['name'] === $name))) {
42
+					continue;
43
+				}
40 44
 
41 45
 				$items[$name] = array_map('strval', $config);
42 46
 			}
@@ -52,11 +56,19 @@  discard block
 block discarded – undo
52 56
 
53 57
 			$name = ''; $param = self::$param[self::$section];
54 58
 
55
-			if (self::exists($name_cookie = Cookie::get($param))) $name = $name_cookie;
59
+			if (self::exists($name_cookie = Cookie::get($param))) {
60
+				$name = $name_cookie;
61
+			}
56 62
 
57
-			if (self::exists($name_get = Request::get(self::$name))) $name = $name_get;
63
+			if (self::exists($name_get = Request::get(self::$name))) {
64
+				$name = $name_get;
65
+			}
58 66
 
59
-			if ('' !== $name) Cookie::set($param, $name, self::$cookie_expires); else return false;
67
+			if ('' !== $name) {
68
+				Cookie::set($param, $name, self::$cookie_expires);
69
+			} else {
70
+				return false;
71
+			}
60 72
 
61 73
 			# ------------------------
62 74
 
@@ -90,7 +102,9 @@  discard block
 block discarded – undo
90 102
 
91 103
 			$section = self::getSection($section); $dir_name = self::getDirName($section);
92 104
 
93
-			if (!Explorer::isDir($dir_name)) throw new Exception\General(self::$error_directory);
105
+			if (!Explorer::isDir($dir_name)) {
106
+				throw new Exception\General(self::$error_directory);
107
+			}
94 108
 
95 109
 			self::$section = $section; self::$dir_name = $dir_name; self::$items = self::getItems($dir_name);
96 110
 
@@ -98,11 +112,15 @@  discard block
 block discarded – undo
98 112
 
99 113
 			$primary = (self::exists(self::$default[$section]) ? self::$default[$section] : false);
100 114
 
101
-			if ($selectable && (false !== ($name = self::getUserDefined()))) $name_valid = true;
102
-
103
-			else $name_valid = (self::exists($name = Settings::get($param)) || (false !== ($name = $primary)));
115
+			if ($selectable && (false !== ($name = self::getUserDefined()))) {
116
+				$name_valid = true;
117
+			} else {
118
+				$name_valid = (self::exists($name = Settings::get($param)) || (false !== ($name = $primary)));
119
+			}
104 120
 
105
-			if (!($name_valid || (null !== ($name = key(self::$items))))) throw new Exception\General(self::$error_select);
121
+			if (!($name_valid || (null !== ($name = key(self::$items))))) {
122
+				throw new Exception\General(self::$error_select);
123
+			}
106 124
 
107 125
 			# ------------------------
108 126
 
@@ -127,7 +145,9 @@  discard block
 block discarded – undo
127 145
 
128 146
 		public static function items(string $section = null) {
129 147
 
130
-			if (null === $section) return self::$items;
148
+			if (null === $section) {
149
+				return self::$items;
150
+			}
131 151
 
132 152
 			$section = self::getSection($section); $dir_name = self::getDirName($section);
133 153
 
@@ -152,7 +172,9 @@  discard block
 block discarded – undo
152 172
 
153 173
 		public static function path() {
154 174
 
155
-			if ('' === self::$active) return false;
175
+			if ('' === self::$active) {
176
+				return false;
177
+			}
156 178
 
157 179
 			return (self::$dir_name . self::$active . '/');
158 180
 		}
@@ -161,7 +183,9 @@  discard block
 block discarded – undo
161 183
 
162 184
 		public static function pathPrimary() {
163 185
 
164
-			if ('' === self::$primary) return false;
186
+			if ('' === self::$primary) {
187
+				return false;
188
+			}
165 189
 
166 190
 			return (self::$dir_name . self::$primary . '/');
167 191
 		}
@@ -170,9 +194,13 @@  discard block
 block discarded – undo
170 194
 
171 195
 		public static function data(string $name = null) {
172 196
 
173
-			if ('' === self::$active) return false;
197
+			if ('' === self::$active) {
198
+				return false;
199
+			}
174 200
 
175
-			if (null === $name) return self::$items[self::$active];
201
+			if (null === $name) {
202
+				return self::$items[self::$active];
203
+			}
176 204
 
177 205
 			return (isset(self::$items[self::$active][$name]) ? self::$items[self::$active][$name] : false);
178 206
 		}
Please login to merge, or discard this patch.
www/engine/System/Classes/Modules/Settings/Settings.php 1 patch
Braces   +39 added lines, -13 removed lines patch added patch discarded remove patch
@@ -37,7 +37,9 @@  discard block
 block discarded – undo
37 37
 
38 38
 		public static function __autoload() {
39 39
 
40
-			if (self::$loaded || empty($host = getenv('HTTP_HOST'))) return;
40
+			if (self::$loaded || empty($host = getenv('HTTP_HOST'))) {
41
+				return;
42
+			}
41 43
 
42 44
 			self::$settings['system_url'] = ((Request::isSecure() ? 'https://' : 'http://') . $host);
43 45
 
@@ -52,7 +54,9 @@  discard block
 block discarded – undo
52 54
 
53 55
 			self::$loaded = (false !== ($settings = Explorer::json($settings_file)));
54 56
 
55
-			if (self::$loaded) foreach ($settings as $name => $value) self::set($name, $value);
57
+			if (self::$loaded) {
58
+				foreach ($settings as $name => $value) self::set($name, $value);
59
+			}
56 60
 
57 61
 			# ------------------------
58 62
 
@@ -65,9 +69,13 @@  discard block
 block discarded – undo
65 69
 
66 70
 			$settings_file = (DIR_SYSTEM_DATA . 'Settings.json');
67 71
 
68
-			if (false === ($settings = json_encode(self::$settings, JSON_PRETTY_PRINT))) return false;
72
+			if (false === ($settings = json_encode(self::$settings, JSON_PRETTY_PRINT))) {
73
+				return false;
74
+			}
69 75
 
70
-			if (false === Explorer::save($settings_file, $settings, true)) return false;
76
+			if (false === Explorer::save($settings_file, $settings, true)) {
77
+				return false;
78
+			}
71 79
 
72 80
 			# ------------------------
73 81
 
@@ -85,34 +93,44 @@  discard block
 block discarded – undo
85 93
 
86 94
 		public static function set(string $name, $value) {
87 95
 
88
-			if (!isset(self::$settings[$name]) || !is_scalar($value)) return false;
96
+			if (!isset(self::$settings[$name]) || !is_scalar($value)) {
97
+				return false;
98
+			}
89 99
 
90 100
 			# Validate language
91 101
 
92 102
 			if (($name === 'admin_language') || ($name === 'site_language')) {
93 103
 
94
-				if (false === ($value = Extend\Languages::validate($value))) return false;
104
+				if (false === ($value = Extend\Languages::validate($value))) {
105
+					return false;
106
+				}
95 107
 			}
96 108
 
97 109
 			# Validate template
98 110
 
99 111
 			else if (($name === 'admin_template') || ($name === 'site_template')) {
100 112
 
101
-				if (false === ($value = Extend\Templates::validate($value))) return false;
113
+				if (false === ($value = Extend\Templates::validate($value))) {
114
+					return false;
115
+				}
102 116
 			}
103 117
 
104 118
 			# Validate site title
105 119
 
106 120
 			else if ($name === 'site_title') {
107 121
 
108
-				if ('' === ($value = strval($value))) return false;
122
+				if ('' === ($value = strval($value))) {
123
+					return false;
124
+				}
109 125
 			}
110 126
 
111 127
 			# Validate site status
112 128
 
113 129
 			else if ($name === 'site_status') {
114 130
 
115
-				if (false === ($value = Lister\Status::validate($value))) return false;
131
+				if (false === ($value = Lister\Status::validate($value))) {
132
+					return false;
133
+				}
116 134
 			}
117 135
 
118 136
 			# Validate site slogan/description/keywords
@@ -126,21 +144,27 @@  discard block
 block discarded – undo
126 144
 
127 145
 			else if ($name === 'system_url') {
128 146
 
129
-				if (false === ($value = Validate::url($value))) return false;
147
+				if (false === ($value = Validate::url($value))) {
148
+					return false;
149
+				}
130 150
 			}
131 151
 
132 152
 			# Validate system email
133 153
 
134 154
 			else if ($name === 'system_email') {
135 155
 
136
-				if (false === ($value = Validate::email($value))) return false;
156
+				if (false === ($value = Validate::email($value))) {
157
+					return false;
158
+				}
137 159
 			}
138 160
 
139 161
 			# Validate system timezone
140 162
 
141 163
 			else if ($name === 'system_timezone') {
142 164
 
143
-				if (false === ($value = Timezone::validate($value))) return false;
165
+				if (false === ($value = Timezone::validate($value))) {
166
+					return false;
167
+				}
144 168
 			}
145 169
 
146 170
 			# Validate users registration
@@ -163,7 +187,9 @@  discard block
 block discarded – undo
163 187
 
164 188
 		public static function get(string $name = null) {
165 189
 
166
-			if (null === $name) return self::$settings;
190
+			if (null === $name) {
191
+				return self::$settings;
192
+			}
167 193
 
168 194
 			return (isset(self::$settings[$name]) ? self::$settings[$name] : null);
169 195
 		}
Please login to merge, or discard this patch.
www/engine/System/Classes/Utils/Map.php 1 patch
Braces   +18 added lines, -6 removed lines patch added patch discarded remove patch
@@ -14,7 +14,9 @@  discard block
 block discarded – undo
14 14
 
15 15
 			$parts = preg_split('/\//', $string, 0, PREG_SPLIT_NO_EMPTY);
16 16
 
17
-			foreach ($parts as $name) if (!preg_match($regex, $name)) return false;
17
+			foreach ($parts as $name) {
18
+				if (!preg_match($regex, $name)) return false;
19
+			}
18 20
 
19 21
 			# ------------------------
20 22
 
@@ -27,9 +29,13 @@  discard block
 block discarded – undo
27 29
 
28 30
 			$data = Arr::select($item, ['path', 'handler']);
29 31
 
30
-			if (false === ($path = $this->parseString($data['path'], REGEX_MAP_ITEM_PATH))) return;
32
+			if (false === ($path = $this->parseString($data['path'], REGEX_MAP_ITEM_PATH))) {
33
+				return;
34
+			}
31 35
 
32
-			if (false === ($handler = $this->parseString($data['handler'], REGEX_MAP_ITEM_HANDLER))) return;
36
+			if (false === ($handler = $this->parseString($data['handler'], REGEX_MAP_ITEM_HANDLER))) {
37
+				return;
38
+			}
33 39
 
34 40
 			$this->map[] = ['path' => $path, 'handler' => implode('\\', $handler)];
35 41
 		}
@@ -40,16 +46,22 @@  discard block
 block discarded – undo
40 46
 
41 47
 			$file_name = (DIR_SYSTEM_DATA . 'Map.json');
42 48
 
43
-			if (false === ($map = Explorer::json($file_name))) return;
49
+			if (false === ($map = Explorer::json($file_name))) {
50
+				return;
51
+			}
44 52
 
45
-			foreach ($map as $item) if (is_array($item)) $this->parseItem($item);
53
+			foreach ($map as $item) {
54
+				if (is_array($item)) $this->parseItem($item);
55
+			}
46 56
 		}
47 57
 
48 58
 		# Get handler by url
49 59
 
50 60
 		public function handler(Url $url) {
51 61
 
52
-			foreach ($this->map as $item) if ($item['path'] === $url->path()) return $item['handler'];
62
+			foreach ($this->map as $item) {
63
+				if ($item['path'] === $url->path()) return $item['handler'];
64
+			}
53 65
 
54 66
 			return false;
55 67
 		}
Please login to merge, or discard this patch.
www/engine/System/System.php 1 patch
Braces   +9 added lines, -3 removed lines patch added patch discarded remove patch
@@ -18,12 +18,16 @@  discard block
 block discarded – undo
18 18
 
19 19
 			foreach (array_keys($this->database) as $key) {
20 20
 
21
-				if (is_scalar($value = Arr::get($data, ['database', $key]))) $this->database[$key] = strval($value);
21
+				if (is_scalar($value = Arr::get($data, ['database', $key]))) {
22
+					$this->database[$key] = strval($value);
23
+				}
22 24
 			}
23 25
 
24 26
 			# Parse installation details
25 27
 
26
-			if (is_numeric($time = Arr::get($data, ['time']))) $this->time = intval($time);
28
+			if (is_numeric($time = Arr::get($data, ['time']))) {
29
+				$this->time = intval($time);
30
+			}
27 31
 		}
28 32
 
29 33
 		# Constructor
@@ -32,7 +36,9 @@  discard block
 block discarded – undo
32 36
 
33 37
 			$system_file = (DIR_SYSTEM_DATA . 'System.json');
34 38
 
35
-			if (false !== ($data = Explorer::json($system_file))) $this->parse($data);
39
+			if (false !== ($data = Explorer::json($system_file))) {
40
+				$this->parse($data);
41
+			}
36 42
 		}
37 43
 	}
38 44
 }
Please login to merge, or discard this patch.
www/engine/Framework/Classes/Explorer/Explorer.php 1 patch
Braces   +28 added lines, -10 removed lines patch added patch discarded remove patch
@@ -12,9 +12,13 @@  discard block
 block discarded – undo
12 12
 
13 13
 				while (false !== ($name = readdir($handler))) {
14 14
 
15
-					if (($name === '.') || ($name === '..')) continue;
15
+					if (($name === '.') || ($name === '..')) {
16
+						continue;
17
+					}
16 18
 
17
-					if ($dirs ? @is_dir($dir_name . $name) : @is_file($dir_name . $name)) yield $name;
19
+					if ($dirs ? @is_dir($dir_name . $name) : @is_file($dir_name . $name)) {
20
+						yield $name;
21
+					}
18 22
 				}
19 23
 
20 24
 				closedir($handler);
@@ -25,7 +29,9 @@  discard block
 block discarded – undo
25 29
 
26 30
 		private static function getInfo(string $file_name, int $param, bool $check_exists = true) {
27 31
 
28
-			if ($check_exists && !self::isFile($file_name)) return false;
32
+			if ($check_exists && !self::isFile($file_name)) {
33
+				return false;
34
+			}
29 35
 
30 36
 			return pathinfo($file_name, $param);
31 37
 		}
@@ -68,9 +74,11 @@  discard block
 block discarded – undo
68 74
 
69 75
 					$name = ($dir_name . '/' . $name);
70 76
 
71
-					if (@is_file($name)) self::removeFile($name);
72
-
73
-					else if (@is_dir($name)) self::removeDir($name, true);
77
+					if (@is_file($name)) {
78
+						self::removeFile($name);
79
+					} else if (@is_dir($name)) {
80
+						self::removeDir($name, true);
81
+					}
74 82
 				}
75 83
 			}
76 84
 
@@ -146,7 +154,9 @@  discard block
 block discarded – undo
146 154
 
147 155
 		public static function json(string $file_name) {
148 156
 
149
-			if ((strtolower(self::extension($file_name)) !== 'json')) return false;
157
+			if ((strtolower(self::extension($file_name)) !== 'json')) {
158
+				return false;
159
+			}
150 160
 
151 161
 			return (json_decode(@file_get_contents($file_name), true) ?? false);
152 162
 		}
@@ -155,7 +165,9 @@  discard block
 block discarded – undo
155 165
 
156 166
 		public static function php(string $file_name) {
157 167
 
158
-			if ((strtolower(self::extension($file_name)) !== 'php')) return false;
168
+			if ((strtolower(self::extension($file_name)) !== 'php')) {
169
+				return false;
170
+			}
159 171
 
160 172
 			return include $file_name;
161 173
 		}
@@ -164,7 +176,9 @@  discard block
 block discarded – undo
164 176
 
165 177
 		public static function xml(string $file_name) {
166 178
 
167
-			if ((strtolower(self::extension($file_name)) !== 'xml')) return false;
179
+			if ((strtolower(self::extension($file_name)) !== 'xml')) {
180
+				return false;
181
+			}
168 182
 
169 183
 			return @simplexml_load_file($file_name);
170 184
 		}
@@ -173,7 +187,11 @@  discard block
 block discarded – undo
173 187
 
174 188
 		public static function save(string $file_name, string $contents, bool $force = false) {
175 189
 
176
-			if (self::isFile($file_name)) if (!$force) return false; else if (!@unlink($file_name)) return false;
190
+			if (self::isFile($file_name)) {
191
+				if (!$force) return false;
192
+			} else if (!@unlink($file_name)) {
193
+				return false;
194
+			}
177 195
 
178 196
 			return @file_put_contents($file_name, $contents);
179 197
 		}
Please login to merge, or discard this patch.
www/engine/Framework/Classes/Template/Template.php 1 patch
Braces   +15 added lines, -5 removed lines patch added patch discarded remove patch
@@ -10,18 +10,26 @@  discard block
 block discarded – undo
10 10
 
11 11
 		public static function global(string $name, string $value = null) {
12 12
 
13
-			if (null === $value) return (self::$globals[$name] ?? false);
13
+			if (null === $value) {
14
+				return (self::$globals[$name] ?? false);
15
+			}
14 16
 
15
-			if (!isset(self::$globals[$name])) self::$globals[$name] = $value;
17
+			if (!isset(self::$globals[$name])) {
18
+				self::$globals[$name] = $value;
19
+			}
16 20
 		}
17 21
 
18 22
 		# Set/get widget
19 23
 
20 24
 		public static function widget(string $name, Template\Asset\Block $block = null) {
21 25
 
22
-			if (null === $block) return (self::$widgets[$name] ?? false);
26
+			if (null === $block) {
27
+				return (self::$widgets[$name] ?? false);
28
+			}
23 29
 
24
-			if (!isset(self::$widgets[$name])) self::$widgets[$name] = $block;
30
+			if (!isset(self::$widgets[$name])) {
31
+				self::$widgets[$name] = $block;
32
+			}
25 33
 		}
26 34
 
27 35
 		# Create block
@@ -56,7 +64,9 @@  discard block
 block discarded – undo
56 64
 
57 65
 		public static function output(Template\Asset\Block $block, $status = null) {
58 66
 
59
-			if ((null === $status) || !Headers::isStatusCode($status)) $status = STATUS_CODE_200;
67
+			if ((null === $status) || !Headers::isStatusCode($status)) {
68
+				$status = STATUS_CODE_200;
69
+			}
60 70
 
61 71
 			Headers::nocache(); Headers::status($status); Headers::content(MIME_TYPE_HTML);
62 72
 
Please login to merge, or discard this patch.
www/engine/Framework/Classes/Arr/Arr.php 1 patch
Braces   +20 added lines, -5 removed lines patch added patch discarded remove patch
@@ -10,7 +10,12 @@  discard block
 block discarded – undo
10 10
 
11 11
 			$value = false;
12 12
 
13
-			foreach ($path as $item) if (isset($array[$item])) $value = ($array = $array[$item]); else return false;
13
+			foreach ($path as $item) {
14
+				if (isset($array[$item])) $value = ($array = $array[$item]);
15
+			}
16
+			else {
17
+				return false;
18
+			}
14 19
 
15 20
 			# ------------------------
16 21
 
@@ -23,10 +28,12 @@  discard block
 block discarded – undo
23 28
 
24 29
 			$result = [];
25 30
 
26
-			foreach ($keys as $key) if (is_scalar($key)) {
31
+			foreach ($keys as $key) {
32
+				if (is_scalar($key)) {
27 33
 
28 34
 				$result[$key] = ((isset($array[$key]) && is_scalar($array[$key])) ? $array[$key] : false);
29 35
 			}
36
+			}
30 37
 
31 38
 			# ------------------------
32 39
 
@@ -39,7 +46,9 @@  discard block
 block discarded – undo
39 46
 
40 47
 			$result = [];
41 48
 
42
-			foreach ($array as $key => $value) $result[] = [$key_name => $key, $value_name => $value];
49
+			foreach ($array as $key => $value) {
50
+				$result[] = [$key_name => $key, $value_name => $value];
51
+			}
43 52
 
44 53
 			# ------------------------
45 54
 
@@ -57,9 +66,15 @@  discard block
 block discarded – undo
57 66
 
58 67
 			$column = array_map($select_key, $array);
59 68
 
60
-			if (!$descending) asort($column); else arsort($column);
69
+			if (!$descending) {
70
+				asort($column);
71
+			} else {
72
+				arsort($column);
73
+			}
61 74
 
62
-			$result = []; foreach (array_keys($column) as $key) $result[$key] = $array[$key];
75
+			$result = []; foreach (array_keys($column) as $key) {
76
+				$result[$key] = $array[$key];
77
+			}
63 78
 
64 79
 			# ------------------------
65 80
 
Please login to merge, or discard this patch.