Adds manage to contact group

Author: elixirprogrammer

web/commanders/contact_commander.ex

@@ -68,27 +68,6 @@ defmodule Crm.ContactCommander do
     end
   end
 
-  def edit_contact_group(socket, sender) do
-    group_id = socket |> select(data: "group", from: this(sender))
-    group = Repo.get!(ContactGroup, group_id)
-    form = "<input class='form-control' name='group' value='#{group.name}'>"
-    note = case socket |> alert("Edit Group", form, buttons: [ok: "Save", cancel: "Cancel"]) do
-      { :ok, params } -> edit_group(socket, group, params["group"])
-      { :cancel, _ }  -> "anonymous"
-    end
-  end
-
-  def edit_group(socket, group, group_param) do
-    changeset = ContactGroup.changeset(group, %{name: group_param})
-    case Repo.update(changeset) do
-      {:ok, group} ->
-        socket
-        |> update(:text, set: group.name, on: "#group_#{group.id}")
-      {:error, changeset} ->
-        socket |> exec_js("alert('The name cannot be empty')")
-    end
-  end
-
   def delete_contact_note(socket, sender) do
     note_id = socket |> select(data: "noteId", from: this(sender))
     note = Repo.get!(Note, note_id)
@@ -103,19 +82,4 @@ defmodule Crm.ContactCommander do
     socket
     socket |> delete("#article_note_id_#{note.id}")
   end
-
-  def delete_contact_group(socket, sender) do
-    group_id = socket |> select(data: "group", from: this(sender))
-    group = Repo.get!(ContactGroup, group_id)
-    button = case socket |> alert("Message", "Are you Sure?", buttons: [ok: "YES", cancel: "NO"]) do
-      { :ok, params } -> delete_group(socket, group)
-      { :cancel, _ }  -> "anonymous"
-    end
-  end
-
-  def delete_group(socket, group) do
-    Repo.delete!(group)
-    socket
-    socket |> delete("#new_group_list_#{group.id}")
-  end
 end

web/controllers/contact_group_controller.ex

@@ -0,0 +1,26 @@
+defmodule Crm.ContactGroupController do
+  use Crm.Web, :controller
+
+  alias Crm.{ContactGroup}
+
+  def index(conn, _params) do
+    groups = ContactGroup.all(conn.assigns.current_user.id)
+
+    render(conn, :index, groups: groups)
+  end
+
+  def delete(conn, %{"id" => id}) do
+    group = Repo.get!(ContactGroup, id) |> Repo.preload(:user)
+    if group.user !== conn.assigns.current_user do
+      conn
+      |> put_flash(:error, "You are not the owner of that group.")
+      |> redirect(to: "/")
+    else
+      Repo.delete!(group)
+
+      conn
+      |> put_flash(:info, "Group #{group.name} deleted successfully.")
+      |> redirect(to: contact_group_path(conn, :index))
+    end
+  end
+end

web/router.ex

@@ -28,7 +28,13 @@ defmodule Crm.Router do
     resources "/contacts", ContactController
     resources "/users", UserController, only: [:edit, :update]
     get "/groups/:id", ContactController, :groups
-    get "/stories/search", ContactController, :search
+    get "/search/contacts", ContactController, :search
+  end
+
+  scope "/manage", Crm do
+    pipe_through [:browser, :authenticate_user]
+
+    resources "/groups", ContactGroupController, only: [:index, :delete]
   end
 
   # Other scopes may use custom stacks.

web/templates/contact/_new_group.html.eex

@@ -1,5 +1,6 @@
 <div class="col-md-3">
   <div class="list-group">
+    <%= link "My Groups", to: contact_group_path(@conn, :index), class: "list-group-item" %>
     <%= link to: contact_path(@conn, :index), class: active_group?(@conn) do %>
       All Contacts
       <span class="badge"><%= length(@groups) %></span>

web/templates/contact/edit.html.eex

@@ -1,6 +1,7 @@
 <div class="row">
   <div class="col-md-3">
     <div class="list-group">
+      <%= link "My Groups", to: contact_group_path(@conn, :index), class: "list-group-item" %>
       <%= link to: contact_path(@conn, :index), class: active_group?(@conn) do %>
         All Contacts
         <span class="badge"><%= @contacts_count %></span>

web/templates/contact/groups.html.eex

@@ -1,6 +1,7 @@
 <div class="row">
   <div class="col-md-3">
     <div class="list-group">
+      <%= link "My Groups", to: contact_group_path(@conn, :index), class: "list-group-item" %>
       <%= link to: contact_path(@conn, :index), class: active_group?(@conn) do %>
         All Contacts
         <span class="badge"><%= length(@contacts) %></span>

web/templates/contact/index.html.eex

@@ -1,11 +1,12 @@
 <div class="col-md-3">
   <div class="list-group">
+    <%= link "My Groups", to: contact_group_path(@conn, :index), class: "list-group-item" %>
     <%= link to: contact_path(@conn, :index), class: active_group?(@conn) do %>
       All Contacts
       <span class="badge"><%= Crm.Contact.count(@current_user.id) %></span>
     <% end %>
     <div id="group_list">
-    <%= render_many @groups, Crm.ContactView, "_new_group.html", as: :group, conn: @conn %>
+    <%= render_many @groups, Crm.ContactView, "_group.html", as: :group, conn: @conn %>
     </div>
   </div>
 </div><!-- /.col-md-3 -->

web/templates/contact/new.html.eex

@@ -1,6 +1,7 @@
 <div class="row">
   <div class="col-md-3">
     <div class="list-group">
+      <%= link "My Groups", to: contact_group_path(@conn, :index), class: "list-group-item" %>
       <%= link to: contact_path(@conn, :index), class: active_group?(@conn) do %>
         All Contacts
         <span class="badge"><%= @contacts_count %></span>

web/templates/contact/search.html.eex

@@ -0,0 +1,22 @@
+<div class="container">
+  <h3>My Groups <small class="pull-right"><%= length(@groups) %> total</small></h3>
+  <table class="table">
+    <thead>
+      <tr>
+        <th>Name</th>
+        <th colspan="3"></th>
+      </tr>
+    </thead>
+    <tbody>
+    <%= for group <- @groups do %>
+        <tr>
+          <td>
+            <%= link group.name, to: contact_path(@conn, :groups, group) %>
+          </td>
+          <td><%= link "Delete", to: contact_group_path(@conn, :delete, group), method: :delete, data: [confirm: "Are you sure?"], class: "text-danger" %></td>
+        </tr>
+      <% end %>
+    </tbody>
+  </table>
+
+</div>