Merge pull request #1901 from tactilenews/1900_fix_undefined_method_t…

…rucate_on_nil_class

Fix ActionView::Template::Error for message with nil text

app/components/chat_form/chat_form.html.erb

@@ -3,6 +3,6 @@
     <h3><%= "Zur Frage: #{reply_to.request.title}" %></h3>
     <%= hidden_field_tag 'message[reply_to_id]', reply_to&.id %>
   <% end %>
-  <%= c 'textarea', id: 'message[text]', value: reply_to && t('.reply_to_reference', text: reply_to.text.truncate(50)), required: true, placeholder: t('components.chat_form.placeholder') %>
+  <%= c 'textarea', id: 'message[text]', value: prefilled_value, required: true, placeholder: t('components.chat_form.placeholder') %>
   <%= c 'button', label: t('components.chat_form.submit'), type: :submit %>
 <% end %>

app/components/chat_form/chat_form.rb

@@ -12,5 +12,13 @@ def initialize(contributor:, reply_to: nil)
     private
 
     attr_reader :contributor, :reply_to
+
+    def prefilled_value
+      return nil unless reply_to
+
+      text = reply_to.text.present? ? reply_to.text.truncate(50) : date_time(reply_to.updated_at)
+
+      t('.reply_to_reference', text: text)
+    end
   end
 end

spec/components/chat_form_spec.rb

@@ -32,5 +32,23 @@
     it 'should have the message prefilled' do
       expect(subject).to have_css('textarea', text: I18n.t('components.chat_form.reply_to_reference', text: reply_to.text.truncate(50)))
     end
+
+    context 'message with nil text' do
+      before { params[:reply_to] = create(:message, text: '') }
+
+      it 'should display the time instead' do
+        expect(subject).to have_css('textarea',
+                                    text: I18n.t('components.chat_form.reply_to_reference', text: date_time(reply_to.updated_at)))
+      end
+    end
+
+    context 'message with nil text' do
+      before { params[:reply_to] = create(:message, text: nil) }
+
+      it 'should display the time instead' do
+        expect(subject).to have_css('textarea',
+                                    text: I18n.t('components.chat_form.reply_to_reference', text: date_time(reply_to.updated_at)))
+      end
+    end
   end
 end