Skip to content

Fix deprecation notices in PHP 8.2 #77

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 19, 2023
Merged

Fix deprecation notices in PHP 8.2 #77

merged 2 commits into from
May 19, 2023

Conversation

srjlewis
Copy link

@srjlewis srjlewis commented Dec 7, 2022

PHP 8.2 add a new deprecation of dynamic properties.
I have added a properties that was assigned dynamically within the class.

@jleonardolemos
Copy link

I have upgraded my application to PHP8.2 and this notices showed up.

   DEPRECATED  Creation of dynamic property Webpatser\Uuid\Uuid::$bytes is deprecated in vendor/webpatser/laravel-uuid/src/Webpatser/Uuid/Uuid.php on line 127.


   DEPRECATED  Creation of dynamic property Webpatser\Uuid\Uuid::$string is deprecated in vendor/webpatser/laravel-uuid/src/Webpatser/Uuid/Uuid.php on line 130.


   DEPRECATED  Creation of dynamic property Webpatser\Uuid\Uuid::$uuid_ordered is deprecated in vendor/webpatser/laravel-uuid/src/Webpatser/Uuid/Uuid.php on line 137.

Would be very nice to see this MR merged.

@webpatser webpatser merged commit f82715c into webpatser:master May 19, 2023
@jahir
Copy link

jahir commented Sep 21, 2023

@webpatser Would you mind packaging this as 4.0.2, now that php 8.2 is in wide use?
#78 is a duplicate and can be closed, btw

@webpatser
Copy link
Owner

Sorry for the delay. But I tagged this to 4.0.2

jahir pushed a commit to volkszaehler/volkszaehler.org that referenced this pull request Dec 24, 2023
…ings"

This reverts commit 7770dc8.
upstream tagged the fixed version of laravel 4.0.2,
see webpatser/laravel-uuid#77
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants