When reviewing the module code, found reference to usage of dynamic variables in file SyncFilesBatch.php. Class class SyncFilesBatch {...} initialises 3 property variables without defining them in the constructor line 27,28 and 29. There is a discussion going to deprecate this behaviour from PHP 8.2 and remove from PHP 9.0 More Info.
Check the code in file SyncFilesBatch.php
Define the variables with proper data types.
Create MR and TEST
N/A
N/A
N/A
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
Hi, I have reviewed the PR, it seems good to me. I am getting minor phpcs errors on the number of spaces before the new lines of code. There are many phpcs errors so perhaps that is another issue in itself.
Automatically closed - issue fixed for 2 weeks with no activity.