Ensure arguments are included in log messages when using decorators. (#11154)

ericsnowcurrently · web-flow · commit d56a65887f61 · 2020-05-05T11:40:04.000-06:00
For #11153.
diff --git a/news/2 Fixes/11153.md b/news/2 Fixes/11153.md
@@ -0,0 +1 @@
+Ensure arguments are included in log messages when using decorators.
diff --git a/src/client/common/logger.ts b/src/client/common/logger.ts
@@ -295,7 +295,7 @@ function trace(message: string, options: LogOptions = LogOptions.None, logLevel?
                         returnValue ? 'truthy' : 'falsy'
                     } return value`
                 );
-                if ((options && LogOptions.Arguments) === LogOptions.Arguments) {
+                if ((options & LogOptions.Arguments) === LogOptions.Arguments) {
                     messagesToLog.push(argsToLogString(args));
                 }
                 if ((options & LogOptions.ReturnValue) === LogOptions.ReturnValue) {

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+Ensure arguments are included in log messages when using decorators.`
Original file line number	Diff line number	Diff line change
`@@ -295,7 +295,7 @@ function trace(message: string, options: LogOptions = LogOptions.None, logLevel?`
`295`	`295`	`returnValue ? 'truthy' : 'falsy'`
`296`	`296`	} return value`
`297`	`297`	`);`
`298`		`- if ((options && LogOptions.Arguments) === LogOptions.Arguments) {`
	`298`	`+ if ((options & LogOptions.Arguments) === LogOptions.Arguments) {`
`299`	`299`	`messagesToLog.push(argsToLogString(args));`
`300`	`300`	`}`
`301`	`301`	`if ((options & LogOptions.ReturnValue) === LogOptions.ReturnValue) {`