From 829c34dd69fc5b5b4356bfa747072464cd323888 Mon Sep 17 00:00:00 2001
From: Matthieu Gautier <mgautier@kymeria.fr>
Date: Tue, 16 Oct 2018 17:35:51 +0200
Subject: [PATCH] Store in the book instance if the given path is valid.

The path may exist and not be valid if the zim file is not truncated
(ie, interrupted download)
---
 include/book.h  | 3 +++
 src/manager.cpp | 4 +++-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/include/book.h b/include/book.h
index ffc9a408b..b79358349 100644
--- a/include/book.h
+++ b/include/book.h
@@ -51,6 +51,7 @@ class Book
   bool readOnly() const { return m_readOnly; }
   const std::string& getId() const { return m_id; }
   const std::string& getPath() const { return m_path; }
+  bool isPathValid() const { return m_pathValid; }
   const std::string& getIndexPath() const { return m_indexPath; }
   const supportedIndexType& getIndexType() const { return m_indexType; }
   const std::string& getTitle() const { return m_title; }
@@ -72,6 +73,7 @@ class Book
   void setReadOnly(bool readOnly) { m_readOnly = readOnly; }
   void setId(const std::string& id) { m_id = id; }
   void setPath(const std::string& path);
+  void setPathValid(bool valid) { m_pathValid = valid; }
   void setIndexPath(const std::string& indexPath);
   void setIndexType(supportedIndexType indexType) { m_indexType = indexType;}
   void setTitle(const std::string& title) { m_title = title; }
@@ -93,6 +95,7 @@ class Book
  protected:
   std::string m_id;
   std::string m_path;
+  bool m_pathValid;
   std::string m_indexPath;
   supportedIndexType m_indexType;
   std::string m_title;
diff --git a/src/manager.cpp b/src/manager.cpp
index 8b646bf37..e85081dc1 100644
--- a/src/manager.cpp
+++ b/src/manager.cpp
@@ -213,8 +213,10 @@ bool Manager::readBookFromPath(const std::string& path, kiwix::Book* book)
   try {
     kiwix::Reader reader(path);
     book->update(reader);
+    book->setPathValid(true);
   } catch (const std::exception& e) {
-    std::cerr << e.what() << std::endl;
+    std::cerr << "Invalid " << path << " : " << e.what() << std::endl;
+    book->setPathValid(false);
     return false;
   }