Remove skipInvalid flag

skipInvalid=false makes little sense, if an invalid book is found, we can simply choose to ignore it rather stopping the whole operation midway.

Author: juuz0

include/manager.h

@@ -159,11 +159,9 @@ class Manager
    * Add all books from the directory tree into the library.
    * 
    * @param path          The path of the directory to scan.
-   * @param skipInvalid          If the function should stop for an invalid file.
    * @param verboseFlag   Verbose logs flag.
    */
   void addBooksFromDirectory(const std::string& path,
-                             const bool skipInvalid = true,
                              const bool verboseFlag = false);
 
   std::string writableLibraryPath;

src/manager.cpp

@@ -257,7 +257,6 @@ bool Manager::addBookFromPath(const std::string& pathToOpen,
 }
 
 void Manager::addBooksFromDirectory(const std::string& path,
-                                    const bool skipInvalid,
                                     const bool verboseFlag)
 {
   std::set<std::string> iteratedDirs;
@@ -285,11 +284,7 @@ void Manager::addBooksFromDirectory(const std::string& path,
           std::cout << "Already iterated over " << pathString << ". Skipping..." << std::endl;
       } else {
         if (!this->addBookFromPath(pathString, pathString, "", false)) {
-          if (skipInvalid)
-            std::cerr << "Skipping invalid file: " << pathString << std::endl;
-          else {
-            throw std::runtime_error("Unable to add file: " + pathString + " into the library.");
-          }
+          std::cerr << "Could not add " << pathString << " into the library." << std::endl;
         } else if (verboseFlag) {
           std::cout << "Added " << pathString << " into the library." << std::endl;
           totalBooksAdded++;