Backport SSR bugfix when manifest files are missing:

vuejs#11609 Also change build.dev.js to avoid the build process of Vue.js
clubdrei · Nov 3, 2020 · 5d99f34 · 5d99f34
1 parent bb253db
commit 5d99f34
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/packages/vue-server-renderer/build.dev.js b/packages/vue-server-renderer/build.dev.js
@@ -8848,8 +8848,12 @@ function mapIdToFile (id, clientManifest) {
     fileIndices.forEach(function (index) {
       var file = clientManifest.all[index];
       // only include async files or non-js, non-css assets
-      if (clientManifest.async.indexOf(file) > -1 || !(/\.(js|css)($|\?)/.test(file))) {
-        files.push(file);
+      if (
+        file &&
+        (clientManifest.async.indexOf(file) > -1 ||
+          !/\.(js|css)($|\?)/.test(file))
+      ) {
+        files.push(file)
       }
     });
   }
@@ -8874,7 +8878,7 @@ var TemplateRenderer = function TemplateRenderer (options) {
   this.inject = options.inject !== false;
   // if no template option is provided, the renderer is created
   // as a utility object for rendering assets like preload links and scripts.
-    
+
   var template = options.template;
   this.parsedTemplate = template
     ? typeof template === 'string'

diff --git a/src/server/template-renderer/create-async-file-mapper.js b/src/server/template-renderer/create-async-file-mapper.js
@@ -44,7 +44,11 @@ function mapIdToFile (id, clientManifest) {
     fileIndices.forEach(index => {
       const file = clientManifest.all[index]
       // only include async files or non-js, non-css assets
-      if (clientManifest.async.indexOf(file) > -1 || !(/\.(js|css)($|\?)/.test(file))) {
+      if (
+        file &&
+        (clientManifest.async.indexOf(file) > -1 ||
+          !/\.(js|css)($|\?)/.test(file))
+      ) {
         files.push(file)
       }
     })