Do not use current_mailbox as begin of association chain for domain, …

…again.

app/controllers/admin/domains_controller.rb

@@ -4,7 +4,7 @@ class Admin::DomainsController < AdminController
 
   inherit_resources
 
-  actions :all, except: :show
+  actions :all, except: [:show, :index]
 
   def create
     create! do |success, error|
@@ -25,17 +25,14 @@ def destroy
   end
 =end
 
+  def index
+    @domains = Domain.managable current_mailbox
+  end
+
   def update
     update! do |success, error|
       success.html { redirect_to [:edit, :admin, resource] }
     end
   end
 
-
-  private
-
-  def begin_of_association_chain
-    current_mailbox
-  end
-
 end

app/helpers/application_helper.rb

@@ -5,7 +5,7 @@ def active_navigation(controller)
   end
 
   def domains
-    @domains ||= current_mailbox.domains
+    @domains ||= Domain.managable current_mailbox
   end
 
   def flash_class(type)

app/models/domain.rb

@@ -62,4 +62,16 @@ def self.default
     domain
   end
 
+  def self.managable(mailbox)
+    if mailbox.admin?
+      Domain
+    else
+      Domain.includes(:permissions).where('
+        (permissions.role = "editor" or permissions.role = "owner")
+        and subject_id = ?',
+        mailbox.id
+      )
+    end
+  end
+
 end

app/models/mailbox.rb

@@ -62,18 +62,6 @@ def mailboxes_for_select
     domains.map(&:mailboxes_for_select).flatten(1)
   end
 
-  def domains
-    if admin?
-      Domain
-    else
-      Domain.includes(:permissions).where('
-        (permissions.role = "editor" or permissions.role = "owner")
-        and subject_id = ?',
-        id
-      )
-    end
-  end
-
   def manager?
     permissions.any? || admin?
   end

app/views/admin/aliases/edit.html.haml

@@ -15,7 +15,7 @@
           .col-md-6
             = f.input :id, disabled: true, label: 'ID'
             = f.input :username
-            = f.association :domain, include_blank: false
+            = f.association :domain, collection: domains.all, include_blank: false
             = f.input :description
             = f.input :active
           .col-md-6

app/views/admin/domains/index.html.haml

@@ -5,7 +5,7 @@
 
 %p
   Got
-  = pluralize domains.count, 'domain'
+  = pluralize collection.count, 'domain'
 
 %table.table.table-striped
   %thead
@@ -22,4 +22,4 @@
     %th Catch-All
     %th
   %tbody
-    = render collection
+    = render collection.all

app/views/admin/mailboxes/edit.html.haml

@@ -14,7 +14,7 @@
           %legend Account Information
           = f.input :id, disabled: true, label: 'ID'
           = f.input :username, hint: 'Note, that changing the username or domain currently requires manually moving the mail_location!'
-          = f.association :domain, include_blank: false
+          = f.association :domain, collection: domains.all, include_blank: false
           = f.input :name
 
       .well