Skip to content

Resolve #2489476 "Pp 1 base fields"

Closes #2489476

  1. Fixes compatibility of datetime views filter plugin with base table fields.
    1. The problem was the use of FieldAPIHandlerTrait . Since the only purpose of using the trait was to load field storage definitions, I removed the trait and injected EntityTypeManager, then loaded the field storage definitions through field_storage_config.
  2. Creates datetime views field plugin.
    1. Not sure why this doesn't already exist. Why would we allow filtering by datetime but not allow displaying a datetime field? Using the date views field plugin is not an alternative, because it expects a timestamp.
    2. Is consideration for timezone necessary or will DrupalDateTime use the correct timezone?

Merge request reports

Loading