Fall back to given url when feed does not contain one

Author: cruessler

lib/ex_rss/feed_adder.ex

@@ -35,6 +35,13 @@ defmodule ExRss.FeedAdder do
          {:ok, raw_feed} <- Feed.parse(response.body) do
       candidate = extract_candidate(raw_feed)
 
+      candidate =
+        if is_nil(candidate.url) do
+          Map.put(candidate, :url, url)
+        else
+          candidate
+        end
+
       {:ok, candidate}
     else
       i when is_integer(i) ->

lib/ex_rss_web/feed_live/new.ex

@@ -40,18 +40,21 @@ defmodule ExRssWeb.FeedLive.New do
       Repo.get!(User, socket.assigns.current_user.id)
       |> FeedAdder.add_feed(feed_params)
 
-    case Repo.transaction(multi) do
-      {:ok, %{feed: added_feed}} ->
-        socket =
+    socket =
+      case Repo.transaction(multi) do
+        {:ok, %{feed: added_feed}} ->
           socket
           |> assign(:candidate, nil)
           |> assign(:added_feed, added_feed)
+          # TODO
+          # Handle case where feed does not have a title.
+          |> put_flash(:info, "Feed “#{added_feed.title}” was added")
 
-        {:noreply, socket}
+        {:error, _error} ->
+          socket
+      end
 
-      {:error, _error} ->
-        {:noreply, socket}
-    end
+    {:noreply, socket}
   end
 
   def format_frequency(%{seconds: seconds, posts: posts}) do

lib/ex_rss_web/feed_live/new.html.heex

@@ -21,4 +21,6 @@
   <h1 class="text-m">This feed has been added</h1>
 
   <p><code>{@added_feed.url}</code></p>
+
+  <a href={~p"/feeds"}>Back to feeds</a>
 <% end %>