fix: filter by uniqueness after filtering by scope

* also, don't filter out external variables

Closes #54

Author: philippfromme

lib/base/VariableResolver.js

@@ -258,24 +258,32 @@ export class BaseVariableResolver {
     const root = getRootElement(bo);
     const allVariables = await this.getProcessVariables(root);
 
-    // keep only unique variables based on name property
-    const uniqueVariables = uniqueBy('name', allVariables.reverse());
-
     // (1) get variables for given scope
-    var scopeVariables = uniqueVariables.filter(function(variable) {
+    var scopeVariables = allVariables.filter(function(variable) {
       return variable.scope.id === bo.id;
     });
 
     // (2) get variables for parent scopes
     var parents = getParents(bo);
 
-    var parentsScopeVariables = uniqueVariables.filter(function(variable) {
+    var parentsScopeVariables = allVariables.filter(function(variable) {
       return parents.find(function(parent) {
         return parent.id === variable.scope.id;
       });
     });
 
-    return [ ...scopeVariables, ...parentsScopeVariables ];
+    const reversedVariables = [ ...scopeVariables, ...parentsScopeVariables ].reverse();
+
+    return reversedVariables.filter(variable => {
+
+      // if external variable, keep
+      if (variable.provider.find(extractor => extractor !== this._baseExtractor)) {
+        return true;
+      }
+
+      // if not external, keep only if first of its name
+      return reversedVariables.find(v => v.name === variable.name) === variable;
+    });
   }
 }