binhend 2.1.9 → 2.1.11

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "binhend",
-  "version": "2.1.9",
+  "version": "2.1.11",
   "description": "",
   "main": "index.js",
   "author": "Nguyen Duc Binh",

package/src/web/code.js

@@ -1,5 +1,6 @@
 
 const path = require('path');
+const Component = require('./component');
 
 function getCodeOfDependencies(component, metadata) {
     var global = metadata || {};
@@ -54,7 +55,10 @@ function getDependencyDelaration(component) {
 
     for (var filePath in dependencies) {
         var { name, url, type } = dependencies[filePath];
-        if (name) {
+        if (name && type === 'service') {
+            codes.push(` ${name} = this.${type}('${url}')`);
+        }
+        else if (name) {
             codes.push(` ${name} = Binh.${type}('${url}')`);
         }
     }
@@ -92,20 +96,32 @@ function getMetadataOfDependencies(component) {
     return component.dependencies;
 }
 
-function generateOptionCode(component, varname) {
-    var options = component.options, urls = [];
+function generateOptionCode(component, baseCode) {
+    var options = component.options;
+
+    if (!options) return ''; // TODO remove this line, seem like always have this object 'options'
 
-    if (!options) return '';
+    var cssUrls = [], serviceUrls = [];
 
+    // CSS
     component.type === 'ui' && options.csses.forEach(function(cssModule) {
-        urls.push(JSON.stringify(cssModule.url));
+        cssUrls.push(JSON.stringify(cssModule.url));
+    });
+
+    if (cssUrls.length) {
+        baseCode += `.style([${cssUrls.join(',')}])`;
+    }
+
+    // Services
+    if (Component.isLazy) options.services.forEach(function(serviceModule) {
+        serviceUrls.push(serviceModule.url);
     });
 
-    if (urls.length) {
-        return `${varname}.style([${urls.join(',')}])`;
+    if (serviceUrls.length) {
+        baseCode += `.service(['${serviceUrls.join("', '")}'])`;
     }
 
-    return '';
+    return baseCode;
 }
 
 function getRelativeFilePath(filePath, rootPath) {

package/src/web/component.build.js

@@ -70,6 +70,7 @@ function lazyload(paths) {
     processEachFile(paths, buildLazyCode, copyExternalComponentFiles);
 }
 
+/** case of components or modules are required from node_modules installed by NPM */
 function copyExternalComponentFiles({ outputRootPath }) {
     var cache = copyExternalComponentFiles.cache,
         csses = copyExternalComponentFiles.csses;

package/src/web/component.method.js

@@ -81,7 +81,7 @@ binh.component = function(context, ui, service, style) {
     component.htmltags = [];
     component.svgtags = [];
     component.links = [];
-    component.options = { csses: [] };
+    component.options = { csses: [], services: [] };
     component.vars = {};
     component.varname = getVariableName(component.filename);
     component.as = alias;
@@ -98,6 +98,10 @@ binh.final = function(module) {
             component.vars[childComponent.filename] = childComponent.alias;
             delete childComponent.alias;
         }
+
+        if (childComponent.type === 'service') {
+            component.options.services.push(childComponent);
+        }
     });
 };
 

package/src/web/index.js

@@ -21,10 +21,12 @@ function WebBuilder(source, { output, external } = {}) {
     var external = mustString(external) || 'NPM';
 
     this.bundle = () => {
+        Component.isLazy = false;
         ComponentFormat.generate(source, module, () => ComponentBuild.bundle({ source, web, module }));
     };
 
     this.lazy = () => {
+        Component.isLazy = true;
         ComponentFormat.generate(source, module, () => ComponentBuild.lazyload({ source, web, module, external }));
     };