Branch: refs/heads/misc-logging-batch4 Home: https://github.com/xwiki/xwiki-platform Commit: d903f8cb2f87607b018d1a7bc9319cd325d8d528 https://github.com/xwiki/xwiki-platform/commit/d903f8cb2f87607b018d1a7bc9319... Author: Vincent Massol <[email protected]> Date: 2026-08-01 (Sat, 01 Aug 2026) Changed paths: M xwiki-platform-core/xwiki-platform-container/xwiki-platform-container-servlet/src/main/java/org/xwiki/container/servlet/HttpServletUtils.java M xwiki-platform-core/xwiki-platform-container/xwiki-platform-container-servlet/src/main/java/org/xwiki/container/servlet/filters/internal/SafeRedirectFilter.java M xwiki-platform-core/xwiki-platform-dashboard/xwiki-platform-dashboard-macro/src/main/java/org/xwiki/rendering/internal/macro/dashboard/DashboardMacro.java M xwiki-platform-core/xwiki-platform-index/xwiki-platform-index-default/src/main/java/org/xwiki/index/internal/DefaultTasksManager.java M xwiki-platform-core/xwiki-platform-javascript/xwiki-platform-javascript-importmap/src/main/java/org/xwiki/javascript/importmap/internal/JavascriptImportmapResolver.java M xwiki-platform-core/xwiki-platform-javascript/xwiki-platform-javascript-importmap/src/test/java/org/xwiki/javascript/importmap/internal/JavascriptImportmapResolverTest.java M xwiki-platform-core/xwiki-platform-localization/xwiki-platform-localization-script/src/main/java/org/xwiki/localization/script/LocalizationScriptService.java M xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/main/java/org/xwiki/refactoring/internal/DefaultModelBridge.java M xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/main/java/org/xwiki/refactoring/internal/DefaultReferenceUpdater.java M xwiki-platform-core/xwiki-platform-sheet/xwiki-platform-sheet-api/src/main/java/org/xwiki/sheet/internal/DefaultSheetManager.java M xwiki-platform-core/xwiki-platform-sheet/xwiki-platform-sheet-api/src/main/java/org/xwiki/sheet/internal/SheetDocumentDisplayer.java M xwiki-platform-core/xwiki-platform-tika/xwiki-platform-tika-detect/src/main/java/org/xwiki/tika/internal/TikaUtils.java M xwiki-platform-core/xwiki-platform-uiextension/xwiki-platform-uiextension-default/src/main/java/org/xwiki/uiextension/internal/WikiUIExtensionParameters.java M xwiki-platform-core/xwiki-platform-uiextension/xwiki-platform-uiextension-default/src/test/java/org/xwiki/uiextension/WikiUIExtensionParametersTest.java M xwiki-platform-core/xwiki-platform-user/xwiki-platform-user-script/src/main/java/org/xwiki/user/script/GroupScriptService.java M xwiki-platform-core/xwiki-platform-user/xwiki-platform-user-script/src/test/java/org/xwiki/user/script/GroupScriptServiceTest.java M xwiki-platform-core/xwiki-platform-wysiwyg/xwiki-platform-wysiwyg-api/src/main/java/org/xwiki/wysiwyg/internal/converter/AbstractRequestParameterConverter.java M xwiki-platform-core/xwiki-platform-wysiwyg/xwiki-platform-wysiwyg-api/src/main/java/org/xwiki/wysiwyg/script/WysiwygEditorScriptService.java M xwiki-platform-core/xwiki-platform-xml/xwiki-platform-xml-script/src/main/java/org/xwiki/xml/script/XMLScriptService.java M xwiki-platform-core/xwiki-platform-zipexplorer/src/main/java/com/xpn/xwiki/plugin/zipexplorer/ZipExplorerPlugin.java M xwiki-platform-distribution/xwiki-platform-distribution-flavor/xwiki-platform-distribution-flavor-test/xwiki-platform-distribution-flavor-test-webstandards/src/test/it/org/xwiki/test/webstandards/framework/AbstractValidationTest.java Log Message: ----------- [Misc] Apply the logging best practices to 15 more platform modules Last batch of the logging best-practices audit: the 15 modules that had only one or two offending call sites each. * `warn()` no longer passes a Throwable: the stack trace leaves the SLF4J slot and the cause is surfaced through `ExceptionUtils.getRootCauseMessage()`. * `String.format()` and string concatenation replaced with `{}` placeholders and arguments. * `warn()` calls using `e.getMessage()` switched to `ExceptionUtils.getRootCauseMessage()`, which surfaces the underlying cause. * Values wrapped in `[]`, and trailing whitespace trimmed from messages. `DashboardMacro` also stopped discarding its `ComponentLookupException` entirely: the log line now names its root cause. The two `LogCaptureExtension`-based tests that assert the rendered message were updated. Their exception fixtures had no message, which would have rendered an uninformative `[GroupException: ]`, so they now carry a distinct message each. Co-Authored-By: Claude Opus 5 (1M context) <[email protected]> To unsubscribe from these emails, change your notification settings at https://github.com/xwiki/xwiki-platform/settings/notifications