Revert behavior of listnum/emlistnum in htmlbuilder without highlight (…

…#449)

lib/review/htmlbuilder.rb

@@ -467,10 +467,18 @@ def listnum(lines, id, caption, lang = nil)
     end
 
     def listnum_body(lines, lang)
-      body = lines.inject(''){|i, j| i + detab(j) + "\n"}
-      lexer = lang
-      puts highlight(:body => body, :lexer => lexer, :format => 'html',
-                     :options => {:linenos => 'inline', :nowrap => false})
+      if highlight?
+        body = lines.inject(''){|i, j| i + detab(j) + "\n"}
+        lexer = lang
+        puts highlight(:body => body, :lexer => lexer, :format => 'html',
+                       :options => {:linenos => 'inline', :nowrap => false})
+      else
+        print '<pre class="list">'
+        lines.each_with_index do |line, i|
+          puts detab((i+1).to_s.rjust(2) + ": " + line)
+        end
+        puts '</pre>'
+      end
     end
 
     def emlist(lines, caption = nil, lang = nil)
@@ -491,10 +499,20 @@ def emlistnum(lines, caption = nil, lang = nil)
       if caption.present?
         puts %Q(<p class="caption">#{compile_inline(caption)}</p>)
       end
-      body = lines.inject(''){|i, j| i + detab(j) + "\n"}
-      lexer = lang
-      puts highlight(:body => body, :lexer => lexer, :format => 'html',
-                     :options => {:linenos => 'inline', :nowrap => false})
+
+      if highlight?
+        body = lines.inject(''){|i, j| i + detab(j) + "\n"}
+        lexer = lang
+        puts highlight(:body => body, :lexer => lexer, :format => 'html',
+                       :options => {:linenos => 'inline', :nowrap => false})
+      else
+        print '<pre class="emlist">'
+        lines.each_with_index do |line, i|
+          puts detab((i+1).to_s.rjust(2) + ": " + line)
+        end
+        puts '</pre>'
+      end
+
       puts '</div>'
     end
 

lib/review/htmlutils.rb

@@ -40,7 +40,7 @@ def escape_comment(str)
       str.gsub('-', '-')
     end
 
-    def highlight_pygments?
+    def highlight?
       @book.config["pygments"].present? ||
         @book.config["highlight"] && @book.config["highlight"]["html"] == "pygments"
     end
@@ -58,7 +58,7 @@ def highlight(ops)
       if ops[:options] && ops[:options].kind_of?(Hash)
         options.merge!(ops[:options])
       end
-      return body if !highlight_pygments?
+      return body if !highlight?
 
       begin
         require 'pygments'

test/test_htmlbuilder.rb

@@ -520,6 +520,35 @@ def @chapter.list(id)
     assert_equal "<div class=\"caption-code\">\n<p class=\"caption\">リスト1.1: this is <b>test</b>&lt;&amp;&gt;_</p>\n<pre class=\"list\">def foo(a1, a2=:test)\n  (1..3).times{|i| a.include?(:foo)}\n  return true\nend\n</pre>\n</div>\n", actual
   end
 
+  def test_listnum
+    def @chapter.list(id)
+      Book::ListIndex::Item.new("samplelist",1)
+    end
+
+    @book.config["highlight"] = false
+    actual = compile_block(<<-EOS)
+//listnum[samplelist][this is @<b>{test}<&>_][ruby]{
+def foo(a1, a2=:test)
+  (1..3).times{|i| a.include?(:foo)}
+  return true
+end
+//}
+EOS
+
+    expected =<<-EOS
+<div class="code">
+<p class="caption">リスト1.1: this is <b>test</b>&lt;&amp;&gt;_</p>
+<pre class="list"> 1: def foo(a1, a2=:test)
+ 2:   (1..3).times{|i| a.include?(:foo)}
+ 3:   return true
+ 4: end
+</pre>
+</div>
+EOS
+
+    assert_equal expected, actual
+  end
+
   def test_listnum_pygments_lang
     def @chapter.list(id)
       Book::ListIndex::Item.new("samplelist",1)
@@ -582,6 +611,19 @@ def test_emlist_with_tab
     assert_equal %Q|<div class="emlist-code">\n<pre class="emlist">        lineA\n                lineB\n        lineC\n</pre>\n</div>\n|, actual
   end
 
+  def test_emlistnum
+    @book.config["highlight"] = false
+    actual = compile_block("//emlistnum{\nlineA\nlineB\n//}\n")
+    expected =<<-EOS
+<div class="emlistnum-code">
+<pre class="emlist"> 1: lineA
+ 2: lineB
+</pre>
+</div>
+EOS
+    assert_equal expected, actual
+  end
+
   def test_emlist_with_4tab
     @config["tabwidth"] = 4
     actual = compile_block("//emlist{\n\tlineA\n\t\tlineB\n\tlineC\n//}\n")