Completed
Branch master (ebb499)
by Alexey
04:15
created
system/modules/Exchange1c/objects/Mode/Init.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -31,7 +31,8 @@
 block discarded – undo
31 31
             } catch (\PDOException $exc) {
32 32
                 if ($exc->getCode() == '42S02') {
33 33
                     \Exchange1c\Exchange\File::createTable();
34
-                } elseif ($exc->getCode() == '42S22') {
34
+                }
35
+                elseif ($exc->getCode() == '42S22') {
35 36
                     $cols = \Exchange1c\Exchange\File::cols();
36 37
                     foreach (\Exchange1c\Exchange\File::$cols as $colName => $params) {
37 38
                         if (!isset($cols[\Exchange1c\Exchange\File::colPrefix() . $colName])) {
Please login to merge, or discard this patch.
system/modules/Callbacks/Callbacks.php 1 patch
Braces   +6 added lines, -3 removed lines patch added patch discarded remove patch
@@ -18,19 +18,22 @@
 block discarded – undo
18 18
             if (empty($callbacksData['text'])) {
19 19
                 $error = true;
20 20
                 Msg::add('Вы не написали текст отзыва');
21
-            } else {
21
+            }
22
+            else {
22 23
                 $callback->text = nl2br(htmlspecialchars($callbacksData['text']));
23 24
             }
24 25
             if (empty($callbacksData['name'])) {
25 26
                 $error = true;
26 27
                 Msg::add('Вы не указали свое имя');
27
-            } else {
28
+            }
29
+            else {
28 30
                 $callback->name = htmlspecialchars($callbacksData['name']);
29 31
             }
30 32
             if (empty($callbacksData['phone'])) {
31 33
                 $error = true;
32 34
                 Msg::add('Вы не указали свой номер телефона');
33
-            } else {
35
+            }
36
+            else {
34 37
                 $callback->phone = htmlspecialchars($callbacksData['phone']);
35 38
             }
36 39
             $files = filter_var($_FILES['Callbacks'], FILTER_REQUIRE_ARRAY);
Please login to merge, or discard this patch.
system/modules/Users/appAdminControllers/content/edit.php 1 patch
Braces   +6 added lines, -4 removed lines patch added patch discarded remove patch
@@ -42,10 +42,12 @@
 block discarded – undo
42 42
                             <select class="form-control" name = 'user_role_id'>
43 43
                               <?php
44 44
                               foreach ($roles as $role) {
45
-                                  if ($role['role_id'] == $user->user_role_id)
46
-                                      $selected = 'selected = "selected"';
47
-                                  else
48
-                                      $selected = '';
45
+                                  if ($role['role_id'] == $user->user_role_id) {
46
+                                                                        $selected = 'selected = "selected"';
47
+                                  }
48
+                                  else {
49
+                                                                        $selected = '';
50
+                                  }
49 51
                                   echo "<option {$selected} value = '{$role['role_id']}'>{$role['role_name']}</option>";
50 52
                               }
51 53
                               ?>
Please login to merge, or discard this patch.
system/modules/Users/Users.php 1 patch
Braces   +33 added lines, -19 removed lines patch added patch discarded remove patch
@@ -98,7 +98,8 @@  discard block
 block discarded – undo
98 98
                     setcookie($this->cookiePrefix . "_user_id", '', 0, "/");
99 99
                 }
100 100
                 Tools::redirect('/', 'Этот аккаунт ещё не активирован. <br />Если вы не получали письмо с ссылкой для активации, нажмите на - <a href = "/users/resendActivation/' . $session->user->id . '"><b>повторно выслать ссылку активации</b></a>');
101
-            } elseif ($session->user->activation) {
101
+            }
102
+            elseif ($session->user->activation) {
102 103
                 Msg::add('Этот аккаунт ещё не активирован, не все функции могут быть доступны. <br />Если вы не получали письмо с ссылкой для активации, нажмите на - <a href = "/users/resendActivation/' . $session->user->id . '"><b>повторно выслать ссылку активации</b></a>');
103 104
             }
104 105
             if (!$session->user->mail && !empty($this->config['noMailNotify'])) {
@@ -107,7 +108,8 @@  discard block
 block discarded – undo
107 108
             Users\User::$cur = $session->user;
108 109
             Users\User::$cur->date_last_active = 'CURRENT_TIMESTAMP';
109 110
             Users\User::$cur->save();
110
-        } else {
111
+        }
112
+        else {
111 113
             if (!headers_sent()) {
112 114
                 setcookie($this->cookiePrefix . "_user_session_hash", '', 0, "/");
113 115
                 setcookie($this->cookiePrefix . "_user_id", '', 0, "/");
@@ -160,7 +162,8 @@  discard block
 block discarded – undo
160 162
         if ($user && $this->verifypass($pass, $user->pass) && !$user->blocked) {
161 163
             if (!empty($this->config['needActivation']) && $user->activation) {
162 164
                 Tools::redirect('/', 'Этот аккаунт ещё не активирован. <br />Если вы не получали письмо с ссылкой для активации, нажмите на - <a href = "/users/resendActivation/' . $user->id . '"><b>повторно выслать ссылку активации</b></a>');
163
-            } elseif ($user->activation) {
165
+            }
166
+            elseif ($user->activation) {
164 167
                 Msg::add('Этот аккаунт ещё не активирован, не все функции могут быть доступны. <br />Если вы не получали письмо с ссылкой для активации, нажмите на - <a href = "/users/resendActivation/' . $user->id . '"><b>повторно выслать ссылку активации</b></a>');
165 168
             }
166 169
             if (!$user->mail && !empty($this->config['noMailNotify'])) {
@@ -179,9 +182,11 @@  discard block
 block discarded – undo
179 182
         if (!$noMsg) {
180 183
             if ($user && $user->blocked) {
181 184
                 Msg::add('Вы заблокированы', 'danger');
182
-            } elseif ($user) {
185
+            }
186
+            elseif ($user) {
183 187
                 Msg::add('Вы ошиблись при наборе пароля или логина, попробуйте ещё раз или воспользуйтесь <a href = "?passre=1&user_mail=' . $user->mail . '">Восстановлением пароля</a>', 'danger');
184
-            } else {
188
+            }
189
+            else {
185 190
                 Msg::add('Данный почтовый ящик не зарегистрирован в системе', 'danger');
186 191
             }
187 192
         }
@@ -204,7 +209,8 @@  discard block
 block discarded – undo
204 209
         if (!headers_sent()) {
205 210
             setcookie($this->cookiePrefix . "_user_session_hash", $session->hash, time() + 360000, "/");
206 211
             setcookie($this->cookiePrefix . "_user_id", $session->user_id, time() + 360000, "/");
207
-        } else {
212
+        }
213
+        else {
208 214
             Msg::add('Не удалось провести авторизацию. Попробуйте позже', 'info');
209 215
         }
210 216
     }
@@ -217,17 +223,22 @@  discard block
 block discarded – undo
217 223
      * @return boolean|\User\User
218 224
      */
219 225
     public function get($idn, $ltype = 'id') {
220
-        if (!$idn)
221
-            return false;
226
+        if (!$idn) {
227
+                    return false;
228
+        }
222 229
 
223
-        if (is_numeric($idn) && $ltype != 'login')
224
-            $user = Users\User::get($idn, 'id');
225
-        elseif ($ltype == 'login')
226
-            $user = Users\User::get($idn, 'login');
227
-        else
228
-            $user = Users\User::get($idn, 'mail');
229
-        if (!$user)
230
-            return [];
230
+        if (is_numeric($idn) && $ltype != 'login') {
231
+                    $user = Users\User::get($idn, 'id');
232
+        }
233
+        elseif ($ltype == 'login') {
234
+                    $user = Users\User::get($idn, 'login');
235
+        }
236
+        else {
237
+                    $user = Users\User::get($idn, 'mail');
238
+        }
239
+        if (!$user) {
240
+                    return [];
241
+        }
231 242
 
232 243
         return $user;
233 244
     }
@@ -309,7 +320,8 @@  discard block
 block discarded – undo
309 320
                 return false;
310 321
             }
311 322
             $pass = $data['user_pass'][0];
312
-        } else {
323
+        }
324
+        else {
313 325
             $pass = Tools::randomString(10);
314 326
         }
315 327
 
@@ -355,7 +367,8 @@  discard block
 block discarded – undo
355 367
             $text .= 'Для активации вашего аккаунта перейдите по ссылке <a href = "http://' . INJI_DOMAIN_NAME . '/users/activation/' . $user->id . '/' . $user->activation . '">http://' . idn_to_utf8(INJI_DOMAIN_NAME) . '/users/activation/' . $user->id . '/' . $user->activation . '</a>';
356 368
             Tools::sendMail($from, $to, $subject, $text);
357 369
             Msg::add('Вы были зарегистрированы. На указанный почтовый ящик был выслан ваш пароль и ссылка для активации', 'success');
358
-        } else {
370
+        }
371
+        else {
359 372
             $from = 'noreply@' . INJI_DOMAIN_NAME;
360 373
             $to = $data['user_mail'];
361 374
             $subject = 'Регистрация на сайте ' . idn_to_utf8(INJI_DOMAIN_NAME);
@@ -387,7 +400,8 @@  discard block
 block discarded – undo
387 400
             if (!$userIds && $levels) {
388 401
                 $levels[$i] = [];
389 402
                 continue;
390
-            } elseif (!$userIds && !$levels) {
403
+            }
404
+            elseif (!$userIds && !$levels) {
391 405
                 break;
392 406
             }
393 407
             $usersLevel = \Users\User::getList(['where' => [['parent_id', $userIds, 'IN']]]);
Please login to merge, or discard this patch.
system/modules/Users/appControllers/UsersController.php 1 patch
Braces   +10 added lines, -5 removed lines patch added patch discarded remove patch
@@ -22,7 +22,8 @@  discard block
 block discarded – undo
22 22
             $this->view->setTitle($sections[$activeSection]['name'] . ' - Личный кабинет');
23 23
             $bread[] = ['text' => 'Личный кабинет', 'href' => '/users/cabinet'];
24 24
             $bread[] = ['text' => $sections[$activeSection]['name']];
25
-        } else {
25
+        }
26
+        else {
26 27
             $this->view->setTitle('Личный кабинет');
27 28
             $bread[] = ['text' => 'Личный кабинет'];
28 29
         }
@@ -57,7 +58,8 @@  discard block
 block discarded – undo
57 58
                         Msg::add('Вы не прошли проверку на робота', 'danger');
58 59
                         $error = true;
59 60
                     }
60
-                } else {
61
+                }
62
+                else {
61 63
                     Msg::add('Произошла ошибка, попробуйте ещё раз');
62 64
                     $error = true;
63 65
                 }
@@ -93,11 +95,13 @@  discard block
 block discarded – undo
93 95
             $user_mail = trim($_POST['mail']);
94 96
             if (!filter_var($user_mail, FILTER_VALIDATE_EMAIL)) {
95 97
                 Msg::add('Вы ввели не корректный E-mail', 'danger');
96
-            } else {
98
+            }
99
+            else {
97 100
                 $user = Users\User::get($user_mail, 'mail');
98 101
                 if ($user && $user->id != Users\User::$cur->id) {
99 102
                     Msg::add('Данный E-mail уже привязан к другому аккаунту', 'danger');
100
-                } else {
103
+                }
104
+                else {
101 105
                     Users\User::$cur->mail = $user_mail;
102 106
                     if (!empty($this->module->config['needActivation'])) {
103 107
                         Users\User::$cur->activation = Tools::randomString();
@@ -107,7 +111,8 @@  discard block
 block discarded – undo
107 111
                         $text = 'Для активации вашего аккаунта перейдите по ссылке <a href = "http://' . INJI_DOMAIN_NAME . '/users/activation/' . Users\User::$cur->id . '/' . Users\User::$cur->activation . '">http://' . idn_to_utf8(INJI_DOMAIN_NAME) . '/users/activation/' . Users\User::$cur->id . '/' . Users\User::$cur->activation . '</a>';
108 112
                         Tools::sendMail($from, $to, $subject, $text);
109 113
                         Msg::add('На указанный почтовый ящик была выслана ваша ссылка для подтверждения E-Mail', 'success');
110
-                    } else {
114
+                    }
115
+                    else {
111 116
                         Msg::add('Вы успешно привязали E-Mail к своему аккаунту', 'success');
112 117
                     }
113 118
                     Users\User::$cur->save();
Please login to merge, or discard this patch.
system/modules/Access/Access.php 1 patch
Braces   +10 added lines, -6 removed lines patch added patch discarded remove patch
@@ -20,8 +20,9 @@  discard block
 block discarded – undo
20 20
         if (!$app) {
21 21
             $app = $this->app->type;
22 22
         }
23
-        if (!empty($this->config['access']['accessTree'][$app]['deniedUrl']))
24
-            return $this->config['access']['accessTree'][$app]['deniedUrl'];
23
+        if (!empty($this->config['access']['accessTree'][$app]['deniedUrl'])) {
24
+                    return $this->config['access']['accessTree'][$app]['deniedUrl'];
25
+        }
25 26
 
26 27
         return '/';
27 28
     }
@@ -46,8 +47,9 @@  discard block
 block discarded – undo
46 47
             return true;
47 48
         }
48 49
 
49
-        if ((!$user->group_id && !empty($access)) || ($user->group_id && !empty($access) && !in_array($user->group_id, $access)))
50
-            return false;
50
+        if ((!$user->group_id && !empty($access)) || ($user->group_id && !empty($access) && !in_array($user->group_id, $access))) {
51
+                    return false;
52
+        }
51 53
 
52 54
         return true;
53 55
     }
@@ -66,9 +68,11 @@  discard block
 block discarded – undo
66 68
     public function pathWalker($array, $path) {
67 69
         if ($path && isset($array[$path[0]])) {
68 70
             return $this->pathWalker($array[$path[0]], array_slice($path, 1));
69
-        } elseif (!$path) {
71
+        }
72
+        elseif (!$path) {
70 73
             return $array;
71
-        } else {
74
+        }
75
+        else {
72 76
             return NULL;
73 77
         }
74 78
     }
Please login to merge, or discard this patch.
system/modules/Ui/objects/ActiveForm/Input/DynamicList.php 1 patch
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -46,7 +46,8 @@  discard block
 block discarded – undo
46 46
                             foreach ($relModels as $model) {
47 47
                                 if (empty($rels[$model->{$relation['model']::index()}])) {
48 48
                                     $model->delete();
49
-                                } else {
49
+                                }
50
+                                else {
50 51
                                     unset($rels[$model->{$relation['model']::index()}]);
51 52
                                 }
52 53
                             }
@@ -78,7 +79,8 @@  discard block
 block discarded – undo
78 79
                             foreach ($relModels as $model) {
79 80
                                 if (empty($rels[$model->pk()])) {
80 81
                                     $model->delete();
81
-                                } else {
82
+                                }
83
+                                else {
82 84
                                     $model->setParams($rels[$model->pk()]);
83 85
                                     $model->save();
84 86
                                 }
Please login to merge, or discard this patch.
system/modules/Ui/objects/ActiveForm/Input/DynamicType.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -43,7 +43,8 @@
 block discarded – undo
43 43
                         }
44 44
 
45 45
                         $inputOptions['values'] = $item->{$relationName}(['forSelect' => true]);
46
-                    } else {
46
+                    }
47
+                    else {
47 48
                         $inputOptions['values'] = $this->activeForm->model->{$type['relation']}(['forSelect' => true]);
48 49
                     }
49 50
                 }
Please login to merge, or discard this patch.
system/modules/Ui/objects/ActiveForm/Input.php 1 patch
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -45,7 +45,8 @@  discard block
 block discarded – undo
45 45
         $colName = empty($this->colParams['col']) ? $this->colName : $this->colParams['col'];
46 46
         if (isset($request[$this->colName])) {
47 47
             $this->activeForm->model->{$colName} = $request[$this->colName];
48
-        } else {
48
+        }
49
+        else {
49 50
             $this->activeForm->model->{$colName} = 0;
50 51
             $this->activeForm->model->{$colName} = '';
51 52
         }
@@ -70,7 +71,8 @@  discard block
 block discarded – undo
70 71
             $value = '';
71 72
             if (!empty($preset['value'])) {
72 73
                 $value = $preset['value'];
73
-            } elseif (!empty($preset['userCol'])) {
74
+            }
75
+            elseif (!empty($preset['userCol'])) {
74 76
                 if (strpos($preset['userCol'], ':')) {
75 77
                     $rel = substr($preset['userCol'], 0, strpos($preset['userCol'], ':'));
76 78
                     $param = substr($preset['userCol'], strpos($preset['userCol'], ':') + 1);
Please login to merge, or discard this patch.