fix(scrobble.php): refactor isTrackAlreadyScrobbled method to avoid triggering warnings
This commit is contained in:
parent
bcb89ceb95
commit
7b2584367c
3 changed files with 8 additions and 3 deletions
|
@ -89,7 +89,12 @@ class NavidromeScrobbleHandler extends ApiHandler
|
||||||
|
|
||||||
private function isTrackAlreadyScrobbled(array $track): bool
|
private function isTrackAlreadyScrobbled(array $track): bool
|
||||||
{
|
{
|
||||||
|
if (empty($track["playDate"])) return false;
|
||||||
|
|
||||||
$playDate = strtotime($track["playDate"]);
|
$playDate = strtotime($track["playDate"]);
|
||||||
|
|
||||||
|
if ($playDate === false) return false;
|
||||||
|
|
||||||
$existingListen = $this->fetchFromApi("listens", "listened_at=eq.{$playDate}&limit=1");
|
$existingListen = $this->fetchFromApi("listens", "listened_at=eq.{$playDate}&limit=1");
|
||||||
|
|
||||||
return !empty($existingListen);
|
return !empty($existingListen);
|
||||||
|
|
4
package-lock.json
generated
4
package-lock.json
generated
|
@ -1,12 +1,12 @@
|
||||||
{
|
{
|
||||||
"name": "coryd.dev",
|
"name": "coryd.dev",
|
||||||
"version": "8.3.2",
|
"version": "8.3.3",
|
||||||
"lockfileVersion": 3,
|
"lockfileVersion": 3,
|
||||||
"requires": true,
|
"requires": true,
|
||||||
"packages": {
|
"packages": {
|
||||||
"": {
|
"": {
|
||||||
"name": "coryd.dev",
|
"name": "coryd.dev",
|
||||||
"version": "8.3.2",
|
"version": "8.3.3",
|
||||||
"license": "MIT",
|
"license": "MIT",
|
||||||
"dependencies": {
|
"dependencies": {
|
||||||
"minisearch": "^7.1.2",
|
"minisearch": "^7.1.2",
|
||||||
|
|
|
@ -1,6 +1,6 @@
|
||||||
{
|
{
|
||||||
"name": "coryd.dev",
|
"name": "coryd.dev",
|
||||||
"version": "8.3.2",
|
"version": "8.3.3",
|
||||||
"description": "The source for my personal site. Built using 11ty (and other tools).",
|
"description": "The source for my personal site. Built using 11ty (and other tools).",
|
||||||
"type": "module",
|
"type": "module",
|
||||||
"engines": {
|
"engines": {
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue