Order Assign Event Error if not a ticket

The website encountered an unexpected error. Please try again later. InvalidArgumentException: Field tickets is unknown. in Drupal\Core\Entity\ContentEntityBase->getTranslatedField() (line 587 of core/lib/Drupal/Core/Entity/ContentEntityBase.php). Drupal\Core\Entity\ContentEntityBase->get('tickets') (Line: 194) Drupal\commerce_ticketing\EventSubscriber\OrderEventSubscriber->onOrderAssign(Object, 'commerce_order.order.assign', Object) call_user_func(Array, Object, 'commerce_order.order.assign', Object) (Line: 142) Drupal\Component\EventDispatcher\ContainerAwareEventDispatcher->dispatch(Object, 'commerce_order.order.assign') (Line: 48)

Comments & Activities

Not all content is available!

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

First commit to issue fork.
Merge request !30Adds testing the events with both order types: tickets enabled, and tickets disabled → (Open) created by aleix
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 7.4 & MySQL 5.7
last update 4 months ago
11 pass, 4 fail
Comment 4 months ago →
🇪🇸Spain aleix
In fact it doesn't affect only the assign event, it will affect the events without the conditional that checks if order entity hasn't the tickets field. The test in MR shows just the assign and paid event. But It's clear that the subscription will affect orders without tickets enabled, and testing all events will add noise to test files.
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 7.4 & MySQL 5.7
last update 4 months ago
5 pass, 8 fail
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 7.4 & MySQL 5.7
last update 4 months ago
16 pass, 2 fail
Status changed to Needs review 4 months ago9:35pm 12 March 2024
Comment 4 months ago →
🇪🇸Spain aleix
After adding the conditional, tests confirm that tickets Field tickets is unknown errors are not there anymore :) .

Adding critical as this issue breaks some sites that are mixing orders with enabled and disabled tickets...
Open on Drupal.org →
Core: 9.5.x + Environment: PHP 7.4 & MySQL 5.7
last update 4 months ago
Waiting for branch to pass
Comment 4 months ago →
🇪🇸Spain aleix
(patch for composer purposes)

Order Assign Event Error if not a ticket

Problem/Motivation

Steps to reproduce

Proposed resolution

Merge Requests

!30Order Assign Event Error if not a ticket
Open

Comments & Activities

Order Assign Event Error if not a ticket

Problem/Motivation

Steps to reproduce

Proposed resolution

Merge Requests

!30Order Assign Event Error if not a ticketOpen

Comments & Activities

!30Order Assign Event Error if not a ticket
Open