Merge pull request #370 from PDOK/truncate-keywords

feat: truncate text potentially overly long list of keywords in collections

Author: rkettelerij

internal/engine/templatefuncs.go

@@ -30,6 +30,7 @@ func init() {
 		"markdown":      markdown,
 		"unmarkdown":    unmarkdown,
 		"truncate":      truncateText,
+		"truncateslice": truncateSlice,
 		"humansize":     humanSize,
 		"bytessize":     bytesSize,
 		"isdate":        isDate,
@@ -102,6 +103,15 @@ func truncateText(s *string, limit int) *string {
 	return s
 }
 
+// truncateSlice truncate text separated by commas to avoid overly long text on overview pages.
+func truncateSlice(s string, limit int) string {
+	if s == "" {
+		return s
+	}
+	result := truncateText(&s, limit)
+	return *result
+}
+
 // humanSize converts size in bytes to a human-readable size.
 func humanSize(a any) string {
 	if i, ok := a.(int64); ok {

internal/engine/templatefuncs_test.go

@@ -62,6 +62,24 @@ func TestTruncateText(t *testing.T) {
 	}
 }
 
+func TestTruncateSlice(t *testing.T) {
+	tests := []struct {
+		input    string
+		limit    int
+		expected string
+	}{
+		{"This text is not too long.", 50, "This text is not too long."},
+		{"", 50, ""},
+		{"This text is longer than the configured limit allows it to be.", 50, "This text is longer than the configured limit..."},
+	}
+
+	for _, tt := range tests {
+		t.Run("", func(t *testing.T) {
+			assert.Equal(t, tt.expected, truncateSlice(tt.input, tt.limit))
+		})
+	}
+}
+
 func TestHumanSize(t *testing.T) {
 	tests := []struct {
 		input    any

internal/ogc/common/geospatial/templates/collections.go.html

@@ -50,7 +50,7 @@ <h3 class="card-title h5">Downloads</h3>
                     {{ end }}
                     {{ if and $coll.Metadata $coll.Metadata.Keywords }}
                     <li class="list-group-item">
-                        <strong>{{ i18n "Keywords" }}</strong>: {{ $coll.Metadata.Keywords | join ", " }}
+                        <strong>{{ i18n "Keywords" }}</strong>: {{ truncateslice ($coll.Metadata.Keywords | join ", ") 400 }}
                     </li>
                     {{ end }}
                     {{ if and $cfg.OgcAPI.Features $cfg.OgcAPI.Features.Collections }}