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

New [mass] SMS is broken - error in CRM_Mailing_BAO_Mailing::getRecipients query

          Details

          • Type: Bug
          • Status: Done/Fixed
          • Priority: Blocker
          • Resolution: Fixed/Completed
          • Affects Version/s: 4.7.31
          • Fix Version/s: 4.7.31
          • Component/s: None
          • Labels:
            None
          • Versioning Impact:
            Patch (backwards-compatible bug fixes)
          • Documentation Required?:
            None
          • Funding Source:
            Contributed Code
          • Verified?:
            No

            Description

            Sending a Mass SMS doesn't work, most likely after merging this commit: https://github.com/civicrm/civicrm-core/commit/6dd717a6572e06770dd7b518ba5b8f264c05944e.

            The fix appears to be simple but I haven't tested it yet. I am guessing this slipped through because we don't have a lot of SMS tests. I'm happy to write one as part of this fix though could do with a pointer on where to start / what unit to test.

              Attachments

                Activity

                  People

                  • Assignee:
                    Unassigned
                    Reporter:
                    michaelmcandrew Michael McAndrew
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    5 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: