fix(content-drive): truncate long-text field values in listing rows (#37185)

dotCMS/src/main/java/com/dotcms/browser/BrowserAPIImpl.java

@@ -3042,7 +3042,11 @@ private Map<String,Object> dotAssetMap(final Contentlet dotAsset) throws DotStat
     } // dotAssetMap.
 
     private Map<String, Object> dotContentMap(final Contentlet dotAsset) throws DotStateException {
-        return new DotTransformerBuilder().defaultOptions().content(dotAsset).build().toMaps().get(0);
+        // issue #37185: opt-in only at this call site -- never added to defaultOptions -- so no
+        // other consumer of DotTransformerBuilder#defaultOptions() (ContentResource, GraphQL, the
+        // Content Editor, etc.) is affected.
+        return new DotTransformerBuilder().defaultOptions().longTextPreview().content(dotAsset)
+                .build().toMaps().get(0);
     } // dotAssetMap.
 
 

dotCMS/src/main/java/com/dotcms/rest/api/v1/drive/ContentDriveResource.java

@@ -67,7 +67,7 @@ public class ContentDriveResource {
                     description = "Drive search results retrieved successfully",
                     content = @Content(mediaType = "application/json",
                         schema = @Schema(type = "object",
-                        description = "Drive search response containing filtered assets, folders, and navigation metadata with content type filtering")
+                        description = "Drive search response containing filtered assets, folders, and navigation metadata with content type filtering. WYSIWYG/TextArea/Story Block field values on each listing row are a <=150-character extracted plain-text preview, not the full stored value (issue #37185).")
                     )
         ),
         @ApiResponse(responseCode = "401",

dotCMS/src/main/java/com/dotmarketing/portlets/contentlet/transform/DotTransformerBuilder.java

@@ -246,6 +246,20 @@ public DotTransformerBuilder defaultOptions(){
         return this;
     }
 
+    /**
+     * Opts into replacing WYSIWYG/TextArea/Story Block field values with a <=150-character
+     * extracted plain-text preview (issue #37185). Additive -- chain it after any of this
+     * builder's other option methods (e.g. {@link #defaultOptions()}) without disturbing their
+     * options. Never added to {@link DotContentletTransformerImpl#defaultOptions} itself, so this
+     * remains strictly opt-in per call site.
+     *
+     * @return The {@link DotTransformerBuilder} instance.
+     */
+    public DotTransformerBuilder longTextPreview(){
+        optionsHolder.add(TransformOptions.LONG_TEXT_PREVIEW);
+        return this;
+    }
+
     /**
      * This transformer provides a view for the History of a Contentlet. It exposes a minified map
      * of properties, just like the data you can see in the History tab in the Content Editor page.

dotCMS/src/main/java/com/dotmarketing/portlets/contentlet/transform/strategy/LongTextPreviewStrategy.java

@@ -0,0 +1,156 @@
+package com.dotmarketing.portlets.contentlet.transform.strategy;
+
+import com.dotcms.api.APIProvider;
+import com.dotcms.contenttype.model.field.Field;
+import com.dotcms.contenttype.model.field.StoryBlockField;
+import com.dotcms.contenttype.model.field.TextAreaField;
+import com.dotcms.contenttype.model.field.WysiwygField;
+import com.dotcms.contenttype.model.type.ContentType;
+import com.dotcms.repackage.org.jsoup.Jsoup;
+import com.dotmarketing.exception.DotDataException;
+import com.dotmarketing.exception.DotSecurityException;
+import com.dotmarketing.portlets.contentlet.model.Contentlet;
+import static com.dotmarketing.portlets.contentlet.model.Contentlet.TITTLE_KEY;
+import com.dotmarketing.util.Logger;
+import com.dotmarketing.util.UtilMethods;
+import com.liferay.portal.model.User;
+import com.liferay.util.StringPool;
+import io.vavr.control.Try;
+import java.util.List;
+import java.util.Map;
+import java.util.Set;
+import java.util.function.Function;
+
+/**
+ * Replaces WYSIWYG, TextArea and Story Block field values in a transformed map with a
+ * <=150-character extracted plain-text preview, instead of the raw stored value.
+ * <p>
+ * WYSIWYG/TextArea values store raw HTML; the preview is {@code Jsoup.parse(html).text()},
+ * truncated to 150 characters. Story Block values are, by the time this strategy runs, already
+ * the {@link Map} (or raw-string/{@code null} fallback) that {@link StoryBlockViewStrategy}
+ * produces -- this strategy walks that structure's {@code content} arrays recursively, collecting
+ * every {@code text} leaf value, and truncates the concatenation to 150 characters. This is why
+ * {@link TransformOptions#LONG_TEXT_PREVIEW} must be declared after {@code STORY_BLOCK_VIEW} and
+ * {@code JSON_VIEW} in the enum -- {@code EnumSet} iteration order runs this strategy last.
+ *
+ * @since 25.xx
+ */
+public class LongTextPreviewStrategy extends AbstractTransformStrategy<Contentlet> {
+
+    static final int MAX_PREVIEW_LENGTH = 150;
+
+    LongTextPreviewStrategy(final APIProvider toolBox) {
+        super(toolBox);
+    }
+
+    @Override
+    protected Map<String, Object> transform(final Contentlet source, final Map<String, Object> map,
+            final Set<TransformOptions> options, final User user)
+            throws DotDataException, DotSecurityException {
+        final ContentType contentType = source.getContentType();
+        if (null == contentType || UtilMethods.isNotSet(contentType.id())) {
+            throw new DotDataException(
+                    String.format("Content Type in Contentlet '%s' is not set", source.getIdentifier()));
+        }
+
+        applyPreview(contentType.fields(WysiwygField.class), map, LongTextPreviewStrategy::extractHtmlPreview);
+        applyPreview(contentType.fields(TextAreaField.class), map, LongTextPreviewStrategy::extractHtmlPreview);
+        applyPreview(contentType.fields(StoryBlockField.class), map, LongTextPreviewStrategy::extractStoryBlockPreview);
+
+        return map;
+    }
+
+    private void applyPreview(final List<Field> fields, final Map<String, Object> map,
+            final Function<Object, String> extractor) {
+        if (!UtilMethods.isSet(fields)) {
+            return;
+        }
+        fields.stream()
+                // AC-008: the "title" key is independently populated by COMMON_PROPS from
+                // Contentlet#getTitle() -- never overwrite it with a truncated preview, even when
+                // the content type's title-source field is itself WYSIWYG/TextArea/Story Block.
+                .filter(field -> !TITTLE_KEY.equals(field.variable()))
+                .forEach(field -> Try.run(() ->
+                        map.put(field.variable(), extractor.apply(map.get(field.variable()))))
+                        .onFailure(e -> Logger.warn(LongTextPreviewStrategy.class, String.format(
+                                "An error occurred extracting a long-text preview for field '%s' [%s]: %s",
+                                field.variable(), field.id(), e.getMessage()))));
+    }
+
+    /** WYSIWYG/TextArea: strip HTML via Jsoup, then truncate the plain text. */
+    private static String extractHtmlPreview(final Object rawValue) {
+        if (!(rawValue instanceof String) || ((String) rawValue).isEmpty()) {
+            return rawValue instanceof String ? (String) rawValue : StringPool.BLANK;
+        }
+        return truncate(Jsoup.parse((String) rawValue).text());
+    }
+
+    /**
+     * Story Block: the map already holds {@link StoryBlockViewStrategy}'s output -- a
+     * {@link Map} (parsed JSON), a raw {@link String} (non-JSON fallback) or {@code null}
+     * (parse-failure fallback). Extract and truncate text from whichever shape it is.
+     */
+    private static String extractStoryBlockPreview(final Object storyBlockValue) {
+        if (null == storyBlockValue) {
+            return StringPool.BLANK;
+        }
+        if (storyBlockValue instanceof String) {
+            return truncate((String) storyBlockValue);
+        }
+        final StringBuilder textBuilder = new StringBuilder();
+        collectText(storyBlockValue, textBuilder);
+        return truncate(textBuilder.toString());
+    }
+
+    /**
+     * Recursively walks a Story Block JSON-tree node, collecting every {@code text} leaf value.
+     * Stops once enough text has been collected for the preview bound, so a large story block is
+     * not fully traversed/concatenated just to be truncated away afterward (found in review).
+     */
+    private static void collectText(final Object node, final StringBuilder out) {
+        if (out.length() >= MAX_PREVIEW_LENGTH) {
+            return;
+        }
+        if (node instanceof Map) {
+            final Map<?, ?> nodeMap = (Map<?, ?>) node;
+            final Object text = nodeMap.get("text");
+            if (text instanceof String) {
+                if (out.length() > 0) {
+                    out.append(' ');
+                }
+                out.append((String) text);
+            }
+            final Object content = nodeMap.get("content");
+            if (content instanceof List) {
+                for (final Object child : (List<?>) content) {
+                    if (out.length() >= MAX_PREVIEW_LENGTH) {
+                        break;
+                    }
+                    collectText(child, out);
+                }
+            }
+        } else if (node instanceof List) {
+            for (final Object child : (List<?>) node) {
+                if (out.length() >= MAX_PREVIEW_LENGTH) {
+                    break;
+                }
+                collectText(child, out);
+            }
+        }
+    }
+
+    private static String truncate(final String text) {
+        if (null == text) {
+            return StringPool.BLANK;
+        }
+        if (text.length() <= MAX_PREVIEW_LENGTH) {
+            return text;
+        }
+        // Avoid splitting a UTF-16 surrogate pair (e.g. an emoji) at the boundary -- that would
+        // leave a lone high surrogate at the end of the preview (found in review).
+        final int cutIndex = Character.isHighSurrogate(text.charAt(MAX_PREVIEW_LENGTH - 1))
+                ? MAX_PREVIEW_LENGTH - 1 : MAX_PREVIEW_LENGTH;
+        return text.substring(0, cutIndex);
+    }
+
+}

dotCMS/src/main/java/com/dotmarketing/portlets/contentlet/transform/strategy/StrategyResolverImpl.java

@@ -22,6 +22,7 @@
 import static com.dotmarketing.portlets.contentlet.transform.strategy.TransformOptions.JSON_VIEW;
 import static com.dotmarketing.portlets.contentlet.transform.strategy.TransformOptions.KEY_VALUE_VIEW;
 import static com.dotmarketing.portlets.contentlet.transform.strategy.TransformOptions.LANGUAGE_VIEW;
+import static com.dotmarketing.portlets.contentlet.transform.strategy.TransformOptions.LONG_TEXT_PREVIEW;
 import static com.dotmarketing.portlets.contentlet.transform.strategy.TransformOptions.RENDER_FIELDS;
 import static com.dotmarketing.portlets.contentlet.transform.strategy.TransformOptions.SITE_VIEW;
 import static com.dotmarketing.portlets.contentlet.transform.strategy.TransformOptions.STORY_BLOCK_VIEW;
@@ -102,6 +103,7 @@ private static Map<TransformOptions, Supplier<AbstractTransformStrategy>> getStr
         strategyTriggeredByOptionMap.put(JSON_VIEW, () -> new JSONViewStrategy(toolBox));
         strategyTriggeredByOptionMap.put(DATETIME_FIELDS_TO_TIMESTAMP, () -> new DateTimeFieldsToTimeStampStrategy(toolBox));
         strategyTriggeredByOptionMap.put(HISTORY_VIEW, () -> new HistoryViewStrategy(toolBox));
+        strategyTriggeredByOptionMap.put(LONG_TEXT_PREVIEW, () -> new LongTextPreviewStrategy(toolBox));
         return strategyTriggeredByOptionMap;
     }
 

dotCMS/src/main/java/com/dotmarketing/portlets/contentlet/transform/strategy/TransformOptions.java

@@ -110,7 +110,15 @@ public enum TransformOptions {
     HISTORY_VIEW,
     /** Instructs the Strategy to clear all existing data in the Contentlet Map before applying a
      * specific Strategy. */
-    CLEAR_EXISTING_DATA;
+    CLEAR_EXISTING_DATA,
+    /**
+     * Instructs the Strategy to replace WYSIWYG/TextArea/Story Block field values with a
+     * <=150-character extracted plain-text preview. Declared last (after {@link #STORY_BLOCK_VIEW}
+     * and {@link #JSON_VIEW}) so {@code EnumSet} iteration order in
+     * {@link StrategyResolverImpl#resolveStrategies} runs this strategy after those have already
+     * decorated the map -- see issue #37185.
+     */
+    LONG_TEXT_PREVIEW;
 
     // -----------------------------------------------------------------------------------------
     // Plug additional Transform Options to manipulate the outcome as a particular type of view