fix bug and add --html

README.md

@@ -47,6 +47,7 @@ nhentai
 
 ![](./images/search.png)  
 ![](./images/download.png)  
+![](./images/viewer.png)  
 
 ### License  
 MIT

nhentai/__init__.py

@@ -1,3 +1,3 @@
-__version__ = '0.2.7'
+__version__ = '0.2.9'
 __author__ = 'Ricter'
 __email__ = 'ricterzheng@gmail.com'

nhentai/cmdline.py

@@ -8,7 +8,7 @@ except ImportError:
     pass
 
 import nhentai.constant as constant
-from nhentai.utils import urlparse
+from nhentai.utils import urlparse, generate_html
 from nhentai.logger import logger
 
 try:
@@ -49,6 +49,7 @@ def cmd_parser():
                       help='timeout of download doujinshi')
     parser.add_option('--proxy', type='string', dest='proxy', action='store', default='',
                       help='use proxy, example: http://127.0.0.1:1080')
+    parser.add_option('--html', dest='html_viewer', action='store_true', help='generate a html viewer at current directory')
 
     try:
         sys.argv = list(map(lambda x: unicode(x.decode(sys.stdin.encoding)), sys.argv))
@@ -59,6 +60,10 @@ def cmd_parser():
 
     args, _ = parser.parse_args(sys.argv[1:])
 
+    if args.html_viewer:
+        generate_html()
+        exit(0)
+
     if args.tags:
         logger.warning('`--tags` is under construction')
         exit(0)

nhentai/command.py

@@ -11,6 +11,7 @@ from nhentai.doujinshi import Doujinshi
 from nhentai.downloader import Downloader
 from nhentai.logger import logger
 from nhentai.constant import BASE_URL
+from nhentai.utils import generate_html
 
 
 def main():
@@ -43,30 +44,7 @@ def main():
         for doujinshi in doujinshi_list:
             doujinshi.downloader = downloader
             doujinshi.download()
-
+            generate_html(doujinshi, output_dir)
-            image_html = ''
-            previous = ''
-
-            doujinshi_dir = os.path.join(options.output_dir, str(doujinshi.id))
-            file_list = os.listdir(doujinshi_dir)
-            for index, image in enumerate(file_list):
-                try:
-                    next_ = file_list[file_list.index(image) + 1]
-                except IndexError:
-                    next_ = ''
-
-                image_html += '<img src="{0}" class="image-item {1}" attr-prev="{2}" attr-next="{3}">\n'\
-                    .format(image, 'current' if index == 0 else '', previous, next_)
-                previous = image
-
-            with open(os.path.join(os.path.dirname(__file__), 'doujinshi.html'), 'r') as template:
-                html = template.read()
-
-            data = html.format(TITLE=doujinshi.name, IMAGES=image_html)
-            with open(os.path.join(doujinshi_dir, 'index.html'), 'w') as f:
-                f.write(data)
-
-            logger.log(15, 'HTML Viewer has been write to \'{0}\''.format(os.path.join(doujinshi_dir, 'index.html')))
 
         if not platform.system() == 'Windows':
             logger.log(15, '🍺 All done.')

nhentai/downloader.py

@@ -67,8 +67,8 @@ class Downloader(Singleton):
     def _download_callback(self, request, result):
         result, data = result
         if result == 0:
-            logger.critical('fatal errors occurred, quit.')
+            logger.warning('fatal errors occurred, ignored')
-            exit(1)
+            # exit(1)
         elif result == -1:
             logger.warning('url {} return status code 404'.format(data))
         else:

nhentai/utils.py

@@ -1,6 +1,9 @@
 # coding: utf-8
 from __future__ import unicode_literals, print_function
 
+import os
+from nhentai.logger import logger
+
 
 class _Singleton(type):
     """ A metaclass that creates a Singleton base class when called. """
@@ -23,3 +26,43 @@ def urlparse(url):
         from urllib.parse import urlparse
 
     return urlparse(url)
+
+
+def generate_html(output_dir='.', doujinshi_obj=None):
+    image_html = ''
+    previous = ''
+
+    if doujinshi_obj is not None:
+        doujinshi_dir = os.path.join(output_dir, str(doujinshi_obj.id))
+    else:
+        doujinshi_dir = '.'
+
+    file_list = os.listdir(doujinshi_dir)
+    file_list.sort()
+
+    for index, image in enumerate(file_list):
+        if not os.path.splitext(image)[1] in ('.jpg', '.png'):
+            continue
+
+        try:
+            next_ = file_list[file_list.index(image) + 1]
+        except IndexError:
+            next_ = ''
+
+        image_html += '<img src="{0}" class="image-item {1}" attr-prev="{2}" attr-next="{3}">\n'\
+            .format(image, 'current' if index == 0 else '', previous, next_)
+        previous = image
+
+    with open(os.path.join(os.path.dirname(__file__), 'doujinshi.html'), 'r') as template:
+        html = template.read()
+
+    if doujinshi_obj is not None:
+        title = doujinshi_obj.name
+    else:
+        title = 'nHentai HTML Viewer'
+
+    data = html.format(TITLE=title, IMAGES=image_html)
+    with open(os.path.join(doujinshi_dir, 'index.html'), 'w') as f:
+        f.write(data)
+
+    logger.log(15, 'HTML Viewer has been write to \'{0}\''.format(os.path.join(doujinshi_dir, 'index.html')))

setup.py

@@ -9,7 +9,7 @@ with open('requirements.txt') as f:
 
 
 def long_description():
-    with codecs.open('README.md', 'rb') as f:
+    with codecs.open('README.md', 'r') as f:
         return str(f.read())
 
 setup(