[bug]Fatal error: Declaration of Detection\Cache\CacheItem::expiresAt(?DateTimeInterface $expiration): Theme must be compatible with Psr\Cache\CacheItemInterface::expiresAt($expiration)

[BackuPs]

Describe the bug
A clear and concise description of what the bug is.
Include steps to reproduce the behavior.
A clear and concise description of what you expected to happen.

User-agent/Device/Phone/Tablet (please complete the following information):

• User-agent [e.g. "Mozilla/5.0 (Linux; Android 12; SAMSUNG SM-F926B) AppleWebKit/537.36..."]

I get this error just on one website. PHP 8.2.27

Fatal error: Declaration of Detection\Cache\CacheItem::expiresAt(?DateTimeInterface $expiration): Theme must be compatible with Psr\Cache\CacheItemInterface::expiresAt($expiration) n /home/public_html/wp-content/themes/champion/framework/plugins/mobile-detect/src/Cache/CacheItem.php on line 100

Any idea how to fix this?

Using the standalone version.

[ArchBlood]

@BackuPs seems to me that it isn't accepting DateTime properly, you could offset this by changing it to one of the following;

public function expiresAt(?DateTimeInterface $expiration): static
{
    $this->expiresAt = $expiration;
    return $this;
}

Or properly use the following;

public function expiresAt(?DateTimeInterface $expiration): static
{
    $this->expiresAt = $expiration instanceof DateTimeImmutable 
        ? DateTime::createFromImmutable($expiration) 
        : $expiration;
    return $this;
}

The second option however may include performance overhead due to conversion. Overall the package in many areas either doesn't fully comply with PSR-16 or ignores requirements when implementing an interface. 🤔

[BackuPs]

@ArchBlood

On the same server many website are using the same code and do not have the issue. I stripped out the caching from the plugin as i dont need it.

Thank anyway for your solution. I will try it in that website that has the error.

[orionchikby]

I have the same problem
Upgrading mobiledetect/mobiledetectlib (2.8.45 => 4.8.07): Extracting archive
$detect = new MobileDetect();
$isMobile = $detect->isMobile();

[BackuPs]

You can use these...

https://github.com/BackuPs/Mobile-Detect/tree/4.8.x/src-nocache/mobile-detect

it uses no caching.