injitools /
cms-Inji
| Conditions | 41 |
| Paths | > 20000 |
| Total Lines | 144 |
| Code Lines | 107 |
| Lines | 0 |
| Ratio | 0 % |
| Changes | 0 | ||
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
| 1 | <?php |
||
| 280 | public function registration($data, $autorization = false, $msg = true) { |
||
| 281 | |||
| 282 | if (empty($data['user_mail'])) { |
||
| 283 | return $this->msgOrErr('Вы не ввели E-mail', $msg); |
||
| 284 | } |
||
| 285 | if (!empty($this->config['csrf']) && !App::$cur->server->checkCsrfForm($data)) { |
||
| 286 | return $this->msgOrErr('Ошибка безопасности, повторите запрос еще раз', $msg); |
||
| 287 | } |
||
| 288 | $data['user_mail'] = trim($data['user_mail']); |
||
| 289 | if (!filter_var($data['user_mail'], FILTER_VALIDATE_EMAIL)) { |
||
| 290 | return $this->msgOrErr(\I18n\Text::module('Users', 'Вы ввели не корректный E-mail'), $msg); |
||
| 291 | |||
| 292 | } |
||
| 293 | |||
| 294 | $user = $this->get($data['user_mail'], 'mail'); |
||
| 295 | if ($user) { |
||
| 296 | return $this->msgOrErr(\I18n\Text::module('Users', 'Введенный вами E-mail зарегистрирован в нашей системе, войдите или введите другой E-mail'), $msg); |
||
| 297 | } |
||
| 298 | if (empty($data['user_login'])) { |
||
| 299 | $data['user_login'] = $data['user_mail']; |
||
| 300 | } |
||
| 301 | $data['user_login'] = trim($data['user_login']); |
||
| 302 | $user = $this->get($data['user_login'], 'login'); |
||
| 303 | if ($user) { |
||
| 304 | return $this->msgOrErr('Введенный вами логин зарегистрирован в нашей системе, войдите или введите другой логин', $msg); |
||
| 305 | } |
||
| 306 | if (empty($data['first_name'])) { |
||
| 307 | $data['first_name'] = ''; |
||
| 308 | } |
||
| 309 | if (empty($data['last_name'])) { |
||
| 310 | $data['last_name'] = ''; |
||
| 311 | } |
||
| 312 | if (!empty($data['user_name'])) { |
||
| 313 | $data['first_name'] = $data['user_name']; |
||
| 314 | } |
||
| 315 | if (empty($data['user_city'])) { |
||
| 316 | $data['user_city'] = ''; |
||
| 317 | } |
||
| 318 | if (empty($data['user_birthday'])) { |
||
| 319 | $data['user_birthday'] = ''; |
||
| 320 | } |
||
| 321 | if (empty($data['user_phone'])) { |
||
| 322 | $data['user_phone'] = ''; |
||
| 323 | } |
||
| 324 | |||
| 325 | if (empty($data['user_avatar_id']) || !is_numeric($data['user_avatar_id'])) { |
||
| 326 | $data['user_avatar_id'] = 0; |
||
| 327 | } else { |
||
| 328 | $avatar = \Users\User\Avatar::get($data['user_avatar_id']); |
||
| 329 | if ($avatar) { |
||
| 330 | $data['user_avatar_id'] = $avatar->id; |
||
| 331 | } else { |
||
| 332 | $data['user_avatar_id'] = 0; |
||
| 333 | } |
||
| 334 | } |
||
| 335 | $invite_code = (!empty($data['invite_code']) ? $data['invite_code'] : (!empty($_POST['invite_code']) ? $_POST['invite_code'] : ((!empty($_COOKIE['invite_code']) ? $_COOKIE['invite_code'] : ((!empty($_GET['invite_code']) ? $_GET['invite_code'] : '')))))); |
||
| 336 | if (!empty($invite_code)) { |
||
| 337 | $invite = Users\User\Invite::get($invite_code, 'code'); |
||
| 338 | if (!$invite) { |
||
| 339 | return $this->msgOrErr('Такой код приглашения не найден', $msg); |
||
| 340 | } |
||
| 341 | if ($invite->limit && !($invite->limit - $invite->count)) { |
||
| 342 | return $this->msgOrErr('Лимит приглашений для данного кода исчерпан', $msg); |
||
| 343 | } |
||
| 344 | $data['parent_id'] = $invite->user_id; |
||
| 345 | $inviter = $data['parent_id']; |
||
| 346 | $invite->count++; |
||
| 347 | $invite->save(); |
||
| 348 | } |
||
| 349 | if (empty($data['parent_id']) && !empty($this->config['defaultPartner'])) { |
||
| 350 | $data['parent_id'] = $this->config['defaultPartner']; |
||
| 351 | } |
||
| 352 | if (!empty($this->config['passwordManualSetup']) && !empty($data['user_pass'])) { |
||
| 353 | if (empty($data['user_pass'][0])) { |
||
| 354 | return $this->msgOrErr('Введите пароль', $msg); |
||
| 355 | } |
||
| 356 | if (empty($data['user_pass'][1])) { |
||
| 357 | return $this->msgOrErr('Повторите ввод пароля', $msg); |
||
| 358 | } |
||
| 359 | if ($data['user_pass'][0] != $data['user_pass'][1]) { |
||
| 360 | return $this->msgOrErr('Введенные пароли несовпадают', $msg); |
||
| 361 | } |
||
| 362 | $pass = $data['user_pass'][0]; |
||
| 363 | } else { |
||
| 364 | $pass = Tools::randomString(10); |
||
| 365 | } |
||
| 366 | $user = new Users\User([ |
||
| 367 | 'pass' => $this->hashpass($pass), |
||
| 368 | 'mail' => $data['user_mail'], |
||
| 369 | 'login' => htmlspecialchars($data['user_login']), |
||
| 370 | 'role_id' => 2, |
||
| 371 | 'group_id' => 2, |
||
| 372 | 'user_avatar_id' => $data['user_avatar_id'], |
||
| 373 | 'parent_id' => !empty($data['parent_id']) ? $data['parent_id'] : 0 |
||
| 374 | ]); |
||
| 375 | if (!empty($this->config['needActivation'])) { |
||
| 376 | $user->activation = Tools::randomString(); |
||
| 377 | } |
||
| 378 | $user->save(); |
||
| 379 | if (!$user->id) { |
||
| 380 | return $this->msgOrErr('Не удалось зарегистрировать', $msg); |
||
| 381 | } |
||
| 382 | $info = new \Users\User\Info([ |
||
| 383 | 'user_id' => $user->id, |
||
| 384 | 'first_name' => htmlspecialchars($data['first_name']), |
||
| 385 | 'last_name' => htmlspecialchars($data['last_name']), |
||
| 386 | 'city' => htmlspecialchars($data['user_city']), |
||
| 387 | 'bday' => htmlspecialchars($data['user_birthday']), |
||
| 388 | 'phone' => htmlspecialchars($data['user_phone']), |
||
| 389 | 'photo_file_id' => !empty($_FILES['user_photo']['tmp_name']) ? $this->files->upload($_FILES['user_photo']) : 0 |
||
| 390 | ]); |
||
| 391 | $info->save(); |
||
| 392 | if (isset($inviter)) { |
||
| 393 | $this->AddUserActivity($inviter, 2, "У вас зарегистрировался новый партнер, {$info->first_name} {$info->last_name} (id: {$user->id}, email: {$user->mail})"); |
||
| 394 | } |
||
| 395 | if ($autorization) { |
||
| 396 | $this->autorization($data['user_mail'], $pass, 'mail'); |
||
| 397 | } |
||
| 398 | if (!empty($this->config['needActivation'])) { |
||
| 399 | $from = 'noreply@' . INJI_DOMAIN_NAME; |
||
| 400 | $to = $data['user_mail']; |
||
| 401 | $subject = 'Регистрация на сайте ' . idn_to_utf8(INJI_DOMAIN_NAME); |
||
| 402 | $text = 'Вы были зарегистрированы на сайте ' . idn_to_utf8(INJI_DOMAIN_NAME) . '<br />для входа используйте ваш почтовый ящик в качестве логина и пароль: ' . $pass; |
||
| 403 | $text .= '<br />'; |
||
| 404 | $text .= '<br />'; |
||
| 405 | $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>'; |
||
| 406 | Tools::sendMail($from, $to, $subject, $text); |
||
| 407 | if ($msg) { |
||
| 408 | \Inji\Msg::add('Вы были зарегистрированы. На указанный почтовый ящик был выслан ваш пароль и ссылка для активации', 'success'); |
||
| 409 | } |
||
| 410 | } else { |
||
| 411 | $from = 'noreply@' . INJI_DOMAIN_NAME; |
||
| 412 | $to = $data['user_mail']; |
||
| 413 | $subject = \I18n\Text::module('Users', 'Регистрация на сайте ${sitename}', ['sitename' => idn_to_utf8(INJI_DOMAIN_NAME)]); |
||
| 414 | $text = \I18n\Text::module('Users', 'sucregmsg', [ |
||
| 415 | 'sitename' => idn_to_utf8(INJI_DOMAIN_NAME), |
||
| 416 | 'pass' => $pass |
||
| 417 | ]); |
||
| 418 | Tools::sendMail($from, $to, $subject, $text); |
||
| 419 | if ($msg) { |
||
| 420 | \Inji\Msg::add(\I18n\Text::module('Users', 'Вы были зарегистрированы. На указанный почтовый ящик был выслан ваш пароль'), 'success'); |
||
| 421 | } |
||
| 422 | } |
||
| 423 | return $user->id; |
||
| 424 | } |
||
| 470 | } |
Loading history...