tiararodney/esm-logging
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(handler): add formatter getter, fix format() return, fix level setter

Browse source 
Add missing getter for formatter property. Fix format() to return
the formatted string instead of discarding it. Fix level setter
to assign to _level instead of recursing infinitely.
This commit is contained in:
Tiara Rodney 2026-03-13 22:40:05 +01:00
parent 03e3641e03
commit 340b31bc50
No known key found for this signature in database
GPG key ID: 5CD8EC1D46106723
1 changed files with 5 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

11
src/handler.ts
View file

@ -78,7 +78,7 @@ export class Handler extends Filterer {
}
get level(): number { return this._level }
set level(level: LogLevel|string) { this.level = checkLevel(level) }
set level(level: LogLevel|string) { this._level = checkLevel(level) }
get scope(): string|null { return this._scope }
set scope(scope: string) { this._scope = scope }
@ -90,11 +90,9 @@ export class Handler extends Filterer {
* If a formatter is set, use it. Otherwise, use the default formatter for
* the module.
*/
format(record: LogRecord) {
var fmt: Formatter|null = null;
if (this.formatter) { fmt = this.formatter }
else { fmt = DEFAULT_FORMATTER }
format(record: LogRecord): string {
const fmt = this.formatter ?? DEFAULT_FORMATTER;
return fmt.format(record);
}
/**
@ -159,6 +157,7 @@ export class Handler extends Filterer {
)
}
get formatter(): Formatter|null { return this._formatter }
set formatter(fmt: Formatter) { this._formatter = fmt }
}