feat: also replace % in aggregatedValues

[patrickskowronekdkfz]

General Summary

This PR replace subgroup placeholders in aggregateValues. This is needed for the catalogue of CCP-Explorer and future Projects

[timakro]

Very nice change, now the % replacement is all in one place.

This means we can also get rid of this call:

lens/packages/lib/src/cql-translator-service/ast-to-cql-translator.ts

Line 29 in 42fe931

criteria = getCriteria("diagnosis");

Maybe we could also completely get rid of getCritera() and its helper functions? That would only work though insofar nobody is using this interface:

lens/packages/lib/src/components/DataPasser.wc.svelte

Lines 69 to 71 in 42fe931

	export const getCriteriaAPI = (category: string): string[] => {
	return getCriteria(category);
	};

[patrickskowronekdkfz]

Very nice change, now the % replacement is all in one place.

This means we can also get rid of this call:

lens/packages/lib/src/cql-translator-service/ast-to-cql-translator.ts

Line 29 in 42fe931

criteria = getCriteria("diagnosis");

You're right. This is then no longer needed and was only for a workaround. I will remove it.

Maybe we could also completely get rid of getCritera() and its helper functions? That would only work though insofar nobody is using this interface:

lens/packages/lib/src/components/DataPasser.wc.svelte

Lines 69 to 71 in 42fe931

	export const getCriteriaAPI = (category: string): string[] => {
	return getCriteria(category);
	};

I would not remove it yet. This is no longer used in the core, however it might be used in a project. Therefore I would maybe mark it as deprecated.

[timakro]

Marking getCriteriaAPI as deprecated sounds good to me.