fix(@angular-devkit/build-webpack): only open browser on first build

clydin · hansl · commit 5fc59245d93e · 2018-06-05T17:55:17.000-07:00
diff --git a/packages/angular_devkit/build_webpack/src/dev-server/index.ts b/packages/angular_devkit/build_webpack/src/dev-server/index.ts
@@ -169,6 +169,7 @@ export class DevServerBuilder implements Builder<DevServerBuilderOptions> {
 
         const server = new WebpackDevServer(webpackCompiler, webpackDevServerConfig);
 
+        let first = true;
         // tslint:disable-next-line:no-any
         (webpackCompiler as any).hooks.done.tap('angular-cli', (stats: webpack.Stats) => {
           if (!browserOptions.verbose) {
@@ -183,7 +184,8 @@ export class DevServerBuilder implements Builder<DevServerBuilderOptions> {
           }
           obs.next({ success: true });
 
-          if (options.open) {
+          if (first && options.open) {
+            first = false;
             opn(serverAddress + webpackDevServerConfig.publicPath);
           }
         });

Original file line number	Diff line number	Diff line change
`@@ -169,6 +169,7 @@ export class DevServerBuilder implements Builder<DevServerBuilderOptions> {`
`169`	`169`
`170`	`170`	`const server = new WebpackDevServer(webpackCompiler, webpackDevServerConfig);`
`171`	`171`
	`172`	`+ let first = true;`
`172`	`173`	`// tslint:disable-next-line:no-any`
`173`	`174`	`(webpackCompiler as any).hooks.done.tap('angular-cli', (stats: webpack.Stats) => {`
`174`	`175`	`if (!browserOptions.verbose) {`
`@@ -183,7 +184,8 @@ export class DevServerBuilder implements Builder<DevServerBuilderOptions> {`
`183`	`184`	`}`
`184`	`185`	`obs.next({ success: true });`
`185`	`186`
`186`		`- if (options.open) {`
	`187`	`+ if (first && options.open) {`
	`188`	`+ first = false;`
`187`	`189`	`opn(serverAddress + webpackDevServerConfig.publicPath);`
`188`	`190`	`}`
`189`	`191`	`});`