Completed
Push — master ( fbe197...e7e29c )
by Alexey
05:05
created
system/modules/Users/Users.php 1 patch
Spacing   +34 added lines, -34 removed lines patch added patch discarded remove patch
@@ -24,8 +24,8 @@  discard block
 block discarded – undo
24 24
         if (isset($_GET['logout'])) {
25 25
             return $this->logOut();
26 26
         }
27
-        if (filter_input(INPUT_COOKIE, $this->cookiePrefix . '_user_session_hash') && filter_input(INPUT_COOKIE, $this->cookiePrefix . '_user_id')) {
28
-            return $this->cuntinueSession(filter_input(INPUT_COOKIE, $this->cookiePrefix . '_user_session_hash'), filter_input(INPUT_COOKIE, $this->cookiePrefix . '_user_id'));
27
+        if (filter_input(INPUT_COOKIE, $this->cookiePrefix.'_user_session_hash') && filter_input(INPUT_COOKIE, $this->cookiePrefix.'_user_id')) {
28
+            return $this->cuntinueSession(filter_input(INPUT_COOKIE, $this->cookiePrefix.'_user_session_hash'), filter_input(INPUT_COOKIE, $this->cookiePrefix.'_user_id'));
29 29
         }
30 30
         if (isset($_POST['autorization']) && trim(filter_input(INPUT_POST, 'user_login')) && trim(filter_input(INPUT_POST, 'user_pass'))) {
31 31
             unset($_POST['autorization']);
@@ -41,18 +41,18 @@  discard block
 block discarded – undo
41 41
 
42 42
     public function logOut($redirect = true)
43 43
     {
44
-        if (!empty($_COOKIE[$this->cookiePrefix . "_user_session_hash"]) && !empty($_COOKIE[$this->cookiePrefix . "_user_id"])) {
44
+        if (!empty($_COOKIE[$this->cookiePrefix."_user_session_hash"]) && !empty($_COOKIE[$this->cookiePrefix."_user_id"])) {
45 45
             $session = Users\Session::get([
46
-                        ['user_id', $_COOKIE[$this->cookiePrefix . "_user_id"]],
47
-                        ['hash', $_COOKIE[$this->cookiePrefix . "_user_session_hash"]]
46
+                        ['user_id', $_COOKIE[$this->cookiePrefix."_user_id"]],
47
+                        ['hash', $_COOKIE[$this->cookiePrefix."_user_session_hash"]]
48 48
             ]);
49 49
             if ($session) {
50 50
                 $session->delete();
51 51
             }
52 52
         }
53 53
         if (!headers_sent()) {
54
-            setcookie($this->cookiePrefix . "_user_session_hash", '', 0, "/");
55
-            setcookie($this->cookiePrefix . "_user_id", '', 0, "/");
54
+            setcookie($this->cookiePrefix."_user_session_hash", '', 0, "/");
55
+            setcookie($this->cookiePrefix."_user_id", '', 0, "/");
56 56
         }
57 57
         if ($redirect) {
58 58
             if (!empty($this->config['logoutUrl'][$this->app->type])) {
@@ -70,25 +70,25 @@  discard block
 block discarded – undo
70 70
         ]);
71 71
         if ($session && $session->user && $session->user->blocked) {
72 72
             if (!headers_sent()) {
73
-                setcookie($this->cookiePrefix . "_user_session_hash", '', 0, "/");
74
-                setcookie($this->cookiePrefix . "_user_id", '', 0, "/");
73
+                setcookie($this->cookiePrefix."_user_session_hash", '', 0, "/");
74
+                setcookie($this->cookiePrefix."_user_id", '', 0, "/");
75 75
             }
76 76
             Msg::add('Ваш аккаунт заблокирован', 'info');
77 77
             return;
78 78
         }
79 79
         if ($session && $session->user && !$session->user->blocked) {
80 80
             if (!headers_sent()) {
81
-                setcookie($this->cookiePrefix . "_user_session_hash", $session->hash, time() + 360000, "/");
82
-                setcookie($this->cookiePrefix . "_user_id", $session->user_id, time() + 360000, "/");
81
+                setcookie($this->cookiePrefix."_user_session_hash", $session->hash, time() + 360000, "/");
82
+                setcookie($this->cookiePrefix."_user_id", $session->user_id, time() + 360000, "/");
83 83
             }
84 84
             if (!empty($this->config['needActivation']) && $session->user->activation) {
85 85
                 if (!headers_sent()) {
86
-                    setcookie($this->cookiePrefix . "_user_session_hash", '', 0, "/");
87
-                    setcookie($this->cookiePrefix . "_user_id", '', 0, "/");
86
+                    setcookie($this->cookiePrefix."_user_session_hash", '', 0, "/");
87
+                    setcookie($this->cookiePrefix."_user_id", '', 0, "/");
88 88
                 }
89
-                Tools::redirect('/', 'Этот аккаунт ещё не активирован. <br />Если вы не получали письмо с ссылкой для активации, нажмите на - <a href = "/users/resendActivation/' . $session->user->id . '"><b>повторно выслать ссылку активации</b></a>');
89
+                Tools::redirect('/', 'Этот аккаунт ещё не активирован. <br />Если вы не получали письмо с ссылкой для активации, нажмите на - <a href = "/users/resendActivation/'.$session->user->id.'"><b>повторно выслать ссылку активации</b></a>');
90 90
             } elseif ($session->user->activation) {
91
-                Msg::add('Этот аккаунт ещё не активирован, не все функции могут быть доступны. <br />Если вы не получали письмо с ссылкой для активации, нажмите на - <a href = "/users/resendActivation/' . $session->user->id . '"><b>повторно выслать ссылку активации</b></a>');
91
+                Msg::add('Этот аккаунт ещё не активирован, не все функции могут быть доступны. <br />Если вы не получали письмо с ссылкой для активации, нажмите на - <a href = "/users/resendActivation/'.$session->user->id.'"><b>повторно выслать ссылку активации</b></a>');
92 92
             }
93 93
             if (!$session->user->mail && !empty($this->config['noMailNotify'])) {
94 94
                 Msg::add($this->config['noMailNotify']);
@@ -98,8 +98,8 @@  discard block
 block discarded – undo
98 98
             Users\User::$cur->save();
99 99
         } else {
100 100
             if (!headers_sent()) {
101
-                setcookie($this->cookiePrefix . "_user_session_hash", '', 0, "/");
102
-                setcookie($this->cookiePrefix . "_user_id", '', 0, "/");
101
+                setcookie($this->cookiePrefix."_user_session_hash", '', 0, "/");
102
+                setcookie($this->cookiePrefix."_user_id", '', 0, "/");
103 103
             }
104 104
             Msg::add('Ваша сессия устарела или более недействительна, вам необходимо пройти <a href = "/users/login">авторизацию</a> заново', 'info');
105 105
         }
@@ -109,7 +109,7 @@  discard block
 block discarded – undo
109 109
     {
110 110
         $user = $this->get($user_mail, 'mail');
111 111
         if (!$user) {
112
-            Msg::add('Пользователь ' . $user_mail . ' не найден, проверьте првильность ввода e-mail или зарегистрируйтесь', 'danger');
112
+            Msg::add('Пользователь '.$user_mail.' не найден, проверьте првильность ввода e-mail или зарегистрируйтесь', 'danger');
113 113
             return false;
114 114
         }
115 115
         $passre = Users\Passre::get([['user_id', $user->id], ['status', 1]]);
@@ -117,10 +117,10 @@  discard block
 block discarded – undo
117 117
             $passre->status = 2;
118 118
             $passre->save();
119 119
         }
120
-        $hash = $user->id . '_' . Tools::randomString(50);
120
+        $hash = $user->id.'_'.Tools::randomString(50);
121 121
         $passre = new Users\Passre(['user_id' => $user->id, 'status' => 1, 'hash' => $hash]);
122 122
         $passre->save();
123
-        Tools::sendMail('noreply@' . INJI_DOMAIN_NAME, $user_mail, 'Восстановление пароля на сайте ' . idn_to_utf8(INJI_DOMAIN_NAME), 'Было запрошено восстановление пароля на сайте ' . idn_to_utf8(INJI_DOMAIN_NAME) . '<br />для продолжения восстановления пароля перейдите по ссылке: <a href = "http://' . idn_to_utf8(INJI_DOMAIN_NAME) . '/?passrecont=1&hash=' . $hash . '">' . idn_to_utf8(INJI_DOMAIN_NAME) . '/?passrecont=1&hash=' . $hash . '</a>');
123
+        Tools::sendMail('noreply@'.INJI_DOMAIN_NAME, $user_mail, 'Восстановление пароля на сайте '.idn_to_utf8(INJI_DOMAIN_NAME), 'Было запрошено восстановление пароля на сайте '.idn_to_utf8(INJI_DOMAIN_NAME).'<br />для продолжения восстановления пароля перейдите по ссылке: <a href = "http://'.idn_to_utf8(INJI_DOMAIN_NAME).'/?passrecont=1&hash='.$hash.'">'.idn_to_utf8(INJI_DOMAIN_NAME).'/?passrecont=1&hash='.$hash.'</a>');
124 124
         Tools::redirect('/', 'На указанный почтовый ящик была выслана инструкция по восстановлению пароля', 'success');
125 125
     }
126 126
 
@@ -138,7 +138,7 @@  discard block
 block discarded – undo
138 138
             $user->pass = $this->hashpass($pass);
139 139
             $user->save();
140 140
             $this->autorization($user->mail, $user->pass, 'mail');
141
-            Tools::sendMail('noreply@' . INJI_DOMAIN_NAME, $user->mail, 'Новый пароль на сайте ' . idn_to_utf8(INJI_DOMAIN_NAME), 'Было запрошено восстановление пароля на сайте ' . idn_to_utf8(INJI_DOMAIN_NAME) . '<br />Ваш новый пароль: ' . $pass);
141
+            Tools::sendMail('noreply@'.INJI_DOMAIN_NAME, $user->mail, 'Новый пароль на сайте '.idn_to_utf8(INJI_DOMAIN_NAME), 'Было запрошено восстановление пароля на сайте '.idn_to_utf8(INJI_DOMAIN_NAME).'<br />Ваш новый пароль: '.$pass);
142 142
             Tools::redirect('/', 'На указанный почтовый ящик был выслан новый пароль', 'success');
143 143
         }
144 144
     }
@@ -151,9 +151,9 @@  discard block
 block discarded – undo
151 151
         $user = $this->get($login, $ltype);
152 152
         if ($user && $this->verifypass($pass, $user->pass) && !$user->blocked) {
153 153
             if (!empty($this->config['needActivation']) && $session->user->activation) {
154
-                Tools::redirect('/', 'Этот аккаунт ещё не активирован. <br />Если вы не получали письмо с ссылкой для активации, нажмите на - <a href = "/users/resendActivation/' . $session->user->id . '"><b>повторно выслать ссылку активации</b></a>');
154
+                Tools::redirect('/', 'Этот аккаунт ещё не активирован. <br />Если вы не получали письмо с ссылкой для активации, нажмите на - <a href = "/users/resendActivation/'.$session->user->id.'"><b>повторно выслать ссылку активации</b></a>');
155 155
             } elseif ($user->activation) {
156
-                Msg::add('Этот аккаунт ещё не активирован, не все функции могут быть доступны. <br />Если вы не получали письмо с ссылкой для активации, нажмите на - <a href = "/users/resendActivation/' . $user->id . '"><b>повторно выслать ссылку активации</b></a>');
156
+                Msg::add('Этот аккаунт ещё не активирован, не все функции могут быть доступны. <br />Если вы не получали письмо с ссылкой для активации, нажмите на - <a href = "/users/resendActivation/'.$user->id.'"><b>повторно выслать ссылку активации</b></a>');
157 157
             }
158 158
             if (!$user->mail && !empty($this->config['noMailNotify'])) {
159 159
                 Msg::add($this->config['noMailNotify']);
@@ -172,7 +172,7 @@  discard block
 block discarded – undo
172 172
             if ($user && $user->blocked) {
173 173
                 Msg::add('Вы заблокированы', 'danger');
174 174
             } elseif ($user) {
175
-                Msg::add('Вы ошиблись при наборе пароля или логина, попробуйте ещё раз или воспользуйтесь <a href = "?passre=1&user_mail=' . $user->mail . '">Восстановлением пароля</a>', 'danger');
175
+                Msg::add('Вы ошиблись при наборе пароля или логина, попробуйте ещё раз или воспользуйтесь <a href = "?passre=1&user_mail='.$user->mail.'">Восстановлением пароля</a>', 'danger');
176 176
             } else {
177 177
                 Msg::add('Данный почтовый ящик не зарегистрирован в системе', 'danger');
178 178
             }
@@ -195,8 +195,8 @@  discard block
 block discarded – undo
195 195
         $session->save();
196 196
 
197 197
         if (!headers_sent()) {
198
-            setcookie($this->cookiePrefix . "_user_session_hash", $session->hash, time() + 360000, "/");
199
-            setcookie($this->cookiePrefix . "_user_id", $session->user_id, time() + 360000, "/");
198
+            setcookie($this->cookiePrefix."_user_session_hash", $session->hash, time() + 360000, "/");
199
+            setcookie($this->cookiePrefix."_user_id", $session->user_id, time() + 360000, "/");
200 200
         } else {
201 201
             Msg::add('Не удалось провести авторизацию. Попробуйте позже', 'info');
202 202
         }
@@ -318,20 +318,20 @@  discard block
 block discarded – undo
318 318
             $this->autorization($user_mail, $pass, 'mail');
319 319
         }
320 320
         if (!empty($this->config['needActivation'])) {
321
-            $from = 'noreply@' . INJI_DOMAIN_NAME;
321
+            $from = 'noreply@'.INJI_DOMAIN_NAME;
322 322
             $to = $user_mail;
323
-            $subject = 'Регистрация на сайте ' . idn_to_utf8(INJI_DOMAIN_NAME);
324
-            $text = 'Вы были зарегистрированы на сайте ' . idn_to_utf8(INJI_DOMAIN_NAME) . '<br />для входа используйте ваш почтовый ящик в качестве логина и пароль: ' . $pass;
325
-            $text .='<br />';
323
+            $subject = 'Регистрация на сайте '.idn_to_utf8(INJI_DOMAIN_NAME);
324
+            $text = 'Вы были зарегистрированы на сайте '.idn_to_utf8(INJI_DOMAIN_NAME).'<br />для входа используйте ваш почтовый ящик в качестве логина и пароль: '.$pass;
326 325
             $text .= '<br />';
327
-            $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>';
326
+            $text .= '<br />';
327
+            $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>';
328 328
             Tools::sendMail($from, $to, $subject, $text);
329 329
             Msg::add('Вы были зарегистрированы. На указанный почтовый ящик был выслан ваш пароль и ссылка для активации', 'success');
330 330
         } else {
331
-            $from = 'noreply@' . INJI_DOMAIN_NAME;
331
+            $from = 'noreply@'.INJI_DOMAIN_NAME;
332 332
             $to = $user_mail;
333
-            $subject = 'Регистрация на сайте ' . idn_to_utf8(INJI_DOMAIN_NAME);
334
-            $text = 'Вы были зарегистрированы на сайте ' . idn_to_utf8(INJI_DOMAIN_NAME) . '<br />для входа используйте ваш почтовый ящик в качестве логина и пароль: ' . $pass;
333
+            $subject = 'Регистрация на сайте '.idn_to_utf8(INJI_DOMAIN_NAME);
334
+            $text = 'Вы были зарегистрированы на сайте '.idn_to_utf8(INJI_DOMAIN_NAME).'<br />для входа используйте ваш почтовый ящик в качестве логина и пароль: '.$pass;
335 335
             Tools::sendMail($from, $to, $subject, $text);
336 336
             Msg::add('Вы были зарегистрированы. На указанный почтовый ящик был выслан ваш пароль', 'success');
337 337
         }
Please login to merge, or discard this patch.
system/modules/Db/objects/Mysql/Query.php 1 patch
Braces   +45 added lines, -34 removed lines patch added patch discarded remove patch
@@ -126,12 +126,14 @@  discard block
 block discarded – undo
126 126
         if (!is_array($order)) {
127 127
             $this->order[] = "{$order} {$type}";
128 128
         } else {
129
-            foreach ($order as $item)
130
-                if (!is_array($item)) {
129
+            foreach ($order as $item) {
130
+                            if (!is_array($item)) {
131 131
                     call_user_func_array(array($this, 'order'), $order);
132
+            }
132 133
                     break;
133
-                } else
134
-                    $this->order($item);
134
+                } else {
135
+                                    $this->order($item);
136
+                }
135 137
         }
136 138
     }
137 139
 
@@ -140,8 +142,9 @@  discard block
 block discarded – undo
140 142
         $start = intval($start);
141 143
         $len = intval($len);
142 144
         $this->limit = "LIMIT {$start}";
143
-        if ($len !== 0)
144
-            $this->limit .= ",{$len}";
145
+        if ($len !== 0) {
146
+                    $this->limit .= ",{$len}";
147
+        }
145 148
     }
146 149
 
147 150
     public function buildJoin($table, $where = false, $type = 'LEFT', $alias = '')
@@ -154,10 +157,12 @@  discard block
 block discarded – undo
154 157
             }
155 158
         } else {
156 159
             $join .= " {$type} JOIN {$this->curInstance->table_prefix}{$table}";
157
-            if ($alias)
158
-                $join .= " AS `{$alias}`";
159
-            if ($where)
160
-                $join .= " ON {$where}";
160
+            if ($alias) {
161
+                            $join .= " AS `{$alias}`";
162
+            }
163
+            if ($where) {
164
+                            $join .= " ON {$where}";
165
+            }
161 166
         }
162 167
         return $join;
163 168
     }
@@ -177,13 +182,15 @@  discard block
 block discarded – undo
177 182
                 $operation = '=';
178 183
             }
179 184
 
180
-            if ($concatenation === false)
181
-                $concatenation = 'AND';
182
-            elseif ($concatenation === true)
183
-                $concatenation = '';
185
+            if ($concatenation === false) {
186
+                            $concatenation = 'AND';
187
+            } elseif ($concatenation === true) {
188
+                            $concatenation = '';
189
+            }
184 190
 
185
-            if ($this->whereString == NULL)
186
-                $this->whereString = ' WHERE ';
191
+            if ($this->whereString == NULL) {
192
+                            $this->whereString = ' WHERE ';
193
+            }
187 194
 
188 195
             if (stristr($operation, 'IN') || stristr($operation, 'NOT IN')) {
189 196
                 if (is_array($value)) {
@@ -209,12 +216,12 @@  discard block
 block discarded – undo
209 216
                 $value = "?";
210 217
             }
211 218
 
212
-            if (substr($this->whereString, -1, 1) == '(' || substr($this->whereString, -2, 2) == 'E ')
213
-                $this->whereString .= " {$where} {$operation} {$value} ";
214
-            else
215
-                $this->whereString .= "{$concatenation} {$where} {$operation} {$value} ";
216
-        }
217
-        else {
219
+            if (substr($this->whereString, -1, 1) == '(' || substr($this->whereString, -2, 2) == 'E ') {
220
+                            $this->whereString .= " {$where} {$operation} {$value} ";
221
+            } else {
222
+                            $this->whereString .= "{$concatenation} {$where} {$operation} {$value} ";
223
+            }
224
+        } else {
218 225
             $i = -1;
219 226
             while (isset($where[++$i])) {
220 227
                 $item = $where[$i];
@@ -229,28 +236,32 @@  discard block
 block discarded – undo
229 236
                         $this->whereString .= "{$concatenation} ";
230 237
                     }
231 238
 
232
-                    if ($this->whereString != NULL)
233
-                        $this->whereString .= '(';
234
-                    else
235
-                        $this->whereString = 'WHERE (';
239
+                    if ($this->whereString != NULL) {
240
+                                            $this->whereString .= '(';
241
+                    } else {
242
+                                            $this->whereString = 'WHERE (';
243
+                    }
236 244
                 }
237 245
 
238 246
                 if (!is_array($item)) {
239 247
                     call_user_func_array(array($this, 'buildWhere'), $where);
240 248
                     break;
241 249
                 } else {
242
-                    if ($this->whereString != NULL && substr($this->whereString, -1, 1) != '(')
243
-                        if (!isset($item[3]))
250
+                    if ($this->whereString != NULL && substr($this->whereString, -1, 1) != '(') {
251
+                                            if (!isset($item[3]))
244 252
                             $concatenation = 'AND';
245
-                        else
246
-                            $concatenation = $item[3];
247
-                    elseif (substr($this->whereString, -1, 1) != '(')
248
-                        $this->whereString = 'WHERE ';
253
+                    } else {
254
+                                                    $concatenation = $item[3];
255
+                        }
256
+                    elseif (substr($this->whereString, -1, 1) != '(') {
257
+                                            $this->whereString = 'WHERE ';
258
+                    }
249 259
 
250 260
                     $this->buildWhere($item);
251 261
                 }
252
-                if (!isset($where[$i + 1]) && isset($where[$i - 1]))
253
-                    $this->whereString .= ') ';
262
+                if (!isset($where[$i + 1]) && isset($where[$i - 1])) {
263
+                                    $this->whereString .= ') ';
264
+                }
254 265
             }
255 266
         }
256 267
     }
Please login to merge, or discard this patch.