Merge pull request #91 from epochtalk/polls

Polls

Author: unenglishable

PortRoadmap.md

@@ -80,7 +80,7 @@
 | bct-trust                | posts.byThread.post            | userTrust                | :white_check_mark: |
 | bct-trust                | posts.byThread.post            | showTrust                | :white_check_mark: |
 | ept-auto-moderation      | posts.create.pre               | moderate                 | :white_check_mark: |
-| ept-auto-moderation      | posts.update.pre               | moderate                 | :x: |
+| ept-auto-moderation      | posts.update.pre               | moderate                 | :white_check_mark: |
 | ept-auto-moderation      | threads.create.pre             | moderate                 | :white_check_mark: |
 | ept-ignore-users         | users.find.post                | userIgnored              | :x: |
 | ept-ignore-users         | posts.patroller.post           | isIgnored                | :x: |
@@ -93,14 +93,14 @@
 | ept-mentions             | posts.search.post              | userIdToUsername         | :x: |
 | ept-mentions             | mentions.page.post             | userIdToUsername         | :x: |
 | ept-mentions             | portal.view.post               | userIdToUsername         | :x: |
-| ept-mentions             | posts.update.post              | userIdToUsername         | :x: |
-| ept-mentions             | posts.update.post              | correctTextSearchVector  | :x: |
+| ept-mentions             | posts.update.post              | userIdToUsername         | :white_check_mark: |
+| ept-mentions             | posts.update.post              | correctTextSearchVector  | :white_check_mark: |
 | ept-mentions             | posts.create.pre               | usernameToUserId         | :white_check_mark: |
-| ept-mentions             | posts.update.pre               | usernameToUserId         | :x: |
+| ept-mentions             | posts.update.pre               | usernameToUserId         | :white_check_mark: |
 | ept-mentions             | threads.create.pre             | usernameToUserId         | :white_check_mark: |
 | ept-mentions             | posts.create.post              | createMention            | :white_check_mark: |
 | ept-mentions             | posts.create.post              | correctTextSearchVector  | :white_check_mark: |
-| ept-mentions             | posts.update.post              | removeMentionIds         | :x: |
+| ept-mentions             | posts.update.post              | removeMentionIds         | **N/A** |
 | ept-mentions             | threads.create.post            | createMention            | :white_check_mark: |
 | ept-mentions             | threads.create.post            | correctTextSearchVector  | :white_check_mark: |
 | ept-mentions             | users.find.post                | userIgnoredMentions      | :x: |

lib/epochtalk_server/models/board.ex

@@ -334,6 +334,46 @@ defmodule EpochtalkServer.Models.Board do
       else: {:error, :board_does_not_exist}
   end
 
+  @doc """
+  Returns boolean indicating if `Board` has `Post` edit disabled, which is determined by
+  the `post_edit_diabled` setting within the `meta` field of the `Board`
+  """
+  @spec is_post_edit_disabled_by_thread_id(
+          thread_id :: non_neg_integer,
+          post_created_at :: NaiveDateTime.t()
+        ) :: boolean
+  def is_post_edit_disabled_by_thread_id(thread_id, post_created_at) when is_integer(thread_id) do
+    query =
+      from b in Board,
+        left_join: t in Thread,
+        on: t.board_id == b.id,
+        where: t.id == ^thread_id,
+        select: b.meta["disable_post_edit"]
+
+    disable_post_edit = Repo.one(query)
+
+    # check time elapsed if post edit is disabled
+    editing_disabled =
+      if is_integer(disable_post_edit) and disable_post_edit > -1 do
+        current_time =
+          NaiveDateTime.utc_now()
+          |> DateTime.from_naive!("Etc/UTC")
+          |> DateTime.to_unix(:millisecond)
+
+        post_created_at =
+          post_created_at |> DateTime.from_naive!("Etc/UTC") |> DateTime.to_unix(:millisecond)
+
+        editable_for_mins = disable_post_edit * 60 * 1000
+
+        current_time - post_created_at >= editable_for_mins
+      else
+        # diable_post_edit not set
+        false
+      end
+
+    editing_disabled
+  end
+
   @doc """
   Used to obtain breadcrumb data for a specific `Board` given it's `slug`
   """

lib/epochtalk_server/models/board_ban.ex

@@ -47,8 +47,8 @@ defmodule EpochtalkServer.Models.BoardBan do
   Returns `true` if the user is banned from the specified `Board` or `false`
   otherwise.
   """
-  @spec is_banned_from_board(user :: User.t(), opts :: list) :: {:ok, banned :: boolean}
-  def is_banned_from_board(%{id: user_id} = _user, opts) when is_integer(user_id) do
+  @spec banned_from_board?(user :: User.t(), opts :: list) :: {:ok, banned :: boolean}
+  def banned_from_board?(%{id: user_id} = _user, opts) when is_integer(user_id) do
     board_id = Keyword.get(opts, :board_id)
     post_id = Keyword.get(opts, :post_id)
     thread_id = Keyword.get(opts, :thread_id)
@@ -91,7 +91,7 @@ defmodule EpochtalkServer.Models.BoardBan do
     {:ok, result}
   end
 
-  def is_banned_from_board(nil, _opts), do: {:ok, false}
+  def banned_from_board?(nil, _opts), do: {:ok, false}
 
   @doc """
   Used to check if `User` with specified `user_id` is not banned from board.
@@ -100,13 +100,13 @@ defmodule EpochtalkServer.Models.BoardBan do
   Returns `true` if the user is **not** banned from the specified `Board` or `false`
   otherwise.
   """
-  @spec is_not_banned_from_board(user :: User.t(), opts :: list) :: {:ok, not_banned :: boolean}
-  def is_not_banned_from_board(user, opts \\ [])
+  @spec not_banned_from_board?(user :: User.t(), opts :: list) :: {:ok, not_banned :: boolean}
+  def not_banned_from_board?(user, opts \\ [])
 
-  def is_not_banned_from_board(%{id: user_id} = user, opts) when is_integer(user_id) do
-    {:ok, banned} = is_banned_from_board(user, opts)
+  def not_banned_from_board?(%{id: user_id} = user, opts) when is_integer(user_id) do
+    {:ok, banned} = banned_from_board?(user, opts)
     {:ok, !banned}
   end
 
-  def is_not_banned_from_board(nil, _opts), do: {:ok, true}
+  def not_banned_from_board?(nil, _opts), do: {:ok, true}
 end

lib/epochtalk_server/models/poll.ex

@@ -6,6 +6,7 @@ defmodule EpochtalkServer.Models.Poll do
   alias EpochtalkServer.Repo
   alias EpochtalkServer.Models.Poll
   alias EpochtalkServer.Models.PollAnswer
+  alias EpochtalkServer.Models.PollResponse
   alias EpochtalkServer.Models.Thread
 
   @moduledoc """
@@ -24,11 +25,14 @@ defmodule EpochtalkServer.Models.Poll do
   @derive {Jason.Encoder,
            only: [
              :thread_id,
+             :id,
              :question,
              :locked,
              :max_answers,
              :expiration,
              :change_vote,
+             :has_voted,
+             :answers,
              :display_mode
            ]}
   schema "polls" do
@@ -39,6 +43,8 @@ defmodule EpochtalkServer.Models.Poll do
     field :expiration, :naive_datetime
     field :change_vote, :boolean
     field :display_mode, Ecto.Enum, values: [:always, :voted, :expired]
+    field :has_voted, :boolean, virtual: true
+    field :answers, {:array, :map}, virtual: true
     has_many :poll_answers, PollAnswer
   end
 
@@ -142,6 +148,25 @@ defmodule EpochtalkServer.Models.Poll do
     end)
   end
 
+  @doc """
+  Returns boolean indicating if the specified `User` has voted on the specified `Thread`
+  """
+  @spec has_voted(thread_id :: integer, user_id :: integer) :: boolean
+  def has_voted(thread_id, user_id) do
+    query =
+      from p in Poll,
+        left_join: pr in PollResponse,
+        on: true,
+        left_join: pa in PollAnswer,
+        on: true,
+        where:
+          p.id == pa.poll_id and pr.answer_id == pa.id and p.thread_id == ^thread_id and
+            pr.user_id == ^user_id,
+        select: pr.user_id
+
+    Repo.exists?(query)
+  end
+
   @doc """
   Queries `Poll` Data by thread
   """

lib/epochtalk_server/models/poll_answer.ex

@@ -15,10 +15,11 @@ defmodule EpochtalkServer.Models.PollAnswer do
           poll_id: non_neg_integer | nil,
           answer: String.t() | nil
         }
-  @derive {Jason.Encoder, only: [:poll_id, :answer]}
+  @derive {Jason.Encoder, only: [:id, :answer, :votes]}
   schema "poll_answers" do
     belongs_to :poll, Poll
     field :answer, :string
+    field :votes, :integer, virtual: true, default: 0
     has_many :poll_responses, PollResponse, foreign_key: :answer_id
   end
 

lib/epochtalk_server/models/poll_response.ex

@@ -2,8 +2,8 @@ defmodule EpochtalkServer.Models.PollResponse do
   use Ecto.Schema
   import Ecto.Changeset
   # import Ecto.Query
-  # alias EpochtalkServer.Repo
-  # alias EpochtalkServer.Models.PollResponse
+  alias EpochtalkServer.Repo
+  alias EpochtalkServer.Models.PollResponse
   alias EpochtalkServer.Models.PollAnswer
   alias EpochtalkServer.Models.User
 
@@ -50,4 +50,21 @@ defmodule EpochtalkServer.Models.PollResponse do
     |> foreign_key_constraint(:answer_id, name: :poll_responses_answer_id_fkey)
     |> foreign_key_constraint(:user_id, name: :poll_responses_user_id_fkey)
   end
+
+  ## === Database Functions ===
+
+  @doc """
+  Create on `PollResponse` for specific poll. Used to vote for a `Poll`
+  """
+  @spec create(attrs :: map, user_id :: integer) :: :ok
+  def create(%{"answer_ids" => answer_ids} = attrs, user_id)
+      when is_map(attrs) and is_integer(user_id),
+      do: Enum.each(answer_ids, &PollResponse.create(&1, user_id))
+
+  def create(answer_id, user_id) do
+    poll_response_cs =
+      create_changeset(%PollResponse{}, %{answer_id: answer_id, user_id: user_id})
+
+    Repo.insert(poll_response_cs)
+  end
 end