Fix: JS Map needs a set call rather than assignment (getredash#3464)

arikfr · web-flow · commit fe10b06928a8 · 2019-02-20T12:07:46.000+02:00
diff --git a/client/app/components/DynamicComponent.jsx b/client/app/components/DynamicComponent.jsx
@@ -6,8 +6,8 @@ const componentsRegistry = new Map();
 const activeInstances = new Set();
 
 export function registerComponent(name, component) {
-  if (isString(name) && (name !== '')) {
-    componentsRegistry[name] = isFunction(component) ? component : null;
+  if (isString(name) && name !== '') {
+    componentsRegistry.set(name, isFunction(component) ? component : null);
     // Refresh active DynamicComponent instances which use this component
     activeInstances.forEach((dynamicComponent) => {
       if (dynamicComponent.props.name === name) {
