fix(logging): restore two changes lost in the merge resolution

- semantic_search: test the query limit again, not SearchParam.limit
- modify_search_results: restore the modify_single_result definition
- wtsite: log the slot content via pformat instead of printing it
- refs #130

Author: LukasGold

src/osw/wiki_tools.py

@@ -323,7 +323,10 @@ def semantic_search_(single_query):
                 _logger.debug(f"Query '{single_query}' returned no results")
             else:
                 _logger.debug(f"Query '{single_query}' returned {n} results")
-        if n >= query.limit:
+        # No limit in force, or 'limit=0' asking for no results at all as a
+        # count format does, means the result count says nothing about
+        # truncation
+        if limit and n >= limit:
             warnings.warn(
                 f"Query '{single_query}' returned {n} results, which meets the "
                 f"requested limit of {limit}. Results are truncated - raise "

src/osw/wtsite.py

@@ -14,7 +14,7 @@
 from datetime import datetime
 from io import StringIO
 from pathlib import Path
-from pprint import pprint
+from pprint import pformat
 from time import sleep
 from typing import Any, Dict, List, Optional, Union
 from warnings import warn
@@ -643,19 +643,21 @@ def modify_search_results(
             titles = titles[0:limit]
         if param.log:
             _logger.debug(f"Found: {titles}")
-        for title in titles:
+
+        def modify_single_result(title: str):
             wtpage = self.get_page(WtSite.GetPageParam(titles=[title])).pages[0]
             modify_page(wtpage)
-            if log:
+            if param.log:
                 _logger.debug(f"\n======= {title} =======")
                 for slot in wtpage._slots:
                     content = wtpage.get_slot_content(slot)
                     # if isinstance(content, dict): content = json.dumps(content)
                     _logger.debug(f"   ==== {title}:{slot} ====   ")
-                    pprint(content)
+                    _logger.debug(pformat(content))
                     _logger.debug("\n")
             if not param.dryrun:
-                wtpage.edit(comment)
+                wtpage.edit(param.comment)
+
         if param.parallel:
             _ = parallelize(modify_single_result, titles, flush_at_end=param.log)
         else: