Revert "Fix PostProcess regression for hugo server"

This reverts commit 4deb5c6066.

commands/commandeer.go

@@ -395,23 +395,23 @@ func (c *commandeer) loadConfig() error {
 	}
 
 	c.fsCreate.Do(func() {
-		// Assume both source and destination are using same filesystem.
-		fs := hugofs.NewFromSourceAndDestination(sourceFs, sourceFs, config)
+		fs := hugofs.NewFrom(sourceFs, config)
 
 		if c.publishDirFs != nil {
 			// Need to reuse the destination on server rebuilds.
 			fs.PublishDir = c.publishDirFs
 			fs.PublishDirServer = c.publishDirServerFs
 		} else {
-			if c.renderStaticToDisk {
-				publishDirStatic := config.GetString("publishDirStatic")
-				workingDir := config.GetString("workingDir")
-				absPublishDirStatic := paths.AbsPathify(workingDir, publishDirStatic)
+			publishDir := config.GetString("publishDir")
+			publishDirStatic := config.GetString("publishDirStatic")
+			workingDir := config.GetString("workingDir")
+			absPublishDir := paths.AbsPathify(workingDir, publishDir)
+			absPublishDirStatic := paths.AbsPathify(workingDir, publishDirStatic)
 
-				fs = hugofs.NewFromSourceAndDestination(sourceFs, afero.NewMemMapFs(), config)
-				// Writes the dynamic output to memory,
+			if c.renderStaticToDisk {
+				// Writes the dynamic output oton memory,
 				// while serve others directly from /public on disk.
-				dynamicFs := fs.PublishDir
+				dynamicFs := afero.NewMemMapFs()
 				staticFs := afero.NewBasePathFs(afero.NewOsFs(), absPublishDirStatic)
 
 				// Serve from both the static and dynamic fs,
@@ -427,10 +427,18 @@ func (c *commandeer) loadConfig() error {
 						},
 					},
 				)
+				fs.PublishDir = dynamicFs
 				fs.PublishDirStatic = staticFs
 			} else if createMemFs {
 				// Hugo writes the output to memory instead of the disk.
-				fs = hugofs.NewFromSourceAndDestination(sourceFs, afero.NewMemMapFs(), config)
+				fs.PublishDir = new(afero.MemMapFs)
+				fs.PublishDirServer = fs.PublishDir
+				fs.PublishDirStatic = fs.PublishDir
+			} else {
+				// Write everything to disk.
+				fs.PublishDir = afero.NewBasePathFs(afero.NewOsFs(), absPublishDir)
+				fs.PublishDirServer = fs.PublishDir
+				fs.PublishDirStatic = fs.PublishDir
 			}
 		}
 

commands/commands_test.go

@@ -375,10 +375,6 @@ Single: {{ .Title }}
 List: {{ .Title }}
 Environment: {{ hugo.Environment }}
 
-For issue 9788:
-{{ $foo :="abc" | resources.FromString "foo.css" | minify | resources.PostProcess }}
-PostProcess: {{ $foo.RelPermalink }}
-
 `)
 
 	return dir

commands/server_test.go

@@ -31,6 +31,16 @@ import (
 	qt "github.com/frankban/quicktest"
 )
 
+func TestServer(t *testing.T) {
+	c := qt.New(t)
+
+	r := runServerTest(c, true, "")
+
+	c.Assert(r.err, qt.IsNil)
+	c.Assert(r.homeContent, qt.Contains, "List: Hugo Commands")
+	c.Assert(r.homeContent, qt.Contains, "Environment: development")
+}
+
 // Issue 9518
 func TestServerPanicOnConfigError(t *testing.T) {
 	c := qt.New(t)
@@ -91,42 +101,6 @@ baseURL="https://example.org"
 
 }
 
-func TestServerBugs(t *testing.T) {
-	c := qt.New(t)
-
-	for _, test := range []struct {
-		name   string
-		flag   string
-		assert func(c *qt.C, r serverTestResult)
-	}{
-		// Issue 9788
-		{"PostProcess, memory", "", func(c *qt.C, r serverTestResult) {
-			c.Assert(r.err, qt.IsNil)
-			c.Assert(r.homeContent, qt.Contains, "PostProcess: /foo.min.css")
-		}},
-		{"PostProcess, disk", "--renderToDisk", func(c *qt.C, r serverTestResult) {
-			c.Assert(r.err, qt.IsNil)
-			c.Assert(r.homeContent, qt.Contains, "PostProcess: /foo.min.css")
-		}},
-	} {
-		c.Run(test.name, func(c *qt.C) {
-			config := `
-baseURL="https://example.org"
-`
-
-			var args []string
-			if test.flag != "" {
-				args = strings.Split(test.flag, "=")
-			}
-			r := runServerTest(c, true, config, args...)
-			test.assert(c, r)
-
-		})
-
-	}
-
-}
-
 type serverTestResult struct {
 	err            error
 	homeContent    string