Skip to content

Commit 31e49d7

Browse files
Merge pull request #4601 from NativeScript/vladimirov/fix-logger
fix: logger changes broke trace and some hooks
2 parents 09b6908 + ae4abbd commit 31e49d7

File tree

1 file changed

+11
-1
lines changed

1 file changed

+11
-1
lines changed

lib/common/logger/logger.ts

+11-1
Original file line numberDiff line numberDiff line change
@@ -81,6 +81,15 @@ export class Logger implements ILogger {
8181
this.logMessage(args, LoggerLevel.INFO);
8282
}
8383

84+
/**
85+
* DEPRECATED
86+
* Present only for backwards compatibility as some plugins (nativescript-plugin-firebase)
87+
* use $logger.out in their hooks
88+
*/
89+
out(...args: any[]): void {
90+
this.info(args);
91+
}
92+
8493
debug(...args: any[]): void {
8594
const encodedArgs: string[] = this.getPasswordEncodedArguments(args);
8695
this.logMessage(encodedArgs, LoggerLevel.DEBUG);
@@ -159,7 +168,8 @@ export class Logger implements ILogger {
159168
const result: any = {};
160169
const cleanedData = _.cloneDeep(data);
161170

162-
const dataToCheck = data.filter(el => typeof el === "object");
171+
// objects created with Object.create(null) do not have `hasOwnProperty` function
172+
const dataToCheck = data.filter(el => typeof el === "object" && el.hasOwnProperty && typeof el.hasOwnProperty === "function");
163173

164174
for (const element of dataToCheck) {
165175
if (opts.length === 0) {

0 commit comments

Comments
 (0)