- Issue created by @mandclu
- Status changed to Needs review
9 months ago 5:18pm 2 September 2024 - 🇨🇦Canada mandclu
Here is an MR to add Smart Date compatibility. Something to note: The intent here is to use this with view field display setting "Display all values in the same row" turned off, so that only the data required for a given time period can be retrieved and displayed. In this configuration, I see a couple of warnings when the view is first rendered:
Warning: Undefined property: Drupal\views\ViewExecutable::$row_index in Drupal\views\Plugin\views\field\EntityField->prepareItemsByDelta() (line 743 of core/modules/views/src/Plugin/views/field/EntityField.php). Warning: Undefined array key "" in Drupal\views\Plugin\views\field\EntityField->prepareItemsByDelta() (line 743 of core/modules/views/src/Plugin/views/field/EntityField.php).I suspect these errors have more to do with how this module handles multivalue fields in general, but it warrants more investigation.
- Status changed to Fixed
9 months ago 5:16pm 7 September 2024 - 🇨🇦Canada mandclu
I decided to take a simpler approach and only worry about properly handling values from Smart Date fields. The latest commit addresses that, including fields with multiple values, which in turn means that recurring events are properly handled.
Merged in, and I will open a separate issue about properly handling views without grouped rows.
Automatically closed - issue fixed for 2 weeks with no activity.