- Issue created by @gcb
- @gcb opened merge request.
- Status changed to Needs work
over 1 year ago 7:10am 4 July 2023 - π¬π§United Kingdom joachim
Thanks for the patch!
I don't think the change in the return value is a problem -- it's a bug that it's not returning something in the right format for those settings.
Just one small tweak needed.
- Status changed to Needs review
over 1 year ago 5:41pm 5 July 2023 - πΊπΈUnited States gcb
Thanks for the prompt review! Fixed that syntax in the branch.
- π¬π§United Kingdom joachim
I can't actually see my computed reverse entity reference field available as a block to layout builder! Is there an additional step needed to make it show?
- π¬π§United Kingdom joachim
I've made a few more tweaks to your branch.
-
joachim β
committed b1a9c705 on 4.0.x
Issue #3371474 by gcb: Fixed reverse entity reference returning badly...
-
joachim β
committed b1a9c705 on 4.0.x
- Status changed to Fixed
over 1 year ago 9:28am 6 July 2023 - π¬π§United Kingdom joachim
I can't see a block for the computed field, but I can reproduce the problem and see the fix, so committed the branch.
Thanks!
- πΊπΈUnited States gcb
Ah hell, that's bad reporting on my part. I was using it in a display mode that was being used as a rendered entity.
Maybe I'll make it up to you by figuring out how to get computed fields to show up in the field list in layout builder ;)
Thanks for the persistence and prompt review/help!
- Status changed to Active
over 1 year ago 8:29pm 7 July 2023 - π¬π§United Kingdom joachim
> Maybe I'll make it up to you by figuring out how to get computed fields to show up in the field list in layout builder ;)
Beat you to it! It was π config computed fields aren't listed in the field map Fixed :)
However, I'm going to reopen this as the fix needs to more comprehensively work for all plugin types, and there's a way to do it using the defaults that field types provide. I'm currently working on it.
- Status changed to Fixed
over 1 year ago 8:52pm 9 July 2023 - π¬π§United Kingdom joachim
Ah, I'd already filed π use $field_type_manager->getDefaultFieldSettings() in ComputedFieldBase::getFieldDefinitionSetting() Fixed to cover the more general case.
Automatically closed - issue fixed for 2 weeks with no activity.