|
1
|
|
|
<?php |
|
2
|
|
|
|
|
3
|
|
|
namespace CSlant\Blog\Api\Services; |
|
4
|
|
|
|
|
5
|
|
|
use Carbon\Carbon; |
|
6
|
|
|
use CSlant\Blog\Api\Models\VisitorLogs; |
|
7
|
|
|
use CSlant\Blog\Core\Models\Post; |
|
8
|
|
|
use Illuminate\Support\Facades\DB; |
|
9
|
|
|
|
|
10
|
|
|
class VisitorLogsService |
|
11
|
|
|
{ |
|
12
|
|
|
/** |
|
13
|
|
|
* Track a view for a post by ID. |
|
14
|
|
|
* |
|
15
|
|
|
* @param int $postId The post ID |
|
16
|
|
|
* @param null|string $ipAddress The viewer's IP address |
|
17
|
|
|
* @param null|string $userAgent The viewer's user agent |
|
18
|
|
|
* @return Post The updated post |
|
19
|
|
|
*/ |
|
20
|
|
|
public function trackPostView(int $postId, ?string $ipAddress, ?string $userAgent = null): Post |
|
21
|
|
|
{ |
|
22
|
|
|
$expirationMinutes = (int) config('blog-core.expiration_view_time', 60); |
|
|
|
|
|
|
23
|
|
|
$ipAddress = $ipAddress ?? ''; |
|
24
|
|
|
|
|
25
|
|
|
/** @var Post */ |
|
26
|
|
|
return DB::transaction(function () use ($postId, $ipAddress, $userAgent, $expirationMinutes): Post { |
|
27
|
|
|
$post = Post::findOrFail($postId); |
|
28
|
|
|
$entityType = get_class($post); |
|
29
|
|
|
$entityId = $post->getKey(); |
|
30
|
|
|
$now = Carbon::now(); |
|
31
|
|
|
|
|
32
|
|
|
/** @var null|VisitorLogs $existingView */ |
|
33
|
|
|
$existingView = VisitorLogs::query() |
|
34
|
|
|
->where('viewable_id', '=', $entityId) |
|
35
|
|
|
->where('viewable_type', '=', $entityType) |
|
36
|
|
|
->where('ip_address', '=', $ipAddress) |
|
37
|
|
|
->first(); |
|
38
|
|
|
|
|
39
|
|
|
if ($existingView === null) { |
|
40
|
|
|
VisitorLogs::create([ |
|
41
|
|
|
'viewable_id' => $entityId, |
|
42
|
|
|
'viewable_type' => $entityType, |
|
43
|
|
|
'ip_address' => $ipAddress, |
|
44
|
|
|
'user_agent' => $userAgent, |
|
45
|
|
|
'expired_at' => $now->copy()->addMinutes($expirationMinutes), |
|
46
|
|
|
]); |
|
47
|
|
|
|
|
48
|
|
|
$post->increment('views'); |
|
49
|
|
|
} else { |
|
50
|
|
|
if ($now->isAfter($existingView->expired_at)) { |
|
51
|
|
|
$existingView->update([ |
|
52
|
|
|
'user_agent' => $userAgent, |
|
53
|
|
|
'expired_at' => $now->copy()->addMinutes($expirationMinutes), |
|
54
|
|
|
]); |
|
55
|
|
|
|
|
56
|
|
|
$post->increment('views'); |
|
57
|
|
|
} |
|
58
|
|
|
} |
|
59
|
|
|
|
|
60
|
|
|
return $post->refresh(); |
|
61
|
|
|
}); |
|
62
|
|
|
} |
|
63
|
|
|
} |
|
64
|
|
|
|
cslant /
blog-api-package
This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.
If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. Please note the @ignore annotation hint above.