- Issue created by @mlncn
- Merge request !96Issue #3480035 by mlncn: After upgrading to Mailchimp 2.2.5, failure to get... → (Merged) created by mlncn
- 🇳🇿New Zealand ericgsmith
Patch correctly identifies and fixes the issue - just a minor suggestion to the fix on the MR but really will leave that to maintainers as its more a code style suggestion to keep the number of returns in the function the same.
- 🇺🇸United States xenophyle
@mlncn does @ericgsmith's modification work for you?
-
xenophyle →
committed 667d1f29 on 2.x authored by
mlncn →
Issue #3480035 by mlncn, ericgsmith, dewancodes: After upgrading to...
-
xenophyle →
committed 667d1f29 on 2.x authored by
mlncn →
-
xenophyle →
committed 0139aeda on 3.x authored by
mlncn →
Issue #3480035 by mlncn, ericgsmith, dewancodes: After upgrading to...
-
xenophyle →
committed 0139aeda on 3.x authored by
mlncn →
- 🇮🇳India sriharsha.uppuluri Hyderabad
This issue still replicating to me even though patch applied successfully on both 2.x and 3.x failing after removal of following condition
// There was a problem getting lists, which was probably already logged.
if (is_null($lists)) {
return [];
}https://git.drupalcode.org/project/mailchimp/-/merge_requests/96/diffs?c...
- 🇺🇸United States xenophyle
@sriharsha.uppuluri It is not clear to me what changes you applied. Did you use https://git.drupalcode.org/project/mailchimp/-/merge_requests/96.diff?
- 🇨🇭Switzerland nadja_stu
the error is still appearing for my site too. I have drupal 11.0.9 installed and mailchimp 3.0.0.
I get this error: TypeError: array_key_exists(): Argument #2 ($array) must be of type array, null given in mailchimp_get_lists() (line 176 of modules\contrib\mailchimp\mailchimp.module).
My whole site is not working because of this.
- Status changed to Fixed
4 months ago 10:55am 11 December 2024 - 🇮🇳India sriharsha.uppuluri Hyderabad
Uploading the patch with the condition, which is a revert of 6 days back code change
if (is_null($lists)) { return []; }
-
xenophyle →
committed f4669bb3 on 3.x
Issue #3480035 by sriharsha.uppuluri, nadja_stu: After upgrading to...
-
xenophyle →
committed f4669bb3 on 3.x
-
xenophyle →
committed aa917dc1 on 2.x
Issue #3480035 by sriharsha.uppuluri, nadja_stu: After upgrading to...
-
xenophyle →
committed aa917dc1 on 2.x
- 🇺🇸United States xenophyle
@nadja_stu @sriharsha.uppuluri It sounds like the additional check on $lists was necessary. Let me know if the latest changes helps.
Automatically closed - issue fixed for 2 weeks with no activity.