Merge branch 'refactor/patch-bookmeta-20170304' into travis

cps/book_formats.py

@@ -64,7 +64,6 @@ def default_meta(tmp_file_path, original_file_name, original_file_extension):
         tags="",
         series="",
         series_id="",
-        comments="",
         languages="")
 
 
@@ -94,7 +93,6 @@ def pdf_meta(tmp_file_path, original_file_name, original_file_extension):
         tags="",
         series="",
         series_id="",
-        comments="",
         languages="")
 
 

cps/epub.py

@@ -41,38 +41,40 @@ def get_epub_info(tmp_file_path, original_file_name, original_file_extension):
     p = tree.xpath('/pkg:package/pkg:metadata', namespaces=ns)[0]
 
     epub_metadata = {}
-    try:#maybe description isn't present
-        comments = tree.xpath("//*[local-name() = 'description']/text()")[0]
-        epub_metadata['comments'] = comments
-    except:
-        epub_metadata['comments'] = ""
 
-    for s in ['title', 'description', 'creator']:
+    for s in ['title', 'description', 'creator', 'language']:
         tmp = p.xpath('dc:%s/text()' % s, namespaces=ns)
         if len(tmp) > 0:
             epub_metadata[s] = p.xpath('dc:%s/text()' % s, namespaces=ns)[0]
         else:
             epub_metadata[s] = "Unknown"
-    #detect lang need futher modification in web.py /upload
-    try:#maybe dc:language isn't present, less possible but possible
-        lang = p.xpath('dc:language/text()', namespaces=ns)[0]
-        lang = lang.split('-', 1)[0]
-        lang.lower()
+
+    if epub_metadata['description'] == "Unknown":
+        description = tree.xpath("//*[local-name() = 'description']/text()")
+        if len(description) > 0:
+            epub_metadata['description'] = description
+        else:
+            epub_metadata['description'] = ""
+
+    if epub_metadata['language'] == "Unknown":
+        epub_metadata['language'] == ""
+    else:
+        lang = epub_metadata['language'].split('-', 1)[0].lower()
         if len(lang) == 2:
-            epub_metadata['languages'] = isoLanguages.get(part1=lang).name
+            epub_metadata['language'] = isoLanguages.get(part1=lang).name
         elif len(lang) == 3:
-            epub_metadata['languages'] = isoLanguages.get(part3=lang).name
+            epub_metadata['language'] = isoLanguages.get(part3=lang).name
         else:
-            epub_metadata['languages'] = ""
-    except:
-        epub_metadata['languages'] = ""
+            epub_metadata['language'] = ""
 
     coversection = tree.xpath("/pkg:package/pkg:manifest/pkg:item[@id='cover-image']/@href", namespaces=ns)
     coverfile = None
     if len(coversection) > 0:
         coverfile = extractCover(zip, coversection[0], coverpath, tmp_file_path)
     else:
-        coversection = tree.xpath("/pkg:package/pkg:manifest/pkg:item[@id='cover']/@href", namespaces=ns)
+        meta_cover = tree.xpath("/pkg:package/pkg:metadata/pkg:meta[@name='cover']/@content", namespaces=ns)
+        if len(meta_cover) > 0:
+            coversection = tree.xpath("/pkg:package/pkg:manifest/pkg:item[@id='"+meta_cover[0]+"']/@href", namespaces=ns)
             if len(coversection) > 0:
                 filetype = coversection[0].rsplit('.',1)[-1]
                 if filetype == "xhtml" or filetype == "html": #if cover is (x)html format
@@ -85,12 +87,6 @@ def get_epub_info(tmp_file_path, original_file_name, original_file_extension):
                     coverfile = extractCover(zip, filename, "", tmp_file_path)
                 else:
                     coverfile = extractCover(zip, coversection[0], coverpath, tmp_file_path)
-        else:
-            meta_cover = tree.xpath("/pkg:package/pkg:metadata/pkg:meta[@name='cover']/@content", namespaces=ns)
-            if len(meta_cover) > 0:
-                meta_cover_content = tree.xpath("/pkg:package/pkg:manifest/pkg:item[@id='"+meta_cover[0]+"']/@href", namespaces=ns)
-                if len(meta_cover_content) > 0:
-                    coverfile = extractCover(zip, meta_cover_content[0], coverpath, tmp_file_path)
             
     if epub_metadata['title'] is None:
         title = original_file_name
@@ -107,5 +103,4 @@ def get_epub_info(tmp_file_path, original_file_name, original_file_extension):
         tags="",
         series="",
         series_id="",
-        comments=epub_metadata['comments'],
-        languages=epub_metadata['languages'])
+        languages=epub_metadata['language'])

cps/uploader.py

@@ -7,7 +7,7 @@ import hashlib
 from collections import namedtuple
 import book_formats
 
-BookMeta = namedtuple('BookMeta', 'file_path, extension, title, author, cover, description, tags, series, series_id, comments, languages')
+BookMeta = namedtuple('BookMeta', 'file_path, extension, title, author, cover, description, tags, series, series_id, languages')
 
 """
  :rtype: BookMeta

cps/web.py

@@ -2234,7 +2234,7 @@ def upload():
         db.session.add(db_book)
         db.session.flush()# flush content get db_book.id avalible
        #add comment
-        upload_comment = Markup(meta.comments).unescape()
+        upload_comment = Markup(meta.description).unescape()
         db_comment = None
         if upload_comment != "":
             db_comment = db.Comments(upload_comment, db_book.id)