mudhorn/n8n
1
0
Fork 0
mirror of https://github.com/n8n-io/n8n.git synced 2024-09-20 14:57:31 -07:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

perf(editor): Memoize locale translate calls during actions generation (#6773)

Browse source 
performance(editor): Memoize locale translate calls during actions generation

Signed-off-by: Oleg Ivaniv <me@olegivaniv.com>
This commit is contained in:
OlegIvaniv 2023-07-28 10:22:30 +02:00 committed by GitHub
parent dd6a4c956a
commit 2d47e8dc4a
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 15 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

24
packages/editor-ui/src/components/Node/NodeCreator/composables/useActionsGeneration.ts
View file

@ -1,11 +1,10 @@
import { startCase } from 'lodash-es';
import { memoize, startCase } from 'lodash-es';
import type {
INodePropertyCollection,
INodePropertyOptions,
INodeProperties,
INodeTypeDescription,
} from 'n8n-workflow';
import { deepCopy } from 'n8n-workflow';
import { CUSTOM_API_CALL_KEY } from '@/constants';
import type { ActionTypeDescription, SimplifiedNodeType, ActionsRecord } from '@/Interface';
@ -13,6 +12,13 @@ import { i18n } from '@/plugins/i18n';
const PLACEHOLDER_RECOMMENDED_ACTION_KEY = 'placeholder_recommended';
function translate(...args: Parameters<typeof i18n.baseText>) {
return i18n.baseText(...args);
}
// Memoize the translation function so we don't have to re-translate the same string
// multiple times when generating the actions
const cachedBaseText = memoize(translate, (...args) => JSON.stringify(args));
const customNodeActionsParsers: {
[key: string]: (
matchedProperty: INodeProperties,
@ -26,7 +32,7 @@ const customNodeActionsParsers: {
(categoryItem): ActionTypeDescription => ({
...getNodeTypeBase(nodeTypeDescription),
actionKey: categoryItem.value as string,
displayName: i18n.baseText('nodeCreator.actionsCategory.onEvent', {
displayName: cachedBaseText('nodeCreator.actionsCategory.onEvent', {
interpolate: { event: startCase(categoryItem.name) },
}),
description: categoryItem.description || '',
@ -40,8 +46,8 @@ const customNodeActionsParsers: {
function getNodeTypeBase(nodeTypeDescription: INodeTypeDescription, label?: string) {
const isTrigger = nodeTypeDescription.group.includes('trigger');
const category = isTrigger
? i18n.baseText('nodeCreator.actionsCategory.triggers')
: i18n.baseText('nodeCreator.actionsCategory.actions');
? cachedBaseText('nodeCreator.actionsCategory.triggers')
: cachedBaseText('nodeCreator.actionsCategory.actions');
return {
name: nodeTypeDescription.name,
group: nodeTypeDescription.group,
@ -101,7 +107,7 @@ function triggersCategory(nodeTypeDescription: INodeTypeDescription): ActionType
{
...getNodeTypeBase(nodeTypeDescription),
actionKey: PLACEHOLDER_RECOMMENDED_ACTION_KEY,
displayName: i18n.baseText('nodeCreator.actionsCategory.onNewEvent', {
displayName: cachedBaseText('nodeCreator.actionsCategory.onNewEvent', {
interpolate: { event: nodeTypeDescription.displayName.replace('Trigger', '').trimEnd() },
}),
description: '',
@ -125,7 +131,7 @@ function triggersCategory(nodeTypeDescription: INodeTypeDescription): ActionType
actionKey: categoryItem.value as string,
displayName:
categoryItem.action ??
i18n.baseText('nodeCreator.actionsCategory.onEvent', {
cachedBaseText('nodeCreator.actionsCategory.onEvent', {
interpolate: { event: startCase(categoryItem.name) },
}),
description: categoryItem.description || '',
@ -175,7 +181,7 @@ function resourceCategories(nodeTypeDescription: INodeTypeDescription): ActionTy
return {
...getNodeTypeBase(
nodeTypeDescription,
`${resourceOption.name} ${i18n.baseText('nodeCreator.actionsCategory.actions')}`,
`${resourceOption.name} ${cachedBaseText('nodeCreator.actionsCategory.actions')}`,
),
actionKey: operationOption.value as string,
description: operationOption?.description ?? '',
@ -234,7 +240,7 @@ export function useActionsGenerator() {
}
function generateMergedNodesAndActions(nodeTypes: INodeTypeDescription[]) {
const visibleNodeTypes = deepCopy(nodeTypes);
const visibleNodeTypes = [...nodeTypes];
const actions: ActionsRecord<typeof mergedNodes> = {};
const mergedNodes: SimplifiedNodeType[] = [];