remove offending print

[wolnelektury.git] / apps / catalogue / management / commands / importbooks.py

diff --git a/apps/catalogue/management/commands/importbooks.py b/apps/catalogue/management/commands/importbooks.py
index 699f2c9..93c68a2 100644 (file)
--- a/apps/catalogue/management/commands/importbooks.py
+++ b/apps/catalogue/management/commands/importbooks.py
@@ -6,7 +6,7 @@ import os
 import sys
 import time
 from optparse import make_option
-
+from django.conf import settings
 from django.core.management.base import BaseCommand
 from django.core.management.color import color_style
 from django.core.files import File
@@ -14,13 +14,14 @@ from django.core.files import File
 from catalogue.models import Book
 from picture.models import Picture
 
+from search import Index
 
 class Command(BaseCommand):
     option_list = BaseCommand.option_list + (
         make_option('-q', '--quiet', action='store_false', dest='verbose', default=True,
             help='Verbosity level; 0=minimal output, 1=normal output, 2=all output'),
         make_option('-f', '--force', action='store_true', dest='force', default=False,
-            help='Print status messages to stdout'),
+            help='Overwrite works already in the catalogue'),
         make_option('-E', '--no-build-epub', action='store_false', dest='build_epub', default=True,
             help='Don\'t build EPUB file'),
         make_option('-M', '--no-build-mobi', action='store_false', dest='build_mobi', default=True,
@@ -29,11 +30,12 @@ class Command(BaseCommand):
             help='Don\'t build TXT file'),
         make_option('-P', '--no-build-pdf', action='store_false', dest='build_pdf', default=True,
             help='Don\'t build PDF file'),
+        make_option('-S', '--no-search-index', action='store_false', dest='search_index', default=True,
+            help='Skip indexing imported works for search'),
         make_option('-w', '--wait-until', dest='wait_until', metavar='TIME',
             help='Wait until specified time (Y-M-D h:m:s)'),
         make_option('-p', '--picture', action='store_true', dest='import_picture', default=False,
             help='Import pictures'),
-        
     )
     help = 'Imports books from the specified directories.'
     args = 'directory [directory ...]'
@@ -42,27 +44,20 @@ class Command(BaseCommand):
         verbose = options.get('verbose')
         file_base, ext = os.path.splitext(file_path)
         book = Book.from_xml_file(file_path, overwrite=options.get('force'),
-                                  build_epub=options.get('build_epub'),
-                                  build_txt=options.get('build_txt'),
-                                  build_pdf=options.get('build_pdf'),
-                                  build_mobi=options.get('build_mobi'))
-
-        if os.path.isfile(file_base + '.pdf'):
-            book.pdf_file.save('%s.pdf' % book.slug, File(file(file_base + '.pdf')))
-            if verbose:
-                print "Importing %s.pdf" % file_base
-        if os.path.isfile(file_base + '.mobi'):
-            book.mobi_file.save('%s.mobi' % book.slug, File(file(file_base + '.mobi')))
-            if verbose:
-                print "Importing %s.mobi" % file_base
-        if os.path.isfile(file_base + '.epub'):
-            book.epub_file.save('%s.epub' % book.slug, File(file(file_base + '.epub')))
-            if verbose:
-                print "Importing %s.epub" % file_base
-        if os.path.isfile(file_base + '.txt'):
-            book.txt_file.save('%s.txt' % book.slug, File(file(file_base + '.txt')))
-            if verbose:
-                print "Importing %s.txt" % file_base
+                                                    build_epub=options.get('build_epub'),
+                                                    build_txt=options.get('build_txt'),
+                                                    build_pdf=options.get('build_pdf'),
+                                                    build_mobi=options.get('build_mobi'),
+                                                    search_index=options.get('search_index'),
+                                                    search_index_tags=False)
+        for ebook_format in Book.ebook_formats:
+            if os.path.isfile(file_base + '.' + ebook_format):
+                getattr(book, '%s_file' % ebook_format).save(
+                    '%s.%s' % (book.slug, ebook_format),
+                    File(file(file_base + '.' + ebook_format)))
+                if verbose:
+                    print "Importing %s.%s" % (file_base, ebook_format)
+
         book.save()
 
     def import_picture(self, file_path, options):
@@ -87,6 +82,16 @@ class Command(BaseCommand):
                     time.strftime('%Y-%m-%d %H:%M:%S',
                     time.localtime(wait_until)), wait_until - time.time())
 
+        index = None
+        if options.get('search_index') and not settings.NO_SEARCH_INDEX:
+            index = Index()
+            try:
+                index.index_tags()
+                index.index.commit()
+            except Exception, e:
+                index.index.rollback()
+                raise e
+
         # Start transaction management.
         transaction.commit_unless_managed()
         transaction.enter_transaction_management()
@@ -124,6 +129,7 @@ class Command(BaseCommand):
                         else:
                             self.import_book(file_path, options)
                         files_imported += 1
+                        transaction.commit()
 
                     except (Book.AlreadyExists, Picture.AlreadyExists):
                         print self.style.ERROR('%s: Book or Picture already imported. Skipping. To overwrite use --force.' %
@@ -156,4 +162,3 @@ class Command(BaseCommand):
 
         transaction.commit()
         transaction.leave_transaction_management()
-