Total Complexity | 79 |
Total Lines | 543 |
Duplicated Lines | 0 % |
Changes | 0 |
Complex classes like User often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes.
Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.
While breaking up the class, it is a good idea to analyze how other classes use User, and based on these observations, apply Extract Interface, too.
1 | <?php |
||
149 | class User extends Authenticatable |
||
150 | { |
||
151 | use HasRoles, InviteTrait, Notifiable, SoftDeletes, UserVerification; |
||
152 | |||
153 | public const ERR_SIGNUP_BADUNAME = -1; |
||
154 | |||
155 | public const ERR_SIGNUP_BADPASS = -2; |
||
156 | |||
157 | public const ERR_SIGNUP_BADEMAIL = -3; |
||
158 | |||
159 | public const ERR_SIGNUP_UNAMEINUSE = -4; |
||
160 | |||
161 | public const ERR_SIGNUP_EMAILINUSE = -5; |
||
162 | |||
163 | public const ERR_SIGNUP_BADINVITECODE = -6; |
||
164 | |||
165 | public const SUCCESS = 1; |
||
166 | |||
167 | public const ROLE_USER = 1; |
||
168 | |||
169 | public const ROLE_ADMIN = 2; |
||
170 | |||
171 | /** |
||
172 | * Users SELECT queue type. |
||
173 | */ |
||
174 | public const QUEUE_NONE = 0; |
||
175 | |||
176 | public const QUEUE_SABNZBD = 1; |
||
177 | |||
178 | public const QUEUE_NZBGET = 2; |
||
179 | |||
180 | /** |
||
181 | * @var string |
||
182 | */ |
||
183 | |||
184 | /** |
||
185 | * @var bool |
||
186 | */ |
||
187 | protected $dateFormat = false; |
||
188 | |||
189 | /** |
||
190 | * @var array |
||
191 | */ |
||
192 | protected $hidden = ['remember_token', 'password']; |
||
193 | |||
194 | /** |
||
195 | * @var array |
||
196 | */ |
||
197 | protected $guarded = []; |
||
198 | |||
199 | protected function getDefaultGuardName(): string |
||
200 | { |
||
201 | return 'web'; |
||
202 | } |
||
203 | |||
204 | public function role(): BelongsTo |
||
205 | { |
||
206 | return $this->belongsTo(Role::class, 'roles_id'); |
||
207 | } |
||
208 | |||
209 | public function request(): HasMany |
||
210 | { |
||
211 | return $this->hasMany(UserRequest::class, 'users_id'); |
||
212 | } |
||
213 | |||
214 | public function download(): HasMany |
||
215 | { |
||
216 | return $this->hasMany(UserDownload::class, 'users_id'); |
||
217 | } |
||
218 | |||
219 | public function release(): HasMany |
||
220 | { |
||
221 | return $this->hasMany(UsersRelease::class, 'users_id'); |
||
222 | } |
||
223 | |||
224 | public function series(): HasMany |
||
225 | { |
||
226 | return $this->hasMany(UserSerie::class, 'users_id'); |
||
227 | } |
||
228 | |||
229 | public function invitation(): HasMany |
||
230 | { |
||
231 | return $this->hasMany(Invitation::class, 'users_id'); |
||
232 | } |
||
233 | |||
234 | public function failedRelease(): HasMany |
||
235 | { |
||
236 | return $this->hasMany(DnzbFailure::class, 'users_id'); |
||
237 | } |
||
238 | |||
239 | public function comment(): HasMany |
||
240 | { |
||
241 | return $this->hasMany(ReleaseComment::class, 'users_id'); |
||
242 | } |
||
243 | |||
244 | /** |
||
245 | * @throws \Exception |
||
246 | */ |
||
247 | public static function deleteUser($id): void |
||
248 | { |
||
249 | self::find($id)->delete(); |
||
250 | } |
||
251 | |||
252 | public static function getCount(?string $role = null, ?string $username = '', ?string $host = '', ?string $email = ''): int |
||
253 | { |
||
254 | $res = self::query()->where('email', '<>', '[email protected]'); |
||
255 | |||
256 | if (! empty($role)) { |
||
257 | $res->where('roles_id', $role); |
||
258 | } |
||
259 | |||
260 | if ($username !== '') { |
||
261 | $res->where('username', 'like', '%'.$username.'%'); |
||
262 | } |
||
263 | |||
264 | if ($host !== '') { |
||
265 | $res->where('host', 'like', '%'.$host.'%'); |
||
266 | } |
||
267 | |||
268 | if ($email !== '') { |
||
269 | $res->where('email', 'like', '%'.$email.'%'); |
||
270 | } |
||
271 | |||
272 | return $res->count(['id']); |
||
273 | } |
||
274 | |||
275 | public static function updateUser(int $id, string $userName, ?string $email, int $grabs, int $role, ?string $notes, int $invites, int $movieview, int $musicview, int $gameview, int $xxxview, int $consoleview, int $bookview, string $style = 'None'): int |
||
276 | { |
||
277 | $userName = trim($userName); |
||
278 | |||
279 | $rateLimit = Role::query()->where('id', $role)->first(); |
||
280 | |||
281 | $sql = [ |
||
282 | 'username' => $userName, |
||
283 | 'grabs' => $grabs, |
||
284 | 'roles_id' => $role, |
||
285 | 'notes' => substr($notes, 0, 255), |
||
286 | 'invites' => $invites, |
||
287 | 'movieview' => $movieview, |
||
288 | 'musicview' => $musicview, |
||
289 | 'gameview' => $gameview, |
||
290 | 'xxxview' => $xxxview, |
||
291 | 'consoleview' => $consoleview, |
||
292 | 'bookview' => $bookview, |
||
293 | 'style' => $style, |
||
294 | 'rate_limit' => $rateLimit ? $rateLimit['rate_limit'] : 60, |
||
295 | ]; |
||
296 | |||
297 | if (! empty($email)) { |
||
298 | $email = trim($email); |
||
299 | $sql += ['email' => $email]; |
||
300 | } |
||
301 | |||
302 | $user = self::find($id); |
||
303 | $user->update($sql); |
||
304 | $user->syncRoles([$rateLimit['name']]); |
||
305 | |||
306 | return self::SUCCESS; |
||
307 | } |
||
308 | |||
309 | /** |
||
310 | * @return User|Builder|Model|object|null |
||
311 | */ |
||
312 | public static function getByUsername(string $userName) |
||
313 | { |
||
314 | return self::whereUsername($userName)->first(); |
||
315 | } |
||
316 | |||
317 | /** |
||
318 | * @return Model|static |
||
319 | * |
||
320 | * @throws ModelNotFoundException |
||
321 | */ |
||
322 | public static function getByEmail(string $email) |
||
323 | { |
||
324 | return self::whereEmail($email)->first(); |
||
325 | } |
||
326 | |||
327 | public static function updateUserRole(int $uid, int|string $role): bool |
||
328 | { |
||
329 | if (is_int($role)) { |
||
330 | $roleQuery = Role::query()->where('id', $role)->first(); |
||
331 | } else { |
||
332 | $roleQuery = Role::query()->where('name', $role)->first(); |
||
333 | } |
||
334 | $roleName = $roleQuery->name; |
||
335 | |||
336 | $user = self::find($uid); |
||
337 | $user->syncRoles([$roleName]); |
||
338 | |||
339 | return self::find($uid)->update(['roles_id' => $roleQuery->id]); |
||
340 | } |
||
341 | |||
342 | public static function updateUserRoleChangeDate(int $uid, $date = '', int $addYear = 0): void |
||
343 | { |
||
344 | $user = self::find($uid); |
||
345 | $currRoleExp = $user->rolechangedate ?? now()->toDateTimeString(); |
||
346 | if (! empty($date)) { |
||
347 | $user->update(['rolechangedate' => $date]); |
||
348 | } |
||
349 | if (empty($date) && ! empty($addYear)) { |
||
350 | $user->update(['rolechangedate' => Carbon::createFromDate($currRoleExp)->addYears($addYear)]); |
||
351 | } |
||
352 | } |
||
353 | |||
354 | public static function updateExpiredRoles(): void |
||
355 | { |
||
356 | $now = CarbonImmutable::now(); |
||
357 | $period = [ |
||
358 | 'day' => $now->addDay(), |
||
359 | 'week' => $now->addWeek(), |
||
360 | 'month' => $now->addMonth(), |
||
361 | ]; |
||
362 | |||
363 | foreach ($period as $value) { |
||
364 | $users = self::query()->whereDate('rolechangedate', '=', $value)->get(); |
||
365 | $days = $now->diffInDays($value, true); |
||
366 | foreach ($users as $user) { |
||
367 | SendAccountWillExpireEmail::dispatch($user, $days)->onQueue('emails'); |
||
368 | } |
||
369 | } |
||
370 | foreach (self::query()->whereDate('rolechangedate', '<', $now)->get() as $expired) { |
||
371 | $expired->update(['roles_id' => self::ROLE_USER, 'rolechangedate' => null]); |
||
372 | $expired->syncRoles(['User']); |
||
373 | SendAccountExpiredEmail::dispatch($expired)->onQueue('emails'); |
||
374 | } |
||
375 | } |
||
376 | |||
377 | /** |
||
378 | * @throws \Throwable |
||
379 | */ |
||
380 | public static function getRange($start, $offset, $orderBy, ?string $userName = '', ?string $email = '', ?string $host = '', ?string $role = '', bool $apiRequests = false): Collection |
||
381 | { |
||
382 | if ($apiRequests) { |
||
383 | UserRequest::clearApiRequests(false); |
||
384 | $query = " |
||
385 | SELECT users.*, roles.name AS rolename, COUNT(user_requests.id) AS apirequests |
||
386 | FROM users |
||
387 | INNER JOIN roles ON roles.id = users.roles_id |
||
388 | LEFT JOIN user_requests ON user_requests.users_id = users.id |
||
389 | WHERE users.id != 0 %s %s %s %s |
||
390 | AND email != '[email protected]' |
||
391 | GROUP BY users.id |
||
392 | ORDER BY %s %s %s "; |
||
393 | } else { |
||
394 | $query = ' |
||
395 | SELECT users.*, roles.name AS rolename |
||
396 | FROM users |
||
397 | INNER JOIN roles ON roles.id = users.roles_id |
||
398 | WHERE 1=1 %s %s %s %s |
||
399 | ORDER BY %s %s %s'; |
||
400 | } |
||
401 | $order = self::getBrowseOrder($orderBy); |
||
402 | |||
403 | return self::fromQuery( |
||
404 | sprintf( |
||
405 | $query, |
||
406 | ! empty($userName) ? 'AND users.username '.'LIKE '.escapeString('%'.$userName.'%') : '', |
||
407 | ! empty($email) ? 'AND users.email '.'LIKE '.escapeString('%'.$email.'%') : '', |
||
408 | ! empty($host) ? 'AND users.host '.'LIKE '.escapeString('%'.$host.'%') : '', |
||
409 | (! empty($role) ? ('AND users.roles_id = '.$role) : ''), |
||
410 | $order[0], |
||
411 | $order[1], |
||
412 | ($start === false ? '' : ('LIMIT '.$offset.' OFFSET '.$start)) |
||
413 | ) |
||
414 | ); |
||
415 | } |
||
416 | |||
417 | /** |
||
418 | * Get sort types for sorting users on the web page user list. |
||
419 | * |
||
420 | * @return string[] |
||
421 | */ |
||
422 | public static function getBrowseOrder($orderBy): array |
||
423 | { |
||
424 | $order = (empty($orderBy) ? 'username_desc' : $orderBy); |
||
425 | $orderArr = explode('_', $order); |
||
426 | $orderField = match ($orderArr[0]) { |
||
427 | 'email' => 'email', |
||
428 | 'host' => 'host', |
||
429 | 'createdat' => 'created_at', |
||
430 | 'lastlogin' => 'lastlogin', |
||
431 | 'apiaccess' => 'apiaccess', |
||
432 | 'grabs' => 'grabs', |
||
433 | 'role' => 'rolename', |
||
434 | 'rolechangedate' => 'rolechangedate', |
||
435 | 'verification' => 'verified', |
||
436 | default => 'username', |
||
437 | }; |
||
438 | $orderSort = (isset($orderArr[1]) && preg_match('/^asc|desc$/i', $orderArr[1])) ? $orderArr[1] : 'desc'; |
||
439 | |||
440 | return [$orderField, $orderSort]; |
||
441 | } |
||
442 | |||
443 | /** |
||
444 | * Verify a password against a hash. |
||
445 | * |
||
446 | * Automatically update the hash if it needs to be. |
||
447 | * |
||
448 | * @param string $password Password to check against hash. |
||
449 | * @param bool|string $hash Hash to check against password. |
||
450 | * @param int $userID ID of the user. |
||
451 | */ |
||
452 | public static function checkPassword(string $password, bool|string $hash, int $userID = -1): bool |
||
453 | { |
||
454 | if (Hash::check($password, $hash) === false) { |
||
455 | return false; |
||
456 | } |
||
457 | |||
458 | // Update the hash if it needs to be. |
||
459 | if (is_numeric($userID) && $userID > 0 && Hash::needsRehash($hash)) { |
||
460 | $hash = self::hashPassword($password); |
||
461 | |||
462 | if ($hash !== false) { |
||
463 | self::find($userID)->update(['password' => $hash]); |
||
464 | } |
||
465 | } |
||
466 | |||
467 | return true; |
||
468 | } |
||
469 | |||
470 | public static function updateRssKey($uid): int |
||
471 | { |
||
472 | self::find($uid)->update(['api_token' => md5(Password::getRepository()->createNewToken())]); |
||
473 | |||
474 | return self::SUCCESS; |
||
475 | } |
||
476 | |||
477 | public static function updatePassResetGuid($id, $guid): int |
||
482 | } |
||
483 | |||
484 | public static function updatePassword(int $id, string $password): int |
||
485 | { |
||
486 | self::find($id)->update(['password' => self::hashPassword($password)]); |
||
487 | |||
488 | return self::SUCCESS; |
||
489 | } |
||
490 | |||
491 | public static function hashPassword($password): string |
||
492 | { |
||
493 | return Hash::make($password); |
||
494 | } |
||
495 | |||
496 | /** |
||
497 | * @return Model|static |
||
498 | * |
||
499 | * @throws ModelNotFoundException |
||
500 | */ |
||
501 | public static function getByPassResetGuid(string $guid) |
||
502 | { |
||
503 | return self::whereResetguid($guid)->first(); |
||
504 | } |
||
505 | |||
506 | public static function incrementGrabs(int $id, int $num = 1): void |
||
507 | { |
||
508 | self::find($id)->increment('grabs', $num); |
||
509 | } |
||
510 | |||
511 | /** |
||
512 | * @return Model|null|static |
||
513 | */ |
||
514 | public static function getByRssToken(string $rssToken) |
||
515 | { |
||
516 | return self::whereApiToken($rssToken)->first(); |
||
517 | } |
||
518 | |||
519 | public static function isValidUrl($url): bool |
||
520 | { |
||
521 | return (! preg_match('/^(http|https|ftp):\/\/([A-Z0-9][A-Z0-9_-]*(?:\.[A-Z0-9][A-Z0-9_-]*)+):?(\d+)?\/?/i', $url)) ? false : true; |
||
522 | } |
||
523 | |||
524 | /** |
||
525 | * @throws \Exception |
||
526 | */ |
||
527 | public static function generatePassword(int $length = 15): string |
||
528 | { |
||
529 | return Str::password($length); |
||
530 | } |
||
531 | |||
532 | /** |
||
533 | * @throws \Exception |
||
534 | */ |
||
535 | public static function signUp($userName, $password, $email, $host, $notes, int $invites = Invitation::DEFAULT_INVITES, string $inviteCode = '', bool $forceInviteMode = false, int $role = self::ROLE_USER, bool $validate = true): bool|int|string |
||
536 | { |
||
537 | $user = [ |
||
538 | 'username' => trim($userName), |
||
539 | 'password' => trim($password), |
||
540 | 'email' => trim($email), |
||
541 | ]; |
||
542 | |||
543 | if ($validate) { |
||
544 | $validator = Validator::make($user, [ |
||
545 | 'username' => ['required', 'string', 'min:5', 'max:255', 'unique:users'], |
||
546 | 'email' => ['required', 'string', 'email', 'max:255', 'unique:users', 'indisposable'], |
||
547 | 'password' => ['required', 'string', 'min:8', 'confirmed', 'regex:/^(?=.*?[A-Z])(?=.*?[a-z])(?=.*?[0-9])(?=.*?[#?!@$%^&*-]).{8,}$/'], |
||
548 | ]); |
||
549 | |||
550 | if ($validator->fails()) { |
||
551 | return implode('', Arr::collapse($validator->errors()->toArray())); |
||
552 | } |
||
553 | } |
||
554 | |||
555 | // Make sure this is the last check, as if a further validation check failed, the invite would still have been used up. |
||
556 | $invitedBy = 0; |
||
557 | if (! $forceInviteMode && (int) Settings::settingValue('registerstatus') === Settings::REGISTER_STATUS_INVITE) { |
||
558 | if ($inviteCode === '') { |
||
559 | return self::ERR_SIGNUP_BADINVITECODE; |
||
560 | } |
||
561 | |||
562 | $invitedBy = self::checkAndUseInvite($inviteCode); |
||
563 | if ($invitedBy < 0) { |
||
564 | return self::ERR_SIGNUP_BADINVITECODE; |
||
565 | } |
||
566 | } |
||
567 | |||
568 | return self::add($user['username'], $user['password'], $user['email'], $role, $notes, $host, $invites, $invitedBy); |
||
569 | } |
||
570 | |||
571 | /** |
||
572 | * If a invite is used, decrement the person who invited's invite count. |
||
573 | */ |
||
574 | public static function checkAndUseInvite(string $inviteCode): int |
||
575 | { |
||
576 | $invite = Invitation::getInvite($inviteCode); |
||
577 | if (! $invite) { |
||
578 | return -1; |
||
579 | } |
||
580 | |||
581 | self::query()->where('id', $invite['users_id'])->decrement('invites'); |
||
582 | Invitation::deleteInvite($inviteCode); |
||
583 | |||
584 | return $invite['users_id']; |
||
585 | } |
||
586 | |||
587 | /** |
||
588 | * @return false|int|mixed |
||
589 | */ |
||
590 | public static function add(string $userName, string $password, string $email, int $role, ?string $notes = '', string $host = '', int $invites = Invitation::DEFAULT_INVITES, int $invitedBy = 0) |
||
591 | { |
||
592 | $password = self::hashPassword($password); |
||
593 | if (! $password) { |
||
594 | return false; |
||
595 | } |
||
596 | |||
597 | $storeips = config('nntmux:settings.store_user_ip') === true ? $host : ''; |
||
598 | |||
599 | $user = self::create( |
||
600 | [ |
||
601 | 'username' => $userName, |
||
602 | 'password' => $password, |
||
603 | 'email' => $email, |
||
604 | 'host' => $storeips, |
||
605 | 'roles_id' => $role, |
||
606 | 'invites' => $invites, |
||
607 | 'invitedby' => (int) $invitedBy === 0 ? null : $invitedBy, |
||
608 | 'notes' => $notes, |
||
609 | ] |
||
610 | ); |
||
611 | |||
612 | return $user->id; |
||
613 | } |
||
614 | |||
615 | /** |
||
616 | * Get the list of categories the user has excluded. |
||
617 | * |
||
618 | * @param int $userID ID of the user. |
||
619 | * |
||
620 | * @throws \Exception |
||
621 | */ |
||
622 | public static function getCategoryExclusionById(int $userID): array |
||
623 | { |
||
624 | $ret = []; |
||
625 | |||
626 | $user = self::find($userID); |
||
627 | |||
628 | $userAllowed = $user->getDirectPermissions()->pluck('name')->toArray(); |
||
629 | $roleAllowed = $user->getAllPermissions()->pluck('name')->toArray(); |
||
630 | |||
631 | $allowed = array_intersect($roleAllowed, $userAllowed); |
||
632 | |||
633 | $cats = ['view console', 'view movies', 'view audio', 'view tv', 'view pc', 'view adult', 'view books', 'view other']; |
||
634 | |||
635 | if (! empty($allowed)) { |
||
636 | foreach ($cats as $cat) { |
||
637 | if (! \in_array($cat, $allowed, false)) { |
||
638 | $ret[] = match ($cat) { |
||
639 | 'view console' => 1000, |
||
640 | 'view movies' => 2000, |
||
641 | 'view audio' => 3000, |
||
642 | 'view pc' => 4000, |
||
643 | 'view tv' => 5000, |
||
644 | 'view adult' => 6000, |
||
645 | 'view books' => 7000, |
||
646 | 'view other' => 1, |
||
647 | }; |
||
648 | } |
||
649 | } |
||
650 | } |
||
651 | |||
652 | return Category::query()->whereIn('root_categories_id', $ret)->pluck('id')->toArray(); |
||
653 | } |
||
654 | |||
655 | /** |
||
656 | * @throws \Exception |
||
657 | */ |
||
658 | public static function getCategoryExclusionForApi(Request $request): array |
||
664 | } |
||
665 | |||
666 | /** |
||
667 | * @throws \Exception |
||
668 | */ |
||
669 | public static function sendInvite($serverUrl, $uid, $emailTo): string |
||
670 | { |
||
671 | $user = self::find($uid); |
||
672 | $token = Invite::invite($emailTo, $user->id); |
||
673 | $url = $serverUrl.'/register?invitecode='.$token; |
||
674 | |||
675 | Invitation::addInvite($uid, $token); |
||
676 | SendInviteEmail::dispatch($emailTo, $user, $url)->onQueue('emails'); |
||
677 | |||
678 | return $url; |
||
679 | } |
||
680 | |||
681 | /** |
||
682 | * Deletes users that have not verified their accounts for 3 or more days. |
||
683 | */ |
||
684 | public static function deleteUnVerified(): void |
||
685 | { |
||
686 | static::whereVerified(0)->where('created_at', '<', now()->subDays(3))->delete(); |
||
687 | } |
||
688 | |||
689 | public function passwordSecurity(): HasOne |
||
692 | } |
||
693 | } |
||
694 |
Let?s assume that you have a directory layout like this:
and let?s assume the following content of
Bar.php
:If both files
OtherDir/Foo.php
andSomeDir/Foo.php
are loaded in the same runtime, you will see a PHP error such as the following:PHP Fatal error: Cannot use SomeDir\Foo as Foo because the name is already in use in OtherDir/Foo.php
However, as
OtherDir/Foo.php
does not necessarily have to be loaded and the error is only triggered if it is loaded beforeOtherDir/Bar.php
, this problem might go unnoticed for a while. In order to prevent this error from surfacing, you must import the namespace with a different alias: