Merge pull request #2891 from quainjn/page_title_config

Allow index page_title to be set in controller

features/index/page_title.feature

@@ -28,3 +28,14 @@ Feature: Index - Page Title
       end
     """
     Then I should see the page title "List of posts"
+
+  Scenario: Set the title in controller
+    Given an index configuration of:
+    """
+      ActiveAdmin.register Post do
+        controller do
+          before_filter { @page_title = "List of #{resource_class.model_name.plural}" }
+        end
+      end
+    """
+    Then I should see the page title "List of posts"

features/show/page_title.feature

@@ -45,3 +45,14 @@ Feature: Show - Page Title
       ActiveAdmin.register Tag
     """
     Then I should see the page title "Tag #"
+
+  Scenario: Set the title in controller
+    Given a show configuration of:
+    """
+      ActiveAdmin.register Post do
+        controller do
+          before_filter { @page_title = "List of #{resource_class.model_name.plural}" }
+        end
+      end
+    """
+    Then I should see the page title "List of posts"

lib/active_admin/views/pages/index.rb

@@ -11,7 +11,7 @@ def title
           when Proc   then controller.instance_exec &config[:title]
           when String then config[:title]
           else
-            active_admin_config.plural_resource_label
+            assigns[:page_title] || active_admin_config.plural_resource_label
           end
         end
 

lib/active_admin/views/pages/show.rb

@@ -11,7 +11,7 @@ def title
           if config[:title]
             render_or_call_method_or_proc_on(resource, config[:title])
           else
-            default_title
+            assigns[:page_title] || default_title
           end
         end