search

SpartnerNL / Laravel-Sidebar

A Laravel Sidebar builder
MIT License
111 stars 58 forks source link

LOG.warning: implements the Serializable interface #41

Closed wikigods closed 12 months ago

wikigods commented 2 years ago

I'm getting LOG.warning: implements the Serializable interface using laravel 9 but I think it's more due to the PHP 8.1.2 version I'm currently using.

"Warning. As of PHP 8.1.0, a class that implements Serializable without also implementing serialize() and unserialize() will generate a deprecation warning."

My solution is to implement serialize() and unserialize() , so as not to have that warning.

Screenshot_6 @patrickbrouwers

patrickbrouwers commented 2 years ago

Hey @wikigods, we don't actively support this package anymore. We merge fixes as long as people are still using it and provide the PR themselves. If you want to keep using this package, feel free to provide a PR that fixes the warnings.

wikigods commented 2 years ago

Good morning, can you help me with a problem I have when I run the tests (I only need this one test) for laravel = 9 php = 8.1 phpunit/phpunit = 9.5.10 mockery/mockery = 1.4.4 attached changes I made to the package with commit: "update implements the Serializable interface". https://github.com/wikigods/Laravel-Sidebar/commits/master Sorry for my english

@patrickbrouwers

Screenshot_15 Screenshot_16