CSlant\Blog\Api\Services\VisitorLogsService

Complexity

Total Complexity

3

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Importance

Changes	5
Bugs	1	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	trackPostView()	0	38	3

<?php

namespace CSlant\Blog\Api\Services;

use Carbon\Carbon;
use CSlant\Blog\Api\Models\VisitorLog;
use CSlant\Blog\Core\Models\Post;

class VisitorLogsService
{
    /**
     * Track a view for a post by ID.
     *
     * @param int $postId The post ID
     * @param null|string $ipAddress The viewer's IP address
     * @param null|string $userAgent The viewer's user agent
     * @return Post The updated post
     */
    public function trackPostView(int $postId, ?string $ipAddress, ?string $userAgent = null): Post
    {
        $expirationMinutes = (int) config('blog-core.expiration_view_time', 60);

The call to config() has too many arguments starting with 60.

        $ipAddress = $ipAddress ?? '';

        $post = Post::findOrFail($postId);
        $entityType = get_class($post);
        $entityId = $post->getKey();
        $now = Carbon::now();

        /** @var null|VisitorLog $existingView */
        $existingView = VisitorLog::query()
            ->where('viewable_id', '=', $entityId)
            ->where('viewable_type', '=', $entityType)
            ->where('ip_address', '=', $ipAddress)
            ->first();

        $expiredAt = $now->copy()->addMinutes($expirationMinutes);

        $visitorData = [
            'user_agent' => $userAgent,
            'expired_at' => $expiredAt,
        ];

        if (!$existingView instanceof VisitorLog) {
            VisitorLog::create([
                'viewable_id' => $entityId,
                'viewable_type' => $entityType,
                'ip_address' => $ipAddress,
            ] + $visitorData);

            $post->increment('views');
        } elseif ($now->isAfter($existingView->expired_at)) {
            $existingView->update($visitorData);
            $post->increment('views');
        }

        return $post->refresh();
    }
}