Merge pull request activeadmin#3855 from davydovanton/block

timoschilling · timoschilling · commit 908e38dbfb4d · 2015-03-30T09:06:00.000+02:00
Change block.call to yield and delete excess argument in block
diff --git a/lib/active_admin/namespace.rb b/lib/active_admin/namespace.rb
@@ -110,13 +110,13 @@ def reset_menu!
     # Add a callback to be ran when we build the menu
     #
     # @param [Symbol] name The name of the menu. Default: :default
-    # @param [Proc] block The block to be ran when the menu is built
+    # @yield [ActiveAdmin::Menu] The block to be ran when the menu is built
     #
     # @return [void]
-    def build_menu(name = DEFAULT_MENU, &block)
+    def build_menu(name = DEFAULT_MENU)
       @menus.before_build do |menus|
         menus.menu name do |menu|
-          block.call(menu)
+          yield menu
         end
       end
     end
@@ -157,7 +157,7 @@ def add_current_user_to_menu(menu, priority = 10, html_options = {})
     def build_menu_collection
       @menus = MenuCollection.new
 
-      @menus.on_build do |menus|
+      @menus.on_build do
         build_default_utility_nav
 
         resources.each do |resource|
