[#6778] updated the excerpt modifier to properly account for multibyte characters
This commit is contained in:
parent
9bee3bd0fd
commit
5713cf422b
|
@ -1,3 +1,8 @@
|
||||||
|
## v0.27.2 (WIP)
|
||||||
|
|
||||||
|
- Updated the excerpt modifier to properly account for multibyte characters ([#6778](https://github.com/pocketbase/pocketbase/issues/6778)).
|
||||||
|
|
||||||
|
|
||||||
## v0.27.1
|
## v0.27.1
|
||||||
|
|
||||||
- Updated example `geoPoint` API preview body data.
|
- Updated example `geoPoint` API preview body data.
|
||||||
|
|
|
@ -135,10 +135,14 @@ func (m *excerptModifier) Modify(value any) (any, error) {
|
||||||
result := strings.TrimSpace(builder.String())
|
result := strings.TrimSpace(builder.String())
|
||||||
|
|
||||||
if len(result) > m.max {
|
if len(result) > m.max {
|
||||||
result = strings.TrimSpace(result[:m.max])
|
// note: casted to []rune to properly account for multi-byte chars
|
||||||
|
runes := []rune(result)
|
||||||
if m.withEllipsis {
|
if len(runes) > m.max {
|
||||||
result += "..."
|
result = string(runes[:m.max])
|
||||||
|
result = strings.TrimSpace(result)
|
||||||
|
if m.withEllipsis {
|
||||||
|
result += "..."
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -134,6 +134,20 @@ func TestExcerptModifierModify(t *testing.T) {
|
||||||
html,
|
html,
|
||||||
plainText,
|
plainText,
|
||||||
},
|
},
|
||||||
|
|
||||||
|
// multibyte chars
|
||||||
|
{
|
||||||
|
"mutibyte chars <= max",
|
||||||
|
[]string{"4", "t"},
|
||||||
|
"аб\nв ",
|
||||||
|
"аб в",
|
||||||
|
},
|
||||||
|
{
|
||||||
|
"mutibyte chars > max",
|
||||||
|
[]string{"3", "t"},
|
||||||
|
"аб\nв ",
|
||||||
|
"аб...",
|
||||||
|
},
|
||||||
}
|
}
|
||||||
|
|
||||||
for _, s := range scenarios {
|
for _, s := range scenarios {
|
||||||
|
|
Loading…
Reference in New Issue