inteligr8/alfresco-ng2-components
1
0
Fork 0
mirror of https://github.com/Alfresco/alfresco-ng2-components.git synced 2025-07-24 17:32:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

AAE-21967 Use column id to reference process variables (#9604)

Browse Source
This commit is contained in:
Wojciech Duda
2024-04-25 14:43:31 +02:00
committed by GitHub
parent d8861de6e0
commit 7ac7db4945
2 changed files with 22 additions and 23 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
lib/process-services-cloud/src/lib/services/variable-mapper.sevice.spec.ts
View File

@@ -32,12 +32,11 @@ describe('VariableMapperService', () => {
const viewModel = service.mapVariablesByColumnTitle([objectWithVariables], [column]); const viewModel = service.mapVariablesByColumnTitle([objectWithVariables], [column]);
expect(viewModel[0].variablesMap[column.title].type).toEqual(expectedColumnType); expect(viewModel[0].variablesMap[column.id].type).toEqual(expectedColumnType);
}; };
beforeEach(() => { beforeEach(() => {
service = new VariableMapperService(); service = new VariableMapperService();
variable = getProcessInstanceVariableMock({ variable = getProcessInstanceVariableMock({
processDefinitionKey: 'processKey', processDefinitionKey: 'processKey',
name: 'variableName' name: 'variableName'
@@ -45,6 +44,7 @@ describe('VariableMapperService', () => {
column = getDataColumnMock<ProcessListDataColumnCustomData>({ column = getDataColumnMock<ProcessListDataColumnCustomData>({
title: 'Column Name', title: 'Column Name',
id: 'columnId',
key: '', key: '',
customData: { customData: {
variableDefinitionsPayload: ['processKey/variableName'], variableDefinitionsPayload: ['processKey/variableName'],
@@ -58,11 +58,11 @@ describe('VariableMapperService', () => {
}; };
}); });
it('should map variables by column title', () => { it('should map variables by column id', () => {
const expectedObjectWithVariableMap = { const expectedObjectWithVariableMap = {
...objectWithVariables, ...objectWithVariables,
variablesMap: { variablesMap: {
[column.title]: variable [column.id]: variable
} }
}; };

17
lib/process-services-cloud/src/lib/services/variable-mapper.sevice.ts
View File

@@ -34,7 +34,8 @@ export class VariableMapperService {
return instance; return instance;
} }
const variablesMap = (instance.variables ?? []).reduce<{[columnTitle: string]: ProcessInstanceVariable}>((variableAccumulator, variable) => { const variablesMap = (instance.variables ?? []).reduce<{ [columnTitle: string]: ProcessInstanceVariable }>(
(variableAccumulator, variable) => {
const processVariableDefinitionPayload = `${variable.processDefinitionKey}/${variable.name}`; const processVariableDefinitionPayload = `${variable.processDefinitionKey}/${variable.name}`;
const column = columnsByVariables[processVariableDefinitionPayload]; const column = columnsByVariables[processVariableDefinitionPayload];
@@ -46,7 +47,9 @@ export class VariableMapperService {
} }
return variableAccumulator; return variableAccumulator;
}, {}); },
{}
);
return { return {
...instance, ...instance,
@@ -57,20 +60,16 @@ export class VariableMapperService {
return rowsViewModel; return rowsViewModel;
} }
private mapColumnKeysByVariable( private mapColumnKeysByVariable(columnsSchema: DataColumn<ProcessListDataColumnCustomData>[]): { [key: string]: string } {
columnsSchema: DataColumn<ProcessListDataColumnCustomData>[]
): { [key: string]: string } {
const columnsByVariables = columnsSchema const columnsByVariables = columnsSchema
.filter(column => !!column.customData) .filter((column) => !!column.customData)
.reduce<{ [key: string]: string }>((columnsByVariable, column) => { .reduce<{ [key: string]: string }>((columnsByVariable, column) => {
const columnTitle = column.title;
const variables = column.customData.variableDefinitionsPayload; const variables = column.customData.variableDefinitionsPayload;
variables.forEach((key) => { variables.forEach((key) => {
columnsByVariable[key] = columnTitle; columnsByVariable[key] = column.id;
}); });
return columnsByVariable; return columnsByVariable;
}, {}); }, {});
return columnsByVariables; return columnsByVariables;