(causeway) branch CAUSEWAY-3751 updated: CAUSEWAY-3751: extends PageRenderLogger also

[danhaywood]

This is an automated email from the ASF dual-hosted git repository.

danhaywood pushed a commit to branch CAUSEWAY-3751
in repository https://gitbox.apache.org/repos/asf/causeway.git


The following commit(s) were added to refs/heads/CAUSEWAY-3751 by this push:
     new 7003ac1164 CAUSEWAY-3751: extends PageRenderLogger also
7003ac1164 is described below

commit 7003ac11641c6e99f432f15c22130c56082c023d
Author: Dan Haywood <d...@haywood-associates.co.uk>
AuthorDate: Tue Jun 4 17:42:55 2024 +0100

    CAUSEWAY-3751: extends PageRenderLogger also
---
 .../services/publishing/log/PageRenderLogger.java  | 37 +++++++++++++++++-----
 1 file changed, 29 insertions(+), 8 deletions(-)

diff --git 
a/api/applib/src/main/java/org/apache/causeway/applib/services/publishing/log/PageRenderLogger.java
 
b/api/applib/src/main/java/org/apache/causeway/applib/services/publishing/log/PageRenderLogger.java
index f1a7251e3d..a2a4eb9439 100644
--- 
a/api/applib/src/main/java/org/apache/causeway/applib/services/publishing/log/PageRenderLogger.java
+++ 
b/api/applib/src/main/java/org/apache/causeway/applib/services/publishing/log/PageRenderLogger.java
@@ -59,6 +59,25 @@ public class PageRenderLogger implements 
PageRenderSubscriber {
         return log.isDebugEnabled();
     }
 
+    @Override
+    public void onRenderingDomainObject(final Bookmark bookmark) {
+        log.debug("rendering object: [ {} ]", 
doubleQuoted(bookmark.stringify()));
+    }
+
+    @Override
+    public void onRenderingCollection(final Supplier<List<Bookmark>> 
bookmarkSupplier) {
+
+        final var bookmarksStringified = 
bookmarksStringified(bookmarkSupplier);
+
+        log.debug("rendering collection: [ {} ]", bookmarksStringified);
+    }
+
+
+    @Override
+    public void onRenderingValue(final Object value) {
+        log.debug("rendering value: [ {} ]", doubleQuoted(value));
+    }
+
     @Override
     public void onRenderedDomainObject(final Bookmark bookmark) {
         log.debug("rendered object: [ {} ]", 
doubleQuoted(bookmark.stringify()));
@@ -67,12 +86,7 @@ public class PageRenderLogger implements 
PageRenderSubscriber {
     @Override
     public void onRenderedCollection(final Supplier<List<Bookmark>> 
bookmarkSupplier) {
 
-        val bookmarksStringified =
-            _NullSafe.stream(bookmarkSupplier.get())
-            .filter(Objects::nonNull)
-            .map(Bookmark::stringify)
-            .map(this::doubleQuoted)
-            .collect(Collectors.joining(", "));
+        final var bookmarksStringified = 
bookmarksStringified(bookmarkSupplier);
 
         log.debug("rendered collection: [ {} ]", bookmarksStringified);
     }
@@ -85,9 +99,16 @@ public class PageRenderLogger implements 
PageRenderSubscriber {
 
     // -- HELPER
 
-    private String doubleQuoted(final @Nullable Object obj) {
-        return "\"" + obj + "\"";
+    private static String bookmarksStringified(Supplier<List<Bookmark>> 
bookmarkSupplier) {
+        return _NullSafe.stream(bookmarkSupplier.get())
+                .filter(Objects::nonNull)
+                .map(Bookmark::stringify)
+                .map(PageRenderLogger::doubleQuoted)
+                .collect(Collectors.joining(", "));
     }
 
+    private static String doubleQuoted(final @Nullable Object obj) {
+        return "\"" + obj + "\"";
+    }
 
 }