Code blocks escaping bug fixed. Some test suite modifications.

9 files changed, 35 insertions, 56 deletions

diff --git a/markdown.py b/markdown.py
index 56b4285..7a1e9d2 100644
--- a/markdown.py
+++ b/markdown.py
@@ -1315,7 +1315,7 @@ class InlineStash:
     def _genPlaceholder(self, type):
         """ Generates placeholder """
         id = "%04d" % len(self._nodes)
-        hash = "%s%s:%s%s" % (self.prefix, type[0:3], id, 
+        hash = "%s%s:%s%s" % (self.prefix, type, id, 
                                 self.suffix) 
         return hash, id
     
@@ -1910,7 +1910,6 @@ class Markdown:
         result = []
         prefix = self.inlineStash.prefix
         strartIndex = 0
-  
         while data:
             
             index = data.find(prefix, strartIndex)
@@ -1938,6 +1937,7 @@ class Markdown:
                                                                 child.value)
                                 pos = node.childNodes.index(child)
                                 node.removeChild(child)
+                                childResult.reverse()
                                 for newChild in childResult:
                                     node.insertChild(pos, newChild)
                                 
@@ -1986,7 +1986,7 @@ class Markdown:
         if not node:
             return data, False
         
-        if isinstance(node, Element):
+        if isinstance(node, Element) and not node.nodeName in ["code", "pre"]:
             for child in node.childNodes:
                 if isinstance(child, TextNode):
                     child.value = self._handleInline(child.value, patternIndex)
diff --git a/test-markdown.py b/test-markdown.py
index 3007d72..68a10e6 100644
--- a/test-markdown.py
+++ b/test-markdown.py
@@ -244,9 +244,9 @@ def testDirectory(dir, measure_time=False, safe_mode=False) :
         t = time.clock()
         for x in repeat: 
             actual_output = md.convert(input)
-
         conversion_time = time.clock() - t
         conversion_mem = memory(mem)
+        md.reset()
         
         expected_lines = [x.encode("utf8") for x in smart_split(expected_output)]
         actual_lines = [x.encode("utf8") for x in smart_split(actual_output)]
diff --git a/tests/markdown-test/auto-links.html b/tests/markdown-test/auto-links.html
index 229ebb4..976d22b 100644
--- a/tests/markdown-test/auto-links.html
+++ b/tests/markdown-test/auto-links.html
@@ -11,8 +11,7 @@
      In a list?
  </li>
 
- <li>
-     <a href="http://example.com/">http://example.com/</a>
+ <li><a href="http://example.com/">http://example.com/</a>
  </li>
 
  <li>
@@ -24,4 +23,4 @@
 </blockquote><p>Auto-links should not occur here: <code>&lt;http://example.com/&gt;</code>
 </p>
 <pre><code>or here: &lt;http://example.com/&gt;
-</code></pre>
\ No newline at end of file
+</code></pre>
diff --git a/tests/markdown-test/markdown-syntax.html b/tests/markdown-test/markdown-syntax.html
index c789d95..40f9547 100644
--- a/tests/markdown-test/markdown-syntax.html
+++ b/tests/markdown-test/markdown-syntax.html
@@ -8,81 +8,62 @@
 </ul>
 
 <ul>
- <li>
-     <a href="#overview">Overview</a><ul>
- <li>
-     <a href="#philosophy">Philosophy</a>
+ <li><a href="#overview">Overview</a><ul>
+ <li><a href="#philosophy">Philosophy</a>
  </li>
 
- <li>
-     <a href="#html">Inline HTML</a>
+ <li><a href="#html">Inline HTML</a>
  </li>
 
- <li>
-     <a href="#autoescape">Automatic Escaping for Special Characters</a>
+ <li><a href="#autoescape">Automatic Escaping for Special Characters</a>
  </li>
 </ul>
 
  </li>
 
- <li>
-     <a href="#block">Block Elements</a><ul>
- <li>
-     <a href="#p">Paragraphs and Line Breaks</a>
+ <li><a href="#block">Block Elements</a><ul>
+ <li><a href="#p">Paragraphs and Line Breaks</a>
  </li>
 
- <li>
-     <a href="#header">Headers</a>
+ <li><a href="#header">Headers</a>
  </li>
 
- <li>
-     <a href="#blockquote">Blockquotes</a>
+ <li><a href="#blockquote">Blockquotes</a>
  </li>
 
- <li>
-     <a href="#list">Lists</a>
+ <li><a href="#list">Lists</a>
  </li>
 
- <li>
-     <a href="#precode">Code Blocks</a>
+ <li><a href="#precode">Code Blocks</a>
  </li>
 
- <li>
-     <a href="#hr">Horizontal Rules</a>
+ <li><a href="#hr">Horizontal Rules</a>
  </li>
 </ul>
 
  </li>
 
- <li>
-     <a href="#span">Span Elements</a><ul>
- <li>
-     <a href="#link">Links</a>
+ <li><a href="#span">Span Elements</a><ul>
+ <li><a href="#link">Links</a>
  </li>
 
- <li>
-     <a href="#em">Emphasis</a>
+ <li><a href="#em">Emphasis</a>
  </li>
 
- <li>
-     <a href="#code">Code</a>
+ <li><a href="#code">Code</a>
  </li>
 
- <li>
-     <a href="#img">Images</a>
+ <li><a href="#img">Images</a>
  </li>
 </ul>
 
  </li>
 
- <li>
-     <a href="#misc">Miscellaneous</a><ul>
- <li>
-     <a href="#backslash">Backslash Escapes</a>
+ <li><a href="#misc">Miscellaneous</a><ul>
+ <li><a href="#backslash">Backslash Escapes</a>
  </li>
 
- <li>
-     <a href="#autolink">Automatic Links</a>
+ <li><a href="#autolink">Automatic Links</a>
  </li>
 </ul>
 
@@ -849,4 +830,4 @@ _   underscore
 -   minus sign (hyphen)
 .   dot
 !   exclamation mark
-</code></pre>
\ No newline at end of file
+</code></pre>
diff --git a/tests/misc/arabic.html b/tests/misc/arabic.html
index fa27f2e..8f3e9c6 100644
--- a/tests/misc/arabic.html
+++ b/tests/misc/arabic.html
@@ -27,9 +27,8 @@ else:
 </code></pre>
 <h2 dir="rtl">وصلات خارجية</h2>
 <ul dir="rtl">
- <li dir="rtl">
-     <a href="http://www.python.org">الموقع الرسمي للغة بايثون</a>
+ <li dir="rtl"><a href="http://www.python.org">الموقع الرسمي للغة بايثون</a>
  </li>
 </ul>
 <p dir="rtl"> بذرة حاس 
-</p>
\ No newline at end of file
+</p>
diff --git a/tests/misc/bidi.html b/tests/misc/bidi.html
index 74d4374..2b55061 100644
--- a/tests/misc/bidi.html
+++ b/tests/misc/bidi.html
@@ -48,8 +48,7 @@ else:
 </code></pre>
 <h2 dir="rtl">وصلات خارجية</h2>
 <ul dir="rtl">
- <li dir="rtl">
-     <a href="http://www.python.org">الموقع الرسمي للغة بايثون</a>
+ <li dir="rtl"><a href="http://www.python.org">الموقع الرسمي للغة بايثون</a>
  </li>
 </ul>
 <p dir="rtl"> بذرة حاس 
@@ -57,4 +56,4 @@ else:
 <p><strong>Недвард «Нед» Фландерс</strong> (Nedward «Ned» Flanders) — вымышленный персонаж мультсериала «[Симпсоны][]», озвученный Гарри Ширером. Он и его семья живут по соседству от семьи Симпсонов. Набожный христианин, Нед является одним из столпов морали Спрингфилда. В эпизоде «Alone Again, Natura-Diddily»  он овдовел, его жена Мод погибла в результате несчастного случая. 
 </p>
 <p>Нед был одним из первых персонажей в мультсериале, который не был членом семьи Симпсонов. Начиная с первых серий, он регулярно появляется в «Симпсонах». Считается, что Нед Фландерс был назван в честь улицы <em>Northeast Flanders St.</em> в <a href="http://www.portland.gov">Портленде</a>, Орегон, родном городе создателя мультсериала Мэтта Грейнинга]]. Надпись на указателе улицы <em>NE Flanders St.</em> хулиганы часто исправляли на <em>NED Flanders St.</em>
-</p>
\ No newline at end of file
+</p>
diff --git a/tests/misc/image-2.html b/tests/misc/image-2.html
index ddc8f3f..79bd7de 100644
--- a/tests/misc/image-2.html
+++ b/tests/misc/image-2.html
@@ -1,7 +1,7 @@
 
 <p><a href="http://src.com/"><em>link!</em></a>
 </p>
-<p>*<a href="http://www.freewisdom.org">link</a>*
+<p><em><a href="http://www.freewisdom.org">link</a></em>
 </p>
 
 
diff --git a/tests/misc/stronintags.html b/tests/misc/stronintags.html
index 8f64e04..d17fa18 100644
--- a/tests/misc/stronintags.html
+++ b/tests/misc/stronintags.html
@@ -1,7 +1,7 @@
 
 <p>this is a <a href="http://example.com/"><strong>test</strong></a>
 </p>
-<p>this is a second <em></em><a href="http://example.com">test</a><em></em>
+<p>this is a second <strong><a href="http://example.com">test</a></strong>
 </p>
 <p>reference <strong>[test][]</strong>
    reference [<strong>test</strong>][]
diff --git a/tests/safe_mode/unsafe_urls.html b/tests/safe_mode/unsafe_urls.html
index 8eda30d..fe37840 100644
--- a/tests/safe_mode/unsafe_urls.html
+++ b/tests/safe_mode/unsafe_urls.html
@@ -12,7 +12,8 @@
    <a href="">link</a>
    <a href="">link</a>
 </p>
-<p><img src="" alt="img"/><a href="">ref</a>
+<p><img src="" alt="img"/>
+   <a href="">ref</a>
    <img src="" alt="imgref"/>
 </p>
 <p>These should work regardless:
@@ -21,4 +22,4 @@
    <a href="mailto:foo@bar.com">email</a>
    <a href="news:some.news.group.com">news scheme</a>
    <a href="http://example.com">http link</a>
-</p>
\ No newline at end of file
+</p>