Cutlery/calibre
1
0
Fork 0
mirror of https://github.com/kovidgoyal/calibre.git synced 2025-07-09 03:04:10 -04:00
Code Issues Packages Projects Releases Wiki Activity

Merge branch 'master' of https://github.com/cbhaley/calibre

Browse Source
This commit is contained in:
Kovid Goyal 2022-09-20 15:27:20 +05:30
commit f85e7ce880
No known key found for this signature in database
GPG Key ID: 06BC317B515ACE7C
2 changed files with 19 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/calibre/db/view.py
View File

@ -41,8 +41,11 @@ class InTagBrowserVirtualField:
self._ids = _ids
def iter_searchable_values(self, get_metadata, candidates, default_value=None):
# The returned value can be any string. For example it could be book_id
# as a string, but there is little point in spending the cpu cycles for
# that as no one will do a search like in_tag_browser:1544 (a book id)
for book_id in candidates:
yield str(book_id) if self._ids is None or book_id in self._ids else default_value, {book_id}
yield 'A' if self._ids is None or book_id in self._ids else default_value, {book_id}
def sort_keys_for_books(self, get_metadata, lang_map):
null = sys.maxsize

25
src/calibre/gui2/tag_browser/model.py
View File

@ -811,16 +811,21 @@ class TagsModel(QAbstractItemModel): # {{{
self.research_required.emit()
def set_in_tag_browser(self):
# rebuild the list even if there is no filter. This lets us support
# in_tag_browser:true or :false based on the displayed categories. It is
# a form of shorthand for searching for all the visible categories with
# category1:true OR category2:true etc. The cost: walking the tree and
# building the set for a case that will certainly rarely be different
# from all books because all books have authors.
id_set = set()
for x in (a for a in self.root_item.children if a.category_key != 'search' and not a.is_gst):
for t in x.child_tags():
id_set |= t.tag.id_set
# If the filter isn't set then don't build the list, improving
# performance significantly for large libraries or libraries with lots
# of categories. This means that in_tag_browser:true with no filter will
# return all books. This is incorrect in the rare case where the
# category list in the tag browser doesn't contain a category like
# authors that by definition matches all books because all books have an
# author. If really needed the user can work around this 'error' by
# clicking on the categories of interest with the connector set to 'or'.
if self.filter_categories_by:
id_set = set()
for x in (a for a in self.root_item.children if a.category_key != 'search' and not a.is_gst):
for t in x.child_tags():
id_set |= t.tag.id_set
else:
id_set = None
changed = self.db.data.get_in_tag_browser() != id_set
self.db.data.set_in_tag_browser(id_set)
return changed