Answers.Body: <p>patch:</p>
  
  <pre><code>index 5b82f65e..76c816dc 100644
  --- a/uk.co.vedaconsulting.mailchimp-2.0.1/CRM/Mailchimp/Permission.php
  +++ b/uk.co.vedaconsulting.mailchimp-2.0.1/CRM/Mailchimp/Permission.php
  @@ -1,6 +1,6 @@
   &lt;?php
  
  -class CRM_Mailchimp_Permission extends CRM_Core_Permission {
  +class CRM_Mailchimp_Permission {
  
     /**
      * Returns an array of permissions defined by this extension. Modeled off of
  @@ -35,6 +35,6 @@ class CRM_Mailchimp_Permission extends CRM_Core_Permission {
           }
         });
       }
  -    return parent::check($permissions);
  +    return CRM_Core_Permission::check($permissions,NULL);
     }
   }
  </code></pre>
  
Answers.ContentLicense: CC BY-SA 4.0
Answers.CreationDate: 2019-03-26T19:09:02.883Z
Answers.Text: and *finally* I spent the 10 minutes to learn how to do a real pull request. 
  https://github.com/veda-consulting/uk.co.vedaconsulting.mailchimp/pull/320
Answers.UserDisplayName: ericG
Answers.ContentLicense: CC BY-SA 4.0
Answers.CreationDate: 2021-08-29T19:17:45.617Z
Answers.Text: This patch does not seem to applied to the current distribution of the extension.
Answers.UserDisplayName: yktdan
Answers.ContentLicense: CC BY-SA 4.0
Answers.CreationDate: 2021-08-30T21:33:32.440Z
Answers.Text: they have not rolled a new release since 2.0.1, but they did fix this issue (with a better solution than mine) in the code available via their github repo 
  https://github.com/veda-consulting-company/uk.co.vedaconsulting.mailchimp
Answers.UserDisplayName: ericG
Answers.ContentLicense: CC BY-SA 4.0
Answers.CreationDate: 2019-03-19T21:12:53.267Z
Answers.LastActivityDate: 2019-03-19T21:12:53.267Z
Answers.UserDisplayName: ericG
Body: <p>with civi 5.10.4, Drupal 7.x and php 7.1 there is a warning that in some cases turns into a fatal error that will prevent the webhooks from working</p>
  
  <p>Declaration of CRM_Mailchimp_Permission::check($permissions) should be compatible with CRM_Core_Permission::check($permissions, $contactId = NULL) in /path/to/uk.co.vedaconsulting.mailchimp-2.0.1/CRM/Mailchimp/Permission.php</p>
  
  <p>CiviVolunteer had a similar issue, following the example in that fix I was able to get mailchimp extension working. <a href="https://github.com/civicrm/org.civicrm.volunteer/pull/477/files" rel="nofollow noreferrer">https://github.com/civicrm/org.civicrm.volunteer/pull/477/files</a></p>
  
  <p>I'll submit a pull request soon but in the meantime if anyone is having the same issue the fix is</p>
  
  <p>remove:    class CRM_Mailchimp_Permission extends CRM_Core_Permission {</p>
  
  <p>add:    class CRM_Mailchimp_Permission {</p>
  
  <p>and</p>
  
  <p>remove:    return parent::check($permissions);</p>
  
  <p>add:    return CRM_Core_Permission::check($permissions,NULL);</p>
  
  <p>if interested, issue in the extension's github is <a href="https://github.com/veda-consulting/uk.co.vedaconsulting.mailchimp/issues/319" rel="nofollow noreferrer">https://github.com/veda-consulting/uk.co.vedaconsulting.mailchimp/issues/319</a> </p>
  
Tags: <mailchimp><php7>
Title: mailchimp extension error, php 7.1, civi 5.10.4, drupal 7.x
ThreadText: # mailchimp extension error, php 7.1, civi 5.10.4, drupal 7.x
  
  (Asked by: username_0 on 2019-03-19)
  
  with civi 5.10.4, Drupal 7.x and php 7.1 there is a warning that in some cases turns into a fatal error that will prevent the webhooks from working
  
  Declaration of CRM\_Mailchimp\_Permission::check($permissions) should be compatible with CRM\_Core\_Permission::check($permissions, $contactId = NULL) in /path/to/uk.co.vedaconsulting.mailchimp-2.0.1/CRM/Mailchimp/Permission.php
  
  CiviVolunteer had a similar issue, following the example in that fix I was able to get mailchimp extension working. <https://github.com/civicrm/org.civicrm.volunteer/pull/477/files>
  
  I'll submit a pull request soon but in the meantime if anyone is having the same issue the fix is
  
  remove: class CRM\_Mailchimp\_Permission extends CRM\_Core\_Permission {
  
  add: class CRM\_Mailchimp\_Permission {
  
  and
  
  remove: return parent::check($permissions);
  
  add: return CRM\_Core\_Permission::check($permissions,NULL);
  
  if interested, issue in the extension's github is <https://github.com/veda-consulting/uk.co.vedaconsulting.mailchimp/issues/319>
  
  **Tags:** mailchimp, php7
  
  
  ---
  
  ## 1 answers
  
  ### Answer by username_0 on 2019-03-19. Score: 3 | Accepted answer
  
  patch:
  index 5b82f65e..76c816dc 100644
  --- a/uk.co.vedaconsulting.mailchimp-2.0.1/CRM/Mailchimp/Permission.php
  +++ b/uk.co.vedaconsulting.mailchimp-2.0.1/CRM/Mailchimp/Permission.php
  @@ -1,6 +1,6 @@
  
  > **Comments:**
  > **username_0**: and *finally* I spent the 10 minutes to learn how to do a real pull request.  https://github.com/veda-consulting/uk.co.vedaconsulting.mailchimp/pull/320  
  > **username_1**: This patch does not seem to applied to the current distribution of the extension.  
  > **username_0**: they have not rolled a new release since 2.0.1, but they did fix this issue (with a better solution than mine) in the code available via their github repo  https://github.com/veda-consulting-company/uk.co.vedaconsulting.mailchimp  
  
  ---
  
  