Contrib.social

Does not work with Group Module v2.0.x

Open on Drupal.org →
Created on 1 November 2022, over 1 year ago
Updated 12 June 2024, 16 days ago

Problem/Motivation

If I try to run Recurring Events together with the Group Module 2.0.x it throws fatal errors. I strongly suspect that this is, at least in part, because the Group Module API has changed from Group v8.x-1.5

Steps to reproduce

  • Set up with versions Drupal 9.4.8, Group 2.0.0-beta3, Recurring Events 2.0.0-rc5
  • Enable all except the Group Recurring Events module. Both Group and Recurring Events work correctly, but separately.
  • Enable the Group Recurring Events module. It throws the first error below when trying to open a Group page.
  • In the file /recurring_events/modules/group_recurring_events_series/config/optional/views.view.group_recurring_events_series.yml change lines
    438, 491 and 542 to read table: group_relationship instead of table: group_content. (This is copied from the Group Module MR#57)
  • It then throws the second error shown below instead.

First Error
Error: Class 'Drupal\group\Entity\Controller\GroupContentController' not found in include() (line 17 of /home/lalg/www/d9dev2/web/modules/contrib/recurring_events/modules/group_recurring_events_series/src/Controller/GroupEventSeriesController.php)
Backtrace file attached.

Second Error
Symfony\Component\Routing\Exception\RouteNotFoundException: Route "view.group_recurring_events_series.page_1" does not exist. in Drupal\Core\Routing\RouteProvider->getRouteByName() (line 206 of /home/lalg/www/d9dev2/web/core/lib/Drupal/Core/Routing/RouteProvider.php).

Proposed resolution

I do not know the cause or fix for the second error, but implementation of the fix for the first will need to allow for running with either Group v8.x-1.5 or v2.0.x, as present at the time.

API changes

See Group module documentation.

🐛 Bug report
Status

Needs review

Version

2.0

Component

Recurring Events (Main module)

Created by

mbkaba22

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

Merge Requests

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • Comment over 1 year ago →
    🇺🇸United States Chris Dart

    Chris Dart → made their first commit to this issue’s fork.

  • Comment 11 months ago →
    🇺🇸United States msielski

    Attaching patch I created and am testing to use recurring events with group 2.x. I believe more work will still be needed.

  • First commit to issue fork.
  • Merge request !90Issue #3318490: Support Group 2.x. → (Open) created by muriqui
  • Pipeline finished with Failed
    about 1 month ago
    Total: 151s
    #181568
  • Status changed to Needs work about 1 month ago
  • Comment about 1 month ago →
    🇺🇸United States muriqui

    Created a merge request with the changes from msielski's patch, plus converting the GroupEventSeriesPermissionProvider. Haven't had a chance to test this yet, so marking as "needs work." (Also hiding the patch file from issue description so as to direct people's attention to the MR for latest changes.)

  • Status changed to Needs review 25 days ago
  • Comment 25 days ago →
    🇺🇸United States muriqui
  • Pipeline finished with Failed
    25 days ago
    Total: 211s
    #190211
  • Comment 16 days ago →
    🇬🇧United Kingdom Rotekian

    This fixed things for me.

  • Pipeline finished with Success
    4 days ago
    Total: 183s
    #206885
  • Pipeline finished with Success
    4 days ago
    Total: 180s
    #207123
contrib.social Blog FAQ Discussions
Production build 0.69.0 2024