trim pre/post spaces in ruby. Cloess: #1837

kmuto · Jul 25, 2022 · 0c1cc13 · 0c1cc13
1 parent ec6df4f
commit 0c1cc13
Show file tree

Hide file tree

Showing 11 changed files with 31 additions and 6 deletions.
diff --git a/doc/format.ja.md b/doc/format.ja.md
@@ -1067,7 +1067,7 @@ UL1-OL1-PARAGRAPH
 * `@<column>{ラベルまたは見出し}` : コラムを参照します。
 
 ### その他
-* `@<ruby>{親文字, ルビ}` : ルビを振ります。たとえば `@<ruby>{愕然, がくぜん}` のように表記します。
+* `@<ruby>{親文字,ルビ}` : ルビを振ります。たとえば `@<ruby>{愕然,がくぜん}` のように表記します。
 * `@<br>{}` : 段落途中で改行します。濫用は避けたいところですが、表のセル内や箇条書き内などで必要になることもあります。
 * `@<uchar>{番号}` : Unicode文字を出力します。引数は16進数で指定します。
 * `@<href>{URL}`, `@<href>{URL, 文字表現}` : ハイパーリンクを作成します（後述）。

diff --git a/doc/format.md b/doc/format.md
@@ -1113,7 +1113,7 @@ Output:
 ### Other inline commands
 
 ```
-@<ruby>{Matsumoto, Matz}:: ruby markups
+@<ruby>{Matsumoto,Matz}:: ruby markups
 @<br>{}::  linebreak in paragraph
 @<uchar>{2460}:: Unicode code point
 @<href>{http://www.google.com/, google}:: hyper link(URL)

diff --git a/lib/review/builder.rb b/lib/review/builder.rb
@@ -401,10 +401,10 @@ def inline_bou(str)
     def inline_ruby(arg)
       base, *ruby = *arg.scan(/(?:(?:(?:\\\\)*\\,)|[^,\\]+)+/)
       if base
-        base = base.gsub(/\\,/, ',')
+        base = base.gsub(/\\,/, ',').strip
       end
       if ruby
-        ruby = ruby.join(',').gsub(/\\,/, ',')
+        ruby = ruby.join(',').gsub(/\\,/, ',').strip
       end
       compile_ruby(base, ruby)
     end

diff --git a/lib/review/idgxmlbuilder.rb b/lib/review/idgxmlbuilder.rb
@@ -692,7 +692,7 @@ def endnote_item(id)
     end
 
     def compile_ruby(base, ruby)
-      %Q(<GroupRuby><aid:ruby xmlns:aid="http://ns.adobe.com/AdobeInDesign/3.0/"><aid:rb>#{escape(base.strip)}</aid:rb><aid:rt>#{escape(ruby.strip)}</aid:rt></aid:ruby></GroupRuby>)
+      %Q(<GroupRuby><aid:ruby xmlns:aid="http://ns.adobe.com/AdobeInDesign/3.0/"><aid:rb>#{escape(base)}</aid:rb><aid:rt>#{escape(ruby)}</aid:rt></aid:ruby></GroupRuby>)
     end
 
     def compile_kw(word, alt)

diff --git a/test/test_builder.rb b/test/test_builder.rb
@@ -71,6 +71,8 @@ def @b.compile_ruby(base, ruby)
     end
     str = @b.inline_ruby('foo,bar')
     assert_equal str, ['foo', 'bar']
+    str = @b.inline_ruby('    foo    ,    bar  ')
+    assert_equal str, ['foo', 'bar']
     str = @b.inline_ruby('foo\\,\\,,\\,bar,buz')
     assert_equal str, ['foo,,', ',bar,buz']
   end

diff --git a/test/test_htmlbuilder.rb b/test/test_htmlbuilder.rb
@@ -419,6 +419,9 @@ def test_inline_balloon
   def test_inline_ruby
     actual = compile_inline('@<ruby>{粗雑,クルード}と思われているなら@<ruby>{繊細,テクニカル}にやり、繊細と思われているなら粗雑にやる。')
     assert_equal '<ruby>粗雑<rp>（</rp><rt>クルード</rt><rp>）</rp></ruby>と思われているなら<ruby>繊細<rp>（</rp><rt>テクニカル</rt><rp>）</rp></ruby>にやり、繊細と思われているなら粗雑にやる。', actual
+
+    actual = compile_inline('@<ruby>{  粗雑  ,  クルード  }と思われているなら@<ruby>{  繊細  ,テクニカル  }にやり、繊細と思われているなら粗雑にやる。')
+    assert_equal '<ruby>粗雑<rp>（</rp><rt>クルード</rt><rp>）</rp></ruby>と思われているなら<ruby>繊細<rp>（</rp><rt>テクニカル</rt><rp>）</rp></ruby>にやり、繊細と思われているなら粗雑にやる。', actual
   end
 
   def test_inline_ruby_comma

diff --git a/test/test_idgxmlbuilder.rb b/test/test_idgxmlbuilder.rb
@@ -229,7 +229,10 @@ def test_inline_uchar
   end
 
   def test_inline_ruby
-    actual = compile_inline('@<ruby>{coffin, bed}')
+    actual = compile_inline('@<ruby>{coffin,bed}')
+    assert_equal %Q(<GroupRuby><aid:ruby xmlns:aid="http://ns.adobe.com/AdobeInDesign/3.0/"><aid:rb>coffin</aid:rb><aid:rt>bed</aid:rt></aid:ruby></GroupRuby>), actual
+
+    actual = compile_inline('@<ruby>{     coffin  ,   bed   }')
     assert_equal %Q(<GroupRuby><aid:ruby xmlns:aid="http://ns.adobe.com/AdobeInDesign/3.0/"><aid:rb>coffin</aid:rb><aid:rt>bed</aid:rt></aid:ruby></GroupRuby>), actual
   end
 

diff --git a/test/test_latexbuilder.rb b/test/test_latexbuilder.rb
@@ -327,9 +327,17 @@ def test_inline_pageref
     assert_equal 'test p.\pageref{p1}', actual
   end
 
+  def test_inline_ruby
+    actual = compile_inline('@<ruby>{  foo  ,  bar  }')
+    assert_equal '\\ruby{foo}{bar}', actual
+  end
+
   def test_inline_ruby_comma
     actual = compile_inline('@<ruby>{foo\\, bar\\, buz,フー・バー・バズ}')
     assert_equal '\\ruby{foo, bar, buz}{フー・バー・バズ}', actual
+
+    actual = compile_inline('@<ruby>{foo\\, bar\\, buz    ,  フー・バー・バズ  }')
+    assert_equal '\\ruby{foo, bar, buz}{フー・バー・バズ}', actual
   end
 
   def test_inline_uchar

diff --git a/test/test_plaintextbuilder.rb b/test/test_plaintextbuilder.rb
@@ -100,6 +100,9 @@ def test_inline_raw
   def test_inline_ruby
     actual = compile_inline('@<ruby>{coffin,bed}')
     assert_equal 'coffin', actual
+
+    actual = compile_inline('@<ruby>{  coffin  ,  bed  }')
+    assert_equal 'coffin', actual
   end
 
   def test_inline_kw

diff --git a/test/test_rstbuilder.rb b/test/test_rstbuilder.rb
@@ -110,6 +110,9 @@ def test_inline_raw
   def test_inline_ruby
     actual = compile_inline('@<ruby>{coffin,bed}')
     assert_equal ' :ruby:`coffin`<bed>`_ ', actual
+
+    actual = compile_inline('@<ruby>{  coffin  ,  bed  }')
+    assert_equal ' :ruby:`coffin`<bed>`_ ', actual
   end
 
   def test_inline_kw

diff --git a/test/test_topbuilder.rb b/test/test_topbuilder.rb
@@ -69,6 +69,9 @@ def test_inline_raw
   def test_inline_ruby
     actual = compile_inline('@<ruby>{coffin,bed}')
     assert_equal 'coffin◆→DTP連絡:「coffin」に「bed」とルビ←◆', actual
+
+    actual = compile_inline('@<ruby>{  coffin  ,  bed  }')
+    assert_equal 'coffin◆→DTP連絡:「coffin」に「bed」とルビ←◆', actual
   end
 
   def test_inline_kw