minor organizations code clean up

Author: markets

app/controllers/organizations_controller.rb

@@ -1,20 +1,12 @@
 class OrganizationsController < ApplicationController
-  before_filter :load_resource
-
-  def load_resource
-    if params[:id]
-      @organization = Organization.find(params[:id])
-    else
-      @organizations = Organization.all
-    end
-  end
+  before_filter :load_resource, except: [:new, :index, :create]
 
   def new
     @organization = Organization.new
   end
 
   def index
-    @organizations = @organizations.matching(params[:q]) if params[:q].present?
+    @organizations = Organization.all
   end
 
   def show
@@ -27,7 +19,8 @@ def show
   end
 
   def create
-    @organization = @organizations.build organization_params
+    @organization = Organization.new(organization_params)
+
     if @organization.save
       redirect_to @organization, status: :created
     else
@@ -57,6 +50,10 @@ def set_current
 
   private
 
+  def load_resource
+    @organization = Organization.find(params[:id])
+  end
+
   def organization_params
     params[:organization].permit(*%w[name theme email phone web
                                      public_opening_times description address

app/models/organization.rb

@@ -2,11 +2,7 @@ class Organization < ActiveRecord::Base
   has_many :members, dependent: :destroy
   has_many :users, -> { order "members.created_at DESC" }, through: :members
   has_many :all_accounts, class_name: "Account", inverse_of: :organization
-  has_many :all_movements, class_name: "Movement", through: :all_accounts, source: :movements do
-    def with_transfer
-      joins(transfer: :movements)
-    end
-  end
+  has_many :all_movements, class_name: "Movement", through: :all_accounts, source: :movements
   has_many :all_transfers, class_name: "Transfer", through: :all_movements, source: :transfer
   has_one :account, as: :accountable
   has_many :member_accounts, through: :members, source: :account
@@ -15,11 +11,7 @@ def with_transfer
   has_many :inquiries
   has_many :documents, as: :documentable
 
-  scope :matching, ->(str) {
-    where(Organization.arel_table[:name].matches("%#{str}%"))
-  }
-
-  validates :name, uniqueness: true
+  validates :name, presence: true, uniqueness: true
 
   before_validation :ensure_url
   after_create :create_account

app/views/organizations/_form.html.erb

@@ -1,3 +1,4 @@
+<%= show_error_messages!(@organization) %>
 <%= simple_form_for @organization do |f| %>
   <%= f.input :name %>
   <%= f.input :email %>

app/views/organizations/edit.html.erb

@@ -1,10 +1,3 @@
-<% if @organization.errors.present? %>
-  <% @organization.errors.full_messages.each do |msg| %>
-    <div class="alert alert-danger" role="alert">
-      <%= msg %>
-    </div>
-  <% end %>
-<% end %>
 <h1>
   <%= link_to @organization, organization_path(@organization) %>
   <small><%= t 'global.edit' %></small>

app/views/organizations/index.html.erb

@@ -2,7 +2,7 @@
   <%= Organization.model_name.human(count: :many) %>
 </h1>
 
-<ul class="nav nav-pills">
+<ul class="nav navbar-nav pull-right">
   <% if superadmin? %>
     <li>
       <%= link_to new_organization_path do %>