chore: fix comments

Author: KristianDD

lib/common/decorators.ts

@@ -95,7 +95,7 @@ export function performanceLog(injector?: IInjector): any {
 		const performanceService: IPerformanceService = injector.resolve("performanceService");
 
 		//needed for the returned function to have the same name as the original - used in hooks decorator
-		const tempObject = {
+		const functionWrapper = {
 			[originalMethod.name]: function (...args: Array<any>) {
 				const start = performanceService.now();
 				const result = originalMethod.apply(this, args);
@@ -120,7 +120,7 @@ export function performanceLog(injector?: IInjector): any {
 				return result;
 			}
 		};
-		descriptor.value = tempObject[originalMethod.name];
+		descriptor.value = functionWrapper[originalMethod.name];
 
 		// used to get parameter names in hooks decorator
 		descriptor.value.toString = () => {

lib/services/performance-service.ts

@@ -5,7 +5,7 @@ import * as semver from "semver";
 
 export class PerformanceService implements IPerformanceService {
 	public static LOG_MESSAGE_TEMPLATE = `Execution of method "%s" took %s ms.`;
-	public static FAIL_LOG_MESSAGE_TEMPLATE = `Failed to log pefromance data for method %s.`;
+	public static FAIL_LOG_MESSAGE_TEMPLATE = `Failed to log pefromance data in file for method %s.`;
 	private static MIN_NODE_PERFORMANCE_MODULE_VERSION = "8.5.0";
 	private performance: {now(): number}  = null;
 
@@ -59,7 +59,7 @@ export class PerformanceService implements IPerformanceService {
 		let methodArgs;
 
 		try {
-			methodArgs = JSON.stringify(args, this.getCircularReplacer());
+			methodArgs = JSON.stringify(args, this.getJsonSanitizer());
 		} catch (e) {
 			methodArgs = "cyclic args";
 		}
@@ -75,10 +75,12 @@ export class PerformanceService implements IPerformanceService {
 			this.$fs.appendFile(filePath, `${JSON.stringify(info)}${EOL}`);
 		} catch (e) {
 			this.$logger.trace(PerformanceService.FAIL_LOG_MESSAGE_TEMPLATE, methodInfo);
+			this.$logger.info(PerformanceService.LOG_MESSAGE_TEMPLATE, methodInfo, executionTime);
 		}
 	}
 
-	private getCircularReplacer() {
+	//removes any injected members of the arguments and excludes the options object even if it was renamed
+	private getJsonSanitizer() {
 		const seen = new WeakSet();
 		seen.add(this.$options);
 		return (key: any, value: any) => {