When trying to use display suites 'Views Display' option for a view based on civicrm events the nid field was added as being the base_field for the civicrm_event table.
This table uses 'id' as the base_field.

I have attached a patch to add this base_table/base_field pair to views_plugin_ds_fields_view.inc. CiviCRM uses a lot more tables all with other base_fields, so this patch is incomplete for a complete CiviCRM integration.

But the logic in display suite itself should really be changed here so that these things are not hard coded.

CommentFileSizeAuthor
ds-patch-civicrm-integration.patch502 bytesweseze

Comments

weseze’s picture

weseze commented

#1839254: Entity Construction Kit compatibility for ds_views fixes a similar problem but with other undefined tables.

swentel’s picture

swentel commented
Title: Views integration for CiviCRM bug (base tables/keys not defined) » Fix base table and base id support in views (entities & fields)

I think I know a solution and that is during a full flush cache (or another event, rather hook_modules_enabled/disabled) getting the views base tables and storing that in a variable. I'll figure that out next week.

The patch will indeed fix support for CiviCRM but it's just waiting until a new base table arrives to add it.

I'm changing the title so we make sure this is fixed once and for all :)

swentel’s picture

swentel commented

Committed the civicrm patch for now, will see in the future how to solve this generally.

swentel’s picture

swentel commented
Category: bug » task
swentel’s picture

swentel commented
Status: Active » Closed (works as designed)

This is never going to be abstract anyway for D7.

swentel’s picture

swentel commented
Issue summary: View changes

Added filename and fixed typo.