Skip to content

Commit 177715e

Browse files
authored
Php stan fixes (#1021)
1 parent f92602d commit 177715e

File tree

4 files changed

+15
-15
lines changed

4 files changed

+15
-15
lines changed

src/Audit.php

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -183,7 +183,13 @@ private function castDatetimeUTC($model, $value)
183183
}
184184

185185
if (preg_match('/^(\d{4})-(\d{2})-(\d{2}) (\d{2}):(\d{2}):(\d{2})$/', $value)) {
186-
return Date::instance(Carbon::createFromFormat('Y-m-d H:i:s', $value, Date::now('UTC')->getTimezone()));
186+
$date = Carbon::createFromFormat('Y-m-d H:i:s', $value, Date::now('UTC')->getTimezone());
187+
188+
if (! $date) {
189+
return $value;
190+
}
191+
192+
return Date::instance($date);
187193
}
188194

189195
try {

src/Drivers/Database.php

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -24,21 +24,17 @@ public function prune(Auditable $model): bool
2424
if (($threshold = $model->getAuditThreshold()) > 0) {
2525
$auditClass = get_class($model->audits()->getModel());
2626
$auditModel = new $auditClass;
27+
$keyName = $auditModel->getKeyName();
2728

2829
return $model->audits()
2930
->leftJoinSub(
30-
$model->audits()->getQuery()
31-
->select($auditModel->getKeyName())->limit($threshold)->latest(),
31+
$model->audits()->getQuery()->select($keyName)->limit($threshold)->latest(),
3232
'audit_threshold',
33-
function ($join) use ($auditModel) {
34-
$join->on(
35-
$auditModel->gettable().'.'.$auditModel->getKeyName(),
36-
'=',
37-
'audit_threshold.'.$auditModel->getKeyName()
38-
);
39-
}
33+
fn ($join) => $join->on(
34+
$auditModel->getTable().".$keyName", '=', "audit_threshold.$keyName"
35+
)
4036
)
41-
->whereNull('audit_threshold.'.$auditModel->getKeyName())
37+
->whereNull("audit_threshold.$keyName")
4238
->delete() > 0;
4339
}
4440

src/Resolvers/UrlResolver.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ public static function resolve(Auditable $auditable): string
2525
public static function resolveCommandLine(): string
2626
{
2727
$command = Request::server('argv', null);
28-
if (is_array($command)) {
28+
if (is_array($command)) { // @phpstan-ignore function.impossibleType
2929
return implode(' ', $command);
3030
}
3131

tests/Functional/AuditingTest.php

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -276,9 +276,7 @@ public function test_it_will_remove_older_audits_above_the_threshold(): void
276276
]);
277277

278278
foreach (range(1, 20) as $count) {
279-
if ($count === 11) {
280-
sleep(1);
281-
}
279+
Carbon::setTestNow(now()->addSeconds($count));
282280

283281
$article->update([
284282
'title' => 'Title #'.$count,

0 commit comments

Comments
 (0)