Passed
Push — master ( e0d6a2...2ec881 )
by Anton
05:25 queued 02:20
created
www/engine/System/Classes/Modules/Entitizer/Controller/Variable.php 1 patch
Braces   +15 added lines, -5 removed lines patch added patch discarded remove patch
@@ -29,17 +29,25 @@  discard block
 block discarded – undo
29 29
 
30 30
 			# Validate name
31 31
 
32
-			if (false === ($name = Validate::templateComponentName($name))) return ['name', 'VARIABLE_ERROR_NAME_INVALID'];
32
+			if (false === ($name = Validate::templateComponentName($name))) {
33
+				return ['name', 'VARIABLE_ERROR_NAME_INVALID'];
34
+			}
33 35
 
34 36
 			# Check name reserved
35 37
 
36
-			if (false !== Template::getGlobal($name)) return ['name', 'VARIABLE_ERROR_NAME_RESERVED'];
38
+			if (false !== Template::getGlobal($name)) {
39
+				return ['name', 'VARIABLE_ERROR_NAME_RESERVED'];
40
+			}
37 41
 
38 42
 			# Check name exists
39 43
 
40
-			if (false === ($check_name = $this->variable->check($name, 'name'))) return 'VARIABLE_ERROR_MODIFY';
44
+			if (false === ($check_name = $this->variable->check($name, 'name'))) {
45
+				return 'VARIABLE_ERROR_MODIFY';
46
+			}
41 47
 
42
-			if ($check_name === 1) return ['name', 'VARIABLE_ERROR_NAME_DUPLICATE'];
48
+			if ($check_name === 1) {
49
+				return ['name', 'VARIABLE_ERROR_NAME_DUPLICATE'];
50
+			}
43 51
 
44 52
 			# Modify variable
45 53
 
@@ -51,7 +59,9 @@  discard block
 block discarded – undo
51 59
 
52 60
 			$modifier = ((0 === $this->variable->id) ? 'create' : 'edit');
53 61
 
54
-			if (!$this->variable->$modifier($data)) return 'VARIABLE_ERROR_MODIFY';
62
+			if (!$this->variable->$modifier($data)) {
63
+				return 'VARIABLE_ERROR_MODIFY';
64
+			}
55 65
 
56 66
 			# ------------------------
57 67
 
Please login to merge, or discard this patch.
www/engine/System/Classes/Modules/Entitizer/Utils/Handler.php 1 patch
Braces   +37 added lines, -15 removed lines patch added patch discarded remove patch
@@ -45,7 +45,9 @@  discard block
 block discarded – undo
45 45
 
46 46
 		private function processSelector(Template\Block $selector) {
47 47
 
48
-			if ($this->create) return $selector->disable();
48
+			if ($this->create) {
49
+				return $selector->disable();
50
+			}
49 51
 
50 52
 			$selector->parent_id = $this->entity->parent_id;
51 53
 
@@ -68,25 +70,33 @@  discard block
 block discarded – undo
68 70
 
69 71
 			# Set path / title
70 72
 
71
-			if (static::$nesting) $contents->path = $this->path;
72
-
73
-			else $contents->title = ($this->create ? Language::get(static::$naming_new) : $this->entity->get(static::$naming));
73
+			if (static::$nesting) {
74
+				$contents->path = $this->path;
75
+			} else {
76
+				$contents->title = ($this->create ? Language::get(static::$naming_new) : $this->entity->get(static::$naming));
77
+			}
74 78
 
75 79
 			# Process parent block
76 80
 
77
-			if (static::$nesting) $this->processParent($contents->getBlock('parent'));
81
+			if (static::$nesting) {
82
+				$this->processParent($contents->getBlock('parent'));
83
+			}
78 84
 
79 85
 			# Set link
80 86
 
81 87
 			$link = (INSTALL_PATH . static::$link . '/');
82 88
 
83
-			if (static::$nesting) $contents->link = ($link . ($this->create ? 'create' : 'edit') . '?id=' . $this->parent->id);
84
-
85
-			else $contents->link = ($link . ($this->create ? 'create' : ('edit?id=' . $this->entity->id)));
89
+			if (static::$nesting) {
90
+				$contents->link = ($link . ($this->create ? 'create' : 'edit') . '?id=' . $this->parent->id);
91
+			} else {
92
+				$contents->link = ($link . ($this->create ? 'create' : ('edit?id=' . $this->entity->id)));
93
+			}
86 94
 
87 95
 			# Process selector block
88 96
 
89
-			if (static::$nesting) $this->processSelector($contents->getBlock('selector'));
97
+			if (static::$nesting) {
98
+				$this->processSelector($contents->getBlock('selector'));
99
+			}
90 100
 
91 101
 			# Implement form
92 102
 
@@ -119,11 +129,15 @@  discard block
 block discarded – undo
119 129
 
120 130
 				$parent_id = Number::forceInt(Request::post('parent_id'));
121 131
 
122
-				if (!$this->entity->move($parent_id)) return $ajax->setError(Language::get(static::$message_error_move));
132
+				if (!$this->entity->move($parent_id)) {
133
+					return $ajax->setError(Language::get(static::$message_error_move));
134
+				}
123 135
 
124 136
 			} else if (Request::post('action') === 'remove') {
125 137
 
126
-				if (!$this->entity->remove()) return $ajax->setError(Language::get(static::$message_error_remove));
138
+				if (!$this->entity->remove()) {
139
+					return $ajax->setError(Language::get(static::$message_error_remove));
140
+				}
127 141
 			}
128 142
 
129 143
 			# ------------------------
@@ -135,7 +149,9 @@  discard block
 block discarded – undo
135 149
 
136 150
 		protected function handle(bool $ajax = false) {
137 151
 
138
-			if (!$this->create && $ajax) return $this->handleAjax();
152
+			if (!$this->create && $ajax) {
153
+				return $this->handleAjax();
154
+			}
139 155
 
140 156
 			# Create entity
141 157
 
@@ -143,7 +159,9 @@  discard block
 block discarded – undo
143 159
 
144 160
 			$this->entity = Entitizer::get(static::$table, (!$this->create ? $id : 0));
145 161
 
146
-			if (!$this->create && (0 === $this->entity->id)) return Request::redirect(INSTALL_PATH . static::$link);
162
+			if (!$this->create && (0 === $this->entity->id)) {
163
+				return Request::redirect(INSTALL_PATH . static::$link);
164
+			}
147 165
 
148 166
 			# Create parent entity
149 167
 
@@ -151,7 +169,9 @@  discard block
 block discarded – undo
151 169
 
152 170
 			# Get path
153 171
 
154
-			if (false !== ($path = $this->parent->path())) $this->path = $path;
172
+			if (false !== ($path = $this->parent->path())) {
173
+				$this->path = $path;
174
+			}
155 175
 
156 176
 			# Create form
157 177
 
@@ -161,7 +181,9 @@  discard block
 block discarded – undo
161 181
 
162 182
 			if ($this->form->handle(new static::$controller($this->entity), true)) {
163 183
 
164
-				if ($this->create && (0 !== $this->parent->id)) $this->entity->move($this->parent->id);
184
+				if ($this->create && (0 !== $this->parent->id)) {
185
+					$this->entity->move($this->parent->id);
186
+				}
165 187
 
166 188
 				Request::redirect(INSTALL_PATH . static::$link . '/edit?id=' . $this->entity->id . '&submitted');
167 189
 			}
Please login to merge, or discard this patch.
www/engine/System/Classes/Modules/Entitizer/Utils/Lister.php 1 patch
Braces   +36 added lines, -16 removed lines patch added patch discarded remove patch
@@ -20,15 +20,15 @@  discard block
 block discarded – undo
20 20
 
21 21
 			# Set create button
22 22
 
23
-			if (count($this->path) < CONFIG_ENTITIZER_MAX_DEPTH) $parent->getBlock('create')->id = $this->parent->id;
24
-
25
-			else { $parent->getBlock('create')->disable(); $parent->getBlock('create_disabled')->enable(); }
23
+			if (count($this->path) < CONFIG_ENTITIZER_MAX_DEPTH) {
24
+				$parent->getBlock('create')->id = $this->parent->id;
25
+			} else { $parent->getBlock('create')->disable(); $parent->getBlock('create_disabled')->enable(); }
26 26
 
27 27
 			# Set edit button
28 28
 
29
-			if (0 !== $this->parent->id) $parent->getBlock('edit')->id = $this->parent->id;
30
-
31
-			else { $parent->getBlock('edit')->disable(); $parent->getBlock('edit_disabled')->enable(); }
29
+			if (0 !== $this->parent->id) {
30
+				$parent->getBlock('edit')->id = $this->parent->id;
31
+			} else { $parent->getBlock('edit')->disable(); $parent->getBlock('edit_disabled')->enable(); }
32 32
 
33 33
 			# Add parent additional data
34 34
 
@@ -41,7 +41,9 @@  discard block
 block discarded – undo
41 41
 
42 42
 			foreach ($this->items['list'] as $item) {
43 43
 
44
-				if ((null !== $this->entity) && ($item['dataset']->id === $this->entity->id)) continue;
44
+				if ((null !== $this->entity) && ($item['dataset']->id === $this->entity->id)) {
45
+					continue;
46
+				}
45 47
 
46 48
 				$items->addItem($view = View::get(!$ajax ? static::$view_item : static::$view_ajax_item));
47 49
 
@@ -104,11 +106,15 @@  discard block
 block discarded – undo
104 106
 
105 107
 			# Set path
106 108
 
107
-			if (static::$nesting) $contents->path = $this->path;
109
+			if (static::$nesting) {
110
+				$contents->path = $this->path;
111
+			}
108 112
 
109 113
 			# Process parent block
110 114
 
111
-			if (static::$nesting && !$ajax) $this->processParent($contents->getBlock('parent'));
115
+			if (static::$nesting && !$ajax) {
116
+				$this->processParent($contents->getBlock('parent'));
117
+			}
112 118
 
113 119
 			# Process items block
114 120
 
@@ -116,7 +122,9 @@  discard block
 block discarded – undo
116 122
 
117 123
 			# Set pagination
118 124
 
119
-			if (!$ajax) $contents->pagination = $this->getPaginationBlock();
125
+			if (!$ajax) {
126
+				$contents->pagination = $this->getPaginationBlock();
127
+			}
120 128
 
121 129
 			# ------------------------
122 130
 
@@ -141,15 +149,21 @@  discard block
 block discarded – undo
141 149
 
142 150
 			# Get path and depth
143 151
 
144
-			if (false !== ($path = $this->parent->path())) $this->path = $path;
152
+			if (false !== ($path = $this->parent->path())) {
153
+				$this->path = $path;
154
+			}
145 155
 
146
-			if ((0 !== $this->entity->id) && (false !== ($depth = $this->entity->subtreeDepth()))) $this->depth = $depth;
156
+			if ((0 !== $this->entity->id) && (false !== ($depth = $this->entity->subtreeDepth()))) {
157
+				$this->depth = $depth;
158
+			}
147 159
 
148 160
 			# Get items list
149 161
 
150 162
 			$lister = (static::$nesting ? 'children' : 'items');
151 163
 
152
-			if (false !== ($items = $this->parent->$lister())) $this->items = $items;
164
+			if (false !== ($items = $this->parent->$lister())) {
165
+				$this->items = $items;
166
+			}
153 167
 
154 168
 			# ------------------------
155 169
 
@@ -160,7 +174,9 @@  discard block
 block discarded – undo
160 174
 
161 175
 		protected function handle(bool $ajax = false) {
162 176
 
163
-			if ($ajax) return $this->handleAjax();
177
+			if ($ajax) {
178
+				return $this->handleAjax();
179
+			}
164 180
 
165 181
 			$this->index = Number::forceInt(Request::get('index'), 1, 999999);
166 182
 
@@ -172,13 +188,17 @@  discard block
 block discarded – undo
172 188
 
173 189
 			# Get path
174 190
 
175
-			if (false !== ($path = $this->parent->path())) $this->path = $path;
191
+			if (false !== ($path = $this->parent->path())) {
192
+				$this->path = $path;
193
+			}
176 194
 
177 195
 			# Get items list
178 196
 
179 197
 			$lister = (static::$nesting ? 'children' : 'items'); $display = Settings::get('admin_display_entities');
180 198
 
181
-			if (false !== ($items = $this->parent->$lister([], [], $this->index, $display))) $this->items = $items;
199
+			if (false !== ($items = $this->parent->$lister([], [], $this->index, $display))) {
200
+				$this->items = $items;
201
+			}
182 202
 
183 203
 			# ------------------------
184 204
 
Please login to merge, or discard this patch.
www/engine/System/Classes/Utils/Template/Variables.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -14,7 +14,9 @@
 block discarded – undo
14 14
 
15 15
 			DB::select(TABLE_VARIABLES, ['name', 'value'], null, ['name' => 'ASC']);
16 16
 
17
-			if (!(DB::getLast() && DB::getLast()->status)) return;
17
+			if (!(DB::getLast() && DB::getLast()->status)) {
18
+				return;
19
+			}
18 20
 
19 21
 			# Process results
20 22
 
Please login to merge, or discard this patch.
www/engine/System/Classes/Utils/Template/Widgets.php 1 patch
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -12,11 +12,15 @@
 block discarded – undo
12 12
 
13 13
 			# Process selection
14 14
 
15
-			if (0 === count($name = array_keys(Template::getWidgets()))) return;
15
+			if (0 === count($name = array_keys(Template::getWidgets()))) {
16
+				return;
17
+			}
16 18
 
17 19
 			DB::select(TABLE_WIDGETS, ['name', 'contents'], ['active' => 1, 'name' => $name], ['name' => 'ASC']);
18 20
 
19
-			if (!(DB::getLast() && DB::getLast()->status)) return;
21
+			if (!(DB::getLast() && DB::getLast()->status)) {
22
+				return;
23
+			}
20 24
 
21 25
 			# Process results
22 26
 
Please login to merge, or discard this patch.
www/engine/System/Classes/Utils/SEO.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -71,7 +71,7 @@
 block discarded – undo
71 71
 
72 72
 		public static function robots() {
73 73
 
74
-			return ((self::robotsIndex() ? 'INDEX' : 'NOINDEX') . ',' . (self::robotsFollow() ? 'FOLLOW' : 'NOFOLLOW'));
74
+			return ((self::robotsIndex() ? 'INDEX' : 'NOINDEX').','.(self::robotsFollow() ? 'FOLLOW' : 'NOFOLLOW'));
75 75
 		}
76 76
 	}
77 77
 }
Please login to merge, or discard this patch.
Braces   +18 added lines, -6 removed lines patch added patch discarded remove patch
@@ -17,7 +17,9 @@  discard block
 block discarded – undo
17 17
 
18 18
 		public static function title(string $value = null) {
19 19
 
20
-			if (null === $value) return (self::$data['title'] ?? false);
20
+			if (null === $value) {
21
+				return (self::$data['title'] ?? false);
22
+			}
21 23
 
22 24
 			self::$data['title'] = $value;
23 25
 		}
@@ -26,7 +28,9 @@  discard block
 block discarded – undo
26 28
 
27 29
 		public static function description(string $value = null) {
28 30
 
29
-			if (null === $value) return (self::$data['description'] ?? false);
31
+			if (null === $value) {
32
+				return (self::$data['description'] ?? false);
33
+			}
30 34
 
31 35
 			self::$data['description'] = $value;
32 36
 		}
@@ -35,7 +39,9 @@  discard block
 block discarded – undo
35 39
 
36 40
 		public static function keywords(string $value = null) {
37 41
 
38
-			if (null === $value) return (self::$data['keywords'] ?? false);
42
+			if (null === $value) {
43
+				return (self::$data['keywords'] ?? false);
44
+			}
39 45
 
40 46
 			self::$data['keywords'] = $value;
41 47
 		}
@@ -44,7 +50,9 @@  discard block
 block discarded – undo
44 50
 
45 51
 		public static function robotsIndex(bool $value = null) {
46 52
 
47
-			if (null === $value) return (self::$data['robots_index'] ?? false);
53
+			if (null === $value) {
54
+				return (self::$data['robots_index'] ?? false);
55
+			}
48 56
 
49 57
 			self::$data['robots_index'] = $value;
50 58
 		}
@@ -53,7 +61,9 @@  discard block
 block discarded – undo
53 61
 
54 62
 		public static function robotsFollow(bool $value = null) {
55 63
 
56
-			if (null === $value) return (self::$data['robots_follow'] ?? false);
64
+			if (null === $value) {
65
+				return (self::$data['robots_follow'] ?? false);
66
+			}
57 67
 
58 68
 			self::$data['robots_follow'] = $value;
59 69
 		}
@@ -62,7 +72,9 @@  discard block
 block discarded – undo
62 72
 
63 73
 		public static function canonical(string $value = null) {
64 74
 
65
-			if (null === $value) return (self::$data['canonical'] ?? false);
75
+			if (null === $value) {
76
+				return (self::$data['canonical'] ?? false);
77
+			}
66 78
 
67 79
 			self::$data['canonical'] = $value;
68 80
 		}
Please login to merge, or discard this patch.
www/engine/System/Classes/Frames/Admin/Section.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -33,7 +33,7 @@  discard block
 block discarded – undo
33 33
 
34 34
 			# Check auth
35 35
 
36
-			if (($this instanceof Area\Auth) && Auth::check()) Request::redirect(INSTALL_PATH . '/admin');
36
+			if (($this instanceof Area\Auth) && Auth::check()) Request::redirect(INSTALL_PATH.'/admin');
37 37
 
38 38
 			# Handle request
39 39
 
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
 
55 55
 			if (!Auth::check() || ((false !== Request::get('logout')) && Auth::logout())) {
56 56
 
57
-				Request::redirect(INSTALL_PATH . '/admin/login');
57
+				Request::redirect(INSTALL_PATH.'/admin/login');
58 58
 			}
59 59
 
60 60
 			# Handle request
Please login to merge, or discard this patch.
Braces   +21 added lines, -7 removed lines patch added patch discarded remove patch
@@ -33,11 +33,15 @@  discard block
 block discarded – undo
33 33
 
34 34
 			# Check auth
35 35
 
36
-			if (($this instanceof Area\Auth) && Auth::check()) Request::redirect(INSTALL_PATH . '/admin');
36
+			if (($this instanceof Area\Auth) && Auth::check()) {
37
+				Request::redirect(INSTALL_PATH . '/admin');
38
+			}
37 39
 
38 40
 			# Handle request
39 41
 
40
-			if (Template::isBlock($result = $this->handle())) return (new View\Form($this->_title))->display($result);
42
+			if (Template::isBlock($result = $this->handle())) {
43
+				return (new View\Form($this->_title))->display($result);
44
+			}
41 45
 
42 46
 			# ------------------------
43 47
 
@@ -63,9 +67,13 @@  discard block
 block discarded – undo
63 67
 
64 68
 			$result = $this->handle(Request::isAjax() && ($request === 'display'));
65 69
 
66
-			if (Template::isBlock($result)) return (new View\Panel($this->_title))->$request($result);
70
+			if (Template::isBlock($result)) {
71
+				return (new View\Panel($this->_title))->$request($result);
72
+			}
67 73
 
68
-			if (Ajax::isResponse($result)) return Ajax::output($result);
74
+			if (Ajax::isResponse($result)) {
75
+				return Ajax::output($result);
76
+			}
69 77
 
70 78
 			# ------------------------
71 79
 
@@ -84,14 +92,20 @@  discard block
 block discarded – undo
84 92
 
85 93
 				$ips = preg_split('/ +/', CONFIG_ADMIN_IP, -1, PREG_SPLIT_NO_EMPTY);
86 94
 
87
-				if (!in_array(REQUEST_CLIENT_IP, $ips, true)) return Status::displayError404();
95
+				if (!in_array(REQUEST_CLIENT_IP, $ips, true)) {
96
+					return Status::displayError404();
97
+				}
88 98
 			}
89 99
 
90 100
 			# Handle request
91 101
 
92
-			if (($this instanceof Area\Auth) || ($this instanceof Area\Install)) return $this->handleFormArea();
102
+			if (($this instanceof Area\Auth) || ($this instanceof Area\Install)) {
103
+				return $this->handleFormArea();
104
+			}
93 105
 
94
-			if ($this instanceof Area\Panel) return $this->handlePanelArea();
106
+			if ($this instanceof Area\Panel) {
107
+				return $this->handlePanelArea();
108
+			}
95 109
 
96 110
 			# ------------------------
97 111
 
Please login to merge, or discard this patch.
www/engine/System/Classes/Frames/Admin/View.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@  discard block
 block discarded – undo
21 21
 
22 22
 		private function getTitle() : string {
23 23
 
24
-			return ((('' !== $this->title) ? (Language::get($this->title) . ' | ') : '') . CADMIUM_NAME);
24
+			return ((('' !== $this->title) ? (Language::get($this->title).' | ') : '').CADMIUM_NAME);
25 25
 		}
26 26
 
27 27
 		/**
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 
31 31
 		protected function _display(Template\Block $layout) {
32 32
 
33
-			$view = View::get('Main/' . $this->view);
33
+			$view = View::get('Main/'.$this->view);
34 34
 
35 35
 			# Set language
36 36
 
Please login to merge, or discard this patch.
www/engine/System/Classes/Frames/Section.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
 
35 35
 			foreach (array_unique($languages) as $path) foreach ($phrases as $name) {
36 36
 
37
-				Language::load($path . 'Phrases/' . $name . '.php');
37
+				Language::load($path.'Phrases/'.$name.'.php');
38 38
 			}
39 39
 		}
40 40
 
@@ -44,24 +44,24 @@  discard block
 block discarded – undo
44 44
 
45 45
 		private function setGlobals() {
46 46
 
47
-			Template::setGlobal('cadmium_home',         CADMIUM_HOME);
48
-			Template::setGlobal('cadmium_copy',         CADMIUM_COPY);
49
-			Template::setGlobal('cadmium_name',         CADMIUM_NAME);
50
-			Template::setGlobal('cadmium_version',      CADMIUM_VERSION);
47
+			Template::setGlobal('cadmium_home', CADMIUM_HOME);
48
+			Template::setGlobal('cadmium_copy', CADMIUM_COPY);
49
+			Template::setGlobal('cadmium_name', CADMIUM_NAME);
50
+			Template::setGlobal('cadmium_version', CADMIUM_VERSION);
51 51
 
52
-			Template::setGlobal('template_name',        strtolower(Extend\Templates::active()));
52
+			Template::setGlobal('template_name', strtolower(Extend\Templates::active()));
53 53
 
54
-			Template::setGlobal('site_title',           Settings::get('site_title'));
55
-			Template::setGlobal('site_slogan',          Settings::get('site_slogan'));
54
+			Template::setGlobal('site_title', Settings::get('site_title'));
55
+			Template::setGlobal('site_slogan', Settings::get('site_slogan'));
56 56
 
57
-			Template::setGlobal('system_url',           Settings::get('system_url'));
58
-			Template::setGlobal('system_email',         Settings::get('system_email'));
57
+			Template::setGlobal('system_url', Settings::get('system_url'));
58
+			Template::setGlobal('system_email', Settings::get('system_email'));
59 59
 
60
-			Template::setGlobal('install_path',         INSTALL_PATH);
60
+			Template::setGlobal('install_path', INSTALL_PATH);
61 61
 
62
-			Template::setGlobal('index_page',           (('' !== INSTALL_PATH) ? INSTALL_PATH : '/'));
62
+			Template::setGlobal('index_page', (('' !== INSTALL_PATH) ? INSTALL_PATH : '/'));
63 63
 
64
-			Template::setGlobal('current_year',         Date::getYear());
64
+			Template::setGlobal('current_year', Date::getYear());
65 65
 		}
66 66
 
67 67
 		/**
Please login to merge, or discard this patch.