fix(library): get_tag_by_name (#1006)

Author: Computerdores

src/tagstudio/core/library/alchemy/library.py

@@ -858,7 +858,7 @@ def get_paths(self, glob: str | None = None, limit: int = -1) -> list[str]:
     def search_library(
         self,
         search: BrowsingState,
-        page_size: int,
+        page_size: int | None,
     ) -> SearchResult:
         """Filter library by search query.
 
@@ -903,7 +903,8 @@ def search_library(
                     sort_on = func.lower(Entry.path)
 
             statement = statement.order_by(asc(sort_on) if search.ascending else desc(sort_on))
-            statement = statement.limit(page_size).offset(search.page_index * page_size)
+            if page_size is not None:
+                statement = statement.limit(page_size).offset(search.page_index * page_size)
 
             logger.info(
                 "searching library",
@@ -1425,23 +1426,38 @@ def get_tag(self, tag_id: int) -> Tag | None:
             )
             tag = session.scalar(tags_query.where(Tag.id == tag_id))
 
-            session.expunge(tag)
-            for parent in tag.parent_tags:
-                session.expunge(parent)
+            if tag is not None:
+                session.expunge(tag)
 
-            for alias in tag.aliases:
-                session.expunge(alias)
+                for parent in tag.parent_tags:
+                    session.expunge(parent)
+
+                for alias in tag.aliases:
+                    session.expunge(alias)
 
         return tag
 
     def get_tag_by_name(self, tag_name: str) -> Tag | None:
         with Session(self.engine) as session:
             statement = (
                 select(Tag)
+                .options(selectinload(Tag.parent_tags), selectinload(Tag.aliases))
                 .outerjoin(TagAlias)
                 .where(or_(Tag.name == tag_name, TagAlias.name == tag_name))
             )
-            return session.scalar(statement)
+
+            tag = session.scalar(statement)
+
+            if tag is not None:
+                session.expunge(tag)
+
+                for parent in tag.parent_tags:
+                    session.expunge(parent)
+
+                for alias in tag.aliases:
+                    session.expunge(alias)
+
+        return tag
 
     def get_alias(self, tag_id: int, alias_id: int) -> TagAlias | None:
         with Session(self.engine) as session: