👕 Apply node-param-default-wrong-for-multi-options (#3185)

2025-03-05 20:50:17 -08:00 · 2022-04-22 13:11:40 +02:00 · 2022-04-22 13:11:40 +02:00 · ffc1c05ef4
parent c67d05724a
commit ffc1c05ef4
24 changed files with 41 additions and 40 deletions
--- a/.eslintrc.js
+++ b/.eslintrc.js
@ -383,6 +383,7 @@ module.exports = {
 				"n8n-nodes-base/node-param-default-wrong-for-collection": "error",
 				"n8n-nodes-base/node-param-default-wrong-for-boolean": "error",
 				"n8n-nodes-base/node-param-default-wrong-for-fixed-collection": "error",
+				"n8n-nodes-base/node-param-default-wrong-for-multi-options": "error",
 			 }
 		},
 	],
--- a/packages/nodes-base/nodes/ERPNext/DocumentDescription.ts
+++ b/packages/nodes-base/nodes/ERPNext/DocumentDescription.ts
@ -135,7 +135,7 @@ export const documentFields: INodeProperties[] = [
 						'docType',
 					],
 				},
-				default: '',
+				default: [],
 				description: 'Comma-separated list of fields to return.',
 				placeholder: 'name,country',
 			},
--- a/packages/nodes-base/nodes/FormIo/FormIoTrigger.node.ts
+++ b/packages/nodes-base/nodes/FormIo/FormIoTrigger.node.ts
@ -84,7 +84,7 @@ export class FormIoTrigger implements INodeType {
 					},
 				],
 				required: true,
-				default: '',
+				default: [],
 			},
 			{
 				displayName: 'Simplify Response',
--- a/packages/nodes-base/nodes/GetResponse/ContactDescription.ts
+++ b/packages/nodes-base/nodes/GetResponse/ContactDescription.ts
@ -182,7 +182,7 @@ export const contactFields: INodeProperties[] = [
 				typeOptions: {
 					loadOptionsMethod: 'getTags',
 				},
-				default: '',
+				default: [],
 			},
 		],
 	},
@ -637,7 +637,7 @@ export const contactFields: INodeProperties[] = [
 				typeOptions: {
 					loadOptionsMethod: 'getTags',
 				},
-				default: '',
+				default: [],
 			},
 		],
 	},
--- a/packages/nodes-base/nodes/Google/Contacts/ContactDescription.ts
+++ b/packages/nodes-base/nodes/Google/Contacts/ContactDescription.ts
@ -704,7 +704,7 @@ export const contactFields: INodeProperties[] = [
 				],
 			},
 		},
-		default: '',
+		default: [],
 		description: 'A field mask to restrict which fields on each person are returned. Multiple fields can be specified by separating them with commas.',
 	},
 	{
@ -884,7 +884,7 @@ export const contactFields: INodeProperties[] = [
 				],
 			},
 		},
-		default: '',
+		default: [],
 		description: 'A field mask to restrict which fields on each person are returned. Multiple fields can be specified by separating them with commas.',
 	},
 	{
@ -1128,7 +1128,7 @@ export const contactFields: INodeProperties[] = [
 				],
 			},
 		},
-		default: '',
+		default: [],
 		description: 'A field mask to restrict which fields on each person are returned. Multiple fields can be specified by separating them with commas.',
 	},
 	{
--- a/packages/nodes-base/nodes/Google/Gmail/MessageLabelDescription.ts
+++ b/packages/nodes-base/nodes/Google/Gmail/MessageLabelDescription.ts
@ -59,7 +59,7 @@ export const messageLabelFields: INodeProperties[] = [
 		typeOptions: {
 			loadOptionsMethod: 'getLabels',
 		},
-		default: '',
+		default: [],
 		required: true,
 		displayOptions: {
 			show: {
--- a/packages/nodes-base/nodes/HackerNews/HackerNews.node.ts
+++ b/packages/nodes-base/nodes/HackerNews/HackerNews.node.ts
@ -284,7 +284,7 @@ export class HackerNews implements INodeType {
 								description: 'Returns query results filtered by Front Page tag',
 							},
 						],
-						default: '',
+						default: [],
 						description: 'Tags for filtering the results of the query.',
 					},
 				],
--- a/packages/nodes-base/nodes/Hubspot/CompanyDescription.ts
+++ b/packages/nodes-base/nodes/Hubspot/CompanyDescription.ts
@ -895,7 +895,7 @@ export const companyFields: INodeProperties[] = [
 				typeOptions: {
 					loadOptionsMethod: 'getCompanyProperties',
 				},
-				default: '',
+				default: [],
 				description: `<p>Used to include specific company properties in the results. By default, the results will only include company ID and will not include the values for any properties for your companies.</p><p>Including this parameter will include the data for the specified property in the results. You can include this parameter multiple times to request multiple properties separated by a comma: <code>,</code>.</p>`,
 			},
 			{
@ -1095,7 +1095,7 @@ export const companyFields: INodeProperties[] = [
 				typeOptions: {
 					loadOptionsMethod: 'getCompanyProperties',
 				},
-				default: '',
+				default: [],
 				description: `<p>Used to include specific company properties in the results. By default, the results will only include company ID and will not include the values for any properties for your company.</p><p>Including this parameter will include the data for the specified property in the results. You can include this parameter multiple times to request multiple properties separated by a comma: <code>,</code>.</p>`,
 			},
 		],
--- a/packages/nodes-base/nodes/Hubspot/ContactDescription.ts
+++ b/packages/nodes-base/nodes/Hubspot/ContactDescription.ts
@ -396,7 +396,7 @@ export const contactFields: INodeProperties[] = [
 						],
 					},
 				},
-				default: '',
+				default: [],
 				description: `<p>Used to include specific company properties in the results. By default, the results will only include company ID and will not include the values for any properties for your company.</p><p>Including this parameter will include the data for the specified property in the results. You can include this parameter multiple times to request multiple properties separated by a comma: <code>,</code>.</p>`,
 			},
 			{
@ -576,7 +576,7 @@ export const contactFields: INodeProperties[] = [
 				typeOptions: {
 					loadOptionsMethod: 'getContactProperties',
 				},
-				default: '',
+				default: [],
 				description: `<p>Used to include specific company properties in the results. By default, the results will only include company ID and will not include the values for any properties for your company.</p><p>Including this parameter will include the data for the specified property in the results. You can include this parameter multiple times to request multiple properties separated by a comma: <code>,</code>.</p>`,
 			},
 			{
@ -699,7 +699,7 @@ export const contactFields: INodeProperties[] = [
 				typeOptions: {
 					loadOptionsMethod: 'getContactProperties',
 				},
-				default: '',
+				default: [],
 				description: `<p>Used to include specific company properties in the results. By default, the results will only include company ID and will not include the values for any properties for your company.</p><p>Including this parameter will include the data for the specified property in the results. You can include this parameter multiple times to request multiple properties separated by a comma: <code>,</code>.</p>`,
 			},
 			{
@ -844,7 +844,7 @@ export const contactFields: INodeProperties[] = [
 				typeOptions: {
 					loadOptionsMethod: 'getContactProperties',
 				},
-				default: '',
+				default: [],
 				description: `<p>Used to include specific company properties in the results. By default, the results will only include company ID and will not include the values for any properties for your company.</p><p>Including this parameter will include the data for the specified property in the results. You can include this parameter multiple times to request multiple properties separated by a comma: <code>,</code>.</p>`,
 			},
 			{
--- a/packages/nodes-base/nodes/Jira/IssueDescription.ts
+++ b/packages/nodes-base/nodes/Jira/IssueDescription.ts
@ -679,7 +679,7 @@ export const issueFields: INodeProperties[] = [
 				displayName: 'Expand',
 				name: 'expand',
 				type: 'multiOptions',
-				default: '',
+				default: [],
 				options: [
 					{
 						name: 'Changelog',
--- a/packages/nodes-base/nodes/Mautic/ContactDescription.ts
+++ b/packages/nodes-base/nodes/Mautic/ContactDescription.ts
@ -438,7 +438,7 @@ export const contactFields: INodeProperties[] = [
 				typeOptions: {
 					loadOptionsMethod: 'getTags',
 				},
-				default: '',
+				default: [],
 			},
 			{
 				displayName: 'Social Media',
@ -937,7 +937,7 @@ export const contactFields: INodeProperties[] = [
 				typeOptions: {
 					loadOptionsMethod: 'getTags',
 				},
-				default: '',
+				default: [],
 			},
 			{
 				displayName: 'Title',
--- a/packages/nodes-base/nodes/Microsoft/Dynamics/descriptions/AccountDescription.ts
+++ b/packages/nodes-base/nodes/Microsoft/Dynamics/descriptions/AccountDescription.ts
@ -178,7 +178,7 @@ export const accountFields: INodeProperties[] = [
 				typeOptions: {
 					loadOptionsMethod: 'getAccountFields',
 				},
-				default: '',
+				default: [],
 			},
 			{
 				displayName: 'Expand Fields',
@ -187,7 +187,7 @@ export const accountFields: INodeProperties[] = [
 				typeOptions: {
 					loadOptionsMethod: 'getExpandableAccountFields',
 				},
-				default: '',
+				default: [],
 			},
 		],
 	},
@ -266,7 +266,7 @@ export const accountFields: INodeProperties[] = [
 				typeOptions: {
 					loadOptionsMethod: 'getAccountFields',
 				},
-				default: '',
+				default: [],
 				description: 'Fields the response will include',
 			},
 		],
--- a/packages/nodes-base/nodes/Paddle/CouponDescription.ts
+++ b/packages/nodes-base/nodes/Paddle/CouponDescription.ts
@ -93,7 +93,7 @@ export const couponFields: INodeProperties[] = [
 				],
 			},
 		},
-		default: '',
+		default: [],
 		description: 'Comma-separated list of product IDs. Required if coupon_type is product.',
 		required: true,
 	},
--- a/packages/nodes-base/nodes/PagerDuty/IncidentDescription.ts
+++ b/packages/nodes-base/nodes/PagerDuty/IncidentDescription.ts
@ -370,7 +370,7 @@ export const incidentFields: INodeProperties[] = [
 				typeOptions: {
 					loadOptionsMethod: 'getServices',
 				},
-				default: '',
+				default: [],
 				description: 'Returns only the incidents associated with the passed service(s).',
 			},
 			{
@ -406,7 +406,7 @@ export const incidentFields: INodeProperties[] = [
 						value: 'triggered',
 					},
 				],
-				default: '',
+				default: [],
 				description: 'Returns only the incidents associated with the passed service(s).',
 			},
 			{
@ -447,7 +447,7 @@ export const incidentFields: INodeProperties[] = [
 						value: 'low',
 					},
 				],
-				default: '',
+				default: [],
 				description: 'urgencies of the incidents to be returned. Defaults to all urgencies. Account must have the urgencies ability to do this',
 			},
 			{
--- a/packages/nodes-base/nodes/ProfitWell/MetricDescription.ts
+++ b/packages/nodes-base/nodes/ProfitWell/MetricDescription.ts
@ -215,7 +215,7 @@ export const metricFields: INodeProperties[] = [
 						description: `How much upgrades and plan length increases affect your MRR`,
 					},
 				],
-				default: '',
+				default: [],
 				description: 'Comma-separated list of metric trends to return (the default is to return all metric)',
 			},
 			{
@ -431,7 +431,7 @@ export const metricFields: INodeProperties[] = [
 						description: `Net change in revenue for this plan`,
 					},
 				],
-				default: '',
+				default: [],
 				description: 'Comma-separated list of metric trends to return (the default is to return all metric)',
 			},
 		],
--- a/packages/nodes-base/nodes/Pushcut/Pushcut.node.ts
+++ b/packages/nodes-base/nodes/Pushcut/Pushcut.node.ts
@ -113,7 +113,7 @@ export class Pushcut implements INodeType {
 						typeOptions: {
 							loadOptionsMethod: 'getDevices',
 						},
-						default: '',
+						default: [],
 						description: 'List of devices this notification is sent to. (default is all devices)',
 					},
 					{
--- a/packages/nodes-base/nodes/QuickBooks/descriptions/Transaction/TransactionDescription.ts
+++ b/packages/nodes-base/nodes/QuickBooks/descriptions/Transaction/TransactionDescription.ts
@ -102,7 +102,7 @@ export const transactionFields: INodeProperties[] = [
 				displayName: 'Columns',
 				name: 'columns',
 				type: 'multiOptions',
-				default: '',
+				default: [],
 				description: 'Columns to return',
 				options: TRANSACTION_REPORT_COLUMNS,
 			},
--- a/packages/nodes-base/nodes/Salesforce/CustomObjectDescription.ts
+++ b/packages/nodes-base/nodes/Salesforce/CustomObjectDescription.ts
@ -507,7 +507,7 @@ export const customObjectFields: INodeProperties[] = [
 						'customObject',
 					],
 				},
-				default: '',
+				default: [],
 				description: 'Fields to include separated by ,',
 			},
 		],
--- a/packages/nodes-base/nodes/SendGrid/ContactDescription.ts
+++ b/packages/nodes-base/nodes/SendGrid/ContactDescription.ts
@ -242,7 +242,7 @@ export const contactFields: INodeProperties[] = [
 								typeOptions: {
 									loadOptionsMethod: 'getListIds',
 								},
-								default: '',
+								default: [],
 								description: 'ID of the field to set.',
 							},
 						],
--- a/packages/nodes-base/nodes/SentryIo/ReleaseDescription.ts
+++ b/packages/nodes-base/nodes/SentryIo/ReleaseDescription.ts
@ -251,7 +251,7 @@ export const releaseFields: INodeProperties[] = [
 		typeOptions: {
 			loadOptionsMethod: 'getProjects',
 		},
-		default: '',
+		default: [],
 		displayOptions: {
 			show: {
 				resource: [
--- a/packages/nodes-base/nodes/Slack/ChannelDescription.ts
+++ b/packages/nodes-base/nodes/Slack/ChannelDescription.ts
@ -238,7 +238,7 @@ export const channelFields: INodeProperties[] = [
 		typeOptions: {
 			loadOptionsMethod: 'getUsers',
 		},
-		default: '',
+		default: [],
 		displayOptions: {
 			show: {
 				operation: [
--- a/packages/nodes-base/nodes/UptimeRobot/MonitorDescription.ts
+++ b/packages/nodes-base/nodes/UptimeRobot/MonitorDescription.ts
@ -263,7 +263,7 @@ export const monitorFields: INodeProperties[] = [
 				displayName: 'Statuses',
 				name: 'statuses',
 				type: 'multiOptions',
-				default: '',
+				default: [],
 				options: [
 					{
 						name: 'Paused',
@ -291,7 +291,7 @@ export const monitorFields: INodeProperties[] = [
 				displayName: 'Types',
 				name: 'types',
 				type: 'multiOptions',
-				default: '',
+				default: [],
 				options: [
 					{
 						name: 'Heartbeat',
--- a/packages/nodes-base/nodes/Wekan/CardDescription.ts
+++ b/packages/nodes-base/nodes/Wekan/CardDescription.ts
@ -186,7 +186,7 @@ export const cardFields: INodeProperties[] = [
 				typeOptions: {
 					loadOptionsMethod: 'getUsers',
 				},
-				default: '',
+				default: [],
 				description: 'The new list of assignee IDs attached to the card.',
 			},
 			{
@ -203,7 +203,7 @@ export const cardFields: INodeProperties[] = [
 				typeOptions: {
 					loadOptionsMethod: 'getUsers',
 				},
-				default: '',
+				default: [],
 				description: 'The new list of member IDs attached to the card.',
 			},
 		],
@ -604,7 +604,7 @@ export const cardFields: INodeProperties[] = [
 				typeOptions: {
 					loadOptionsMethod: 'getUsers',
 				},
-				default: '',
+				default: [],
 				description: 'The new list of assignee IDs attached to the card.',
 			},
 			{
@ -764,7 +764,7 @@ export const cardFields: INodeProperties[] = [
 				typeOptions: {
 					loadOptionsMethod: 'getUsers',
 				},
-				default: '',
+				default: [],
 				description: 'The new list of member IDs attached to the card.',
 			},
 			{
--- a/packages/nodes-base/nodes/Zulip/MessageDescription.ts
+++ b/packages/nodes-base/nodes/Zulip/MessageDescription.ts
@ -62,7 +62,7 @@ export const messageFields: INodeProperties[] = [
 			loadOptionsMethod: 'getUsers',
 		},
 		required: true,
-		default: '',
+		default: [],
 		displayOptions: {
 			show: {
 				resource: [