Skip to content

Commit

Permalink
fix: reconfigure logger on settings change
Browse files Browse the repository at this point in the history
  • Loading branch information
dvirtz committed Jul 16, 2023
1 parent 9fa945b commit fbd6bcc
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 6 deletions.
8 changes: 3 additions & 5 deletions src/extension.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
import * as vscode from 'vscode';
import { ParquetEditorProvider } from './parquet-editor-provider';
import { getLogger, initLogger } from './logger';
import { loggingSettings } from './settings';

// this method is called when your extension is activated
// your extension is activated the very first time the command is executed
Expand All @@ -12,11 +13,8 @@ export async function activate(context: vscode.ExtensionContext): Promise<void>
getLogger().info('parquet-viewer activated');

context.subscriptions.push(vscode.workspace.onDidChangeConfiguration((e: vscode.ConfigurationChangeEvent) => {
if (e.affectsConfiguration('logging')
|| e.affectsConfiguration('logLevel')
|| e.affectsConfiguration('logPanel')
|| e.affectsConfiguration('logFolder')) {
// restart logger on log configuration change
if (loggingSettings.some(s => e.affectsConfiguration(s))) {
// restart logger on log configuration change
initLogger(context);
}
}));
Expand Down
6 changes: 5 additions & 1 deletion src/settings.ts
Original file line number Diff line number Diff line change
@@ -1,8 +1,10 @@
import * as vscode from 'vscode';
import { LogLevel } from '@vscode-logging/logger';

const basename = 'parquet-viewer';

function settings() {
return vscode.workspace.getConfiguration('parquet-viewer');
return vscode.workspace.getConfiguration(basename);
}

export function parquetTools(): string | undefined {
Expand Down Expand Up @@ -44,3 +46,5 @@ export function useParquetTools(): boolean {
export function jsonSpace(): number | string | undefined {
return settings().get('jsonSpace');
}

export const loggingSettings = ['logging', 'logLevel', 'logPanel', 'logFolder'].map(s => `${basename}.${s}`);

0 comments on commit fbd6bcc

Please sign in to comment.