I found it was disabled, but as this field doesn't add any query, it should be allowed to be used.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hanoii created an issue. See original summary.

hanoii’s picture

Attached is a patch that enables support but make sure no aggregate function could really be used.

gg24’s picture

Hi @hanoii,

I tried the above patch for the issue mentioned. I am able to reproduce the issue and the patch applies cleanly but after applying the patch when I check the aggregation types for that field I see only Group results together in the option list, no other aggregation options are available there. Attaching a screenshot as well. Please check.

Thanks!

hanoii’s picture

Issue summary: View changes

@gg24 yes, that's on purpose. I mentioned exactly that on #2. Aggregation makes sense on SQL query, this is a compound field that pulls data from other views, so there's no true SQL aggregation function that will work here.

The patch only allows the field to be used on an aggregated query, but you can't apply any aggregation function to this field.

purushotam.rai’s picture

Status: Active » Needs review

I think @hanoii is right here, we definitely need VFV to work in case of aggregation too.

  • hanoii authored 8e9ceaf on 8.x-1.x
    Issue #2915054 by hanoii, gg24: Allow the field to be used on aggregated...
dawehner’s picture

Status: Needs review » Fixed

Thank you @hanoii, the explanation makes sense.

Status: Fixed » Closed (fixed)

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