Steps to reproduce:

- Create a view with field rows, not entity
- Add atleast one field
- Enable 'Use aggregation' setting for the view under advanced fieldset
- click 'Aggregation settings' link for your field
- FATAL

The form is still trying to use field_info['columns'], assuming this is an array. This is a Field config entity now, but also doesn't have a columns property. So we can use the getColumns() method instead.

Here is the fix, don't have time to do test coverage right now, but will do later.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

damiankloip’s picture

Here is a quick test for this in the UI.

Status: Needs review » Needs work

The last submitted patch, 1: 2126973-1-tests-only.patch, failed testing.

damiankloip’s picture

Status: Needs work » Needs review

Not sure, but before that would have stayed as needs review ;)

dawehner’s picture

Priority: Major » Critical
Status: Needs review » Reviewed & tested by the community

A fatal is a fatal, and should be considered critical.

+1 for the patch, just perfect.

catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

Status: Fixed » Closed (fixed)

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