Fix php 8.2 dynamic properties. Thanks to GitHub user @daveromsey for the submission.

MinnPost / object-sync-for-salesforce

WordPress plugin that maps and syncs data between Salesforce objects and WordPress objects.

https://wordpress.org/plugins/object-sync-for-salesforce/

GNU General Public License v2.0

93 stars 48 forks source link

Fix php 8.2 dynamic properties. Thanks to GitHub user @daveromsey for the submission. #521

Closed daveromsey closed 5 months ago

daveromsey commented 6 months ago

This PR adds property declarations to classes in order to clean up some of the deprecation warnings that have cropped up after upgrading to PHP 8.2.

How do I test this Pull Request?

Load the plugin and view the error log. This PR will clean up the deprecation notices for dynamic properties. There are still some other deprecations for other things, but I wanted to limit the scope of this PR and also make sure I didn't break anything :)

jonathanstegall commented 6 months ago

I'm no longer at MinnPost, and I don't think anyone is currently working on the plugin, but I'll try to take a look at this over the weekend and approve it and get a small release out. Hopefully soon!