Passed
Pull Request — master (#11)
by Anton
03:24
created
www/engine/System/Classes/Frames/Site/Area/Auth.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -12,7 +12,7 @@
 block discarded – undo
12 12
 
13 13
 			# Check auth
14 14
 
15
-			if (Modules\Auth::check()) Request::redirect(INSTALL_PATH . '/profile');
15
+			if (Modules\Auth::check()) Request::redirect(INSTALL_PATH.'/profile');
16 16
 
17 17
 			# Handle request
18 18
 
Please login to merge, or discard this 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
 			# Check auth
14 14
 
15
-			if (Modules\Auth::check()) Request::redirect(INSTALL_PATH . '/profile');
15
+			if (Modules\Auth::check()) {
16
+				Request::redirect(INSTALL_PATH . '/profile');
17
+			}
16 18
 
17 19
 			# Handle request
18 20
 
19
-			if (Template::isBlock($result = $this->handle())) return $this->displayPage($result, STATUS_CODE_401);
21
+			if (Template::isBlock($result = $this->handle())) {
22
+				return $this->displayPage($result, STATUS_CODE_401);
23
+			}
20 24
 
21 25
 			# ------------------------
22 26
 
Please login to merge, or discard this patch.
www/engine/System/Classes/Frames/Site/Area/Authorized.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -14,7 +14,7 @@
 block discarded – undo
14 14
 
15 15
 			if (!Modules\Auth::check() || ((false !== Request::get('logout')) && Modules\Auth::logout())) {
16 16
 
17
-				Request::redirect(INSTALL_PATH . '/profile/login');
17
+				Request::redirect(INSTALL_PATH.'/profile/login');
18 18
 			}
19 19
 
20 20
 			# Handle request
Please login to merge, or discard this patch.
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -19,9 +19,13 @@
 block discarded – undo
19 19
 
20 20
 			# Handle request
21 21
 
22
-			if (Template::isBlock($result = $this->handle())) return $this->displayPage($result, STATUS_CODE_200);
22
+			if (Template::isBlock($result = $this->handle())) {
23
+				return $this->displayPage($result, STATUS_CODE_200);
24
+			}
23 25
 
24
-			if (Ajax::isResponse($result)) return Ajax::output($result);
26
+			if (Ajax::isResponse($result)) {
27
+				return Ajax::output($result);
28
+			}
25 29
 
26 30
 			# ------------------------
27 31
 
Please login to merge, or discard this patch.
www/engine/System/Classes/Frames/Site/Section.php 2 patches
Spacing   +5 added lines, -9 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
23 23
 
24 24
 		private function getLayout(Template\Block $contents) {
25 25
 
26
-			$layout = View::get('Layouts/' . $this->layout);
26
+			$layout = View::get('Layouts/'.$this->layout);
27 27
 
28 28
 			# Create layout components
29 29
 
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
 
40 40
 				$layout->getBlock('user')->enable();
41 41
 
42
-				$layout->getBlock('user')->gravatar = Auth::user()->gravatar;;
42
+				$layout->getBlock('user')->gravatar = Auth::user()->gravatar; ;
43 43
 
44 44
 				$layout->getBlock('user')->name = Auth::user()->name;
45 45
 
@@ -50,9 +50,7 @@  discard block
 block discarded – undo
50 50
 
51 51
 			# Set title
52 52
 
53
-			$layout->title = (('' !== SEO::title()) ? SEO::title() :
54
-
55
-				(('' !== $this->title) ? Language::get($this->title) : Settings::get('site_title')));
53
+			$layout->title = (('' !== SEO::title()) ? SEO::title() : (('' !== $this->title) ? Language::get($this->title) : Settings::get('site_title')));
56 54
 
57 55
 			# Set slogan
58 56
 
@@ -95,13 +93,11 @@  discard block
 block discarded – undo
95 93
 
96 94
 			$page->keywords = ('' !== SEO::keywords()) ? SEO::keywords() : Settings::get('site_keywords');
97 95
 
98
-			$page->robots = ((SEO::robotsIndex() ? 'INDEX' : 'NOINDEX') . ',' . (SEO::robotsFollow() ? 'FOLLOW' : 'NOFOLLOW'));
96
+			$page->robots = ((SEO::robotsIndex() ? 'INDEX' : 'NOINDEX').','.(SEO::robotsFollow() ? 'FOLLOW' : 'NOFOLLOW'));
99 97
 
100 98
 			# Set title
101 99
 
102
-			$page->title = (('' !== SEO::title()) ? SEO::title() :
103
-
104
-				((('' !== $this->title) ? (Language::get($this->title) . ' | ') : '') . Settings::get('site_title')));
100
+			$page->title = (('' !== SEO::title()) ? SEO::title() : ((('' !== $this->title) ? (Language::get($this->title).' | ') : '').Settings::get('site_title')));
105 101
 
106 102
 			# Set canonical
107 103
 
Please login to merge, or discard this patch.
Braces   +20 added lines, -8 removed lines patch added patch discarded remove patch
@@ -45,7 +45,9 @@  discard block
 block discarded – undo
45 45
 
46 46
 				$layout->getBlock('auth')->disable();
47 47
 
48
-				if (Auth::user()->rank === RANK_ADMINISTRATOR) $layout->getBlock('admin')->enable();
48
+				if (Auth::user()->rank === RANK_ADMINISTRATOR) {
49
+					$layout->getBlock('admin')->enable();
50
+				}
49 51
 			}
50 52
 
51 53
 			# Set title
@@ -105,9 +107,11 @@  discard block
 block discarded – undo
105 107
 
106 108
 			# Set canonical
107 109
 
108
-			if ('' === SEO::canonical()) $page->getBlock('canonical')->disable();
109
-
110
-			else $page->getBlock('canonical')->link = SEO::canonical();
110
+			if ('' === SEO::canonical()) {
111
+				$page->getBlock('canonical')->disable();
112
+			} else {
113
+				$page->getBlock('canonical')->link = SEO::canonical();
114
+			}
111 115
 
112 116
 			# Set layout
113 117
 
@@ -119,9 +123,13 @@  discard block
 block discarded – undo
119 123
 
120 124
 			# Set global components
121 125
 
122
-			foreach (Variables::generate() as $name => $value) Template::setGlobal($name, $value);
126
+			foreach (Variables::generate() as $name => $value) {
127
+				Template::setGlobal($name, $value);
128
+			}
123 129
 
124
-			foreach (Widgets::generate() as $name => $block) Template::setWidget($name, $block);
130
+			foreach (Widgets::generate() as $name => $block) {
131
+				Template::setWidget($name, $block);
132
+			}
125 133
 
126 134
 			# ------------------------
127 135
 
@@ -134,9 +142,13 @@  discard block
 block discarded – undo
134 142
 
135 143
 			# Check site status
136 144
 
137
-			if (Settings::get('site_status') === STATUS_MAINTENANCE) return Status::maintenance();
145
+			if (Settings::get('site_status') === STATUS_MAINTENANCE) {
146
+				return Status::maintenance();
147
+			}
138 148
 
139
-			if (Settings::get('site_status') === STATUS_UPDATE) return Status::update();
149
+			if (Settings::get('site_status') === STATUS_UPDATE) {
150
+				return Status::update();
151
+			}
140 152
 
141 153
 			# ------------------------
142 154
 
Please login to merge, or discard this patch.
www/engine/System/Classes/Frames/Section.php 2 patches
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -36,22 +36,22 @@
 block discarded – undo
36 36
 
37 37
 			foreach (array_unique($languages) as $path) foreach ($phrases as $name) {
38 38
 
39
-				Language::load($path . 'Phrases/' . $name . '.php');
39
+				Language::load($path.'Phrases/'.$name.'.php');
40 40
 			}
41 41
 
42 42
 			# Set template globals
43 43
 
44
-			Template::setGlobal('template_name',    strtolower(Extend\Templates::active()));
44
+			Template::setGlobal('template_name', strtolower(Extend\Templates::active()));
45 45
 
46
-			Template::setGlobal('site_title',       Settings::get('site_title'));
46
+			Template::setGlobal('site_title', Settings::get('site_title'));
47 47
 
48
-			Template::setGlobal('system_url',       Settings::get('system_url'));
48
+			Template::setGlobal('system_url', Settings::get('system_url'));
49 49
 
50
-			Template::setGlobal('system_email',     Settings::get('system_email'));
50
+			Template::setGlobal('system_email', Settings::get('system_email'));
51 51
 
52
-			Template::setGlobal('install_path',     INSTALL_PATH);
52
+			Template::setGlobal('install_path', INSTALL_PATH);
53 53
 
54
-			Template::setGlobal('index_page',       (('' !== INSTALL_PATH) ? INSTALL_PATH : '/'));
54
+			Template::setGlobal('index_page', (('' !== INSTALL_PATH) ? INSTALL_PATH : '/'));
55 55
 
56 56
 			# Init utils
57 57
 
Please login to merge, or discard this patch.
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -34,10 +34,12 @@  discard block
 block discarded – undo
34 34
 
35 35
 			$phrases = array_merge(static::PHRASES, array_keys(Extend\Addons::items() ?? []));
36 36
 
37
-			foreach (array_unique($languages) as $path) foreach ($phrases as $name) {
37
+			foreach (array_unique($languages) as $path) {
38
+				foreach ($phrases as $name) {
38 39
 
39 40
 				Language::load($path . 'Phrases/' . $name . '.php');
40 41
 			}
42
+			}
41 43
 
42 44
 			# Set template globals
43 45
 
@@ -59,7 +61,9 @@  discard block
 block discarded – undo
59 61
 
60 62
 			# Set timezone
61 63
 
62
-			if (Auth::check() && ('' !== ($timezone = Auth::user()->timezone))) date_default_timezone_set($timezone);
64
+			if (Auth::check() && ('' !== ($timezone = Auth::user()->timezone))) {
65
+				date_default_timezone_set($timezone);
66
+			}
63 67
 
64 68
 			# ------------------------
65 69
 
Please login to merge, or discard this patch.
www/engine/System/Classes/Utils/Form.php 1 patch
Braces   +14 added lines, -4 removed lines patch added patch discarded remove patch
@@ -12,7 +12,11 @@  discard block
 block discarded – undo
12 12
 
13 13
 			$text = Language::get($code);
14 14
 
15
-			if (!$popup) Messages::set('error', $text); else Popup::set('negative', $text);
15
+			if (!$popup) {
16
+				Messages::set('error', $text);
17
+			} else {
18
+				Popup::set('negative', $text);
19
+			}
16 20
 
17 21
 			# ------------------------
18 22
 
@@ -23,15 +27,21 @@  discard block
 block discarded – undo
23 27
 
24 28
 		public function handle(callable $callback, bool $popup = false) {
25 29
 
26
-			if (false === ($post = $this->post())) return false;
30
+			if (false === ($post = $this->post())) {
31
+				return false;
32
+			}
27 33
 
28 34
 			# Check form for errors
29 35
 
30
-			if ($this->hasErrors()) return $this->displayError('FORM_ERROR_REQUIRED', $popup);
36
+			if ($this->hasErrors()) {
37
+				return $this->displayError('FORM_ERROR_REQUIRED', $popup);
38
+			}
31 39
 
32 40
 			# Call controller method
33 41
 
34
-			if (is_string($result = $callback($post))) return $this->displayError($result, $popup);
42
+			if (is_string($result = $callback($post))) {
43
+				return $this->displayError($result, $popup);
44
+			}
35 45
 
36 46
 			if (is_array($result)) { $this->getField($result[0])->error = true; return $this->displayError($result[1], $popup); }
37 47
 
Please login to merge, or discard this patch.
www/engine/System/Classes/Utils/Schema.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -10,7 +10,7 @@
 block discarded – undo
10 10
 
11 11
 		public static function get(string $name) {
12 12
 
13
-			$class_name = ('Schemas\\' . $name);
13
+			$class_name = ('Schemas\\'.$name);
14 14
 
15 15
 			if (!isset(self::$cache[$class_name])) self::$cache[$class_name] = new $class_name;
16 16
 
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -12,7 +12,9 @@
 block discarded – undo
12 12
 
13 13
 			$class_name = ('Schemas\\' . $name);
14 14
 
15
-			if (!isset(self::$cache[$class_name])) self::$cache[$class_name] = new $class_name;
15
+			if (!isset(self::$cache[$class_name])) {
16
+				self::$cache[$class_name] = new $class_name;
17
+			}
16 18
 
17 19
 			# ------------------------
18 20
 
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
 			$selection = ['name', 'value']; $order = ['name' => 'ASC'];
16 16
 
17
-			if (!(DB::select(TABLE_VARIABLES, $selection, null, $order) && DB::getLast()->status)) return;
17
+			if (!(DB::select(TABLE_VARIABLES, $selection, null, $order) && 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   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -14,7 +14,9 @@
 block discarded – undo
14 14
 
15 15
 			$selection = ['name', 'contents']; $condition = ['active' => 1]; $order = ['name' => 'ASC'];
16 16
 
17
-			if (!(DB::select(TABLE_WIDGETS, $selection, $condition, $order) && DB::getLast()->status)) return;
17
+			if (!(DB::select(TABLE_WIDGETS, $selection, $condition, $order) && 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/Menu.php 1 patch
Braces   +9 added lines, -3 removed lines patch added patch discarded remove patch
@@ -20,7 +20,9 @@  discard block
 block discarded – undo
20 20
 
21 21
 				$item->children = ($children = Template::createBlock());
22 22
 
23
-				foreach ($this->menu[$id]['children'] as $child) $children->addItem($this->parseItem($child));
23
+				foreach ($this->menu[$id]['children'] as $child) {
24
+					$children->addItem($this->parseItem($child));
25
+				}
24 26
 
25 27
 			} else {
26 28
 
@@ -44,7 +46,9 @@  discard block
 block discarded – undo
44 46
 
45 47
 			$menu = Entitizer::treeview(TABLE_MENU)->subtree(0, ['active' => true]);
46 48
 
47
-			if (false !== $menu) $this->menu = $menu;
49
+			if (false !== $menu) {
50
+				$this->menu = $menu;
51
+			}
48 52
 		}
49 53
 
50 54
 		# Get block
@@ -53,7 +57,9 @@  discard block
 block discarded – undo
53 57
 
54 58
 			$menu = Template::createBlock();
55 59
 
56
-			foreach ($this->menu[0]['children'] as $id) $menu->addItem($this->parseItem($id));
60
+			foreach ($this->menu[0]['children'] as $id) {
61
+				$menu->addItem($this->parseItem($id));
62
+			}
57 63
 
58 64
 			# ------------------------
59 65
 
Please login to merge, or discard this patch.