Decouple members and users resources

app/controllers/account_controller.rb

@@ -0,0 +1,12 @@
+class AccountController < ApplicationController
+  # GET /account
+  #
+  def show
+  end
+
+  # GET /account/edit
+  #
+  def edit
+    @user = current_user
+  end
+end

app/controllers/application_controller.rb

@@ -47,7 +47,7 @@ def store_location
 
   def after_sign_in_path_for(user)
     if user.members.present?
-      users_path
+      members_path
     else
       page_path("about")
     end

app/controllers/home_controller.rb

@@ -1,5 +1,5 @@
 class HomeController < ApplicationController
   def index
-    redirect_to :users if current_user
+    redirect_to :members if current_user
   end
 end

app/controllers/members_controller.rb

@@ -1,11 +1,38 @@
 class MembersController < ApplicationController
   before_filter :authenticate_user!
 
+  # GET /members
+  #
+  def index
+    @users = current_organization.users
+    @memberships = current_organization
+      .members
+      .where(user_id: @users.map(&:id))
+      .includes(:account)
+      .each_with_object({}) do |mem, ob|
+        ob[mem.user_id] = mem
+      end
+  end
+
+  # GET /members/:member_uid
+  #
+  def show
+    find_member
+    @user = @member.user
+    @movements = @member
+      .movements
+      .order('created_at DESC')
+      .page(params[:page])
+      .per(10)
+  end
+
+  # DELETE /members/:member_uid
+  #
   def destroy
     find_member
     toggle_active_posts
     @member.destroy
-    redirect_to users_path
+    redirect_to members_path
   end
 
   def toggle_manager
@@ -33,8 +60,15 @@ def toggle_active
 
   private
 
+  # TODO: rely on organization scope instead of current_organization
   def find_member
-    @member ||= current_organization.members.find(params[:id])
+    @member ||= Member.where(
+      organization_id: current_organization.id,
+      member_uid: params[:member_uid]
+    ).first
+
+    # TODO: better not found management please
+    raise unless @member
   end
 
   def toggle_active_posts

app/controllers/users_controller.rb

@@ -5,20 +5,12 @@ def scoped_users
     current_organization.users
   end
 
-  def index
-    @users = scoped_users
-    @memberships = current_organization.members.
-                   where(user_id: @users.map(&:id)).
-                   includes(:account).each_with_object({}) do |mem, ob|
-                     ob[mem.user_id] = mem
-                   end
-  end
-
+  # GET /users/:id
+  #
   def show
-    @user = find_user
-    @member = @user.as_member_of(current_organization)
-    @movements = @member.movements.order("created_at DESC").page(params[:page]).
-                 per(10)
+    @user = User.find_by_id(params[:id])
+    raise unless @user
+    authorize @user
   end
 
   def new

app/helpers/users_helper.rb

@@ -7,15 +7,15 @@ def users_as_json
       {
         id: user.id,
         avatar: avatar_url(user),
-        member_id: membership.member_uid,
+        member_uid: membership.member_uid,
         username: user.username,
         email: user.email_if_real,
         unconfirmed_email: user.unconfirmed_email,
         phone: user.phone,
         alt_phone: user.alt_phone,
         balance: membership.account_balance.to_i,
 
-        url: user_path(user),
+        url: member_path(membership.member_uid),
         edit_link: edit_user_path(user),
         cancel_link: cancel_member_path(membership),
         toggle_manager_link: toggle_manager_member_path(membership),

app/policies/user_policy.rb

@@ -3,6 +3,14 @@ def new?
     user.admins?(organization)
   end
 
+  def show?
+    return true if user.id == record.id
+
+    record.organizations.any? do |org|
+      user.admins?(org)
+    end
+  end
+
   def create?
     user.admins?(organization)
   end

app/views/account/edit.html.erb

@@ -0,0 +1,6 @@
+<h1>
+  <%= @user.username %>
+  <small><%= t ".edit_user" %></small>
+</h1>
+
+<%= render "users/form" %>

app/views/account/show.html.erb

@@ -0,0 +1,54 @@
+<div class="panel user-profile">
+  <div class="panel-body">
+    <h1>
+      <%= current_user.username %>
+      <small>
+        <% if current_user.superadmin? %>
+          <span class="label label-important">
+            <%= t ".superadmin" %>
+          </span>
+        <% end %>
+      </small>
+    </h1>
+    <div class="row">
+      <div class="col-sm-3 col-xs-5 text-center">
+        <a href="https://www.gravatar.com" target="_blank">
+          <%= image_tag avatar_url(current_user, 160) %>
+        </a>
+      </div>
+      <div class="col-sm-9 col-xs-7 break-word">
+        <%= m current_user.description %>
+        <ul class="nav nav-pills pull-right">
+          <li>
+            <%= link_to account_edit_path do %>
+              <%= glyph :pencil %>
+              <%= t "global.edit" %>
+            <% end %>
+          </li>
+        </ul>
+        <dl class="dl-horizontal">
+          <h3>
+            <%= t "global.contact_details" %>
+          </h3>
+          <% if current_user.email_if_real != "" %>
+            <dt>
+              <%= User.human_attribute_name(:email) %>
+            </dt>
+            <dd>
+              <%= current_user.email_if_real %>
+            </dd>
+          <% end %>
+          <% phones = [current_user.phone, current_user.alt_phone].select(&:present?) %>
+          <% if phones.present? %>
+            <dt>
+              <%= t(".phone", count: phones.size) %>
+            </dt>
+            <dd>
+              <%= phones.map(&:to_s).join('—') %>
+            </dd>
+          <% end %>
+        </dl>
+      </div>
+    </div>
+  </div>
+</div>

app/views/application/_navbar.html.erb

@@ -48,7 +48,7 @@
       <div class="row">
         <div class="col-xs-12 col-sm-12 col-md-12 col-lg-12">
           <ul class="nav nav-pills actions-menu">
-              <%= render 'application/menus/user_list_link' %>
+              <%= render 'application/menus/members_list_link' %>
               <%= render 'application/menus/offers_dashboard_link' %>
               <%= render 'application/menus/inquiries_list_link' %>
               <%= render 'application/menus/offers_by_tag_link' %>
@@ -63,4 +63,4 @@
     </div>
   </div>
 </nav>
-<% end %>
+<% end %>

app/views/application/menus/_members_list_link.html.erb

@@ -0,0 +1,6 @@
+<li class="<%= "active" if current_page?(members_path) %>">
+  <%= link_to members_path do %>
+    <%= glyph :user %>
+    <%= Member.model_name.human.pluralize %>
+  <% end %>
+</li>

app/views/application/menus/_user_admin_menu.html.erb

@@ -25,7 +25,7 @@
       <li class="divider" role="presentation"></li>
     <% end %>
     <li role="presentation">
-      <%= link_to current_user do %>
+      <%= link_to account_path do %>
         <%= glyph :user %>
         <%= t "layouts.application.edit_profile" %>
       <% end %>

app/views/users/_user_rows.html.erb → app/views/members/_members_rows.html.erb

@@ -3,7 +3,7 @@
   <td>
     <img ng-src="{{user.avatar}}" height: "32px" width: "32px">
   </td>
-  <td>{{user.member_id}}</td>
+  <td>{{user.member_uid}}</td>
   <td>
     <span class="glyphicon glyphicon-time" ng-if="!user.active"></span>
     <a ng-href="{{user.url}}">{{user.username}}</a>

app/views/users/index.html.erb → app/views/members/index.html.erb

@@ -1,6 +1,6 @@
 <div ng-controller="UserListCtrl">
   <h1>
-    <%= User.model_name.human.pluralize %>
+    <%= Member.model_name.human.pluralize %>
     —
     <%= link_to current_organization.name,
                 organization_path(current_organization) %>
@@ -23,7 +23,7 @@
         <li>
           <a href="<%= new_user_path %>">
             <span class="glyphicon glyphicon-plus"></span>
-            <%= t "helpers.submit.create", model: User.model_name.human %>
+            <%= t "helpers.submit.create", model: Member.model_name.human %>
           </a>
         </li>
       <% end %>
@@ -78,7 +78,7 @@
         </tr>
       </thead>
       <tbody>
-        <%= render "user_rows", users: @users %>
+        <%= render "members_rows", users: @users %>
       </tbody>
     </table>
   </div>