core#1568: Show recipientListing to non-admins

[MegaphoneJon]

Overview

There's a bug report that the permissions were wrong when setting scheduled reminders as a non-administrator (via Manage Events) that you can't limit by participant role.

Before

After

Technical Details

There's no reason for this route to require Administer CiviCRM and is surely an artifact of the time before Scheduled Reminders were opened up to non-administrators.

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[jaapjansma]

Betty and I are reviewing PR's and have reviewed this PR see below our findings.

• General standards
  • Explain (r-explain)
    • PASS : The goal/problem/solution have been adequately explained in the PR and in the issue.
  • User impact (r-user)
    • PASS: A field which wasn't visible but should be visible is now visible again.
  • Documentation (r-doc)
    • PASS: The changes do not require documentation.
  • Run it (r-run)
    • PASS: We have tested the patch and it seems to work. We also tested whether the scheduled reminder did send the e-mail and everything seems to work.
• Developer standards
  • Technical impact (r-tech)
    • PASS: The change preserves compatibility with existing callers/code/downstream.
  • Code quality (r-code)
    • PASS: The functionality, purpose, and style of the code seems clear+sensible.
  • Maintainability (r-maint)
    • PASS: The change does not require tests
  • Test results (r-test)
    • PASS: The test results are all-clear.

@mattwire we have reviewed this PR and we think it is good enough to merge it, can you merge this?