Problem/Motivation

When I add the last_updated field, Config Inspector reports schema errors

display.default.display_options.fields.last_updated.date_format
display.default.display_options.fields.last_updated.custom_date_format
display.default.display_options.fields.last_updated.timezone

Steps to reproduce

1. Create a view, for example use nodes
2. Add the field "Comment Statistics: Updated/commented date"
3. Save the view
4. In the Config Inspector module, see schema errors for this view

Proposed resolution

Change views_field to views.field.date in the comment.views.schema.yml file

Issue fork drupal-3550095

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

ivnish created an issue. See original summary.

ivnish’s picture

Assigned: ivnish » Unassigned
Status: Active » Needs review
smustgrave’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: +Needs Review Queue Initiative

Makes sense, it's what the rest of the date schemas are too.

astonvictor’s picture

+1 RTBC

xjm’s picture

I am trying to understand how this has been wrong for 11 years.

Is there a way we can validate the whole file?

catch’s picture

Version: 11.x-dev » 11.3.x-dev
Status: Reviewed & tested by the community » Fixed

We don't have a way to validate config schema files, it would need some kind of config schema schema (not sure if I am half joking here or not). We can validate config against the schema and that will tell us if there's a mis-match but it could be either the config or schema that is wrong.

Might be something we're able to do eventually but I think we should just go ahead here.

Committed/pushed to 11.x and cherry-picked to 11.3.x, thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

  • catch committed b983151e on 11.3.x
    Issue #3550095 by ivnish, xjm: Fix views.field.comment_ces_last_updated...

  • catch committed c29c62b2 on 11.x
    Issue #3550095 by ivnish, xjm: Fix views.field.comment_ces_last_updated...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.