SearchKit - Allow functions in the GROUP BY clause

[colemanw]

Overview

See dev/core#2485

Before

Cannot use functions in the GROUP BY clause.

After

Comments

There's still more to do to make this really useful, like allowing that expression to be shown as a column, but this is a good step and has a test.

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[petednz]

Great to see this Group by (fx) available - are there major reasons why it won't offer 'week' (eg too hard to know what the starting day of the week should be?) - or could there be two options eg Week (Sun - Sat) | Week (Mon - Sun)

[petednz]

also - does it make sense that if i Group By "Birth Date" i can still use Field Transformation to show the date as Month Only, but that if i do this for a Custom Date field, the transformation options are much more limited (only offering Aggregate options)

[colemanw]

@petednz the "Partial Date" field transformation does offer "Weeks" as an option:

civicrm-core/Civi/Api4/Query/SqlFunctionEXTRACT.php

Line 30 in 8dbb6d3

'WEEK' => ts('Weeks'),

[petednz]

Brilliant - thx