cst1/openstreetmap-website
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Truncate long richtext descriptions

Browse source
This commit is contained in:
Anton Khorev 2024-08-07 06:53:43 +03:00
parent e5279dacde
commit 88a7ca5625
2 changed files with 20 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

13
lib/rich_text.rb
View file

@ -3,6 +3,8 @@ module RichText
"Business Description:", "Additional Keywords:"
].freeze
MAX_DESCRIPTION_LENGTH = 500
def self.new(format, text)
case format
when "html" then HTML.new(text || "")
@ -111,7 +113,7 @@ module RichText
def description
@paragraph_element = first_paragraph_element(document.root) unless defined? @paragraph_element
text_content(@paragraph_element) if @paragraph_element
truncated_text_content(@paragraph_element) if @paragraph_element
end
private
@ -138,19 +140,22 @@ module RichText
end
end
def text_content(element)
def truncated_text_content(element)
text = ""
append_text = lambda do |child|
if child.type == :text
text << child.value
else
child.children.each { |c| append_text.call(c) }
child.children.each do |c|
append_text.call(c)
break if text.length > MAX_DESCRIPTION_LENGTH
end
end
end
append_text.call(element)
text
text.truncate(MAX_DESCRIPTION_LENGTH)
end
def image?(element)

11
test/lib/rich_text_test.rb
View file

@ -355,6 +355,17 @@ class RichTextTest < ActiveSupport::TestCase
assert_equal "Can use HTML tags.", r.description
end
def test_markdown_description_max_length
r = RichText.new("markdown", "x" * RichText::MAX_DESCRIPTION_LENGTH)
assert_equal "x" * RichText::MAX_DESCRIPTION_LENGTH, r.description
r = RichText.new("markdown", "y" * (RichText::MAX_DESCRIPTION_LENGTH + 1))
assert_equal "#{'y' * (RichText::MAX_DESCRIPTION_LENGTH - 3)}...", r.description
r = RichText.new("markdown", "*zzzzzzzzz*z" * ((RichText::MAX_DESCRIPTION_LENGTH + 1) / 10.0).ceil)
assert_equal "#{'z' * (RichText::MAX_DESCRIPTION_LENGTH - 3)}...", r.description
end
private
def assert_html(richtext, &block)