Skip to content
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

Creation of dynamic property is deprecated #8

Open
FaycalBoutam opened this issue Sep 11, 2024 · 2 comments
Open

Creation of dynamic property is deprecated #8

FaycalBoutam opened this issue Sep 11, 2024 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@FaycalBoutam
Copy link

Full description

There seem to be a few warning popping up when using the Lite version - nothing seems to be broken yet but just wondering if these can be fixed easily to avoid any issues going forward.

Deprecated: Creation of dynamic property Barn2\Plugin\Document_Library\Document::$post_object is deprecated in ...\public\wp-content\plugins\document-library-lite\src\Document.php on line 36

They must be using PHP 8.2 where dynamic properties are deprecated:
https://php.watch/versions/8.2/dynamic-properties-deprecated

Expected outcome

No response

Observed/Actual outcome

No response

Steps to replicate

No response

Login credentials

No response

Ticket link

https://secure.helpscout.net/conversation/2703952654/117228?viewId=7855239

@FaycalBoutam FaycalBoutam added the bug Something isn't working label Sep 11, 2024
@ejpadero
Copy link

Hi, @amirition. I know you're currently OOO, so am just bumping this up for when you're back. Thanks!

@alessandrotesoro
Copy link
Member

For some reason I was getting notifications regarding issues in this repository, so anyways, because Amir is away and the next release is under QA and has quite a few bugs/tasks reported by Aleem. I just pushed a PR that fixes this hoping to help speed things up a tiny bit.

Please include this in the next release @amirition

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants