Webhook event handling still uses old syntax

Created on 22 March 2024, 8 months ago

Updated 5 April 2024, 7 months ago

Problem/Motivation

Incoming webhooks yield errors like this:

TypeError: Drupal\Component\EventDispatcher\ContainerAwareEventDispatcher::dispatch(): Argument #1 ($event) must be of type object, string given, called in modules/contrib/brandfolder/src/Controller/IncomingWebhookController.php

Also, the webhook event subscriber has an entity query that needs to be updated.

Steps to reproduce

Point a Brandfolder webhook at a Drupal site with the latest version of the Brandfolder module installed.

Proposed resolution

Update syntax to work with current supported Symfony/Drupal versions.

User interface changes

None

API changes

None, except webhook events will be successfully dispatched again.

Data model changes

None

🐛 Bug report

Status

Fixed

Version

5.0

Component

Code

Created by

🇺🇸United States ndewhurst USA

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @ndewhurst
Comment 8 months ago →
🇺🇸United States ndewhurst USA
Comment 8 months ago →
System Message

ndewhurst → committed cfa5a32f on 5.0.x
#3433119 Update webhook event dispatch and subscriber per Drupal 10.
Status changed to Fixed 8 months ago10:45pm 22 March 2024
Comment 8 months ago →
🇺🇸United States ndewhurst USA
Comment 7 months ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024