Merge pull request #65 from doughsay/api-recent-player-states

Recent player states API

lib/ambry_web/controllers/api/book_controller.ex

@@ -1,21 +1,16 @@
 defmodule AmbryWeb.API.BookController do
   use AmbryWeb, :controller
 
+  import AmbryWeb.API.ControllerUtils
+
   alias Ambry.Books
 
   action_fallback AmbryWeb.FallbackController
 
   @limit 10
 
   def index(conn, params) do
-    page =
-      case params |> Map.get("page", "1") |> Integer.parse() do
-        {page, _} when page >= 1 -> page
-        {_bad_page, _} -> 1
-        :error -> 1
-      end
-
-    offset = page_to_offset(page)
+    offset = offset_from_params(params, @limit)
 
     books = Books.get_recent_books!(offset, @limit)
     render(conn, "index.json", books: books)
@@ -25,8 +20,4 @@ defmodule AmbryWeb.API.BookController do
     book = Books.get_book_with_media!(id)
     render(conn, "show.json", book: book)
   end
-
-  defp page_to_offset(page) do
-    page * @limit - @limit
-  end
 end

lib/ambry_web/controllers/api/controller_utils.ex

@@ -0,0 +1,16 @@
+defmodule AmbryWeb.API.ControllerUtils do
+  @moduledoc """
+  Util functions for API controllers.
+  """
+
+  def offset_from_params(params, limit) do
+    page =
+      case params |> Map.get("page", "1") |> Integer.parse() do
+        {page, _} when page >= 1 -> page
+        {_bad_page, _} -> 1
+        :error -> 1
+      end
+
+    page * limit - limit
+  end
+end

lib/ambry_web/controllers/api/player_state_controller.ex

@@ -1,10 +1,21 @@
 defmodule AmbryWeb.API.PlayerStateController do
   use AmbryWeb, :controller
 
+  import AmbryWeb.API.ControllerUtils
+
   alias Ambry.Media
 
   action_fallback AmbryWeb.FallbackController
 
+  @limit 10
+
+  def index(conn, params) do
+    offset = offset_from_params(params, @limit)
+
+    player_states = Media.get_recent_player_states(conn.assigns.api_user.id, offset, @limit)
+    render(conn, "index.json", player_states: player_states)
+  end
+
   def show(conn, %{"id" => media_id}) do
     player_state = Media.get_or_create_player_state!(conn.assigns.api_user.id, media_id)
     render(conn, "show.json", player_state: player_state)

lib/ambry_web/router.ex

@@ -86,7 +86,7 @@ defmodule AmbryWeb.Router do
     resources "/books", BookController, only: [:index, :show]
     resources "/people", PersonController, only: [:show]
     resources "/series", SeriesController, only: [:show]
-    resources "/player_states", PlayerStateController, only: [:show, :update]
+    resources "/player_states", PlayerStateController, only: [:index, :show, :update]
   end
 
   # Other scopes may use custom stacks.

lib/ambry_web/views/api/player_state_view.ex

@@ -3,6 +3,10 @@ defmodule AmbryWeb.API.PlayerStateView do
 
   alias AmbryWeb.API.{BookView, PlayerStateView}
 
+  def render("index.json", %{player_states: player_states}) do
+    %{data: render_many(player_states, PlayerStateView, "player_state.json")}
+  end
+
   def render("show.json", %{player_state: player_state}) do
     %{data: render_one(player_state, PlayerStateView, "player_state.json")}
   end