Skip to content

Commit d73ce84

Browse files
authored
fix: fix logging logic (#25)
1 parent 7bb7145 commit d73ce84

File tree

2 files changed

+17
-24
lines changed

2 files changed

+17
-24
lines changed

src/service/printer/command/LogLevelCommand.ts

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -46,21 +46,25 @@ export default class LogLevelCommand implements GlobalModifierCommand {
4646

4747
switch (logLevel) {
4848
case "DEBUG":
49-
return this.#printerServiceProvider.printerService!.setLevel(
49+
this.#printerServiceProvider.printerService!.setLevel(
5050
Level.DEBUG,
5151
);
52+
break;
5253
case "INFO":
53-
return this.#printerServiceProvider.printerService!.setLevel(
54+
this.#printerServiceProvider.printerService!.setLevel(
5455
Level.INFO,
5556
);
57+
break;
5658
case "WARN":
57-
return this.#printerServiceProvider.printerService!.setLevel(
59+
this.#printerServiceProvider.printerService!.setLevel(
5860
Level.WARN,
5961
);
62+
break;
6063
case "ERROR":
61-
return this.#printerServiceProvider.printerService!.setLevel(
64+
this.#printerServiceProvider.printerService!.setLevel(
6265
Level.ERROR,
6366
);
67+
break;
6468
}
6569

6670
return Promise.resolve();

src/util/logger.ts

Lines changed: 9 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -28,28 +28,17 @@ let maxLoggerNameLength = 0;
2828
const LOGGER_NAME_PADDINGS: Record<string, string> = {};
2929

3030
function getDefaultLogger(): Logger {
31-
if (debugEnabled) {
32-
return {
33-
trace: () => {},
34-
debug: (message, ...optionalParams) => {
35-
console.debug(message, ...optionalParams);
36-
},
37-
info: (message, ...optionalParams) => {
38-
console.info(message, ...optionalParams);
39-
},
40-
warn: (message, ...optionalParams) => {
41-
console.warn(message, ...optionalParams);
42-
},
43-
error: (message, ...optionalParams) => {
44-
console.error(message, ...optionalParams);
45-
},
46-
};
47-
}
4831
return {
4932
trace: () => {},
50-
debug: () => {},
51-
info: () => {},
52-
warn: () => {},
33+
debug: (message, ...optionalParams) => {
34+
console.debug(message, ...optionalParams);
35+
},
36+
info: (message, ...optionalParams) => {
37+
console.info(message, ...optionalParams);
38+
},
39+
warn: (message, ...optionalParams) => {
40+
console.warn(message, ...optionalParams);
41+
},
5342
error: (message, ...optionalParams) => {
5443
console.error(message, ...optionalParams);
5544
},

0 commit comments

Comments
 (0)