From: Marcin Koziej <marcin.koziej@nowoczesnapolska.org.pl>
Date: Fri, 8 Feb 2013 12:38:01 +0000 (+0100)
Subject: More fixes
X-Git-Url: https://git.mdrn.pl/librarian.git/commitdiff_plain/51396a4419bb3d4b89ad65a434dfeb9e81e6f6e2

More fixes
---

diff --git a/librarian/pyhtml.py b/librarian/pyhtml.py
index c76a371..ded327b 100644
--- a/librarian/pyhtml.py
+++ b/librarian/pyhtml.py
@@ -11,6 +11,8 @@ import re
 import random
 
 
+DEFAULT_MATERIAL_FORMAT = 'odt'
+
 
 class EduModule(Xmill):
     def __init__(self, options=None):
@@ -41,6 +43,9 @@ class EduModule(Xmill):
     handle_tytul_dziela = tag('em', 'title')
     handle_slowo_obce = tag('em', 'foreign')
 
+    def handle_uwaga(self, _e):
+        return None
+
     def handle_aktywnosc(self, element):
         self.activity_counter += 1
         self.options = {
@@ -183,12 +188,16 @@ class EduModule(Xmill):
         if 'url' in element.attrib:
             return tag('a', href=element.attrib['url'])(self, element)
         elif 'material' in element.attrib:
-            formats = re.split(r"[, ]+", element.attrib.get('format', 'odt'))
+            formats = re.split(r"[, ]+",
+                               element.attrib.get('format', DEFAULT_MATERIAL_FORMAT))
+            make_url = lambda f: self.options['urlmapper'] \
+              .url_for_material(element.attrib['material'], f)
+            def_href = make_url(formats[0])
             fmt_links = []
-            for f in formats:
-                fmt_links.append(u'<a href="%s">%s</a>' % (self.options['urlmapper'].url_for_material(element.attrib['material'], f), f.upper()))
+            for f in formats[1:]:
+                fmt_links.append(u'<a href="%s">%s</a>' % (make_url(f), f.upper()))
 
-            return u"", u' (%s)' % u' '.join(fmt_links)
+            return u"<a href='%s'>", u'</a> (%s)' % (def_href, u' '.join(fmt_links))
 
 
 class Exercise(EduModule):
diff --git a/librarian/xmlutils.py b/librarian/xmlutils.py
index f5cd213..37a719b 100644
--- a/librarian/xmlutils.py
+++ b/librarian/xmlutils.py
@@ -102,7 +102,7 @@ class Xmill(object):
                 # depending on number of returned values, vals can be None, a value, or a tuple.
                 # how poorly designed is that? 9 lines below are needed just to unpack this.
                 if vals is None:
-                    return []
+                    return [self.filter_text(element.tail)]
                 else:
                     if not isinstance(vals, tuple):
                         return [vals, self.filter_text(element.tail)]