function mailchimp_unsubscribe_process($list_id, $email, $delete, $goodbye, $notify) {

I can't see that these are used at all.

CommentFileSizeAuthor
#5 delete-unused-params-2988575-5.patch1.37 KBrjacobsen0
#2 delete-unused-params-2988575-2.patch849 bytesmsankhala

Comments

joachim created an issue. See original summary.

msankhala’s picture

msankhala commented
Status: Active » Needs review
StatusFileSize
new delete-unused-params-2988575-2.patch849 bytes

Here is the patch.

surbz’s picture

surbz commented

Thanks for the patch @msankhala, I can confirm the patch applies clean!!

msankhala’s picture

msankhala commented
Status: Needs review » Reviewed & tested by the community
rjacobsen0’s picture

rjacobsen0 commented
StatusFileSize
new delete-unused-params-2988575-5.patch1.37 KB

A call to this function was missed and still had the extra arguments in it. I deleted them. It was easy to miss because it was a callback function with an array of args.

  • rjacobsen0 committed 44db7ff on 8.x-1.x 
    Issue #2988575 by msankhala, rjacobsen0: $delete, $goodbye, $notify...
rjacobsen0’s picture

rjacobsen0 commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.