@oroinc/oro-webpack-config-builder 5.1.0-alpha35 → 5.1.0-alpha37

package/modules-config/layout-modules-config-loader.js

@@ -8,7 +8,7 @@ class LayoutModulesConfigLoader extends ModulesConfigLoader {
      */
     loadConfig(theme, filePath) {
         let themeConfig = super.loadConfig(theme, path.join('Resources/views/layouts/', theme, filePath));
-        themeConfig = merge(themeConfig, super.loadConfig(theme, path.join('templates/layouts/', theme, filePath)));
+
         // recursive process parent theme
         const {parent: parentTheme} = this.themes[theme];
         if (typeof parentTheme === 'string') {

package/modules-config/modules-config-loader.js

@@ -38,34 +38,24 @@ class ModulesConfigLoader {
 
     /**
      * @param {Array} bundles Array of ordered symfony bundle paths
-     * @param {string|Array} themesLocation Path inside the bundle, where to find the theme
+     * @param {string} themesLocation Path inside the bundle, where to find the theme
      * @param {string} themeInfoFileName Yaml File name with theme info
      */
     constructor(bundles, themesLocation, themeInfoFileName) {
         this._bundles = bundles;
-        if (!Array.isArray(themesLocation)) {
-            themesLocation = [themesLocation];
-        }
-
-        const themes = {};
-        const self = this;
-        themesLocation.forEach(themesLocation => {
-            self._collectThemes(themes, themesLocation, themeInfoFileName);
-        });
-
-        this._themes = themes;
+        this._themes = this._getThemes(themesLocation, themeInfoFileName);
         this._processedFiles = [];
     }
 
     /**
-     * Collects list of themes with their parents into given storage(themes param)
-     *
-     * @param {Object} themes
+     * Return list of themes with their parents
      * @param {string} themesLocation
      * @param {string} themeInfoFileName
+     * @returns {Object.<string|null>}
      * @private
      */
-    _collectThemes(themes, themesLocation, themeInfoFileName) {
+    _getThemes(themesLocation, themeInfoFileName) {
+        const themes = {};
         this._bundles.forEach(bundle => {
             const source = bundle + themesLocation;
 
@@ -83,6 +73,8 @@ class ModulesConfigLoader {
                 themes[name] = merge(themes[name] || {}, theme, {arrayMerge});
             });
         });
+
+        return themes;
     }
 
     /**

package/oro-webpack-config.js

@@ -32,7 +32,6 @@ require('resolve-url-loader');
 
 class ConfigBuilder {
     constructor() {
-        this._projectPath = '';
         this._enableLayoutThemes = false;
         this._defaultLayoutThemes = null;
         this.emitter = new EventEmitter();
@@ -187,13 +186,6 @@ class ConfigBuilder {
         };
     }
 
-    get resolvedProjectPath() {
-        if (this._resolvedProjectPath == undefined) {
-            this._resolvedProjectPath = path.resolve(this._projectPath);
-        }
-        return this._resolvedProjectPath;
-    }
-
     get resolvedPublicPath() {
         if (this._resolvedPublicPath === undefined) {
             this._resolvedPublicPath = path.resolve(this._publicPath);
@@ -441,14 +433,9 @@ class ConfigBuilder {
         if (this._isAdminTheme(buildName)) {
             themeDefinition = this._modulesConfigLoader.themes[buildName.split('.')[1]];
             buildPublicPath = '/build/admin/';
-            const jsModulesConfig = this._themeConfigFactory.loadConfig(
-                buildName,
-                [
-                    'Resources/config/oro/jsmodules.yml',
-                    'Resources/config/jsmodules.yml',
-                    'config/oro/jsmodules.yml'
-                ]
-            );            validation.jsmodulesValidator.checkFullSchema(
+            const jsModulesConfig = this._themeConfigFactory.loadConfig(buildName,
+                ['Resources/config/oro/jsmodules.yml', 'Resources/config/jsmodules.yml']);
+            validation.jsmodulesValidator.checkFullSchema(
                 jsModulesConfig,
                 this._themeConfigFactory?._configLoader.processedFiles,
                 buildName
@@ -486,7 +473,6 @@ class ConfigBuilder {
             modules: [
                 resolvedBuildPath,
                 this.resolvedPublicPath,
-                path.join(this.resolvedProjectPath, '/assets'),
                 path.join(this.resolvedPublicPath, '/bundles'),
                 path.join(this.resolvedPublicPath, '/js'),
                 this.resolvedNodeModulesPath
@@ -551,11 +537,10 @@ class ConfigBuilder {
         this._isProduction = args.mode === 'production';
         this._symfonyEnv = env.symfony;
         this._appConfig = AppConfigLoader.getConfig(this._cachePath, this._symfonyEnv);
-        this._appConfig.paths.push(this.resolvedProjectPath);
 
         this._modulesConfigLoader = new ModulesConfigLoader(
             this._appConfig.paths,
-            ['/Resources/public/themes/', '/public/themes/admin/'],
+            '/Resources/public/themes/',
             'settings.yml'
         );
         this._adminThemes = this._modulesConfigLoader.themeNames.map(themeName => 'admin.' + themeName);
@@ -569,7 +554,7 @@ class ConfigBuilder {
 
         this._layoutModulesConfigLoader = new LayoutModulesConfigLoader(
             this._appConfig.paths,
-            ['/Resources/views/layouts/', '/templates/layouts/'],
+            '/Resources/views/layouts/',
             'theme.yml'
         );
         this._layoutStyleLoader = new LayoutStyleLoader(this._layoutModulesConfigLoader, entryPointFileWriter);

package/package.json

@@ -1,19 +1,19 @@
 {
   "name": "@oroinc/oro-webpack-config-builder",
-  "version": "5.1.0-alpha35",
+  "version": "5.1.0-alpha37",
   "author": "Oro, Inc (https://www.oroinc.com)",
   "license": "MIT",
   "description": "An integration of OroPlatform based applications with the Webpack.",
   "main": "oro-webpack-config.js",
   "dependencies": {
-    "@babel/core": "~7.17.9",
-    "@babel/plugin-transform-runtime": "~7.17.0",
-    "@babel/preset-env": "~7.16.11",
+    "@babel/core": "~7.20.5",
+    "@babel/plugin-transform-runtime": "~7.19.6",
+    "@babel/preset-env": "~7.20.2",
     "autoprefixer": "~10.4.0",
-    "babel-loader": "~8.2.3",
+    "babel-loader": "~9.1.0",
     "bindings": "~1.5.0",
     "css-loader": "~6.7.1",
-    "css-minimizer-webpack-plugin": "~3.4.1",
+    "css-minimizer-webpack-plugin": "~4.2.2",
     "deepmerge": "~4.2.2",
     "exports-loader": "~3.1.0",
     "expose-loader": "~3.1.0",
@@ -23,26 +23,26 @@
     "html-webpack-plugin": "~5.5.0",
     "imports-loader": "~3.1.1",
     "js-yaml": "~4.1.0",
-    "mini-css-extract-plugin": "~2.6.0",
-    "minimist": "~1.2.3",
-    "nan": "~2.15.0",
+    "mini-css-extract-plugin": "~2.7.1",
+    "minimist": "~1.2.7",
+    "nan": "~2.17.0",
     "path": "0.12.7",
     "postcss": "~8.4.5",
-    "postcss-loader": "~6.2.0",
+    "postcss-loader": "~7.0.2",
     "printf": "~0.6.0",
     "resolve-url-loader": "^5.0.0",
     "rtlcss-webpack-plugin": "~4.0.6",
-    "sass": "~1.50.0",
-    "sass-loader": "~12.6.0",
+    "sass": "~1.56.1",
+    "sass-loader": "~13.2.0",
     "schema-utils": "^4.0.0",
     "style-loader": "~3.3.1",
-    "terser": "~5.12.1",
+    "terser": "^5.16.0",
     "text-loader": "0.0.1",
     "underscore": "~1.13.1",
     "url-loader": "~4.1.1",
-    "webpack": "~5.72.0",
-    "webpack-bundle-analyzer": "~4.5.0",
-    "webpack-cli": "~4.9.1",
+    "webpack": "~5.75.0",
+    "webpack-bundle-analyzer": "~4.7.0",
+    "webpack-cli": "~5.0.0",
     "webpack-dev-server": "^4.7.3",
     "webpack-merge": "~5.8.0",
     "wildcard": "~2.0.0"

package/style/admin-style-loader.js

@@ -9,9 +9,8 @@ class AdminStyleLoader extends StyleLoader {
         const {rtl_support: rtlSupport = false, styles: extraThemeConfig} =
             this._configLoader.loadConfig(themeName, 'Resources/public/themes/' + themeName + '/settings.yml');
         const baseThemeConfig = this._configLoader.loadConfig(themeName, 'Resources/config/oro/assets.yml');
-        const appRootExtraConfig = this._configLoader.loadConfig(themeName, 'config/oro/assets.yml');
         /** @type {Object.<string, ThemeGroupConfig>} */
-        const themeConfig = merge(baseThemeConfig, appRootExtraConfig, extraThemeConfig);
+        const themeConfig = merge(baseThemeConfig, extraThemeConfig);
 
         return {
             themeConfig,

package/style/style-loader.js

@@ -49,7 +49,6 @@ class StyleLoader {
 
             inputs = this._overrideInputs(inputs);
             inputs = this._sortInputs(inputs);
-            inputs = this._applyInputsBasePathPrefix(inputs);
 
             if (settings.rtlSupport) {
                 writingOptions.ignoreRTLInputs = _.difference(inputs, this._matchInputs(rtlMasks, inputs));
@@ -59,7 +58,7 @@ class StyleLoader {
             const filePath = path.join(buildPath, output);
             entryPoints[entryPointName] = [
                 ...entries,
-                this._entryPointFileWriter.write('./../../', inputs, filePath, writingOptions)
+                this._entryPointFileWriter.write('./../../../', inputs, filePath, writingOptions)
             ];
         }
         return entryPoints;
@@ -163,26 +162,6 @@ class StyleLoader {
 
         return _.unique(whiteListedInputs);
     }
-
-    /**
-     * Considering base path as application's root.
-     * Bundles based *.scss sources go with '../' prefix.
-     *
-     * @param {string[]} inputs
-     * @private
-     */
-    _applyInputsBasePathPrefix(inputs) {
-        const processedInputs = [];
-
-        inputs.forEach(input => {
-            if (input.indexOf('bundles') === 0) {
-                input = '../' + input;
-            }
-            processedInputs.push(input);
-        });
-
-        return processedInputs;
-    }
 }
 
 module.exports = StyleLoader;

package/validation/assets-validator.js

@@ -82,11 +82,10 @@ module.exports = Object.assign({}, schemaValidator, {
                     input = newPath;
                 }
 
-                const fullBasePath = path.resolve(input);
                 const fullPath = path.resolve(this._publicPath + input);
                 // skip the path to global node modules,
                 // e.g. '~bootstrap/scss/bootstrap'
-                if (!input.startsWith('~') && !fs.existsSync(fullPath) && !fs.existsSync(fullBasePath)) {
+                if (!input.startsWith('~') && !fs.existsSync(fullPath)) {
                     files.push(input);
                 }
             });