Merge pull request #3 from GeekyAnts/fix/changes

Fix/changes

Author: sanketsahu

example/Plugins/ClickMePlugin.tsx

@@ -9,16 +9,15 @@ class ClickMePlugin implements IPlugin {
     console.log('Inside init');
     this.pluginStore = pluginStore;
   }
+
   activate() {
     this.pluginStore.addFunction('sendAlert', () => {
       alert('Testing');
     });
 
-    this.pluginStore.executeFunction(
-      'RendererPlugin.add',
-      'top',
-      <h1>asjdf</h1>
-    );
+    this.pluginStore.executeFunction('RendererPlugin.add', 'top', () => (
+      <h1>asdjkdas</h1>
+    ));
   }
   deactivate() {
     //

example/index.tsx

@@ -5,13 +5,13 @@ import { createPluginStore, PluginProvider, RendererPlugin } from '../.';
 import ClickMePlugin from './Plugins/ClickMePlugin';
 import Test from './components/Test';
 
-const App = () => {
-  const pluginStore = createPluginStore();
-  pluginStore.install(new RendererPlugin());
-  pluginStore.install(new ClickMePlugin());
+const pluginStore = createPluginStore();
+pluginStore.install('RendererPlugin', new RendererPlugin());
+pluginStore.install('ClickMePlugin', new ClickMePlugin());
 
+const App = () => {
   pluginStore.addFunction('test', (a, b) => {
-    console.log('working', a, b);
+    console.log('working 1', a, b);
   });
   return (
     <PluginProvider pluginStore={pluginStore}>

src/PluginStore.tsx

@@ -2,15 +2,15 @@ import { IPlugin } from './interfaces/IPlugin';
 
 export class PluginStore {
   private functionArray: Map<string, any>;
-  private pluginMap: Array<IPlugin>;
+  private pluginMap: Map<string, IPlugin>;
 
   constructor() {
     this.functionArray = new Map<string, any>();
-    this.pluginMap = [];
+    this.pluginMap = new Map<string, IPlugin>();
   }
 
-  install(plugin: IPlugin) {
-    this.pluginMap.push(plugin);
+  install(key: string, plugin: IPlugin) {
+    this.pluginMap.set(key, plugin);
     plugin.init(this);
     plugin.activate();
   }

src/plugins/RendererPlugin/components/Renderer.tsx

@@ -12,10 +12,10 @@ export const Renderer: React.SFC<{
   );
 
   return (
-    <React.Fragment>
-      {components.map((component: any) => (
-        <>{component}</>
+    <>
+      {components.map((Component: any) => (
+        <Component />
       ))}
-    </React.Fragment>
+    </>
   );
 };