Wrong handle with null values on getRemoteStatus

TypeError: Drupal\widget_type\Entity\WidgetType::getRemoteStatus(): Return value must be of type string, null returned in Drupal\widget_type\Entity\WidgetType->getRemoteStatus() (line 336 of /var/www/html/web/modules/contrib/widget_type/src/Entity/WidgetType.php)

Comments & Activities

Issue created by @isholgueras
Issue was unassigned.
Status changed to Needs review almost 2 years ago6:49am 28 January 2023
Comment almost 2 years ago →
🇪🇸Spain isholgueras
Here the patch to add the nullable string.
Comment almost 2 years ago →
🇪🇸Spain isholgueras
Using a different approach.

Instead of using null, widget_ingestion module will set the status to the new unknown whenever the status is not set in the registry.

The default value for the field remote_widget_status inside the customBaseFieldDefinitions is also set to unknown.

Also, I've fixed one typo.
Comment almost 2 years ago →
🇪🇸Spain isholgueras
Status changed to RTBC almost 2 years ago12:32pm 7 February 2023
Comment almost 2 years ago →
e0ipso Can Picafort
I love this.
Comment almost 2 years ago →
e0ipso Can Picafort
Comment almost 2 years ago →
System Message

e0ipso → committed 68405444 on 1.x authored by isholgueras →
Issue #3337158 by isholgueras, e0ipso: Wrong handle with null values on...
Status changed to Fixed almost 2 years ago10:15am 15 February 2023
Comment almost 2 years ago →
e0ipso Can Picafort
Comment almost 2 years ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

Wrong handle with null values on getRemoteStatus

Problem/Motivation

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Comments & Activities