From bf4c8bc3a218c1ded768a28ca88aba9095fb3b09 Mon Sep 17 00:00:00 2001
From: Jan Oberhauser <jan.oberhauser@gmail.com>
Date: Sat, 4 Jan 2020 22:28:09 -0600
Subject: [PATCH] :sparkles: Make it possible to define dependencies for
 loadOptions

---
 .../src/components/ParameterInput.vue         | 24 +++++++++++++++++++
 .../src/components/ParameterInputList.vue     |  2 +-
 .../nodes-base/nodes/Coda/TableDescription.ts | 12 ++++++++++
 .../Eventbrite/EventbriteTrigger.node.ts      |  3 +++
 packages/workflow/src/Interfaces.ts           |  3 ++-
 5 files changed, 42 insertions(+), 2 deletions(-)

diff --git a/packages/editor-ui/src/components/ParameterInput.vue b/packages/editor-ui/src/components/ParameterInput.vue
index 945be0165f..d430992de8 100644
--- a/packages/editor-ui/src/components/ParameterInput.vue
+++ b/packages/editor-ui/src/components/ParameterInput.vue
@@ -116,6 +116,7 @@
 
 <script lang="ts">
 import Vue from 'vue';
+import { get } from 'lodash';
 
 import {
 	INodeUi,
@@ -206,11 +207,34 @@ export default mixins(
 			};
 		},
 		watch: {
+			dependentParametersValues () {
+				// Reload the remote parameters whenever a parameter
+				// on which the current field depends on changes
+				this.loadRemoteParameterOptions();
+			},
 			value () {
 				this.tempValue = this.displayValue as string;
 			},
 		},
 		computed: {
+			dependentParametersValues (): string | null {
+				const loadOptionsDependsOn = this.getArgument('loadOptionsDependsOn') as string[] | undefined;
+
+				if (loadOptionsDependsOn === undefined) {
+					return null;
+				}
+
+				// Get the resolved parameter values of the current node
+				const currentNodeParameters = this.$store.getters.activeNode.parameters;
+				const resolvedNodeParameters = this.getResolveNodeParameters(currentNodeParameters);
+
+				let returnValues: string[] = [];
+				for (const parameterPath of loadOptionsDependsOn) {
+					returnValues.push(get(resolvedNodeParameters, parameterPath));
+				}
+
+				return returnValues.join('|');
+			},
 			node (): INodeUi | null {
 				if (this.isCredential === true) {
 					return null;
diff --git a/packages/editor-ui/src/components/ParameterInputList.vue b/packages/editor-ui/src/components/ParameterInputList.vue
index e8c5d992a4..4bad3a3417 100644
--- a/packages/editor-ui/src/components/ParameterInputList.vue
+++ b/packages/editor-ui/src/components/ParameterInputList.vue
@@ -124,7 +124,7 @@ export default mixins(
 			getArgument (
 				argumentName: string,
 				parameter: INodeProperties,
-			): string | number | boolean | undefined {
+			): string | string[] | number | boolean | undefined{
 				if (parameter.typeOptions === undefined) {
 					return undefined;
 				}
diff --git a/packages/nodes-base/nodes/Coda/TableDescription.ts b/packages/nodes-base/nodes/Coda/TableDescription.ts
index eccecb2fe9..647d92e1cb 100644
--- a/packages/nodes-base/nodes/Coda/TableDescription.ts
+++ b/packages/nodes-base/nodes/Coda/TableDescription.ts
@@ -70,6 +70,9 @@ export const tableFields = [
 		name: 'tableId',
 		type: 'options',
 		typeOptions: {
+			loadOptionsDependsOn: [
+				'docId',
+			],
 			loadOptionsMethod: 'getTables',
 		},
 		required: true,
@@ -150,6 +153,9 @@ export const tableFields = [
 		name: 'tableId',
 		type: 'options',
 		typeOptions: {
+			loadOptionsDependsOn: [
+				'docId',
+			],
 			loadOptionsMethod: 'getTables',
 		},
 		required: true,
@@ -271,6 +277,9 @@ export const tableFields = [
 		name: 'tableId',
 		type: 'options',
 		typeOptions: {
+			loadOptionsDependsOn: [
+				'docId',
+			],
 			loadOptionsMethod: 'getTables',
 		},
 		required: true,
@@ -438,6 +447,9 @@ export const tableFields = [
 		name: 'tableId',
 		type: 'options',
 		typeOptions: {
+			loadOptionsDependsOn: [
+				'docId',
+			],
 			loadOptionsMethod: 'getTables',
 		},
 		required: true,
diff --git a/packages/nodes-base/nodes/Eventbrite/EventbriteTrigger.node.ts b/packages/nodes-base/nodes/Eventbrite/EventbriteTrigger.node.ts
index 5b25cccabd..8a13bb5fc3 100644
--- a/packages/nodes-base/nodes/Eventbrite/EventbriteTrigger.node.ts
+++ b/packages/nodes-base/nodes/Eventbrite/EventbriteTrigger.node.ts
@@ -63,6 +63,9 @@ export class EventbriteTrigger implements INodeType {
 				type: 'options',
 				required: true,
 				typeOptions: {
+					loadOptionsDependsOn: [
+						'organization',
+					],
 					loadOptionsMethod: 'getEvents'
 				},
 				default: '',
diff --git a/packages/workflow/src/Interfaces.ts b/packages/workflow/src/Interfaces.ts
index 2aa49c1acb..8fd424ba19 100644
--- a/packages/workflow/src/Interfaces.ts
+++ b/packages/workflow/src/Interfaces.ts
@@ -332,6 +332,7 @@ export type EditorTypes = 'code';
 export interface INodePropertyTypeOptions {
 	alwaysOpenEditWindow?: boolean; // Supported by: string
 	editor?: EditorTypes;        // Supported by: string
+	loadOptionsDependsOn?: string[];  // Supported by: options
 	loadOptionsMethod?: string;  // Supported by: options
 	maxValue?: number;           // Supported by: number
 	minValue?: number;           // Supported by: number
@@ -341,7 +342,7 @@ export interface INodePropertyTypeOptions {
 	numberStepSize?: number;     // Supported by: number
 	password?: boolean;          // Supported by: string
 	rows?: number;               // Supported by: string
-	[key: string]: boolean | number | string | EditorTypes | undefined;
+	[key: string]: boolean | number | string | EditorTypes | undefined | string[];
 }
 
 export interface IDisplayOptions {