Merge pull request #1329 from opencomponents/check-exports-data

check for exports data too

Author: ricardo-devis-agullo

src/registry/routes/helpers/get-component.ts

@@ -523,6 +523,7 @@ export default function getComponent(conf: Config, repository: Repository) {
                     module: {
                       exports: {} as Record<string, (...args: any[]) => any>
                     },
+                    exports: {} as Record<string, (...args: any[]) => any>,
                     console: conf.local ? console : noopConsole,
                     setTimeout,
                     Buffer
@@ -559,7 +560,8 @@ export default function getComponent(conf: Config, repository: Repository) {
 
                   try {
                     vm.runInNewContext(dataProvider.content, context, options);
-                    const processData = context.module.exports['data'];
+                    const processData =
+                      context.module.exports['data'] || context.exports['data'];
                     cache.set('file-contents', cacheKey, processData);
 
                     domain.on('error', handleError);

test/fixtures/mocked-components/plugin.js

@@ -21,7 +21,7 @@ module.exports = {
       }
     }
   },
-  data: '"use strict";module.exports.data=function(t,n){n(null,{a:t.plugins.doSomething()})};',
+  data: '"use strict";exports.data=function(t,n){n(null,{a:t.plugins.doSomething()})};',
   view:
     'var oc=oc||{};oc.components=oc.components||{},oc.components["8b3650989d66345eea8152e89ec03e1dad8e8e9b"]' +
     '=function(e){var n,o=[],c=e||{};return function(e){o.push("<div>"+jade.escape(null==(n=e)?"":n)+" John  </div>")}.' +