Uploaded image for project: 'CiviCRM'
  1. CiviCRM
  2. CRM-7921

EmailProcessor.php/CiviMailProcessor.php passing incorrect $params

    Details

    • Type: Bug
    • Status: Done/Fixed
    • Priority: Major
    • Resolution: Fixed/Completed
    • Affects Version/s: 3.3.6, Unscheduled
    • Fix Version/s: 3.4.0
    • Component/s: CiviCRM Profile, CiviMail
    • Labels:
      None

      Description

      I discovered this when trying to get the double opt-in profiles patch (CRM-7465) to work - it appears that the Mailer.php $params signature was changed and SoapServer.php was updated to match, but EmailProcessor.php/CiviMailProcessor.php were left with the old style, which breaks this functionality (double opt-in has no effect on group membership status, as suggested by Alex in CRM-7465). As far as I can tell, this issue is present on both stable and trunk code. I have included a patch for the "confirm" path that works for me, but it looks like some of the other $params need updating also (I would do this, but I am unsure of how to test each case).

        Attachments

          Activity

            People

            • Assignee:
              lobo Donald A. Lobo
              Reporter:
              grugnog Owen Barton
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: