search

3sd / civicrm-recalculate-recipients

Moved to https://lab.civicrm.org/extensions/civicrm-recalculate-recipients
Other
6 stars 5 forks source link

Too many parameters in call to CRM_Mailing_BAO_Mailing::getRecipients #7

Closed lolcodeca closed 6 years ago

lolcodeca commented 6 years ago

As mentioned by Monish in chat: https://chat.civicrm.org/civicrm/pl/3y5s4uxcof81ur553cmeu85tdo there are too many parameters to this function call.

michaelmcandrew commented 6 years ago

thanks for reporting @lolcodeca (and @monishdeb)

following monish's refactor I've pushed this commit https://github.com/3sd/civicrm-recalculate-recipients/commit/37b0cd79fd3ee147cf4fe94c55184fe476e1be8c which should fix the problem (and also removes a now unecessary api call).