diff --git a/src/book.cpp b/src/book.cpp index 234b6a73..4964e359 100644 --- a/src/book.cpp +++ b/src/book.cpp @@ -48,8 +48,8 @@ bool Book::update(const kiwix::Book& other) m_path = other.m_path; m_pathValid = other.m_pathValid; m_url = other.m_url; - m_tags = other.m_tags; m_name = other.m_name; + m_tags = other.m_tags; m_favicon = other.m_favicon; m_faviconMimeType = other.m_faviconMimeType; @@ -59,13 +59,14 @@ bool Book::update(const kiwix::Book& other) void Book::update(const kiwix::Reader& reader) { m_path = reader.getZimFilePath(); + m_pathValid = true; m_id = reader.getId(); + m_title = reader.getTitle(); m_description = reader.getDescription(); m_language = reader.getLanguage(); - m_date = reader.getDate(); m_creator = reader.getCreator(); m_publisher = reader.getPublisher(); - m_title = reader.getTitle(); + m_date = reader.getDate(); m_name = reader.getName(); m_tags = reader.getTags(); m_origId = reader.getOrigId(); @@ -87,14 +88,14 @@ void Book::updateFromXml(const pugi::xml_node& node, const std::string& baseDir) } m_path = path; m_title = ATTR("title"); - m_name = ATTR("name"); - m_tags = ATTR("tags"); m_description = ATTR("description"); m_language = ATTR("language"); - m_date = ATTR("date"); m_creator = ATTR("creator"); m_publisher = ATTR("publisher"); + m_date = ATTR("date"); m_url = ATTR("url"); + m_name = ATTR("name"); + m_tags = ATTR("tags"); m_origId = ATTR("origId"); m_articleCount = strtoull(ATTR("articleCount"), 0, 0); m_mediaCount = strtoull(ATTR("mediaCount"), 0, 0); @@ -124,12 +125,12 @@ void Book::updateFromOpds(const pugi::xml_node& node, const std::string& urlHost m_id.erase(0, 9); } m_title = VALUE("title"); - m_name = VALUE("name"); m_description = VALUE("summary"); m_language = VALUE("language"); - m_date = fromOpdsDate(VALUE("updated")); m_creator = node.child("author").child("name").child_value(); m_publisher = node.child("publisher").child("name").child_value(); + m_date = fromOpdsDate(VALUE("updated")); + m_name = VALUE("name"); m_tags = VALUE("tags"); for(auto linkNode = node.child("link"); linkNode; linkNode = linkNode.next_sibling("link")) { diff --git a/src/libxml_dumper.cpp b/src/libxml_dumper.cpp index b6d3b488..58e35073 100644 --- a/src/libxml_dumper.cpp +++ b/src/libxml_dumper.cpp @@ -53,12 +53,12 @@ void LibXMLDumper::handleBook(Book book, pugi::xml_node root_node) { if (book.getOrigId().empty()) { ADD_ATTR_NOT_EMPTY(entry_node, "title", book.getTitle()); - ADD_ATTR_NOT_EMPTY(entry_node, "name", book.getName()); - ADD_ATTR_NOT_EMPTY(entry_node, "tags", book.getTags()); ADD_ATTR_NOT_EMPTY(entry_node, "description", book.getDescription()); ADD_ATTR_NOT_EMPTY(entry_node, "language", book.getLanguage()); ADD_ATTR_NOT_EMPTY(entry_node, "creator", book.getCreator()); ADD_ATTR_NOT_EMPTY(entry_node, "publisher", book.getPublisher()); + ADD_ATTR_NOT_EMPTY(entry_node, "name", book.getName()); + ADD_ATTR_NOT_EMPTY(entry_node, "tags", book.getTags()); ADD_ATTR_NOT_EMPTY(entry_node, "faviconMimeType", book.getFaviconMimeType()); if (!book.getFavicon().empty()) ADD_ATTRIBUTE(entry_node, "favicon", base64_encode(book.getFavicon())); diff --git a/src/opds_dumper.cpp b/src/opds_dumper.cpp index ceefe0a8..04cdb543 100644 --- a/src/opds_dumper.cpp +++ b/src/opds_dumper.cpp @@ -70,14 +70,14 @@ void OPDSDumper::setOpenSearchInfo(int totalResults, int startIndex, int count) pugi::xml_node OPDSDumper::handleBook(Book book, pugi::xml_node root_node) { auto entry_node = root_node.append_child("entry"); - ADD_TEXT_ENTRY(entry_node, "title", book.getTitle()); - ADD_TEXT_ENTRY(entry_node, "name", book.getName()); ADD_TEXT_ENTRY(entry_node, "id", "urn:uuid:"+book.getId()); - ADD_TEXT_ENTRY(entry_node, "icon", rootLocation + "/meta?name=favicon&content=" + book.getHumanReadableIdFromPath()); - ADD_TEXT_ENTRY(entry_node, "updated", gen_date_from_yyyy_mm_dd(book.getDate())); + ADD_TEXT_ENTRY(entry_node, "title", book.getTitle()); ADD_TEXT_ENTRY(entry_node, "summary", book.getDescription()); - ADD_TEXT_ENTRY(entry_node, "tags", book.getTags()); ADD_TEXT_ENTRY(entry_node, "language", book.getLanguage()); + ADD_TEXT_ENTRY(entry_node, "updated", gen_date_from_yyyy_mm_dd(book.getDate())); + ADD_TEXT_ENTRY(entry_node, "name", book.getName()); + ADD_TEXT_ENTRY(entry_node, "tags", book.getTags()); + ADD_TEXT_ENTRY(entry_node, "icon", rootLocation + "/meta?name=favicon&content=" + book.getHumanReadableIdFromPath()); auto content_node = entry_node.append_child("link"); content_node.append_attribute("type") = "text/html";