js: Remove external source map option

It was merged by accident and the implementation does not look correct. Fixes #7932
gohugoio · Nov 4, 2020 · 944150b · 944150b
1 parent bf2837a
commit 944150b
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 11 deletions.
diff --git a/resources/resource_transformers/js/build.go b/resources/resource_transformers/js/build.go
@@ -78,8 +78,7 @@ func (t *buildTransformation) Transform(ctx *resources.ResourceTransformationCtx
 	}
 
 	opts.sourcefile = ctx.SourcePath
-	opts.workDir = t.c.rs.WorkingDir
-	opts.resolveDir = opts.workDir
+	opts.resolveDir = t.c.rs.WorkingDir
 	opts.contents = string(src)
 	opts.mediaType = ctx.InMediaType
 

diff --git a/resources/resource_transformers/js/options.go b/resources/resource_transformers/js/options.go
@@ -93,7 +93,6 @@ type Options struct {
 	contents   string
 	sourcefile string
 	resolveDir string
-	workDir    string
 	tsConfig   string
 }
 
@@ -365,12 +364,6 @@ func toBuildOptions(opts Options) (buildOptions api.BuildOptions, err error) {
 	switch opts.SourceMap {
 	case "inline":
 		sourceMap = api.SourceMapInline
-	case "external":
-		// When doing external sourcemaps we should specify
-		// out file and no out dir
-		sourceMap = api.SourceMapExternal
-		outFile = filepath.Join(opts.workDir, opts.TargetPath)
-		outDir = ""
 	case "":
 		sourceMap = api.SourceMapNone
 	default:

diff --git a/resources/resource_transformers/js/options_test.go b/resources/resource_transformers/js/options_test.go
@@ -93,7 +93,7 @@ func TestToBuildOptions(t *testing.T) {
 
 	opts, err = toBuildOptions(Options{
 		Target: "es2018", Format: "cjs", Minify: true, mediaType: media.JavascriptType,
-		SourceMap: "external"})
+		SourceMap: "inline"})
 	c.Assert(err, qt.IsNil)
 	c.Assert(opts, qt.DeepEquals, api.BuildOptions{
 		Bundle:            true,
@@ -102,7 +102,7 @@ func TestToBuildOptions(t *testing.T) {
 		MinifyIdentifiers: true,
 		MinifySyntax:      true,
 		MinifyWhitespace:  true,
-		Sourcemap:         api.SourceMapExternal,
+		Sourcemap:         api.SourceMapInline,
 		Stdin: &api.StdinOptions{
 			Loader: api.LoaderJS,
 		},