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 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

Issue fork charts-3335932

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:

About issue forks

Comments

nikathone created an issue. See original summary.

nikathone opened merge request !50

nikathone’s picture

nikathone
Primary language Swahili
Location Ontario
commented
Title: Chart type views field as exposed filter identifier » Issue #3335932: Chart type views field as exposed filter identifier
Assigned: nikathone » Unassigned
Status: Active » Needs review

Made an initial commit that I think should be enough.

andileco made their first commit to this issue’s fork.

  • andileco committed 92a9127d on 5.0.x authored by nikathone 
    Issue #3335932 by nikathone: Issue #3335932: Chart type views field as...
andileco’s picture

andileco commented
Status: Needs review » Fixed

Thank you!

Status: Fixed » Closed (fixed)

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