potsda.mn/mobilizon
10
4
Fork 1
Code Issues 22 Pull requests 1 Projects Releases Wiki Activity

Merge branch 'html_to_text-nil' into 'main'

Browse source 
fix: allow html_to_text/1 to receive nil, e.g. for empty event descriptions

See merge request framasoft/mobilizon!1502
This commit is contained in:
Thomas Citharel 2023-12-11 09:44:06 +00:00
parent 987c5b59d3 5030b755a0
commit f6ff99987f
2 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
lib/service/formatter/html.ex
View file

@ -41,7 +41,9 @@ defmodule Mobilizon.Service.Formatter.HTML do
def strip_tags_and_insert_spaces(html), do: html
@spec html_to_text(String.t()) :: String.t()
@spec html_to_text(String.t() | nil) :: String.t() | nil
def html_to_text(nil), do: nil
def html_to_text(html) do
html
|> String.replace(~r/<li>/, "\\g{1}- ", global: true)

2
test/service/export/icalendar_test.exs
View file

@ -44,11 +44,13 @@ defmodule Mobilizon.Service.ICalendarTest do
%Event{} = event = insert(:event, title: "I'm public")
%Event{} = event2 = insert(:event, visibility: :private, title: "I'm private")
%Event{} = event3 = insert(:event, title: "Another public")
%Event{} = event4 = insert(:event, title: "No description", description: nil)
{:commit, ics} = ICalendarService.create_cache("instance")
assert ics =~ event.title
refute ics =~ event2.title
assert ics =~ event3.title
assert ics =~ event4.title
end
test "with 50 events" do