$cardinality variable being set as string and failing strict type check

Created on 6 March 2024, 9 months ago

Problem/Motivation

The $cardinality variable being set on line 17 of /field_config_cardinality/src/Plugin/Field/FieldWidget/CardinalityMediaLibraryWidget.php needs to be stored as an integer. Currently it is stored as a string, which then fails the strict type check in line 473 of /core/modules/media_library/src/Plugin/Field/FieldWidget/MediaLibraryWidget.php

$cardinality_unlimited = ($element['#cardinality'] === FieldStorageDefinitionInterface::CARDINALITY_UNLIMITED);

Steps to reproduce

Add a media library field with unlimited cardinality
Add a media item to the widget
See that the message that shows is "-1 media items remaining"

Proposed resolution

Cast the $cardinality variable as an integer.

Remaining tasks

Make the changes

User interface changes

None

API changes

None

Data model changes

None

🐛 Bug report

Status

Closed: cannot reproduce

Version

3.0

Component

Code

Created by

🇬🇧United Kingdom hebl

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @hebl
Status changed to Closed: cannot reproduce 9 months ago1:15pm 6 March 2024
Comment 9 months ago →
🇬🇧United Kingdom hebl
My mistake, the issue was coming from a patch for another issue which I've now re-rolled.

https://www.drupal.org/project/field_config_cardinality/issues/3274567#c... 🐛 CardinalityMediaLibraryWidget does not work with base field RTBC

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024