Date's date_views 2.11 breaks exposed form HTML id's and classes

Created on 15 April 2021, over 3 years ago
Updated 8 February 2023, almost 2 years ago

The following lines addition in the date_views module:

cause the exposed filters element wrapper to have useless classes.

Steps to reproduce correct behaviour using Date 2.10:

1. Create a view with exposed date field
2. Assign custom identifier to the exposed date field (e.g. 'my-date').
3. Views generated HTML will use this id for form field and for the wrapper div ('edit-my-date', etc). You can now use this for styling and js.

Steps to reproduce incorrect behaviour using Date 2.11:

1. Create a view with exposed date field
2. Assign custom identifier to the exposed date field (e.g. 'my-date').
3. Views generated HTML will use uncontrollable (and randomized) id for both wrapper div and form field, as caused by the line 327

πŸ› Bug report
Status

Active

Version

2.0

Component

Views Filter

Created by

πŸ‡΅πŸ‡±Poland pawel.traczynski Warsaw

Live updates comments and jobs are added and updated live.
  • Needs tests

    The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024