Added the ID column by default to resources on index tables

gregbell · gregbell · commit e0b83c312a77 · 2010-09-02T17:10:07.000-07:00
diff --git a/lib/active_admin/pages/index.rb b/lib/active_admin/pages/index.rb
@@ -31,6 +31,7 @@ def main_content
       # with each column displayed as well as all the default actions
       def default_index_config
         @default_index_config ||= ::ActiveAdmin::PageConfig.new(:as => :table) do |display|
+          display.id
           resource_class.content_columns.each do |column|
             display.column column.name.to_sym
           end
diff --git a/lib/active_admin/pages/index/table.rb b/lib/active_admin/pages/index/table.rb
@@ -21,6 +21,11 @@ def to_html(page_config, collection)
         #
         class TableBuilder < ::ActiveAdmin::TableBuilder
 
+          # Display a column for the id
+          def id
+            column 'ID', :id
+          end
+
           # Adds links to View, Edit and Delete
           def default_actions(options = {})
             options = {
diff --git a/spec/integration/index_as_table_spec.rb b/spec/integration/index_as_table_spec.rb
@@ -41,6 +41,9 @@
       it "should render a new link" do
         response.should have_tag("a", "New Post", :attributes => {:href => "/admin/posts/new"})
       end
+      it "should render an ID header" do
+        response.should have_tag("th", "ID")
+      end
       it "should render a table with default sortable headers" do
         response.should have_tag("a", :content => "Title", 
                                       :parent => { :tag => "th" })
