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

Warning when running cron job #309

Open
rmniemela opened this issue Dec 30, 2018 · 3 comments
Open

Warning when running cron job #309

rmniemela opened this issue Dec 30, 2018 · 3 comments

Comments

@rmniemela
Copy link

Am testing cron jobs now that I've updated to CiviCRM 5.8.2. Testing the CRON job with https://.../wp-content/plugins/civicrm/civicrm/bin/cron.php?name=xxx&pass=xxx&key=xxx.

Getting

Warning: Declaration of CRM_Mailchimp_Permission::check($permissions) should be compatible with CRM_Core_Permission::check($permissions, $contactId = NULL) in /home/xxx/public_html/wp-content/uploads/civicrm/ext/uk.co.vedaconsulting.mailchimp/CRM/Mailchimp/Permission.php on line 40
Sent Report mail has been sent.

Have not seen this before, although I've not tested the cron job via the http interface in quite a while.

How do I fix the warning?

@rmniemela
Copy link
Author

Update: reset the webhook and nothing worked. No synchronization happening even though testing the webhook shows that it's getting correct Mailchimp account and email.

Also getting

Warning: invalid argument supplied for foreach() in /home/sudbu904/public_html/wp-content/uploads/civicrm/ext/uk.co.vedaconsulting.mailchimp/CRM/Mailchimp/Utils.php on line 228

@sunnycs121
Copy link

sunnycs121 commented Mar 30, 2020

@sunnycs121
Copy link

I fixed it for me by modifying permission.php file, added parameter $contactId in function "check" and initialized it with NULL.

From
public static function check($permissions) {
TO
public static function check($permissions, $contactId = NULL) {

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

No branches or pull requests

2 participants