X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/aa8dd75e5ec959e377f469a83bf67898472a67e5..735a5cf9e25b81d03db80ca4393aa34ea74d612a:/lib/rich_text.rb

diff --git a/lib/rich_text.rb b/lib/rich_text.rb
index b720f33e7..bdf9c37ca 100644
--- a/lib/rich_text.rb
+++ b/lib/rich_text.rb
@@ -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 || "")
@@ -110,8 +112,9 @@ module RichText
     end
 
     def description
-      @paragraph_element = first_paragraph_element(document.root) unless defined? @paragraph_element
-      text_content(@paragraph_element) if @paragraph_element
+      return @description if defined? @description
+
+      @description = first_truncated_text_content(document.root)
     end
 
     private
@@ -129,28 +132,35 @@ module RichText
       end
     end
 
-    def first_paragraph_element(element)
-      return element if paragraph?(element)
-
-      element.children.find do |child|
-        nested_paragraph = first_paragraph_element(child)
-        break nested_paragraph if nested_paragraph
+    def first_truncated_text_content(element)
+      if paragraph?(element)
+        truncated_text_content(element)
+      else
+        element.children.find do |child|
+          text = first_truncated_text_content(child)
+          break text unless text.nil?
+        end
       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
+      return nil if text.blank?
+
+      text.truncate(MAX_DESCRIPTION_LENGTH)
     end
 
     def image?(element)
@@ -158,7 +168,7 @@ module RichText
     end
 
     def paragraph?(element)
-      element.type == :p
+      element.type == :p || (element.type == :html_element && element.value == "p")
     end
   end