fix!: include unscoped variables in getVariables

Author: Buckwich

lib/base/VariableResolver.js

@@ -16,22 +16,21 @@ import { getParents } from './util/elementsUtil';
 /**
  * @typedef {AdditionalVariable} ProcessVariable
  * @property {Array<ModdleElement>} origin
- * @property {ModdleElement} scope
+ * @property {ModdleElement} [scope]
  * @property {Array<Object>} provider
  */
 
 /**
  * Base Class that handles additional variable extractors, variable parsing and caching.
  */
 export class BaseVariableResolver {
-
   constructor(eventBus, bpmnjs) {
     this.providers = [];
     this._eventBus = eventBus;
     this._bpmnjs = bpmnjs;
 
     this.rawVariables = new CachedValue(this._generateRawVariables.bind(this));
-    this._allParsedVariables = new CachedValue(async () => {
+    this.parsedVariables = new CachedValue(async () => {
 
       const rawVariables = await this.getRawVariables();
       const context = { variables: rawVariables };
@@ -40,17 +39,6 @@ export class BaseVariableResolver {
 
       return context.variables;
     });
-    this.parsedVariables = new CachedValue(async () => {
-      const allParsed = await this._allParsedVariables.get();
-
-      // Filter out variables with no scope (e.g. consumed variable markers)
-      const filtered = {};
-      for (const key in allParsed) {
-        filtered[key] = allParsed[key].filter(v => v.scope);
-      }
-
-      return filtered;
-    });
 
     eventBus.on([ 'commandStack.changed', 'diagram.clear', 'import.done', 'variables.changed' ], () => {
       this.invalidateCache();
@@ -112,7 +100,6 @@ export class BaseVariableResolver {
    */
   invalidateCache() {
     this.rawVariables.invalidate();
-    this._allParsedVariables.invalidate();
     this.parsedVariables.invalidate();
   }
 
@@ -338,11 +325,7 @@ export class BaseVariableResolver {
    * @returns {Promise<Array<AvailableVariable>>}
    */
   async getConsumedVariablesForElement(element) {
-    const allVariablesByRoot = await this._allParsedVariables.get()
-      .catch(() => {
-        return {};
-      });
-
+    const allVariablesByRoot = await this.parsedVariables.get();
     const allVariables = Object.values(allVariablesByRoot).flat();
 
     return allVariables.filter(v =>