From 5c62c1cf9508cef98f84aaeae46a7d60b4380a85 2013-01-09 13:31:05 From: Jean-Philippe Lang Date: 2013-01-09 13:31:05 Subject: [PATCH] Merged r11126 from trunk (#12744). git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/branches/2.2-stable@11145 e93f8b46-1217-0410-a6f0-8f06a7374b81 --- diff --git a/lib/redmine/wiki_formatting/macros.rb b/lib/redmine/wiki_formatting/macros.rb index 64b7814..5bafd26 100644 --- a/lib/redmine/wiki_formatting/macros.rb +++ b/lib/redmine/wiki_formatting/macros.rb @@ -147,10 +147,10 @@ module Redmine unless block_given? raise "Can not create a macro without a block!" end - name = name.to_sym if name.is_a?(String) + name = name.to_s.downcase.to_sym available_macros[name] = {:desc => @@desc || ''}.merge(options) @@desc = nil - Definitions.send :define_method, "macro_#{name}".downcase, &block + Definitions.send :define_method, "macro_#{name}", &block end # Sets description for the next macro to be defined diff --git a/test/unit/lib/redmine/wiki_formatting/macros_test.rb b/test/unit/lib/redmine/wiki_formatting/macros_test.rb index cd3ac7a..d014aa5 100644 --- a/test/unit/lib/redmine/wiki_formatting/macros_test.rb +++ b/test/unit/lib/redmine/wiki_formatting/macros_test.rb @@ -78,6 +78,12 @@ class Redmine::WikiFormatting::MacrosTest < ActionView::TestCase assert_equal "

Baz: (arg1,arg2) (String) (line1\nline2)

", textilizable("{{baz(arg1, arg2)\nline1\nline2\n}}") end + def test_macro_name_with_upper_case + Redmine::WikiFormatting::Macros.macro(:UpperCase) {|obj, args| "Upper"} + + assert_equal "

Upper

", textilizable("{{UpperCase}}") + end + def test_multiple_macros_on_the_same_line Redmine::WikiFormatting::Macros.macro :foo do |obj, args| args.any? ? "args: #{args.join(',')}" : "no args"