Uploader is working on windows

cps/templates/detail.html

@@ -106,7 +106,7 @@
             {% endif %}
               <div class="btn-group" role="group">
                 <button id="btnGroupDrop2" type="button" class="btn btn-primary dropdown-toggle" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">
-                  <span class="glyphicon glyphicon-eye-open"></span>Read in browser
+                  <span class="glyphicon glyphicon-eye-open"></span> Read in browser
                   <span class="caret"></span>
                 </button>
                 <ul class="dropdown-menu" aria-labelledby="btnGroupDrop2">

cps/uploader.py

@@ -1,25 +1,24 @@
 import os
+from tempfile import gettempdir
 import hashlib
 from collections import namedtuple
 import book_formats
 
-
-tmp_dir = "/tmp/calibre-web"
-
 BookMeta = namedtuple('BookMeta', 'file_path, extension, title, author, cover, description, tags, series, series_id')
 
-
 """
  :rtype: BookMeta
 """
 def upload(file):
+    tmp_dir = os.path.join(gettempdir(), 'calibre_web')
+
     if not os.path.isdir(tmp_dir):
         os.mkdir(tmp_dir)
 
     filename = file.filename
     filename_root, file_extension = os.path.splitext(filename)
     md5 = hashlib.md5()
-    md5.update(filename)
+    md5.update(filename.encode('utf-8'))
     tmp_file_path = os.path.join(tmp_dir, md5.hexdigest())
     file.save(tmp_file_path)
     meta = book_formats.process(tmp_file_path, filename_root, file_extension)

cps/web.py

@@ -544,6 +544,7 @@ def get_cover(cover_path):
 @login_required
 def read_book(book_id,format):
     book = db.session.query(db.Books).filter(db.Books.id == book_id).first()
+    if book :
         book_dir = os.path.join(config.MAIN_DIR, "cps","static", str(book_id))
         if not os.path.exists(book_dir):
             os.mkdir(book_dir)
@@ -586,6 +587,10 @@ def read_book(book_id,format):
 	            txt_file =  os.path.join(config.DB_ROOT, book.path, book.data[0].name) + ".txt"
 	            copyfile(txt_file,tmp_file)
 	        return render_template('readtxt.html', txtfile=all_name, title="Read a Book")
+    else :
+        flash("Error opening eBook. File does not exist or file is not accessible:", category="error")
+        return redirect('/' or url_for("index", _external=True))
+
 
 @app.route("/download/<int:book_id>/<format>")
 @login_required
@@ -1167,14 +1172,15 @@ def upload():
         file = request.files['btn-upload']
         meta = uploader.upload(file)
 
-        title = meta.title
-        author = meta.author
+        title = meta.title.encode('utf-8')
+        author = meta.author.encode('utf-8')
 
         title_dir = helper.get_valid_filename(title.decode('utf-8'), False)
         author_dir = helper.get_valid_filename(author.decode('utf-8'), False)
         data_name = title_dir
-        filepath = config.DB_ROOT + "/" + author_dir + "/" + title_dir
-        saved_filename = filepath + "/" + data_name + meta.extension
+        filepath = config.DB_ROOT + os.sep + author_dir + os.sep + title_dir
+        saved_filename = filepath + os.sep + data_name + meta.extension
+
         if not os.path.exists(filepath):
             try:
                 os.makedirs(filepath)
@@ -1182,10 +1188,14 @@ def upload():
                 flash("Failed to create path %s (Permission denied)." % filepath, category="error")
                 return redirect(url_for('index', _external=True))
         try:
-            move(meta.file_path, saved_filename)
-        except OSError:
+            copyfile(meta.file_path, saved_filename)
+        except OSError, e:
             flash("Failed to store file %s (Permission denied)." % saved_filename, category="error")
             return redirect(url_for('index', _external=True))
+        try:
+            os.unlink(meta.file_path)
+        except OSError, e:
+            flash("Failed to delete file %s (Permission denied)." % meta.file_path, category="warning")
 
         file_size = os.path.getsize(saved_filename)
         if meta.cover is None: