Moved default table block from Index page to IndexAsTable.

This allows you to pass a set of options to the #index method without
passing in a block. The default block will be rendered.

Author: gregbell

features/index/index_scopes.feature

@@ -49,9 +49,7 @@ Feature: Index Scoping
       """
       ActiveAdmin.register Post do
         scope :all, :default => true
-        index :as => :table, :scope_count => false do
-          column :id
-        end
+        index :as => :table, :scope_count => false
       end
       """
     Then I should see the scope "All" selected

lib/active_admin/views/index_as_table.rb

@@ -102,14 +102,25 @@ def build(page_presenter, collection)
         }
 
         table_for collection, table_options do |t|
-          instance_exec(t, &page_presenter.block)
+          table_config_block = page_presenter.block || default_table
+          instance_exec(t, &table_config_block)
         end
       end
 
       def table_for(*args, &block)
         insert_tag IndexTableFor, *args, &block
       end
 
+      def default_table
+        proc do
+          id_column
+          resource_class.content_columns.each do |col|
+            column col.name.to_sym
+          end
+          default_actions
+        end
+      end
+
       #
       # Extend the default ActiveAdmin::Views::TableFor with some
       # methods for quickly displaying items on the index page

lib/active_admin/views/pages/index.rb

@@ -29,8 +29,7 @@ def main_content
         end
 
         protected
-        
-        
+
         # TODO: Refactor to new HTML DSL
         def build_download_format_links(formats = [:csv, :xml, :json])
           links = formats.collect do |format|
@@ -41,25 +40,20 @@ def build_download_format_links(formats = [:csv, :xml, :json])
 
         def build_scopes
           if active_admin_config.scopes.any?
-            options = {}
-            options[:scope_count] = config[:scope_count].nil? ? true : config[:scope_count]
-            
+            scope_options = {
+              :scope_count => config[:scope_count].nil? ? true : config[:scope_count]
+            }
+
             div :class => "table_tools" do
-              scopes_renderer active_admin_config.scopes, options
+              scopes_renderer active_admin_config.scopes, scope_options
             end
           end
         end
 
         # Creates a default configuration for the resource class. This is a table
         # with each column displayed as well as all the default actions
         def default_index_config
-          @default_index_config ||= ::ActiveAdmin::PagePresenter.new(:as => :table) do |display|
-            id_column
-            resource_class.content_columns.each do |col|
-              column col.name.to_sym
-            end
-            default_actions
-          end
+          @default_index_config ||= ::ActiveAdmin::PagePresenter.new(:as => :table)
         end
 
         # Returns the actual class for renderering the main content on the index