- Issue created by @dsdeiz
- Status changed to Needs review
over 1 year ago 8:02am 25 October 2023 - last update
over 1 year ago 112 pass, 2 fail The last submitted patch, 2: metatag-main-property-name-3396529-1.patch, failed testing. View results →
- Status changed to Needs work
over 1 year ago 8:20am 25 October 2023 - last update
over 1 year ago 113 pass - 🇺🇸United States DamienMcKenna NH, USA
The test failure was unrelated, so I've reran the tests.
- Status changed to Needs review
over 1 year ago 9:58am 30 October 2023 - 🇵🇭Philippines dsdeiz
Yeah, marking back as "Needs review" since the test passes now.
- 🇺🇸United States DamienMcKenna NH, USA
Yes, good to see the tests pass now.
My next question: Do you have any links to documentation to indicate if this is the correct or appropriate implementation of that method?
- 🇵🇭Philippines dsdeiz
Yeah, I mostly just noticed this here - https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Field%21F....
Any field types that don't have "value" as one of the field's properties return a different main property name.
FieldItemBase
by defaults return "value" as the main property. - Status changed to RTBC
12 days ago 12:06pm 25 March 2025 - 🇮🇳India vinaygawade Sawantwadi, Maharashtra
I encountered the same issue, and applying this patch resolved it. Updating the status to RTBC.