From 944150bafbbb5c3e807ba3688174e70764dbdc64 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bj=C3=B8rn=20Erik=20Pedersen?=
 <bjorn.erik.pedersen@gmail.com>
Date: Wed, 4 Nov 2020 19:09:40 +0100
Subject: [PATCH] js: Remove external source map option

It was merged by accident and the implementation does not look correct.

Fixes #7932
---
 resources/resource_transformers/js/build.go        | 3 +--
 resources/resource_transformers/js/options.go      | 7 -------
 resources/resource_transformers/js/options_test.go | 4 ++--
 3 files changed, 3 insertions(+), 11 deletions(-)

diff --git a/resources/resource_transformers/js/build.go b/resources/resource_transformers/js/build.go
index 3a7065e0d..5ff21cf02 100644
--- 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
index 654dbbab9..689efe0b4 100644
--- 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
index abc8091a9..ef8c203ce 100644
--- 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,
 		},