ENGCOM-7447: Fix handling of mixins for modules with no dependencies defined #27690

 - Merge Pull Request #27690 from krzksz/magento2:hotfix-mixins
 - Merged commits:
   1. 97f0a1f
   2. 16444d0

Author: magento-engcom-team

dev/tests/js/jasmine/tests/lib/mage/requirejs/mixins.test.js

@@ -183,5 +183,31 @@ define(['rjsResolver', 'mixins'], function (resolver, mixins) {
 
             require([name], function () {});
         });
+
+        it('applies mixins for modules that have no dependencies', function (done) {
+            var name = 'tests/assets/mixins/mixins-applied-no-dependencies',
+                mixinName = 'tests/assets/mixins/mixins-applied-no-dependencies-ext';
+
+            mixins.hasMixins.and.returnValue(true);
+            mixins.getMixins.and.returnValue([mixinName]);
+
+            define(name, {
+                value: 'original'
+            });
+
+            define(mixinName, [], function () {
+                return function (module) {
+                    module.value = 'changed';
+
+                    return module;
+                };
+            });
+
+            require([name], function (module) {
+                expect(module.value).toBe('changed');
+
+                done();
+            });
+        });
     });
 });

lib/web/mage/requirejs/mixins.js

@@ -232,9 +232,12 @@ require([
      * from it every time require call happens.
      */
     defContext.defQueue.shift = function () {
-        var queueItem = Array.prototype.shift.call(this);
+        var queueItem = Array.prototype.shift.call(this),
+            lastDeps = queueItem && queueItem[1];
 
-        queueItem[1] = processNames(queueItem[1], defContext);
+        if (Array.isArray(lastDeps)) {
+            queueItem[1] = processNames(queueItem[1], defContext);
+        }
 
         return queueItem;
     };