diff --git a/lib/pinchflat/downloading/download_option_builder.ex b/lib/pinchflat/downloading/download_option_builder.ex
index 0bafc2e..3e04472 100644
--- a/lib/pinchflat/downloading/download_option_builder.ex
+++ b/lib/pinchflat/downloading/download_option_builder.ex
@@ -74,6 +74,9 @@ defmodule Pinchflat.Downloading.DownloadOptionBuilder do
{{:download_auto_subs, true}, %{download_subs: true}} ->
acc ++ [:write_auto_subs]
+ {{:download_auto_subs, true}, %{embed_subs: true}} ->
+ acc ++ [:write_auto_subs]
+
{{:embed_subs, true}, %{preferred_resolution: pr}} when pr != :audio ->
acc ++ [:embed_subs]
diff --git a/lib/pinchflat_web/controllers/media_profiles/media_profile_html/output_template_help.html.heex b/lib/pinchflat_web/controllers/media_profiles/media_profile_html/output_template_help.html.heex
index 45ecf4f..e3a1300 100644
--- a/lib/pinchflat_web/controllers/media_profiles/media_profile_html/output_template_help.html.heex
+++ b/lib/pinchflat_web/controllers/media_profiles/media_profile_html/output_template_help.html.heex
@@ -60,14 +60,14 @@
-
<.inline_code>{{ <%= k %> }}
- - <%= raw(v) %>
+ - <%= html_escape({:safe, v}) %>
Other Custom Aliases
-
<.inline_code>{{ <%= k %> }}
- - <%= raw(v) %>
+ - <%= html_escape({:safe, v}) %>
Common Options
diff --git a/test/pinchflat/downloading/download_option_builder_test.exs b/test/pinchflat/downloading/download_option_builder_test.exs
index d31c6c6..5670c38 100644
--- a/test/pinchflat/downloading/download_option_builder_test.exs
+++ b/test/pinchflat/downloading/download_option_builder_test.exs
@@ -93,15 +93,23 @@ defmodule Pinchflat.Downloading.DownloadOptionBuilderTest do
end
test "includes :write_auto_subs option when specified", %{media_item: media_item} do
- media_item = update_media_profile_attribute(media_item, %{download_subs: true, download_auto_subs: true})
+ media_item_1 = update_media_profile_attribute(media_item, %{download_subs: true, download_auto_subs: true})
+ media_item_2 = update_media_profile_attribute(media_item, %{embed_subs: true, download_auto_subs: true})
- assert {:ok, res} = DownloadOptionBuilder.build(media_item)
+ assert {:ok, res_1} = DownloadOptionBuilder.build(media_item_1)
+ assert {:ok, res_2} = DownloadOptionBuilder.build(media_item_2)
- assert :write_auto_subs in res
+ assert :write_auto_subs in res_1
+ assert :write_auto_subs in res_2
end
- test "doesn't include :write_auto_subs option when download_subs is false", %{media_item: media_item} do
- media_item = update_media_profile_attribute(media_item, %{download_subs: false, download_auto_subs: true})
+ test "doesn't include :write_auto_subs option when download_subs and embed_subs is false", %{media_item: media_item} do
+ media_item =
+ update_media_profile_attribute(media_item, %{
+ download_subs: false,
+ embed_subs: false,
+ download_auto_subs: true
+ })
assert {:ok, res} = DownloadOptionBuilder.build(media_item)