Issue #3335932: Chart type views field as exposed filter identifier

Open on Drupal.org →

Created on 23 January 2023, almost 2 years ago

Problem/Motivation

The exposed chart type views field expose a form input to change the chart type. The exposed input identifier is hardcoded as "ct" but not set as one of the defined options.

Steps to reproduce

Add the exposed chart type field and then install a module like better exposed filter along with the patch at https://www.drupal.org/project/better_exposed_filters/issues/2996297 🐛 Reset button does not use AJAX Needs work that expects exposed inputs to have "identifier" and you start to get php errors.

Proposed resolution

Provide a default define options for expose identifier.

Remaining tasks

Patch or merge request

User interface changes

N/A

API changes

N/A

Data model changes

N/A

🐛 Bug report

Status

Fixed

Version

5.0

Component

Views integration

Created by

🇨🇦Canada nikathone Ontario

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Comments & Activities

Issue created by @nikathone
@nikathone opened merge request.
Issue was unassigned.
Status changed to Needs review almost 2 years ago7:50pm 23 January 2023
Comment almost 2 years ago →
🇨🇦Canada nikathone Ontario
Made an initial commit that I think should be enough.
First commit to issue fork.
Comment almost 2 years ago →
System Message

andileco → committed 92a9127d on 5.0.x authored by nikathone →
Issue #3335932 by nikathone: Issue #3335932: Chart type views field as...
Status changed to Fixed almost 2 years ago8:28pm 23 January 2023
Comment almost 2 years ago →
🇺🇸United States andileco
Thank you!
Comment almost 2 years ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024