This is due to not cleaning the available list of fields.

views table display has a call to sanitize fields which we should implement.

I though this would help #1340226: Provide inheriting of method sanitize_columns for views_plugin_style

But we have to build this ourself.

Comments

clemens.tolboom’s picture

clemens.tolboom
Primary language Dutch
Location Groningen, 🇳🇱/🇪🇺
commented
Title: Removing fields from a view makes graphapi preview crash » Removing fields from a view makes graphapi (pre)view crash

Steps to reproduc:
1. create a view from nodes
2. add a relation to comments
3. set display to graphapi
4. change the settings
5. remove a field

clemens.tolboom’s picture

clemens.tolboom
Primary language Dutch
Location Groningen, 🇳🇱/🇪🇺
commented
Assigned: Unassigned » clemens.tolboom
Status: Active » Needs work

The views does not crash anymore.

Notice: Undefined index: uid in _views_graphapi_style_build_graph_data() (line 144 of /Users/clemens/Sites/opendata/dev/www/sites/default/modules/graphapi/views/theme/views_graphapi_style.theme.inc).

But this could use some better error handling.

clemens.tolboom’s picture

clemens.tolboom
Primary language Dutch
Location Groningen, 🇳🇱/🇪🇺
commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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