Suppress deprecation notices in PHP 8.0+

[derekmd]

Alternate to #42

Suppress deprecation notices in PHP 8.0 and 8.1 from internal languages methods missing return typehints.

PHP Deprecated: Return type of Jenssengers\Model\Model::offsetExists($offset) should either be compatible with ArrayAccess::offsetExists(mixed $offset): bool, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in vendor/jenssegers/model/src/Model.php on line 866

Instead of adding those explicit return types (and breaking support with older PHP versions), add attribute #[\ReturnTypeWillChange] to the same Illuminate\Database\Eloquent\Model methods as "laravel/framework" 8.x (https://github.com/laravel/framework/blob/888947cedacbefac4019a163ccf941e592edad82/src/Illuminate/Database/Eloquent/Model.php):

• offsetExists
• offsetGet
• offsetSet
• offsetUnset
• jsonSerialize

---

• "laravel/framework" 9.x eventually replaces #[\ReturnTypeWillChange] with explicit return types for PHP 8.1+: https://github.com/laravel/framework/blob/067c21851f15512f921e899e1ec2d09d6ddff0cd/src/Illuminate/Database/Eloquent/Model.php
• PHP 9+ will require the return types: https://wiki.php.net/rfc/internal_method_return_types

[dreammonkey]

Hi, I would like to check the status on this because I might want to use this package in a project I'm working on.
It looks like a code styling issue is blocking this merge request ? Is that correct?

Please let me know if there is anything I can do to contribute !

[JoshMountain]

Would love to get this merged and get rid of these warnings.

[jarikoff]

Hi everyone !
Any updates on this ?