Passed
Push — master ( c7e98d...e0d6a2 )
by Anton
04:56 queued 01:50
created
www/engine/System/Classes/Modules/Entitizer/Dataset/Page.php 2 patches
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -19,14 +19,18 @@
 block discarded – undo
19 19
 
20 20
 			$this->addHandler('link', function (array $data) {
21 21
 
22
-				if ('' === $data['slug']) return '';
22
+				if ('' === $data['slug']) {
23
+					return '';
24
+				}
23 25
 
24 26
 				return (INSTALL_PATH . '/' . $data['slug']);
25 27
 			});
26 28
 
27 29
 			$this->addHandler('canonical', function (array $data) {
28 30
 
29
-				if ('' === $data['slug']) return '';
31
+				if ('' === $data['slug']) {
32
+					return '';
33
+				}
30 34
 
31 35
 				return (Settings::get('system_url') . (($data['id'] !== 1) ? ('/' . $data['slug']) : ''));
32 36
 			});
Please login to merge, or discard this patch.
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -12,23 +12,23 @@
 block discarded – undo
12 12
 
13 13
 		protected function init() {
14 14
 
15
-			$this->addWorker('active', function (array $data) {
15
+			$this->addWorker('active', function(array $data) {
16 16
 
17 17
 				return (($data['visibility'] === VISIBILITY_PUBLISHED) && !$data['locked']);
18 18
 			});
19 19
 
20
-			$this->addWorker('link', function (array $data) {
20
+			$this->addWorker('link', function(array $data) {
21 21
 
22 22
 				if ('' === $data['slug']) return '';
23 23
 
24
-				return (INSTALL_PATH . '/' . $data['slug']);
24
+				return (INSTALL_PATH.'/'.$data['slug']);
25 25
 			});
26 26
 
27
-			$this->addWorker('canonical', function (array $data) {
27
+			$this->addWorker('canonical', function(array $data) {
28 28
 
29 29
 				if ('' === $data['slug']) return '';
30 30
 
31
-				return (Settings::get('system_url') . (($data['id'] !== 1) ? ('/' . $data['slug']) : ''));
31
+				return (Settings::get('system_url').(($data['id'] !== 1) ? ('/'.$data['slug']) : ''));
32 32
 			});
33 33
 		}
34 34
 	}
Please login to merge, or discard this patch.
www/engine/System/Classes/Modules/Entitizer/Dataset/Menuitem.php 2 patches
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -14,9 +14,13 @@
 block discarded – undo
14 14
 
15 15
 			$this->addHandler('link', function (array $data) {
16 16
 
17
-				if ('' === $data['slug']) return '';
17
+				if ('' === $data['slug']) {
18
+					return '';
19
+				}
18 20
 
19
-				if (false !== Validate::url($data['slug'])) return $data['slug'];
21
+				if (false !== Validate::url($data['slug'])) {
22
+					return $data['slug'];
23
+				}
20 24
 
21 25
 				return (INSTALL_PATH . '/' . $data['slug']);
22 26
 			});
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -12,13 +12,13 @@
 block discarded – undo
12 12
 
13 13
 		protected function init() {
14 14
 
15
-			$this->addWorker('link', function (array $data) {
15
+			$this->addWorker('link', function(array $data) {
16 16
 
17 17
 				if ('' === $data['slug']) return '';
18 18
 
19 19
 				if (false !== Validate::url($data['slug'])) return $data['slug'];
20 20
 
21
-				return (INSTALL_PATH . '/' . $data['slug']);
21
+				return (INSTALL_PATH.'/'.$data['slug']);
22 22
 			});
23 23
 		}
24 24
 	}
Please login to merge, or discard this patch.
www/engine/System/Classes/Modules/Entitizer/Utils/Collection.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
 
33 33
 			$selection = [];
34 34
 
35
-			foreach ($this->definition->paramsSecure() as $field) $selection[] = ('ent.' . $field);
35
+			foreach ($this->definition->paramsSecure() as $field) $selection[] = ('ent.'.$field);
36 36
 
37 37
 			# ------------------------
38 38
 
@@ -45,7 +45,7 @@  discard block
 block discarded – undo
45 45
 
46 46
 			$order_by = [];
47 47
 
48
-			foreach ($this->castOrderBy($data) as $field => $direction) $order_by[] = ('ent.' . $field . ' ' . $direction);
48
+			foreach ($this->castOrderBy($data) as $field => $direction) $order_by[] = ('ent.'.$field.' '.$direction);
49 49
 
50 50
 			# ------------------------
51 51
 
Please login to merge, or discard this patch.
Braces   +9 added lines, -3 removed lines patch added patch discarded remove patch
@@ -16,7 +16,9 @@  discard block
 block discarded – undo
16 16
 
17 17
 			foreach ($data as $field => $direction) {
18 18
 
19
-				if (false === ($param = $this->definition->param($field))) continue;
19
+				if (false === ($param = $this->definition->param($field))) {
20
+					continue;
21
+				}
20 22
 
21 23
 				$order_by[$param->name] = ((strtoupper($direction) !== 'DESC') ? 'ASC' : 'DESC');
22 24
 			}
@@ -32,7 +34,9 @@  discard block
 block discarded – undo
32 34
 
33 35
 			$selection = [];
34 36
 
35
-			foreach ($this->definition->paramsSecure() as $field) $selection[] = ('ent.' . $field);
37
+			foreach ($this->definition->paramsSecure() as $field) {
38
+				$selection[] = ('ent.' . $field);
39
+			}
36 40
 
37 41
 			# ------------------------
38 42
 
@@ -45,7 +49,9 @@  discard block
 block discarded – undo
45 49
 
46 50
 			$order_by = [];
47 51
 
48
-			foreach ($this->castOrderBy($data) as $field => $direction) $order_by[] = ('ent.' . $field . ' ' . $direction);
52
+			foreach ($this->castOrderBy($data) as $field => $direction) {
53
+				$order_by[] = ('ent.' . $field . ' ' . $direction);
54
+			}
49 55
 
50 56
 			# ------------------------
51 57
 
Please login to merge, or discard this patch.
www/engine/System/Classes/Modules/Entitizer/Utils/Treeview.php 3 patches
Indentation   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -12,7 +12,7 @@  discard block
 block discarded – undo
12 12
 
13 13
 			return (0 !== $parent_id) ? (("JOIN " . static::$table_relations . " rel ") .
14 14
 
15
-			       ("ON rel.ancestor = " . $parent_id . " AND rel.descendant = ent.id AND rel.depth >= 1")) :
15
+				   ("ON rel.ancestor = " . $parent_id . " AND rel.descendant = ent.id AND rel.depth >= 1")) :
16 16
 
17 17
 				   ("JOIN " . static::$table_relations . " rel ON rel.descendant = ent.id");
18 18
 		}
@@ -23,13 +23,13 @@  discard block
 block discarded – undo
23 23
 
24 24
 			return ("SELECT " . $this->getSelection() . ", COALESCE(par.ancestor, 0) as parent_id ") .
25 25
 
26
-			       ("FROM " . static::$table . " ent " . $this->getQueryJoin($parent_id) . " ") .
26
+				   ("FROM " . static::$table . " ent " . $this->getQueryJoin($parent_id) . " ") .
27 27
 
28
-			       ("LEFT JOIN " . static::$table_relations . " par ON par.descendant = ent.id AND par.depth = 1 ") .
28
+				   ("LEFT JOIN " . static::$table_relations . " par ON par.descendant = ent.id AND par.depth = 1 ") .
29 29
 
30
-			       (('' !== ($condition = $this->getCondition($config))) ? ("WHERE " . $condition . " ") : "") .
30
+				   (('' !== ($condition = $this->getCondition($config))) ? ("WHERE " . $condition . " ") : "") .
31 31
 
32
-			       ("GROUP BY ent.id ORDER BY MAX(rel.depth) ASC, " . $this->getOrderBy($order_by));
32
+				   ("GROUP BY ent.id ORDER BY MAX(rel.depth) ASC, " . $this->getOrderBy($order_by));
33 33
 		}
34 34
 
35 35
 		# Get count query
@@ -38,7 +38,7 @@  discard block
 block discarded – undo
38 38
 
39 39
 			return ("SELECT COUNT(DISTINCT ent.id) as count FROM " . static::$table . " ent ") .
40 40
 
41
-			       $this->getQueryJoin($parent_id);
41
+				   $this->getQueryJoin($parent_id);
42 42
 		}
43 43
 
44 44
 		# Get depth query
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
 
48 48
 			$query = ("SELECT COUNT(DISTINCT rel.depth) as depth FROM " . static::$table . " ent ") .
49 49
 
50
-			         $this->getQueryJoin($parent_id);
50
+					 $this->getQueryJoin($parent_id);
51 51
 
52 52
 			# ------------------------
53 53
 
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
 
129 129
 			$query = ("SELECT " . $this->getSelection() . " FROM " . static::$table . " ent ") .
130 130
 
131
-			         ("JOIN " . static::$table_relations . " rel ON rel.ancestor = ent.id ") .
131
+					 ("JOIN " . static::$table_relations . " rel ON rel.ancestor = ent.id ") .
132 132
 
133 133
 					 ("WHERE rel.descendant = " . $parent_id . " ORDER BY rel.depth DESC");
134 134
 
Please login to merge, or discard this patch.
Spacing   +12 added lines, -14 removed lines patch added patch discarded remove patch
@@ -10,33 +10,31 @@  discard block
 block discarded – undo
10 10
 
11 11
 		protected function getQueryJoin(int $parent_id) {
12 12
 
13
-			return (0 !== $parent_id) ? (("JOIN " . static::$table_relations . " rel ") .
13
+			return (0 !== $parent_id) ? (("JOIN ".static::$table_relations." rel ").
14 14
 
15
-			       ("ON rel.ancestor = " . $parent_id . " AND rel.descendant = ent.id AND rel.depth >= 1")) :
16
-
17
-				   ("JOIN " . static::$table_relations . " rel ON rel.descendant = ent.id");
15
+			       ("ON rel.ancestor = ".$parent_id." AND rel.descendant = ent.id AND rel.depth >= 1")) : ("JOIN ".static::$table_relations." rel ON rel.descendant = ent.id");
18 16
 		}
19 17
 
20 18
 		# Get select query
21 19
 
22 20
 		private function getSelectQuery(int $parent_id, array $config, array $order_by) {
23 21
 
24
-			return ("SELECT " . $this->getSelection() . ", COALESCE(par.ancestor, 0) as parent_id ") .
22
+			return ("SELECT ".$this->getSelection().", COALESCE(par.ancestor, 0) as parent_id ").
25 23
 
26
-			       ("FROM " . static::$table . " ent " . $this->getQueryJoin($parent_id) . " ") .
24
+			       ("FROM ".static::$table." ent ".$this->getQueryJoin($parent_id)." ").
27 25
 
28
-			       ("LEFT JOIN " . static::$table_relations . " par ON par.descendant = ent.id AND par.depth = 1 ") .
26
+			       ("LEFT JOIN ".static::$table_relations." par ON par.descendant = ent.id AND par.depth = 1 ").
29 27
 
30
-			       (('' !== ($condition = $this->getCondition($config))) ? ("WHERE " . $condition . " ") : "") .
28
+			       (('' !== ($condition = $this->getCondition($config))) ? ("WHERE ".$condition." ") : "").
31 29
 
32
-			       ("GROUP BY ent.id ORDER BY MAX(rel.depth) ASC, " . $this->getOrderBy($order_by));
30
+			       ("GROUP BY ent.id ORDER BY MAX(rel.depth) ASC, ".$this->getOrderBy($order_by));
33 31
 		}
34 32
 
35 33
 		# Get count query
36 34
 
37 35
 		private function getCountQuery(int $parent_id) {
38 36
 
39
-			return ("SELECT COUNT(DISTINCT ent.id) as count FROM " . static::$table . " ent ") .
37
+			return ("SELECT COUNT(DISTINCT ent.id) as count FROM ".static::$table." ent ").
40 38
 
41 39
 			       $this->getQueryJoin($parent_id);
42 40
 		}
@@ -45,7 +43,7 @@  discard block
 block discarded – undo
45 43
 
46 44
 		private function getDepthQuery(int $parent_id) {
47 45
 
48
-			$query = ("SELECT COUNT(DISTINCT rel.depth) as depth FROM " . static::$table . " ent ") .
46
+			$query = ("SELECT COUNT(DISTINCT rel.depth) as depth FROM ".static::$table." ent ").
49 47
 
50 48
 			         $this->getQueryJoin($parent_id);
51 49
 
@@ -126,11 +124,11 @@  discard block
 block discarded – undo
126 124
 
127 125
 			# Process query
128 126
 
129
-			$query = ("SELECT " . $this->getSelection() . " FROM " . static::$table . " ent ") .
127
+			$query = ("SELECT ".$this->getSelection()." FROM ".static::$table." ent ").
130 128
 
131
-			         ("JOIN " . static::$table_relations . " rel ON rel.ancestor = ent.id ") .
129
+			         ("JOIN ".static::$table_relations." rel ON rel.ancestor = ent.id ").
132 130
 
133
-					 ("WHERE rel.descendant = " . $parent_id . " ORDER BY rel.depth DESC");
131
+					 ("WHERE rel.descendant = ".$parent_id." ORDER BY rel.depth DESC");
134 132
 
135 133
 			# Select path
136 134
 
Please login to merge, or discard this patch.
Braces   +27 added lines, -9 removed lines patch added patch discarded remove patch
@@ -58,13 +58,17 @@  discard block
 block discarded – undo
58 58
 
59 59
 		public function subtree(int $parent_id = 0, array $config = [], array $order_by = []) {
60 60
 
61
-			if (!(static::$nesting && ($parent_id >= 0))) return false;
61
+			if (!(static::$nesting && ($parent_id >= 0))) {
62
+				return false;
63
+			}
62 64
 
63 65
 			# Select entities
64 66
 
65 67
 			$query = $this->getSelectQuery($parent_id, $config, $order_by);
66 68
 
67
-			if (!(DB::send($query) && DB::getLast()->status)) return false;
69
+			if (!(DB::send($query) && DB::getLast()->status)) {
70
+				return false;
71
+			}
68 72
 
69 73
 			# Process results
70 74
 
@@ -88,13 +92,17 @@  discard block
 block discarded – undo
88 92
 
89 93
 		public function subtreeCount(int $parent_id = 0) {
90 94
 
91
-			if (!(static::$nesting && ($parent_id >= 0))) return false;
95
+			if (!(static::$nesting && ($parent_id >= 0))) {
96
+				return false;
97
+			}
92 98
 
93 99
 			# Get count
94 100
 
95 101
 			$query = $this->getCountQuery($parent_id);
96 102
 
97
-			if (!(DB::send($query) && DB::getLast()->status)) return false;
103
+			if (!(DB::send($query) && DB::getLast()->status)) {
104
+				return false;
105
+			}
98 106
 
99 107
 			# ------------------------
100 108
 
@@ -105,13 +113,17 @@  discard block
 block discarded – undo
105 113
 
106 114
 		public function subtreeDepth(int $parent_id = 0) {
107 115
 
108
-			if (!(static::$nesting && ($parent_id >= 0))) return false;
116
+			if (!(static::$nesting && ($parent_id >= 0))) {
117
+				return false;
118
+			}
109 119
 
110 120
 			# Get depth
111 121
 
112 122
 			$query = $this->getDepthQuery($parent_id);
113 123
 
114
-			if (!(DB::send($query) && DB::getLast()->status)) return false;
124
+			if (!(DB::send($query) && DB::getLast()->status)) {
125
+				return false;
126
+			}
115 127
 
116 128
 			# ------------------------
117 129
 
@@ -122,7 +134,9 @@  discard block
 block discarded – undo
122 134
 
123 135
 		public function path(int $parent_id = 0) {
124 136
 
125
-			if (!(static::$nesting && ($parent_id >= 0))) return false;
137
+			if (!(static::$nesting && ($parent_id >= 0))) {
138
+				return false;
139
+			}
126 140
 
127 141
 			# Process query
128 142
 
@@ -134,13 +148,17 @@  discard block
 block discarded – undo
134 148
 
135 149
 			# Select path
136 150
 
137
-			if (!(DB::send($query) && DB::getLast()->status)) return false;
151
+			if (!(DB::send($query) && DB::getLast()->status)) {
152
+				return false;
153
+			}
138 154
 
139 155
 			# Process results
140 156
 
141 157
 			$path = [];
142 158
 
143
-			while (null !== ($data = DB::getLast()->getRow())) $path[] = Entitizer::dataset(static::$table, $data)->data();
159
+			while (null !== ($data = DB::getLast()->getRow())) {
160
+				$path[] = Entitizer::dataset(static::$table, $data)->data();
161
+			}
144 162
 
145 163
 			# ------------------------
146 164
 
Please login to merge, or discard this patch.
engine/System/Classes/Modules/Entitizer/Utils/Definition/Group/Foreigns.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -10,7 +10,9 @@
 block discarded – undo
10 10
 
11 11
 		public function add(string $name, string $table, string $field, string $delete = null, string $update = null) {
12 12
 
13
-			if ((false === $this->definition->param($name)) || isset($this->list[$name])) return;
13
+			if ((false === $this->definition->param($name)) || isset($this->list[$name])) {
14
+				return;
15
+			}
14 16
 
15 17
 			$this->list[$name] = new Definition\Item\Foreign($name, $table, $field, $delete, $update);
16 18
 		}
Please login to merge, or discard this patch.
www/engine/System/Classes/Modules/Entitizer/Utils/Factory.php 1 patch
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -12,9 +12,13 @@
 block discarded – undo
12 12
 
13 13
 		public static function get(string $table) {
14 14
 
15
-			if (!isset(static::$classes[$table])) throw new Exception\General(static::$error_message);
15
+			if (!isset(static::$classes[$table])) {
16
+				throw new Exception\General(static::$error_message);
17
+			}
16 18
 
17
-			if (!isset(static::$cache[$table])) static::$cache[$table] = new static::$classes[$table];
19
+			if (!isset(static::$cache[$table])) {
20
+				static::$cache[$table] = new static::$classes[$table];
21
+			}
18 22
 
19 23
 			# ------------------------
20 24
 
Please login to merge, or discard this patch.
www/engine/System/Classes/Modules/Entitizer/Entitizer.php 1 patch
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -25,9 +25,13 @@
 block discarded – undo
25 25
 
26 26
 		public static function get(string $table, int $id = 0) {
27 27
 
28
-			if (!isset(self::$classes[$table])) throw new Exception\General(self::$error_message);
28
+			if (!isset(self::$classes[$table])) {
29
+				throw new Exception\General(self::$error_message);
30
+			}
29 31
 
30
-			if (isset(self::$cache[$table][$id])) return self::$cache[$table][$id];
32
+			if (isset(self::$cache[$table][$id])) {
33
+				return self::$cache[$table][$id];
34
+			}
31 35
 
32 36
 			$entity = new self::$classes[$table]; $entity->init($id);
33 37
 
Please login to merge, or discard this patch.
engine/System/Classes/Modules/Entitizer/Utils/Definition/Group/Indexes.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -10,7 +10,9 @@
 block discarded – undo
10 10
 
11 11
 		public function add(string $name, string $table, string $field, string $delete = null, string $update = null) {
12 12
 
13
-			if ((false === $this->definition->param($name)) || isset($this->list[$name])) return;
13
+			if ((false === $this->definition->param($name)) || isset($this->list[$name])) {
14
+				return;
15
+			}
14 16
 
15 17
 			$this->list[$name] = new Definition\Item\Foreign($name, $table, $field, $delete, $update);
16 18
 		}
Please login to merge, or discard this patch.
www/engine/Framework/Includes/Constants.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -2,23 +2,23 @@
 block discarded – undo
2 2
 
3 3
 # Date formats
4 4
 
5
-define('DATE_FORMAT_STANDART',                      'd.m.Y');
6
-define('DATE_FORMAT_MYSQL',                         'Y-m-d');
7
-define('DATE_FORMAT_DATETIME',                      'd.m.Y, H:i');
8
-define('DATE_FORMAT_W3C',                           'Y-m-d\TH:i:sP');
5
+define('DATE_FORMAT_STANDART', 'd.m.Y');
6
+define('DATE_FORMAT_MYSQL', 'Y-m-d');
7
+define('DATE_FORMAT_DATETIME', 'd.m.Y, H:i');
8
+define('DATE_FORMAT_W3C', 'Y-m-d\TH:i:sP');
9 9
 
10 10
 # Form fields
11 11
 
12
-define('FORM_FIELD_HIDDEN',                         'hidden');
13
-define('FORM_FIELD_PASSWORD',                       'password');
14
-define('FORM_FIELD_CAPTCHA',                        'captcha');
15
-define('FORM_FIELD_TEXT',                           'text');
16
-define('FORM_FIELD_TEXTAREA',                       'textarea');
12
+define('FORM_FIELD_HIDDEN', 'hidden');
13
+define('FORM_FIELD_PASSWORD', 'password');
14
+define('FORM_FIELD_CAPTCHA', 'captcha');
15
+define('FORM_FIELD_TEXT', 'text');
16
+define('FORM_FIELD_TEXTAREA', 'textarea');
17 17
 
18 18
 # String pools
19 19
 
20
-define('STR_POOL_DEFAULT',                          'abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789');
21
-define('STR_POOL_LATIN',                            'abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ');
22
-define('STR_POOL_LATIN_LOWER',                      'abcdefghijklmnopqrstuvwxyz');
23
-define('STR_POOL_LATIN_UPPER',                      'ABCDEFGHIJKLMNOPQRSTUVWXYZ');
24
-define('STR_POOL_NUMERIC',                          '0123456789');
20
+define('STR_POOL_DEFAULT', 'abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789');
21
+define('STR_POOL_LATIN', 'abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ');
22
+define('STR_POOL_LATIN_LOWER', 'abcdefghijklmnopqrstuvwxyz');
23
+define('STR_POOL_LATIN_UPPER', 'ABCDEFGHIJKLMNOPQRSTUVWXYZ');
24
+define('STR_POOL_NUMERIC', '0123456789');
Please login to merge, or discard this patch.