From f49bdc87723f1d1d10d6e5168114607475a8d94b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jo=C3=A3o=20Dinis=20Ferreira?= Date: Sat, 30 May 2026 17:59:19 +0200 Subject: [PATCH] fix: declare loggers with the enclosing class, not another/super class Three loggers were created for the wrong category, so their records show up under an unrelated class: - EObjectDescriptions used getLogger(NameFunctions.class) (unrelated class) - DefaultCheckQuickfixProvider used its superclass DefaultQuickfixProvider - CustomClassAwareEcoreGenerator used its superclass EcoreGenerator Use the enclosing class literal in each, per the LogManager.getLogger convention. Co-Authored-By: Claude Opus 4.8 (1M context) --- .../check/runtime/ui/quickfix/DefaultCheckQuickfixProvider.java | 2 +- .../xtext/generator/util/CustomClassAwareEcoreGenerator.java | 2 +- .../com/avaloq/tools/ddk/xtext/scoping/EObjectDescriptions.java | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/com.avaloq.tools.ddk.check.runtime.ui/src/com/avaloq/tools/ddk/check/runtime/ui/quickfix/DefaultCheckQuickfixProvider.java b/com.avaloq.tools.ddk.check.runtime.ui/src/com/avaloq/tools/ddk/check/runtime/ui/quickfix/DefaultCheckQuickfixProvider.java index a19f1f21fe..ac2efab595 100644 --- a/com.avaloq.tools.ddk.check.runtime.ui/src/com/avaloq/tools/ddk/check/runtime/ui/quickfix/DefaultCheckQuickfixProvider.java +++ b/com.avaloq.tools.ddk.check.runtime.ui/src/com/avaloq/tools/ddk/check/runtime/ui/quickfix/DefaultCheckQuickfixProvider.java @@ -56,7 +56,7 @@ */ public class DefaultCheckQuickfixProvider extends DefaultQuickfixProvider { - private static final Logger LOGGER = LogManager.getLogger(DefaultQuickfixProvider.class); + private static final Logger LOGGER = LogManager.getLogger(DefaultCheckQuickfixProvider.class); @Inject @Named(Constants.LANGUAGE_NAME) diff --git a/com.avaloq.tools.ddk.xtext.generator/src/com/avaloq/tools/ddk/xtext/generator/util/CustomClassAwareEcoreGenerator.java b/com.avaloq.tools.ddk.xtext.generator/src/com/avaloq/tools/ddk/xtext/generator/util/CustomClassAwareEcoreGenerator.java index e33d519c83..b07f7664bb 100644 --- a/com.avaloq.tools.ddk.xtext.generator/src/com/avaloq/tools/ddk/xtext/generator/util/CustomClassAwareEcoreGenerator.java +++ b/com.avaloq.tools.ddk.xtext.generator/src/com/avaloq/tools/ddk/xtext/generator/util/CustomClassAwareEcoreGenerator.java @@ -44,7 +44,7 @@ */ @SuppressWarnings("nls") public class CustomClassAwareEcoreGenerator extends EcoreGenerator { - private static final Logger LOGGER = LogManager.getLogger(EcoreGenerator.class); + private static final Logger LOGGER = LogManager.getLogger(CustomClassAwareEcoreGenerator.class); private String genModel; private final List suppressedSrcPaths = Lists.newLinkedList(); diff --git a/com.avaloq.tools.ddk.xtext/src/com/avaloq/tools/ddk/xtext/scoping/EObjectDescriptions.java b/com.avaloq.tools.ddk.xtext/src/com/avaloq/tools/ddk/xtext/scoping/EObjectDescriptions.java index 559bec58d5..5754ef86d8 100644 --- a/com.avaloq.tools.ddk.xtext/src/com/avaloq/tools/ddk/xtext/scoping/EObjectDescriptions.java +++ b/com.avaloq.tools.ddk.xtext/src/com/avaloq/tools/ddk/xtext/scoping/EObjectDescriptions.java @@ -33,7 +33,7 @@ public final class EObjectDescriptions { /** Class-wide logger. */ - private static final Logger LOGGER = LogManager.getLogger(NameFunctions.class); + private static final Logger LOGGER = LogManager.getLogger(EObjectDescriptions.class); /** No public instantiation. */ private EObjectDescriptions() {