From a1b684509727422630b056f0d30a07e5607d2d45 Mon Sep 17 00:00:00 2001
From: "aliaksei.bykau" <aliaksei.bykau@4science.com>
Date: Tue, 12 Sep 2023 17:17:49 +0200
Subject: [PATCH] [DSC-1234] added cell limit. Now if content of the cell is
 bigger than limit it will be truncated and end with "..."

---
 .../org/dspace/app/bulkimport/model/BulkImportSheet.java  | 8 +++++++-
 .../content/integration/crosswalks/XlsCrosswalk.java      | 3 ++-
 2 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/dspace-api/src/main/java/org/dspace/app/bulkimport/model/BulkImportSheet.java b/dspace-api/src/main/java/org/dspace/app/bulkimport/model/BulkImportSheet.java
index 14fbb60524fb..53c5f9b99166 100644
--- a/dspace-api/src/main/java/org/dspace/app/bulkimport/model/BulkImportSheet.java
+++ b/dspace-api/src/main/java/org/dspace/app/bulkimport/model/BulkImportSheet.java
@@ -15,6 +15,7 @@
 import java.util.List;
 import java.util.Map;
 
+import org.apache.commons.lang.StringUtils;
 import org.apache.poi.ss.usermodel.Row;
 import org.apache.poi.ss.usermodel.Sheet;
 import org.apache.poi.ss.usermodel.Workbook;
@@ -107,7 +108,12 @@ public void appendValueOnLastRow(String header, String value, String separator)
             throw new IllegalArgumentException("Unknown header '" + header + "'");
         }
         String cellContent = WorkbookUtils.getCellValue(lastRow, column);
-        createCell(lastRow, column, isEmpty(cellContent) ? value : cellContent + separator + value);
+        createCell(lastRow, column,
+                getValueLimitedByLength(isEmpty(cellContent) ? value : cellContent + separator + value));
+    }
+
+    private String getValueLimitedByLength(String value) {
+        return StringUtils.length(value) > 32726 ? value.substring(0, 32725) + "…" : value;
     }
 
 }
diff --git a/dspace-api/src/main/java/org/dspace/content/integration/crosswalks/XlsCrosswalk.java b/dspace-api/src/main/java/org/dspace/content/integration/crosswalks/XlsCrosswalk.java
index 026b6f375dfa..cbbfee4fb49b 100644
--- a/dspace-api/src/main/java/org/dspace/content/integration/crosswalks/XlsCrosswalk.java
+++ b/dspace-api/src/main/java/org/dspace/content/integration/crosswalks/XlsCrosswalk.java
@@ -11,6 +11,7 @@
 import java.io.OutputStream;
 import java.util.List;
 
+import org.apache.commons.lang.StringUtils;
 import org.apache.poi.hssf.usermodel.HSSFWorkbook;
 import org.apache.poi.ss.usermodel.Cell;
 import org.apache.poi.ss.usermodel.Row;
@@ -45,7 +46,7 @@ protected void writeRows(List<List<String>> rows, OutputStream out) {
                 int cellCount = 0;
                 for (String field : row) {
                     Cell cell = sheetRow.createCell(cellCount++);
-                    cell.setCellValue(field);
+                    cell.setCellValue(StringUtils.length(field) > 32726 ? field.substring(0, 32725) + "…" : field );
                 }
             }