View GitHub Repository cms-Inji

Schedule Inspection
Subscribe
Completed
Push — master ( 16c7c4...785292 )
by Alexey
05:40
created
Status
Category
Toggle Patches
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']) && $user->activation) {
154 
-                Tools::redirect('/', 'Этот аккаунт ещё не активирован. <br />Если вы не получали письмо с ссылкой для активации, нажмите на - <a href = "/users/resendActivation/' . $user->id . '"><b>повторно выслать ссылку активации</b></a>');
154 
+                Tools::redirect('/', 'Этот аккаунт ещё не активирован. <br />Если вы не получали письмо с ссылкой для активации, нажмите на - <a href = "/users/resendActivation/'.$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.