- Issue created by @jurgenhaas
- @jurgenhaas opened merge request.
- Status changed to Needs review
about 2 years ago 8:53am 24 March 2023 -
matthieuscarset →
committed 79369b21 on 2.0.x
Fix processResults() get field value #3350119
-
matthieuscarset →
committed 79369b21 on 2.0.x
Thank you for reporting the issue and please forgive the inconvenience.
After a second read and more research into how
\Drupal\views\Plugin\views\field\EntityFIeld::getValue()
works, I realized theprocessResult()
method can be even simpler.I've fixed the situation in this commit on
2.0.x
.Would you be able to review and confirm the results are now processed without any exceptions?
Looking forward to reading you.
- Status changed to RTBC
about 2 years ago 4:08pm 24 March 2023 - 🇩🇪Germany jurgenhaas Gottmadingen
This works great, thanks for the very quick fix on that. I saw the processResult with nested foreach loops before too but haven't looked deeper then. Glad you addressed this here too, so that we get an even better performance.
- Status changed to Fixed
about 2 years ago 7:06pm 24 March 2023 Thanks a lot for testing and confirming the fix.
Will be shipped on next release soon.
Marking this issue as fixed for now.
Automatically closed - issue fixed for 2 weeks with no activity.