From d0d49d98f02e23dd4e53b3ccbfdc6b4180aedb93 Mon Sep 17 00:00:00 2001
From: Halil ibrahim Kalkan <hi_kalkan@yahoo.com>
Date: Sun, 17 Jun 2018 17:54:26 +0300
Subject: [PATCH] Automatically discover resource mapping dependencies.

---
 .../aspnetcore.mvc.ui/gulp/copy-resources.js  | 56 +++++++++++++------
 1 file changed, 38 insertions(+), 18 deletions(-)

diff --git a/npm/packs/aspnetcore.mvc.ui/gulp/copy-resources.js b/npm/packs/aspnetcore.mvc.ui/gulp/copy-resources.js
index 2f8f048a3d..dbedeeaa8e 100644
--- a/npm/packs/aspnetcore.mvc.ui/gulp/copy-resources.js
+++ b/npm/packs/aspnetcore.mvc.ui/gulp/copy-resources.js
@@ -10,8 +10,10 @@
         path = require("path"),
         extendObject = require('extend-object');
 
-    function init(rootMappingFilePath) {
-        var resourceMapping = buildResourceMapping(rootMappingFilePath);
+    function init(rootPath) {
+        var investigatedPackagePaths = {};
+
+        var resourceMapping = buildResourceMapping(rootPath);
 
         function replaceAliases(text) {
             if (!resourceMapping.aliases) {
@@ -33,6 +35,15 @@
             return text.replace(new RegExp(search, 'g'), replacement);
         }
 
+        function requireOptional(filePath) {
+            //TODO: Implement this using a library instead of try-catch!
+            try {
+                return require(filePath);
+            } catch (e) {
+                return undefined;
+            } 
+        }
+        
         function cleanFiles() {
             if (resourceMapping.clean) {
                 for (var i = 0; i < resourceMapping.clean.length; i++) {
@@ -41,20 +52,27 @@
             }
         }
 
-        function buildResourceMapping(mappingFilePath) {
-            var resourcemapping = require(mappingFilePath);
+        function buildResourceMapping(packagePath) {
+            if (investigatedPackagePaths[packagePath]) {
+                return {};
+            }
+
+            investigatedPackagePaths[packagePath] = 'OK';
 
-            if (resourcemapping.imports && resourcemapping.imports.length) {
+            var packageJson = requireOptional(path.join(packagePath, 'package.json'));
+            var resourcemapping = requireOptional(path.join(packagePath, 'abp.resourcemapping.js')) || { };
 
+            if (packageJson && packageJson.dependencies) {
                 var aliases = {};
                 var mappings = {};
 
-                for (var i = 0; i < resourcemapping.imports.length; i++) {
-
-                    var importedMappingFilePath = path.join(path.dirname(mappingFilePath), resourcemapping.imports[i]);
-                    var importedResourceMapping = buildResourceMapping(importedMappingFilePath);
-                    extendObject(aliases, importedResourceMapping.aliases);
-                    extendObject(mappings, importedResourceMapping.mappings);
+                for (var dependency in packageJson.dependencies) {
+                    if (packageJson.dependencies.hasOwnProperty(dependency)) {
+                        var dependedPackagePath = path.join(rootPath, 'node_modules', dependency);
+                        var importedResourceMapping = buildResourceMapping(dependedPackagePath);
+                        extendObject(aliases, importedResourceMapping.aliases);
+                        extendObject(mappings, importedResourceMapping.mappings);
+                    }
                 }
 
                 extendObject(aliases, resourcemapping.aliases);
@@ -73,13 +91,15 @@
 
                 var tasks = [];
 
-                for (var mapping in resourceMapping.mappings) {
-                    if (resourceMapping.mappings.hasOwnProperty(mapping)) {
-                        var source = replaceAliases(mapping);
-                        var destination = replaceAliases(resourceMapping.mappings[mapping]);
-                        tasks.push(
-                            gulp.src(source).pipe(gulp.dest(destination))
-                        );
+                if (resourceMapping.mappings) {
+                    for (var mapping in resourceMapping.mappings) {
+                        if (resourceMapping.mappings.hasOwnProperty(mapping)) {
+                            var source = replaceAliases(mapping);
+                            var destination = replaceAliases(resourceMapping.mappings[mapping]);
+                            tasks.push(
+                                gulp.src(source).pipe(gulp.dest(destination))
+                            );
+                        }
                     }
                 }