Merge pull request #6374 from jtwebman/admin_error_no_theme

Fixed ghost admin error when missing theme folder.

core/server/middleware/static-theme.js

@@ -11,10 +11,14 @@ function isBlackListedFileType(file) {
 }
 
 function forwardToExpressStatic(req, res, next) {
-    express.static(
+    if (!req.app.get('activeTheme')) {
-        path.join(config.paths.themePath, req.app.get('activeTheme')),
+        next();
-        {maxAge: utils.ONE_YEAR_MS}
+    } else {
-    )(req, res, next);
+        express.static(
+            path.join(config.paths.themePath, req.app.get('activeTheme')),
+            {maxAge: utils.ONE_YEAR_MS}
+        )(req, res, next);
+    }
 }
 
 function staticTheme() {

core/test/unit/middleware/static-theme_spec.js

@@ -65,4 +65,24 @@ describe('staticTheme', function () {
             done();
         });
     });
+
+    it('should not error if active theme is missing', function (done) {
+        var req = {
+                url: 'myvalidfile.css',
+                app: {
+                    get: function () { return undefined; }
+                }
+            },
+            activeThemeStub,
+            sandbox = sinon.sandbox.create();
+
+        activeThemeStub = sandbox.spy(req.app, 'get');
+
+        staticTheme(null)(req, null, function (reqArg, res, next2) {
+            /*jshint unused:false */
+            sandbox.restore();
+            next.called.should.be.false;
+            done();
+        });
+    });
 });