Contrib.social

Registrants view does not show registrants from all event instances in a "Series Registration" event series

Open on Drupal.org →
Created on 7 February 2024, about 1 year ago

Problem/Motivation

When using the recurring_events_registrations view provided by Recurring Events Views submodule, an incomplete list of Registrants is shown if the series uses Series Registration.

Steps to reproduce

  • enable the Recurring Events Views submodule so that the event instance registrations list at /event/{eventinstance}/registrations is replaced by the recurring_events_registrations view.
  • create an event with Series Registration and more than one instance
  • register one registrant to each instance in newly created series
  • go to the Registrations tab for each event instance and see that the view does not show all the Registrants from the whole series, but just the Registrants from that particular instance

Proposed resolution

Figure out how to configure the view to show all Registrants from the series if set to Series Registration OR show a different view display for Series Registration events and do not show the default, incorrect Registrant view

🐛 Bug report
Status

Active

Version

2.0

Component

Recurring Events Views (Submodule)

Created by

🇨🇦Canada endless_wander

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @endless_wander
  • Comment about 1 year ago →
    🇺🇸United States owenbush Denver, CO

    Thank you for raising this issue. I see the problem now. There is a function `recurring_events_views_views_query_alter` which alters the query for the registrant list view to ensure that it displays registrants for all instances in a series registration series.

    The problem is that previously the view had an id of 'registrations' but that was eventually changed in a later release of recurring events to be 'recurring_events_registrations' so it was better namespaced.

    The code in the function was not updated to use that ID.

    I'll get that addressed for you.

  • Merge request !85Update the view alter hooks to also modify the new view IDs → (Open) created by owenbush
  • Status changed to Needs review about 1 year ago
  • Comment about 1 year ago →
    🇺🇸United States owenbush Denver, CO

    Please try out a patch from the MR and let me know whether it works for you.

  • Comment 3 months ago →
    🇨🇦Canada endless_wander

    Heads up this will no longer apply against latest 2.0.x branch

  • Comment 3 months ago →
    🇨🇦Canada endless_wander

    New patch works against 2.0.2. Sorry can't get access to make MR right now.

    @maintainers, could this get merged into 2.0 branch?

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024