Passed
Push — master ( 1f6010...1bd3c9 )
by Anton
04:57
created
www/engine/System/Classes/Modules/Entitizer/Lister/Pages.php 1 patch
Braces   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -37,9 +37,9 @@
 block discarded – undo
37 37
 
38 38
 		protected function processEntityParent(Template\Block $parent) {
39 39
 
40
-			if ((0 !== $this->parent->id) && $this->parent->active) $parent->getBlock('browse')->link = $this->parent->link;
41
-
42
-			else { $parent->getBlock('browse')->disable(); $parent->getBlock('browse_disabled')->enable(); }
40
+			if ((0 !== $this->parent->id) && $this->parent->active) {
41
+				$parent->getBlock('browse')->link = $this->parent->link;
42
+			} else { $parent->getBlock('browse')->disable(); $parent->getBlock('browse_disabled')->enable(); }
43 43
 		}
44 44
 
45 45
 		/**
Please login to merge, or discard this patch.
www/engine/System/Classes/Modules/Entitizer/Lister/Menuitems.php 1 patch
Braces   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -37,9 +37,9 @@
 block discarded – undo
37 37
 
38 38
 		protected function processEntityParent(Template\Block $parent) {
39 39
 
40
-			if (0 !== $this->parent->id) $parent->getBlock('browse')->link = $this->parent->link;
41
-
42
-			else { $parent->getBlock('browse')->disable(); $parent->getBlock('browse_disabled')->enable(); }
40
+			if (0 !== $this->parent->id) {
41
+				$parent->getBlock('browse')->link = $this->parent->link;
42
+			} else { $parent->getBlock('browse')->disable(); $parent->getBlock('browse_disabled')->enable(); }
43 43
 		}
44 44
 
45 45
 		/**
Please login to merge, or discard this patch.
www/engine/System/Classes/Modules/Entitizer/Handler/Edit/Page.php 1 patch
Braces   +6 added lines, -4 removed lines patch added patch discarded remove patch
@@ -41,9 +41,9 @@  discard block
 block discarded – undo
41 41
 
42 42
 		protected function processEntityParent(Template\Block $parent) {
43 43
 
44
-			if ((0 !== $this->parent->id) && $this->parent->active) $parent->getBlock('browse')->link = $this->parent->link;
45
-
46
-			else { $parent->getBlock('browse')->disable(); $parent->getBlock('browse_disabled')->enable(); }
44
+			if ((0 !== $this->parent->id) && $this->parent->active) {
45
+				$parent->getBlock('browse')->link = $this->parent->link;
46
+			} else { $parent->getBlock('browse')->disable(); $parent->getBlock('browse_disabled')->enable(); }
47 47
 		}
48 48
 
49 49
 		/**
@@ -52,7 +52,9 @@  discard block
 block discarded – undo
52 52
 
53 53
 		protected function processEntity(Template\Block $contents) {
54 54
 
55
-			if (!$this->create && $this->parent->locked) $contents->getBlock('locked')->enable();
55
+			if (!$this->create && $this->parent->locked) {
56
+				$contents->getBlock('locked')->enable();
57
+			}
56 58
 		}
57 59
 	}
58 60
 }
Please login to merge, or discard this patch.
www/engine/System/Classes/Modules/Entitizer/Handler/Edit/Menuitem.php 1 patch
Braces   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -37,9 +37,9 @@
 block discarded – undo
37 37
 
38 38
 		protected function processEntityParent(Template\Block $parent) {
39 39
 
40
-			if (0 !== $this->parent->id) $parent->getBlock('browse')->link = $this->parent->link;
41
-
42
-			else { $parent->getBlock('browse')->disable(); $parent->getBlock('browse_disabled')->enable(); }
40
+			if (0 !== $this->parent->id) {
41
+				$parent->getBlock('browse')->link = $this->parent->link;
42
+			} else { $parent->getBlock('browse')->disable(); $parent->getBlock('browse_disabled')->enable(); }
43 43
 		}
44 44
 
45 45
 		/**
Please login to merge, or discard this patch.
www/engine/System/Classes/Modules/Entitizer/Definition/User/Session.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -23,19 +23,19 @@
 block discarded – undo
23 23
 
24 24
 			# Add params
25 25
 
26
-			$this->params->addTextual       ('code',                true, 40, true, '');
27
-			$this->params->addTextual       ('ip',                  true, 255, false, '');
28
-			$this->params->addInteger       ('time',                false, 10, true, 0);
26
+			$this->params->addTextual('code', true, 40, true, '');
27
+			$this->params->addTextual('ip', true, 255, false, '');
28
+			$this->params->addInteger('time', false, 10, true, 0);
29 29
 
30 30
 			# Add indexes
31 31
 
32
-			$this->indexes->add             ('code',                'UNIQUE');
33
-			$this->indexes->add             ('ip');
34
-			$this->indexes->add             ('time');
32
+			$this->indexes->add('code', 'UNIQUE');
33
+			$this->indexes->add('ip');
34
+			$this->indexes->add('time');
35 35
 
36 36
 			# Add foreign keys
37 37
 
38
-			$this->foreigns->add            ('id',                  TABLE_USERS, 'id', 'CASCADE', 'RESTRICT');
38
+			$this->foreigns->add('id', TABLE_USERS, 'id', 'CASCADE', 'RESTRICT');
39 39
 		}
40 40
 	}
41 41
 }
Please login to merge, or discard this patch.
www/engine/System/Classes/Modules/Filemanager/Utils/Entity.php 3 patches
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -183,7 +183,7 @@  discard block
 block discarded – undo
183 183
 		/**
184 184
 		 * Get the entity creation time
185 185
 		 *
186
-		 * @return int|false : the time or false on failure
186
+		 * @return null|integer : the time or false on failure
187 187
 		 */
188 188
 
189 189
 		public function getCreated() {
@@ -209,7 +209,7 @@  discard block
 block discarded – undo
209 209
 		/**
210 210
 		 * Get the entity modification time
211 211
 		 *
212
-		 * @return int|false : the time or false on failure
212
+		 * @return null|integer : the time or false on failure
213 213
 		 */
214 214
 
215 215
 		public function getModified() {
@@ -235,7 +235,7 @@  discard block
 block discarded – undo
235 235
 		/**
236 236
 		 * Get the entity size
237 237
 		 *
238
-		 * @return int|false : the size or false on failure
238
+		 * @return integer : the size or false on failure
239 239
 		 */
240 240
 
241 241
 		public function getSize() {
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -27,9 +27,9 @@
 block discarded – undo
27 27
 
28 28
 			if ($this->parent->isIgnoreHidden() && preg_match('/^\./', $name)) return false;
29 29
 
30
-			$path = (($this->parent->getPath() ? ($this->parent->getPath() . '/') : '') . $name);
30
+			$path = (($this->parent->getPath() ? ($this->parent->getPath().'/') : '').$name);
31 31
 
32
-			$path_full = ($this->parent->getPathFull() . $name);
32
+			$path_full = ($this->parent->getPathFull().$name);
33 33
 
34 34
 			# ------------------------
35 35
 
Please login to merge, or discard this patch.
Braces   +72 added lines, -24 removed lines patch added patch discarded remove patch
@@ -23,9 +23,13 @@  discard block
 block discarded – undo
23 23
 
24 24
 		private function getPrototype(string $name) {
25 25
 
26
-			if (('' === $name) || preg_match('/[\/\\\\]/', $name)) return false;
26
+			if (('' === $name) || preg_match('/[\/\\\\]/', $name)) {
27
+				return false;
28
+			}
27 29
 
28
-			if ($this->parent->isIgnoreHidden() && preg_match('/^\./', $name)) return false;
30
+			if ($this->parent->isIgnoreHidden() && preg_match('/^\./', $name)) {
31
+				return false;
32
+			}
29 33
 
30 34
 			$path = (($this->parent->getPath() ? ($this->parent->getPath() . '/') : '') . $name);
31 35
 
@@ -53,17 +57,23 @@  discard block
 block discarded – undo
53 57
 
54 58
 		public function init(string $name) : bool {
55 59
 
56
-			if (false === ($prototype = $this->getPrototype($name))) return false;
60
+			if (false === ($prototype = $this->getPrototype($name))) {
61
+				return false;
62
+			}
57 63
 
58 64
 			# Check type
59 65
 
60 66
 			$type = Explorer::getType($prototype['path_full']);
61 67
 
62
-			if (!in_array($type, ['dir', 'file'], true)) return false;
68
+			if (!in_array($type, ['dir', 'file'], true)) {
69
+				return false;
70
+			}
63 71
 
64 72
 			# Set data
65 73
 
66
-			foreach ($prototype as $var => $value) $this->$var = $value;
74
+			foreach ($prototype as $var => $value) {
75
+				$this->$var = $value;
76
+			}
67 77
 
68 78
 			$this->type = $type;
69 79
 
@@ -81,9 +91,13 @@  discard block
 block discarded – undo
81 91
 
82 92
 		public function check(string $name) : bool {
83 93
 
84
-			if (false === ($prototype = $this->getPrototype($name))) return false;
94
+			if (false === ($prototype = $this->getPrototype($name))) {
95
+				return false;
96
+			}
85 97
 
86
-			if (($name !== $this->name) && Explorer::exists($prototype['path_full'])) return false;
98
+			if (($name !== $this->name) && Explorer::exists($prototype['path_full'])) {
99
+				return false;
100
+			}
87 101
 
88 102
 			# ------------------------
89 103
 
@@ -98,19 +112,27 @@  discard block
 block discarded – undo
98 112
 
99 113
 		public function create(string $name, string $type) : bool {
100 114
 
101
-			if (('' !== $this->name) || (false === ($prototype = $this->getPrototype($name)))) return false;
115
+			if (('' !== $this->name) || (false === ($prototype = $this->getPrototype($name)))) {
116
+				return false;
117
+			}
102 118
 
103
-			if (!in_array($type, ['dir', 'file'], true)) return false;
119
+			if (!in_array($type, ['dir', 'file'], true)) {
120
+				return false;
121
+			}
104 122
 
105 123
 			# Create entity
106 124
 
107 125
 			$create = (($type === 'dir') ? 'createDir' : 'createFile');
108 126
 
109
-			if (!Explorer::$create($prototype['path_full'])) return false;
127
+			if (!Explorer::$create($prototype['path_full'])) {
128
+				return false;
129
+			}
110 130
 
111 131
 			# Set data
112 132
 
113
-			foreach ($prototype as $var => $value) $this->$var = $value;
133
+			foreach ($prototype as $var => $value) {
134
+				$this->$var = $value;
135
+			}
114 136
 
115 137
 			$this->type = $type;
116 138
 
@@ -127,15 +149,21 @@  discard block
 block discarded – undo
127 149
 
128 150
 		public function rename(string $name) : bool {
129 151
 
130
-			if (('' === $this->name) || (false === ($prototype = $this->getPrototype($name)))) return false;
152
+			if (('' === $this->name) || (false === ($prototype = $this->getPrototype($name)))) {
153
+				return false;
154
+			}
131 155
 
132 156
 			# Rename entity
133 157
 
134
-			if (!Explorer::rename($this->path_full, $prototype['path_full'])) return false;
158
+			if (!Explorer::rename($this->path_full, $prototype['path_full'])) {
159
+				return false;
160
+			}
135 161
 
136 162
 			# Set data
137 163
 
138
-			foreach ($prototype as $var => $value) $this->$var = $value;
164
+			foreach ($prototype as $var => $value) {
165
+				$this->$var = $value;
166
+			}
139 167
 
140 168
 			# ------------------------
141 169
 
@@ -150,13 +178,17 @@  discard block
 block discarded – undo
150 178
 
151 179
 		public function remove() : bool {
152 180
 
153
-			if ('' === $this->name) return false;
181
+			if ('' === $this->name) {
182
+				return false;
183
+			}
154 184
 
155 185
 			# Remove entity
156 186
 
157 187
 			$remove = (($this->type === 'dir') ? 'removeDir' : 'removeFile');
158 188
 
159
-			if (!Explorer::$remove($this->path_full, true)) return false;
189
+			if (!Explorer::$remove($this->path_full, true)) {
190
+				return false;
191
+			}
160 192
 
161 193
 			# Reset data
162 194
 
@@ -175,7 +207,9 @@  discard block
 block discarded – undo
175 207
 
176 208
 		public function getExtension() {
177 209
 
178
-			if ('' === $this->name) return false;
210
+			if ('' === $this->name) {
211
+				return false;
212
+			}
179 213
 
180 214
 			return strtolower(Explorer::getExtension($this->name, false));
181 215
 		}
@@ -188,7 +222,9 @@  discard block
 block discarded – undo
188 222
 
189 223
 		public function getCreated() {
190 224
 
191
-			if ('' === $this->name) return false;
225
+			if ('' === $this->name) {
226
+				return false;
227
+			}
192 228
 
193 229
 			return Explorer::getCreated($this->path_full);
194 230
 		}
@@ -201,7 +237,9 @@  discard block
 block discarded – undo
201 237
 
202 238
 		public function getAccessed() {
203 239
 
204
-			if ('' === $this->name) return false;
240
+			if ('' === $this->name) {
241
+				return false;
242
+			}
205 243
 
206 244
 			return Explorer::getAccessed($this->path_full);
207 245
 		}
@@ -214,7 +252,9 @@  discard block
 block discarded – undo
214 252
 
215 253
 		public function getModified() {
216 254
 
217
-			if ('' === $this->name) return false;
255
+			if ('' === $this->name) {
256
+				return false;
257
+			}
218 258
 
219 259
 			return Explorer::getModified($this->path_full);
220 260
 		}
@@ -227,7 +267,9 @@  discard block
 block discarded – undo
227 267
 
228 268
 		public function getPermissions() {
229 269
 
230
-			if ('' === $this->name) return false;
270
+			if ('' === $this->name) {
271
+				return false;
272
+			}
231 273
 
232 274
 			return Explorer::getPermissions($this->path_full);
233 275
 		}
@@ -240,7 +282,9 @@  discard block
 block discarded – undo
240 282
 
241 283
 		public function getSize() {
242 284
 
243
-			if ('' === $this->name) return false;
285
+			if ('' === $this->name) {
286
+				return false;
287
+			}
244 288
 
245 289
 			return Explorer::getSize($this->path_full);
246 290
 		}
@@ -253,7 +297,9 @@  discard block
 block discarded – undo
253 297
 
254 298
 		public function getContents() {
255 299
 
256
-			if ('' === $this->name) return false;
300
+			if ('' === $this->name) {
301
+				return false;
302
+			}
257 303
 
258 304
 			return Explorer::getContents($this->path_full);
259 305
 		}
@@ -266,7 +312,9 @@  discard block
 block discarded – undo
266 312
 
267 313
 		public function putContents(string $contents) {
268 314
 
269
-			if ('' === $this->name) return false;
315
+			if ('' === $this->name) {
316
+				return false;
317
+			}
270 318
 
271 319
 			return Explorer::putContents($this->path_full, $contents);
272 320
 		}
Please login to merge, or discard this patch.
www/engine/System/Classes/Modules/Filemanager/Handler/Upload.php 2 patches
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -21,7 +21,9 @@
 block discarded – undo
21 21
 
22 22
 		protected function handle(bool $ajax = false) {
23 23
 
24
-			if (!$ajax) return false;
24
+			if (!$ajax) {
25
+				return false;
26
+			}
25 27
 
26 28
 			# Create response
27 29
 
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -33,9 +33,9 @@
 block discarded – undo
33 33
 
34 34
 			# Save uploaded file
35 35
 
36
-			if (true === ($upload = (Uploader::save('upload', (DIR_UPLOADS . $target_dir))))) {
36
+			if (true === ($upload = (Uploader::save('upload', (DIR_UPLOADS.$target_dir))))) {
37 37
 
38
-				$name = Uploader::getBasename(); $url = (INSTALL_PATH . '/uploads/' . $target_dir . $name);
38
+				$name = Uploader::getBasename(); $url = (INSTALL_PATH.'/uploads/'.$target_dir.$name);
39 39
 
40 40
 				$ajax->set('name', $name)->set('url', $url);
41 41
 
Please login to merge, or discard this patch.
www/engine/System/Classes/Modules/Filemanager/Controller/Rename.php 1 patch
Braces   +12 added lines, -4 removed lines patch added patch discarded remove patch
@@ -44,19 +44,27 @@
 block discarded – undo
44 44
 
45 45
 			# Validate name
46 46
 
47
-			if (false === ($name = Validate::fileName($name))) return ['name', 'FILEMANAGER_ERROR_NAME_INVALID'];
47
+			if (false === ($name = Validate::fileName($name))) {
48
+				return ['name', 'FILEMANAGER_ERROR_NAME_INVALID'];
49
+			}
48 50
 
49
-			if ($this->parent->isIgnoreHidden() && preg_match('/^\./', $name)) return ['name', 'FILEMANAGER_ERROR_HIDDEN'];
51
+			if ($this->parent->isIgnoreHidden() && preg_match('/^\./', $name)) {
52
+				return ['name', 'FILEMANAGER_ERROR_HIDDEN'];
53
+			}
50 54
 
51 55
 			# Check if name is used
52 56
 
53
-			if (!$this->entity->check($name)) return ['name', 'FILEMANAGER_ERROR_EXISTS'];
57
+			if (!$this->entity->check($name)) {
58
+				return ['name', 'FILEMANAGER_ERROR_EXISTS'];
59
+			}
54 60
 
55 61
 			# Rename entity
56 62
 
57
-			if (!$this->entity->rename($name)) return (($this->entity->getType() === 'dir') ?
63
+			if (!$this->entity->rename($name)) {
64
+				return (($this->entity->getType() === 'dir') ?
58 65
 
59 66
 				'FILEMANAGER_ERROR_DIR_RENAME' : 'FILEMANAGER_ERROR_FILE_RENAME');
67
+			}
60 68
 
61 69
 			# ------------------------
62 70
 
Please login to merge, or discard this patch.
www/engine/System/Classes/Modules/Filemanager/Controller/Edit.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -44,7 +44,9 @@
 block discarded – undo
44 44
 
45 45
 			# Rename item
46 46
 
47
-			if (false === $this->entity->putContents($contents)) return 'FILEMANAGER_ERROR_FILE_EDIT';
47
+			if (false === $this->entity->putContents($contents)) {
48
+				return 'FILEMANAGER_ERROR_FILE_EDIT';
49
+			}
48 50
 
49 51
 			# ------------------------
50 52
 
Please login to merge, or discard this patch.