Hi @digitalchild
Can you share with us your system report available from the Matomo Analytics > Diagnostics menu?
Kind regards
Mat
hi @digitalchild
any chances you read my previous comment?
Kind regards
Mat
Hi @mattmary
Apologies for the delay. Is there somewhere I can email the system report?
cheers,
Jamie.
hi @digitalchild
sure. you can email it at [email protected]
Kind regards
Mat
Hi @digitalchild
I cannot reproduce your bug.
Can you try to increase your memory limit ?
Kind regards
Mat
Hi Matt,
How is 256MB of memory not enough for the plugin? I’ve installed new relic and will do a bunch more tests and see what I can find. Will report back when I have something.
cheers,
Jamie.
With new relic enabled, I’m able to replicate the issue and it appears that something within the sync class
in WpMatomo\User\Sync::sync_current_users_1000 called at /wp-content/plugins/matomo/classes/WpMatomo/User/Sync.php (69)
When Matomo is not activated, we get no fatal error, so something in that is causing an issue.
I’ll continue digging to see if I can find what is happening in the sync calls, that might be causing it.
cheers,
jamie.
Hi @digitalchild
Thanks for investigating.
256 M is enough for the plugin to run but the memory consumption depends also of the others plugins installed and the amount of data you have.
Do you have a lot of user accounts?
Kind regards
Mat
Hi Mat,
The site has over 20k users. Without reading the Matomo source, I’m assuming the sync tool is going through all users?
I’ll keep the staging site up if you have anything else you need me to do / break / test.
cheers,
Jamie.
hi @digitalchild
the user accounts are synchronized only if there are less than 1000 accounts. otherwise this is a background task which synchronize the accounts.
My understanding is the SQL query which collect the users accounts number fetch all the user accounts data and this is why you get this error.
I’ve created this issue https://github.com/matomo-org/matomo-for-wordpress/issues/587
feel free to subscribe to it to get notifications when it will be solved
Plugin Author
Thomas
(@tsteur)
@digitalchild any chance you are familiar with patching files in your WordPress installation? We have a patch that may fix this issue but would need to get this confirmed.
In case you are familiar with patching files or PHP, could you try to apply this same change from https://github.com/matomo-org/matomo-for-wordpress/pull/588/files the file wp-content/plugins/matomo/classes/WpMatomo/User/Sync.php ?
Or you could replace the entire content of that file with this content: https://raw.githubusercontent.com/matomo-org/matomo-for-wordpress/acb360040cc57298f8e8d8ef334f3bd09e40fa70/classes/WpMatomo/User/Sync.php
Hi @digitalchild
Any chances you read our previous message?
Kind regards
Mat
Hello,
I’ve patched and confirmed I’m no longer getting the fatal error. Thanks!
cheers,
Jamie.
